Fix saving sitemap definitions (#19)

Fixes #15 Addresses wintercms/winter#1219, wintercms/winter#1222 Replaces #18
wintercms · Oct 9, 2024 · fd6c2e3 · fd6c2e3
1 parent a427498
commit fd6c2e3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 10 deletions.
diff --git a/assets/js/sitemap-definitions.js b/assets/js/sitemap-definitions.js
@@ -49,11 +49,12 @@
             return result
         }
 
-        data.options.data['itemData'] = iterator($items)
+        const fieldName = $('div[data-control=sitemap-item-editor]').data('fieldName')
+        data.options.data[fieldName] = iterator($items);
     }
 
     $(document).ready(function(){
         $.wn.sitemapDefinitions = new SitemapDefinitions()
     })
 
-}(window.jQuery);
+}(window.jQuery);
diff --git a/formwidgets/SitemapItems.php b/formwidgets/SitemapItems.php
@@ -75,14 +75,6 @@ protected function loadAssets()
         $this->addJs('js/sitemap-items-editor.js', 'core');
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public function getSaveValue($value)
-    {
-        return post('itemData');
-    }
-
     //
     // Methods for the internal use
     //

diff --git a/formwidgets/sitemapitems/partials/_sitemapitems.htm b/formwidgets/sitemapitems/partials/_sitemapitems.htm
@@ -3,6 +3,7 @@
     data-control="sitemap-item-editor"
     data-alias="<?= $this->alias ?>"
     data-item-properties="<?= e($itemProperties) ?>"
+    data-field-name="<?= $this->getFieldName() ?>"
 >
     <div class="control-scrollbar" data-control="scrollbar">
         <div class="control-treeview treeview-light"