Changeset 3105623

Timestamp:

06/21/2024 10:59:49 AM (7 weeks ago)

Author:

whiteshadow

Message:

Minor: Remove some unused params and variables

File:

• admin-menu-editor/trunk/js/menu-editor.js (modified) (5 diffs)

admin-menu-editor/trunk/js/menu-editor.js

@@ -411 +411 @@
         this.usesSubmenuContainers = (this.level > 1);
 
-        let isNewContainer = false;
         if ((typeof container === 'undefined') || (container === null)) {
-            isNewContainer = true;
             container = $('#ame-submenu-column-template').first().clone();
             container.attr('id', '');
@@ -4538 +4536 @@
     menuEditorNode.on(
         'adminMenuEditor:action-toggle-all',
-        /**
-         * @param event
-         */
-        function(event) {
+        function() {
             if ( actorSelectorWidget.selectedActor === null ) {
                 alert("This button enables/disables all menus for the selected role. To use it, click a role and then click this button again.");
@@ -4573 +4568 @@
     menuEditorNode.on(
         'adminMenuEditor:action-copy-permissions',
-        /**
-         * @param event
-         * @param {JQuery|null} selectedItem
-         * @param {AmeEditorColumn} column
-         */
-        function(event, selectedItem, column) {
+        function() {
             const previousSource = sourceActorList.val();
 
@@ -4750 +4740 @@
     menuEditorNode.on(
         'adminMenuEditor:action-toggle-toolbar',
-        /**
-         * @param event
-         */
-        function(event) {
+        function() {
             let visible = menuEditorNode.find('.ws_second_toolbar_row').toggle().is(':visible');
             if (typeof $['cookie'] !== 'undefined') {
@@ -5663 +5650 @@
 
         var selectedOption = testMenuItemList.find('option:selected').first(),
-            selectedMenu = selectedOption.data('menu_item'),
-            menuUrl = selectedOption.val();
+            selectedMenu = selectedOption.data('menu_item');
 
         $.ajax(