Skip to content

[5.x] Fix Stache index re-entrancy causing null URIs on cold stache - #14181

Merged
jasonvarga merged 4 commits into
statamic:5.xfrom
o1y:fix-stache-null-uris-5.x
Apr 3, 2026
Merged

[5.x] Fix Stache index re-entrancy causing null URIs on cold stache#14181
jasonvarga merged 4 commits into
statamic:5.xfrom
o1y:fix-stache-null-uris-5.x

Conversation

@o1y

@o1y o1y commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

Backport of #14109 to 5.x.

Fixes statamic/seo-pro#508

@o1y
o1y changed the base branch from 6.x to 5.x March 9, 2026 20:06
@mmodler

mmodler commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

I'm waiting for this one :) @o1y Can you push a empty commit to trigger the pipeline again?

@o1y
o1y force-pushed the fix-stache-null-uris-5.x branch from ecfe1b8 to 6783297 Compare March 18, 2026 09:41
@mmodler

mmodler commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Thanks! Now lets wait for the next 5.x release. Meanwhile i will use this fix via composer patch.

jasonvarga and others added 3 commits April 3, 2026 16:22
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
No internal callers remain after switching to Index::isLoading().
Kept for potential addon usage.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Wrap Index::load() body in try/finally so the static loading
stack is always cleaned up, even if an exception is thrown.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jasonvarga
jasonvarga merged commit e9edd39 into statamic:5.x Apr 3, 2026
26 checks passed
@o1y
o1y deleted the fix-stache-null-uris-5.x branch June 20, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants