Fix error on order change on keywordGroup & fix bad forgotten visible arg on view and edit template

Author: mespeche

AdminIncludes/module-config-js.html

@@ -20,7 +20,7 @@
             placement   : 'left',
             success     : function(response, newValue) {
                 // The URL template
-                var url = "{url path='/admin/module/Keyword/group/update-position' keyword_group_id='__ID__' position='__POS__'}";
+                var url = "{url path='/admin/module/Keyword/group/update-position' keyword_group_id='__ID__' position='__POS__' noamp=1}";
 
                 // Perform subtitutions
                 url = url.replace('__ID__', $(this).data('id'))

AdminIncludes/module_configuration.html

@@ -195,4 +195,5 @@
 
     form_action         = {url path='/admin/module/Keyword/group/delete'}
     form_content        = {$smarty.capture.keyword_group_delete_dialog nofilter}
-}
+}
+

Config/routing.xml

@@ -24,6 +24,10 @@
         <requirement key="product_id">\d+</requirement>
     </route>
 
+    <route id="admin.keyword.group.update-position" path="/admin/module/Keyword/group/update-position">
+        <default key="_controller">Keyword\Controller\Admin\KeywordGroupController::updatePositionAction</default>
+    </route>
+
     <route id="admin.keyword.update-position" path="/admin/module/Keyword/update-position">
         <default key="_controller">Keyword\Controller\Admin\KeywordController::updatePositionAction</default>
     </route>
@@ -77,10 +81,6 @@
         <default key="_controller">Keyword\Controller\Admin\KeywordGroupController::deleteAction</default>
     </route>
 
-    <route id="admin.keyword.group.update-position" path="/admin/module/Keyword/group/update-position">
-        <default key="_controller">Keyword\Controller\Admin\KeywordGroupController::updatePositionAction</default>
-    </route>
-
     <route id="admin.keyword.group.toggle-online" path="/admin/module/Keyword/group/toggle-online">
         <default key="_controller">Keyword\Controller\Admin\KeywordGroupController::setToggleVisibilityAction</default>
     </route>

templates/backOffice/default/keyword-group-view.html

@@ -9,7 +9,7 @@
 <div class="keyword view-keyword">
     <div id="wrapper" class="container">
 
-        {loop name="keyword" type="keyword_group" id="$keyword_group_id"}
+        {loop name="keyword" type="keyword_group" visible="*" id="$keyword_group_id"}
         <ul class="breadcrumb">
             <li><a href="{url path='admin/home'}">{intl l="Home"}</a></li>
             <li><a href="{url path='admin/module/Keyword'}">{intl l="Keywords" d='keyword'}</a></li>
@@ -286,7 +286,7 @@
                     url = url.replace('__ID__', $(this).data('id'))
                             .replace('__KEYWORD_GROUP_ID__', $(this).data('group'))
                             .replace('__POS__', newValue);
-                    
+
                     // Reload the page
                     location.href = url;
                 }

templates/backOffice/default/keyword-view.html

@@ -9,7 +9,7 @@
 <div class="keyword view-keyword">
     <div id="wrapper" class="container">
 
-        {loop name="keyword" type="keyword" id="$keyword_id"}
+        {loop name="keyword" type="keyword" visible="*" id="$keyword_id"}
         <ul class="breadcrumb">
             <li><a href="{url path='admin/home'}">{intl l="Home"}</a></li>
             <li><a href="{url path='admin/module/Keyword'}">{intl l="Keywords" d='keyword'}</a></li>