fix(AjaxNodeTreeController): Fixed non-integer translationId

lib/Rozier/src/AjaxControllers/AjaxNodeTreeController.php

@@ -14,6 +14,9 @@
 use Symfony\Component\HttpFoundation\Request;
 use Themes\Rozier\Widgets\NodeTreeWidget;
 use Themes\Rozier\Widgets\TreeWidgetFactory;
+use Twig\Error\LoaderError;
+use Twig\Error\RuntimeError;
+use Twig\Error\SyntaxError;
 
 /**
  * @package Themes\Rozier\AjaxControllers
@@ -38,22 +41,22 @@ public function __construct(
      * @param Request $request
      *
      * @return JsonResponse
-     * @throws \Twig\Error\LoaderError
-     * @throws \Twig\Error\RuntimeError
-     * @throws \Twig\Error\SyntaxError
+     * @throws LoaderError
+     * @throws RuntimeError
+     * @throws SyntaxError
      */
-    public function getTreeAction(Request $request)
+    public function getTreeAction(Request $request): JsonResponse
     {
         $this->denyAccessUnlessGranted('ROLE_ACCESS_NODES');
 
         $translationId = $request->get('translationId', null);
-        if (null === $translationId) {
-            $translation = $this->em()->getRepository(Translation::class)->findDefault();
-        } else {
+        if (\is_numeric($translationId) && $translationId > 0) {
             $translation = $this->em()->find(
                 Translation::class,
                 $translationId
             );
+        } else {
+            $translation = $this->em()->getRepository(Translation::class)->findDefault();
         }
 
         /** @var NodeTreeWidget|null $nodeTree */

lib/Rozier/src/Resources/app/Rozier.js

@@ -343,20 +343,20 @@ export default class Rozier {
     /**
      * Refresh only main nodeTree.
      *
-     * @param translationId
+     * @param {Number|null|undefined} translationId
      */
     refreshMainNodeTree(translationId) {
         let $currentNodeTree = $('#tree-container').find('.nodetree-widget')
         let $currentRootTree = $currentNodeTree.find('.root-tree').eq(0)
         if ($currentRootTree.length && !translationId) {
-            translationId = parseInt($currentRootTree.attr('data-translation-id'))
+            translationId = $currentRootTree.attr('data-translation-id')
         }
 
         if ($currentNodeTree.length) {
             let postData = {
                 _token: this.ajaxToken,
                 _action: 'requestMainNodeTree',
-                translationId: translationId,
+                translationId: translationId || null,
             }
             let url = this.routes.nodesTreeAjax
 

lib/Rozier/src/Resources/app/widgets/ChildrenNodesField.js

@@ -101,7 +101,7 @@ export default class ChildrenNodesField {
             }
             const rootNodeId = parseInt($rootTree.attr('data-parent-node-id'))
             const linkedTypesRaw = $rootTree.attr('data-linked-types')
-            let translationId = parseInt($rootTree.attr('data-translation-id'))
+            let translationId = $rootTree.attr('data-translation-id')
             if (linkedTypesRaw) {
                 linkedTypes = JSON.parse(linkedTypesRaw)
             }
@@ -112,7 +112,7 @@ export default class ChildrenNodesField {
                 _action: 'requestNodeTree',
                 parentNodeId: rootNodeId,
                 linkedTypes: linkedTypes,
-                translationId: translationId,
+                translationId: translationId || null,
             }
 
             let url = window.Rozier.routes.nodesTreeAjax

lib/Rozier/src/Resources/app/widgets/StackNodeTree.js

@@ -67,9 +67,9 @@ export default class StackNodeTree {
 
         let $link = $(event.currentTarget)
         let parentNodeId = parseInt($link.attr('data-children-parent-node'))
-        let translationId = parseInt($link.attr('data-translation-id'))
+        let translationId = $link.attr('data-translation-id')
         let tagId = $link.attr('data-filter-tag')
-        this.refreshNodeTree(parentNodeId, translationId, tagId)
+        this.refreshNodeTree(parentNodeId, translationId, tagId, 1)
         return false
     }
 
@@ -134,9 +134,9 @@ export default class StackNodeTree {
 
     /**
      *
-     * @param rootNodeId
-     * @param translationId
-     * @param tagId
+     * @param {Number|undefined|null} rootNodeId
+     * @param {Number|undefined|null} translationId
+     * @param {Number|undefined|null} tagId
      * @param page
      */
     refreshNodeTree(rootNodeId, translationId, tagId, page) {

lib/Rozier/src/Resources/views/partials/css-inject.html.twig

@@ -1,7 +1,7 @@
 
 <link href="{{ asset('css/vendor.aaeedaa02e00b12e41cf.css', 'Rozier') }}" rel="stylesheet">
 
-<link href="{{ asset('css/app.6d6a59aa70c093ba0a8c.css', 'Rozier') }}" rel="stylesheet">
+<link href="{{ asset('css/app.c9045ea8fbedd28c00ed.css', 'Rozier') }}" rel="stylesheet">
 
 
 

lib/Rozier/src/Resources/views/partials/js-inject.html.twig

@@ -1,5 +1,5 @@
 
-<script src="{{ asset('js/vendor.2784169093f7024eaf12.js', 'Rozier') }}" defer type="text/javascript"></script>
+<script src="{{ asset('js/vendor.84adcd1151ebc6fc29e9.js', 'Rozier') }}" defer type="text/javascript"></script>
 
-<script src="{{ asset('js/app.2784169093f7024eaf12.js', 'Rozier') }}" defer type="text/javascript"></script>
+<script src="{{ asset('js/app.84adcd1151ebc6fc29e9.js', 'Rozier') }}" defer type="text/javascript"></script>
 

lib/Rozier/src/Resources/views/partials/simple-js-inject.html.twig

@@ -1,3 +1,3 @@
 
-<script src="{{ asset('js/simple.2784169093f7024eaf12.js', 'Rozier') }}" defer type="text/javascript"></script>
+<script src="{{ asset('js/simple.84adcd1151ebc6fc29e9.js', 'Rozier') }}" defer type="text/javascript"></script>