release: Version 1.0.1

skerbis · skerbis · commit e9a5ade4366a · 2026-02-25T13:18:08.000+01:00
- Fix: HTML-Ersetzung id nur aus GET gelesen → leere Felder beim Editieren
- Fix: 'Name existiert bereits' beim Speichern bestehender Ersetzungsregeln
- Fix: Suchwert-Whitespace verhinderte HTML-Ersetzung (trim beim Speichern)
- Fix: i18n-Keys ohne Addon-Prefix in categories.php und listing.php
- Fix: Abbrechen-Button in HTML-Ersetzung zeigte leeres Formular statt Uebersicht
- Feat: HTML-Ersetzung OUTPUT_FILTER mit LATE-Prioritaet als eigener Filter
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -0,0 +1,21 @@
+# Changelog
+
+## [1.0.1] – 2026-02-25
+
+### Behoben
+
+- **HTML-Ersetzung: Editieren zeigte leere Felder** – `id` wurde nur aus GET gelesen, jedoch beim Formular-Submit per POST übertragen; `rex_request::request()` liest nun aus beiden Quellen
+- **HTML-Ersetzung: „Name existiert bereits" beim Speichern einer bestehenden Regel** – Folgefehler der fehlenden `id`, `nameExists()` schloss die eigene ID nun korrekt aus
+- **HTML-Ersetzung: Suchwert mit führenden/nachfolgenden Leerzeichen wurde nicht gefunden** – `trim()` wird jetzt beim Speichern angewendet
+- **Kategorien: `[translate:cancel]` und `[translate:no_data_available]`** – falsche Core-Keys ersetzt durch addon-eigene Keys (`snippets_btn_cancel`, `snippets_category_no_data`)
+- **Kategorien/Listing: `[translate:edit]`** – in `categories.php` und `listing.php` durch `snippets_category_edit` / `snippets_btn_edit` ersetzt
+- **HTML-Ersetzung: Abbrechen-Button** verzweigte zurück zur leeren Bearbeiten-Seite statt zur Übersicht
+- **HTML-Ersetzung: `OUTPUT_FILTER`** in separate Extension mit `rex_extension::LATE` ausgelagert, damit die HTML-Ersetzung garantiert nach allen anderen `NORMAL`-Filtern (z. B. Sprog, YRewrite) ausgeführt wird
+
+### Hinzugefügt
+
+- Neue i18n-Keys: `snippets_category_edit`, `snippets_category_no_data`, `snippets_btn_edit` (de_de + en_gb)
+
+## [1.0.0] – 2026-02-24
+
+- Initiales Release
diff --git a/boot.php b/boot.php
@@ -25,41 +25,38 @@
 // Frontend: Snippet-Replacement
 if (!rex::isBackend()) {
     rex_extension::register('OUTPUT_FILTER', static function (rex_extension_point $ep) {
-        $content = $ep->getSubject();
-
-        // 1. Snippet-Keys ersetzen
-        $content = ReplacementService::replace($content, [
+        return ReplacementService::replace($ep->getSubject(), [
             'clang_id' => rex_clang::getCurrentId(),
         ]);
-
-        // 2. HTML-Ersetzungen anwenden
-        $content = HtmlReplacementService::process($content, 'frontend');
-
-        return $content;
     }, rex_extension::NORMAL);
+
+    // HTML-Ersetzungen immer als letzter Filter ausführen
+    rex_extension::register('OUTPUT_FILTER', static function (rex_extension_point $ep) {
+        return HtmlReplacementService::process($ep->getSubject(), 'frontend');
+    }, rex_extension::LATE);
 }
 
 // Backend: Snippet-Replacement (nur in sicheren Kontexten)
 if (rex::isBackend()) {
     rex_extension::register('OUTPUT_FILTER', static function (rex_extension_point $ep) {
-        // Prüfen, ob wir uns in einem Edit-Kontext befinden
         if (ContextDetector::isEditContext()) {
             return $ep->getSubject();
         }
 
-        $content = $ep->getSubject();
-
-        // 1. Snippet-Keys ersetzen
-        $content = ReplacementService::replace($content, [
+        return ReplacementService::replace($ep->getSubject(), [
             'context' => 'backend',
             'clang_id' => rex_clang::getCurrentId(),
         ]);
+    }, rex_extension::NORMAL);
 
-        // 2. HTML-Ersetzungen anwenden
-        $content = HtmlReplacementService::process($content, 'backend');
+    // HTML-Ersetzungen immer als letzter Filter ausführen
+    rex_extension::register('OUTPUT_FILTER', static function (rex_extension_point $ep) {
+        if (ContextDetector::isEditContext()) {
+            return $ep->getSubject();
+        }
 
-        return $content;
-    }, rex_extension::NORMAL);
+        return HtmlReplacementService::process($ep->getSubject(), 'backend');
+    }, rex_extension::LATE);
 }
 
 // Sprog-Integration (falls Sprog aktiv ist)
diff --git a/fragments/snippets/html_replacement_form.php b/fragments/snippets/html_replacement_form.php
@@ -191,7 +191,7 @@
                 <button type="submit" name="save_and_close" value="1" class="btn btn-apply">
                     <i class="rex-icon rex-icon-apply"></i> <?= $addon->i18n('snippets_save_and_close') ?>
                 </button>
-                <a href="<?= rex_url::currentBackendPage(['func' => '']) ?>" class="btn btn-abort">
+                <a href="<?= rex_url::backendPage('snippets/html_replacement') ?>" class="btn btn-abort">
                     <i class="rex-icon rex-icon-abort"></i> <?= $addon->i18n('snippets_cancel') ?>
                 </a>
             </div>
diff --git a/fragments/snippets/listing.php b/fragments/snippets/listing.php
@@ -171,7 +171,7 @@ class="btn btn-primary">
                         <?php if ($canEditThis): ?>
                         <a href="<?= rex_url::currentBackendPage(['page' => 'snippets/edit', 'func' => 'edit', 'id' => $snippet->getId()]) ?>" 
                            class="btn btn-xs btn-default">
-                            <i class="rex-icon fa-edit"></i> <?= rex_i18n::msg('edit') ?>
+                            <i class="rex-icon fa-edit"></i> <?= rex_i18n::msg('snippets_btn_edit') ?>
                         </a>
                         <?php endif; ?>
                     </td>
diff --git a/lang/de_de.lang b/lang/de_de.lang
@@ -97,6 +97,8 @@ snippets_translation_missing = Noch keine Übersetzung vorhanden
 snippets_save = Speichern
 snippets_search = Suchen
 snippets_delete = Löschen
+snippets_edit_btn = Bearbeiten
+snippets_btn_edit = Bearbeiten
 snippets_required = Pflichtfeld
 snippets_confirm_delete = Wirklich löschen
 snippets_btn_save = Speichern
@@ -122,10 +124,12 @@ perm_general_snippets[viewer] = Snippets: Nur ansehen
 
 # Kategorien
 snippets_category_add = Kategorie hinzufügen
+snippets_category_edit = Kategorie bearbeiten
 snippets_category_name = Name
 snippets_category_icon = Icon
 snippets_category_saved = Kategorie wurde gespeichert
 snippets_category_deleted = Kategorie wurde gelöscht
+snippets_category_no_data = Keine Kategorien vorhanden
 
 # Einstellungen
 snippets_settings_title = Snippet-Einstellungen
diff --git a/lang/en_gb.lang b/lang/en_gb.lang
@@ -96,6 +96,8 @@ snippets_translation_missing = No translation available yet
 snippets_save = Save
 snippets_search = Search
 snippets_delete = Delete
+snippets_edit_btn = Edit
+snippets_btn_edit = Edit
 snippets_required = Required
 snippets_confirm_delete = Really delete
 snippets_btn_save = Save
@@ -121,10 +123,12 @@ perm_general_snippets[viewer] = Snippets: View only
 
 # Categories
 snippets_category_add = Add Category
+snippets_category_edit = Edit Category
 snippets_category_name = Name
 snippets_category_icon = Icon
 snippets_category_saved = Category has been saved
 snippets_category_deleted = Category has been deleted
+snippets_category_no_data = No categories available
 
 # Settings
 snippets_settings_title = Snippet Settings
diff --git a/package.yml b/package.yml
@@ -1,5 +1,5 @@
 package: snippets
-version: '1.0.0'
+version: '1.0.1'
 author: REDAXO Community
 supportpage: https://github.com/FriendsOfREDAXO/snippets
 
diff --git a/pages/categories.php b/pages/categories.php
@@ -84,7 +84,7 @@
     $icon = is_array($category) && isset($category['icon']) ? (string) $category['icon'] : '';
     $sortOrder = is_array($category) && isset($category['sort_order']) ? (int) $category['sort_order'] : 0;
 
-    $formTitle = 'add' === $func ? rex_i18n::msg('snippets_category_add') : rex_i18n::msg('edit');
+    $formTitle = 'add' === $func ? rex_i18n::msg('snippets_category_add') : rex_i18n::msg('snippets_category_edit');
 
     $content = '
     <form method="post" action="' . rex_url::currentBackendPage() . '">
@@ -117,7 +117,7 @@
                         <i class="rex-icon rex-icon-save"></i> ' . rex_i18n::msg('snippets_save') . '
                     </button>
                     <a href="' . rex_url::currentBackendPage() . '" class="btn btn-abort">
-                        ' . rex_i18n::msg('cancel') . '
+                        ' . rex_i18n::msg('snippets_btn_cancel') . '
                     </a>
                 </div>
             </div>
@@ -151,7 +151,7 @@
 if (0 === $sql->getRows()) {
     $listContent .= '
         <tr>
-            <td colspan="5" class="text-center">' . rex_i18n::msg('no_data_available') . '</td>
+            <td colspan="5" class="text-center">' . rex_i18n::msg('snippets_category_no_data') . '</td>
         </tr>';
 } else {
     for ($i = 0; $i < $sql->getRows(); ++$i) {
@@ -169,7 +169,7 @@
             <td>' . $catSort . '</td>
             <td>
                 <a href="' . rex_url::currentBackendPage(['func' => 'edit', 'id' => $catId]) . '" class="btn btn-xs btn-default">
-                    <i class="rex-icon fa-edit"></i> ' . rex_i18n::msg('edit') . '
+                    <i class="rex-icon fa-edit"></i> ' . rex_i18n::msg('snippets_btn_edit') . '
                 </a>
                 <a href="' . rex_url::currentBackendPage(['func' => 'delete', 'id' => $catId] + $csrfToken->getUrlParams()) . '" 
                    class="btn btn-xs btn-delete"
diff --git a/pages/html_replacement.edit.php b/pages/html_replacement.edit.php
@@ -20,7 +20,8 @@
 
 $csrf = rex_csrf_token::factory('snippets_html_replacement_edit');
 
-$id = rex_request::get('id', 'int', 0);
+// id aus GET (initialer Aufruf) ODER aus POST (nach Formular-Submit, da currentBackendPage() kein id enthält)
+$id = rex_request::request('id', 'int', 0);
 $replacement = null;
 
 // Bestehende Regel laden
@@ -40,7 +41,7 @@
         $name = rex_post('name', 'string', '');
         $description = rex_post('description', 'string', '');
         $type = rex_post('type', 'string', HtmlReplacement::TYPE_CSS_SELECTOR);
-        $searchValue = rex_post('search_value', 'string', '');
+        $searchValue = trim(rex_post('search_value', 'string', ''));
         $replacementContent = rex_post('replacement', 'string', '');
         $position = rex_post('position', 'string', HtmlReplacement::POSITION_REPLACE);
         $scopeContext = rex_post('scope_context', 'string', HtmlReplacement::CONTEXT_FRONTEND);
