Skip to content

Entities in Repeater-Werten erhalten (Backport #463), Release 9.5.3 - #465

Merged
skerbis merged 1 commit into
9.xfrom
fix/entities-9x
Oct 7, 2026
Merged

skerbis merged 1 commit into
9.xfrom
fix/entities-9x

Conversation

@skerbis

@skerbis skerbis commented Oct 7, 2026

Copy link
Copy Markdown
Member

Backport von #463 auf die 9.5-Linie, Release 9.5.3.

Die 9.x-Linie hat dieselben Stellen, nur ohne decodeRaw():

  • MFormRepeaterHelper::decode() liest zuerst das rohe JSON, Entities werden nur noch als Rückfall umgewandelt (escapter Platzhalter 'REX_VALUE[1]'). decodeById() läuft darüber.
  • MFormValueHandler::loadRexVars() liest ohne htmlspecialchars_decode().
  • MBlockToRepeaterConverter::convertData() liest zuerst roh (" im Inhalt ließ die Konvertierung sonst scheitern).

#462 betrifft 9.x nicht: Die Tab-Leiste nutzt dort noch Floats, pull-right funktioniert.

9.x hat keine Testsuite. Im Container standalone geprüft: rohe Werte mit "/&/< bleiben erhalten, der escapte Platzhalter mit nl2br() liefert dasselbe wie vorher, Konverter mit rohen und escapten Werten. Gegen den alten Code schlagen die beiden Entity-Fälle fehl.

🤖 Generated with Claude Code

…kport #463), Release 9.5.3

Per Slot-Id kommt der Wert roh aus der Datenbank; decode() wandelte
Entities vor dem json_decode() um. Ein " aus TinyMCE machte das JSON
ungueltig (Modul leer), <h2> als Text kam als Markup zurueck.
decode() und MBlockToRepeaterConverter::convertData() lesen jetzt zuerst
roh und wandeln Entities nur als Rueckfall um; MFormValueHandler liest
ohne htmlspecialchars_decode().

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 108a6558-6e87-44bf-b4d7-cb9ed23347f6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@skerbis
skerbis merged commit f5bb9ab into 9.x Oct 7, 2026
7 checks passed
@skerbis
skerbis deleted the fix/entities-9x branch October 7, 2026 20:09
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.

1 participant