Skip to content

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

Merged
jasonvarga merged 4 commits intostatamic: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 intostatamic:5.xfrom
o1y:fix-stache-null-uris-5.x

Conversation

@o1y
Copy link
Copy Markdown
Contributor

@o1y o1y commented Mar 9, 2026

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
Copy link
Copy Markdown
Contributor

mmodler commented Mar 18, 2026

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
Copy link
Copy Markdown
Contributor

mmodler commented Mar 18, 2026

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sitemap.xml is empty after cache:clear in Multisite

3 participants