Phil/philflow.io
1
0
Fork 0
Code Issues 21 Pull requests Projects Releases Packages Wiki Activity Actions

Sitemap-Index lastmod ueber Composer-Patch korrigieren #152

Merged
Phil merged 1 commit from fix/sitemap-index-lastmod into main 2026-09-10 20:43:37 +00:00
Conversation 0 Commits 1 Files changed 6 +61 -1
Phil commented 2026-09-10 20:43:32 +00:00
Owner
Copy link

Zusammenfassung

  • ether/seos SitemapService::_getUpdated() fehlte ein orderBy, deshalb nahm der Sitemap-Index das dateUpdated des ersten Elements der Default-Sortierung statt das Maximum. Fix: orderBy(['dateUpdated' => SORT_DESC]).
  • Fix als versionierter Composer-Patch (patches/ether-seo-sitemap-index-lastmod.patch) via cweagans/composer-patches, statt direktem vendor/-Edit.
  • patch-Binary fehlte in allen drei Dockerfiles (Alpine-Basisimages) — ergaenzt, sonst schlaegt die Patch-Anwendung bei composer install fehl.
  • Dockerfile.prod kopierte patches/ nicht in den Build-Kontext der Dependencies-Stage, bevor composer install laeuft — nachgezogen.

Verifikation

  • composer install from scratch (vendor geloescht, neu gebautes Image mit patch-Binary): Log zeigt Applying patches for ether/seo ... patches/ether-seo-sitemap-index-lastmod.patch ohne Fehler.
  • Lokal: sitemap.xml lastmod fuer die Blog-Sektion entsprach exakt dem Maximum der lastmod-Werte in sitemap_sections_8_0.xml.
  • Entry neu gespeichert (Touch) -> Index-lastmod bewegte sich exakt auf den neuen dateUpdated-Wert mit.

Closes #151

🤖 Generated with Claude Code

https://claude.ai/code/session_01FJ1qNYn5cu4DL2sWMV9dpE

## Zusammenfassung - `ether/seo`s `SitemapService::_getUpdated()` fehlte ein `orderBy`, deshalb nahm der Sitemap-Index das `dateUpdated` des ersten Elements der Default-Sortierung statt das Maximum. Fix: `orderBy(['dateUpdated' => SORT_DESC])`. - Fix als versionierter Composer-Patch (`patches/ether-seo-sitemap-index-lastmod.patch`) via `cweagans/composer-patches`, statt direktem `vendor/`-Edit. - `patch`-Binary fehlte in allen drei Dockerfiles (Alpine-Basisimages) — ergaenzt, sonst schlaegt die Patch-Anwendung bei `composer install` fehl. - `Dockerfile.prod` kopierte `patches/` nicht in den Build-Kontext der Dependencies-Stage, bevor `composer install` laeuft — nachgezogen. ## Verifikation - `composer install` from scratch (vendor geloescht, neu gebautes Image mit `patch`-Binary): Log zeigt `Applying patches for ether/seo ... patches/ether-seo-sitemap-index-lastmod.patch` ohne Fehler. - Lokal: `sitemap.xml` lastmod fuer die Blog-Sektion entsprach exakt dem Maximum der lastmod-Werte in `sitemap_sections_8_0.xml`. - Entry neu gespeichert (Touch) -> Index-lastmod bewegte sich exakt auf den neuen `dateUpdated`-Wert mit. Closes #151 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01FJ1qNYn5cu4DL2sWMV9dpE
ether/seo's SitemapService::_getUpdated() liest das dateUpdated des
ersten Elements der Default-Sortierung statt des Maximums, weil kein
orderBy gesetzt ist. Dadurch zeigte der Sitemap-Index veraltete
lastmod-Werte, obwohl die referenzierte Sektions-Sitemap frischere
Entries enthielt.

Fix als versionierter Composer-Patch (cweagans/composer-patches) statt
direktem vendor/-Edit, damit composer install ihn reproduzierbar
anwendet. Dafuer patch-Binary in allen drei Dockerfiles ergaenzt
(Alpine-Images hatten sie nicht) und in Dockerfile.prod den fehlenden
COPY-Schritt fuer patches/ vor composer install nachgezogen.

Closes #151

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FJ1qNYn5cu4DL2sWMV9dpE
Phil merged commit e794d2952e into main 2026-09-10 20:43:37 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Phil/philflow.io!152
No description provided.