Skip to content

2.0.0 :: rexstan, namespace, Issues - #86

Closed
skerbis wants to merge 2 commits into
masterfrom
2.0.0
Closed

2.0.0 :: rexstan, namespace, Issues#86
skerbis wants to merge 2 commits into
masterfrom
2.0.0

Conversation

@skerbis

@skerbis skerbis commented Aug 10, 2026

Copy link
Copy Markdown
Member

Umstellung auf Namespace und:
fixed: #85
fixed: #84
fixed: #43

Copilot AI lite review requested due to automatic review settings August 10, 2026 11:34

Copilot AI 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.

Pull request overview

Diese PR hebt das AddOn auf Version 2.0.0 an, migriert die eigenen Klassen in einen Namespace und passt die Metainfo-Tab-Integration an den REDAXO-Core-Workflow an, um die in #85/#84 beschriebenen Probleme (Speichern im Tab / Deprecated-Warnung) zu beheben.

Changes:

  • Namespace-Migration auf FriendsOfREDAXO\StructureTweaks inkl. use-Imports in Page-/Boot-Dateien
  • Metainfo-Tab-Variante: DOM-Verarbeitung ohne mb_convert_encoding(..., 'HTML-ENTITIES', ...), stattdessen PHP-8.4 DOM API
  • UI-/JS-Aufräumen: Kategorien-Seite initialisiert Selectpicker/Logik zentral über assets/script.js, Inline-Scripts reduziert

Reviewed changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
update.php Lädt Lib-Klassen in Update-Kontext per __DIR__/Autoload/require, um Namespace-Klassen verfügbar zu machen.
README.md Aktualisiert Beschreibung zur Metainfo-Tab-Funktion.
pages/settings.php Importiert die namespacete Settings-Page-Klasse.
pages/content.metainfo.php Baut Metainfo-Form im Tab/Container so um, dass der Speichern-Workflow im Tab bleibt.
pages/categories.php Importiert die namespacete Categories-Page-Klasse.
package.yml Version bump + Zielplattform-Anhebung (PJAX, Mindest-PHP/REDAXO) und Subpage-Struktur angepasst.
lib/pages/page_settings.php Namespace + vereinfachte Checkbox-Logik inkl. Fallback von alter Config.
lib/pages/page_categories.php Namespace + Typisierung, Select-Feld für Artikel mit rex_category_select, entfernt Inline-JS.
lib/move_metainfo.php Namespace + Typisierung und kleinere Robustheits-/SQL-Anpassungen.
lib/move_metainfo_to_tab.php Ersetzt DOMDocument/mb_convert_encoding durch Dom\HTMLDocument-Workflow.
lib/hide_startarticle.php Namespace + Typisierung, Nonce für Inline-Script, kleinere JS-Aufräumungen.
lib/hide_category_functions.php Namespace + Typisierung, Nonce für Inline-Scripts, Bereinigung alter Branches.
lib/hide_categories.php Namespace + DOM-Umstellung auf Dom\HTMLDocument und Nonce; DOM-Filter für Select-Options.
lib/category_splitter.php Namespace + Typisierung, Nonce für Inline-Script, JS-Aufräumungen.
lib/base.php Namespace + Typisierung (Rückgabetypen / Casts).
install.php Robusteres Include von update.php via __DIR__.
CHANGELOG.md Neuer Changelog für 2.0.0 inkl. Bezug auf #84/#85.
boot.php Importiert namespacete Klassen und vereinheitlicht Metainfo-Init auf Tab-Variante.
assets/script.js Initialisiert Kategorien-Seite zentral via rex:ready + Selectpicker-Setup.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread lib/hide_categories.php
Comment thread lib/hide_categories.php
@DanielWeitenauer

DanielWeitenauer commented Aug 11, 2026

Copy link
Copy Markdown
Member

Ich muss mir das in Ruhe ansehen.
Ich sehe enorm viele unnötige oder unvollständige Änderungen, die es schwierig machen, zu erkennen was die eigentliche Fehlerbehebungen sind.

@DanielWeitenauer

Copy link
Copy Markdown
Member

@skerbis Hast du die Möglichkeit, den PR aufzuteilen?
Einmal konkrete Bugfixes und dann der Rest?
Die Namespace-Anpassung würde ich vorerst komplett verwerfen, da das Addon sehr REX-5-spezifisch ist und so wahrscheinlich gar nicht für Version 6 benötigt wird. Außerdem sind es alles keine Klassen, die als API genutzt werden sollten.

@skerbis

skerbis commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

Fast alle AddOns sind schon auf Namespace umgestellt.
Eine Aufteilung ist nicht mehr möglich. Der Namespace ist hier eher weniger das Thema. Der stört nicht.
Wichtiger ist ob die Metas sich so verhalten wie sie sollen. Sonst sind halt nur REXSTAN Checks korrigiert worden.
Aber ich versuch mal :-*

@skerbis

skerbis commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

Stacked PR split for easier review:

  1. refactor(core): rexstan + namespace baseline #87 refactor(core): rexstan + namespace baseline (base: master)
  2. fix(backend): category/settings page fixes #88 fix(backend): category/settings page fixes (base: stack/01-core)
  3. docs(ui): changelog/readme/script updates #89 docs(ui): changelog/readme/script updates (base: stack/02-backend)

Top PR #89 represents the full final diff of this branch (2.0.0).

Suggested review order: #87 -> #88 -> #89.

@skerbis

skerbis commented Aug 11, 2026

Copy link
Copy Markdown
Member Author

@DanielWeitenauer ich habe einen Stack PR angelegt

@skerbis skerbis closed this Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants