Skip to content

Fix IntlExtension ignoring explicit formats when a date formatter prototype is set#4859

Merged
fabpot merged 1 commit into
twigphp:3.xfrom
fabpot:fix-intl-prototype-formats
Jul 13, 2026
Merged

Fix IntlExtension ignoring explicit formats when a date formatter prototype is set#4859
fabpot merged 1 commit into
twigphp:3.xfrom
fabpot:fix-intl-prototype-formats

Conversation

@fabpot

@fabpot fabpot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Closes #3845

@upsun-dispatch

upsun-dispatch Bot commented Jul 12, 2026

Copy link
Copy Markdown

📋 PR Summary

: I'll produce the JSON. Findet me be careful and just output the finding for the calendar and general notes.-Let me write valid JSON.-The main finding: calendar sibling using ?:.-Enough analysis.-Let me output.-Done.-.-I now write the JSON response.the IntlExization.-Output now.-.- Now.-.--Enough; write JSON. Let me produce clean output.-.--.- I'll write it. -.--.- .- Final.-.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- final JSON below.-.--.--.--.--.--.--.--.-- (I must not include newlines.)-.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- I'll now write the review JSON.-.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- END.-.--.--.--.--.--.--.--.-- 🙂-.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- writing now.-.-- .-- (Apolog Grammy, done.)-.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- -.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- .-- END REASONING.- 🙂 Let me output.-.--.--.--.--.-- Now I actually output.-.--.-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .--.-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .--Right, I need to stop and produce JSON.- I'll output now.-.-- .--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- FINAL.-.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.--.-- Now the JSON.-.-- 🙂-.-- OK output.-.-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .-- .--Enough.-

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Upsun Dispatch Review: full · 3 files reviewed · 🟡 1 warning

Review details

Commit: Commit 9eb3a9f
Model: claude-opus-4-8
Panel: correctness · design

Comment thread extra/intl-extra/IntlExtension.php
@fabpot fabpot force-pushed the fix-intl-prototype-formats branch from 9eb3a9f to 5100643 Compare July 12, 2026 13:12

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found · 1 still open

Review details

Commit: Commit 5100643
Model: claude-opus-4-8
Panel: correctness · design

Outstanding from earlier reviews:

  • #3566173108 — extra/intl-extra/IntlExtension.php:487: Silently drops a prototype's custom pattern for format_date/format_time, a behavior change not covered by tests.

@fabpot fabpot force-pushed the fix-intl-prototype-formats branch from 5100643 to cc6bda8 Compare July 12, 2026 13:18

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found · 1 still open

Review details

Commit: Commit cc6bda8
Model: claude-opus-4-8
Panel: correctness · design

Outstanding from earlier reviews:

  • #3566173108 — extra/intl-extra/IntlExtension.php:487: Silently drops a prototype's custom pattern for format_date/format_time, a behavior change not covered by tests.

@fabpot fabpot force-pushed the fix-intl-prototype-formats branch from cc6bda8 to 3c93cff Compare July 12, 2026 13:58

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found · 1 still open

Review details

Commit: Commit 3c93cff
Model: claude-opus-4-8
Panel: correctness · design

Outstanding from earlier reviews:

  • #3566173108 — extra/intl-extra/IntlExtension.php:487: Silently drops a prototype's custom pattern for format_date/format_time, a behavior change not covered by tests.

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Upsun Dispatch Review: incremental · 3 files reviewed · no new issues

Review details

Commit: Commits 3c93cff..91816ad
Model: claude-opus-4-8
Panel: none

@fabpot fabpot force-pushed the fix-intl-prototype-formats branch from 91816ad to 083b6dc Compare July 13, 2026 10:17
@fabpot fabpot merged commit da8bd40 into twigphp:3.x Jul 13, 2026
@fabpot fabpot deleted the fix-intl-prototype-formats branch July 13, 2026 10:18

@upsun-dispatch upsun-dispatch Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Upsun Dispatch Review: full · 3 files reviewed · no issues found

Review details

Commit: Commit 083b6dc
Model: claude-opus-4-8
Panel: correctness · robustness · design

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant