MC-41897: Fix jQuery.fn.bind

eliseacornejo · eliseacornejo · commit ffc62d9954f3 · 2021-05-06T14:37:49.000+02:00
- Change deprecated functions bind and unbind
diff --git a/app/code/Magento/CatalogInventory/Block/Adminhtml/Form/Field/Stock.php b/app/code/Magento/CatalogInventory/Block/Adminhtml/Form/Field/Stock.php
@@ -256,8 +256,8 @@ protected function _getJs($quantityFieldId, $inStockFieldId)
                         };
                         $.each(fieldsAssociations, function(generalTabField, advancedTabField) {
                             $('#' + generalTabField + ', #' + advancedTabField)
-                                .bind('focus blur change keyup click', filler)
-                                .bind('keyup change blur', disabler)
+                                .on('focus blur change keyup click', filler)
+                                .on('keyup change blur', disabler)
                                 .trigger('change');
                         });
 
diff --git a/app/code/Magento/Theme/view/adminhtml/templates/tabs/css.phtml b/app/code/Magento/Theme/view/adminhtml/templates/tabs/css.phtml
@@ -34,7 +34,7 @@ require([
         add: function (e, data) {
             var uploadButton = $('#css_uploader_button');
             /** Unbind click event on file change */
-            uploadButton.unbind('click');
+            uploadButton.off('click');
             uploadButton.removeAttr('disabled');
 
             uploadButton.click(function () {
diff --git a/app/code/Magento/Theme/view/adminhtml/templates/tabs/js.phtml b/app/code/Magento/Theme/view/adminhtml/templates/tabs/js.phtml
@@ -124,7 +124,7 @@ require([
     $('#js_files_uploader').click(function () {
         /** Unbind click event on file change */
         $('#js-file-uploader').html('');
-        $('#js_uploader_button').unbind('click');
+        $('#js_uploader_button').off('click');
     });
 
 
diff --git a/app/code/Magento/Ui/view/base/web/js/dynamic-rows/dnd.js b/app/code/Magento/Ui/view/base/web/js/dynamic-rows/dnd.js
@@ -109,7 +109,7 @@ define([
          * @param {Object} data - element data
          */
         initListeners: function (elem, data) {
-            $(elem).on('mousedown touchstart', this.mousedownHandler.bind(this, data, elem));
+            $(elem).on('mousedown touchstart', this.mousedownHandler.on(this, data, elem));
         },
 
         /**
@@ -138,8 +138,8 @@ define([
                 $table.offset().top - originRecord.offset().top + $table.children('thead').outerHeight();
             drEl.maxYpos = drEl.minYpos + $table.children('tbody').outerHeight() - originRecord.outerHeight();
             $tableWrapper.append(recordNode);
-            this.body.bind('mousemove touchmove', this.mousemoveHandler);
-            this.body.bind('mouseup touchend', this.mouseupHandler);
+            this.body.on('mousemove touchmove', this.mousemoveHandler);
+            this.body.on('mouseup touchend', this.mouseupHandler);
         },
 
         /**
@@ -199,8 +199,8 @@ define([
 
             drEl.originRow.removeClass(this.draggableElementClass);
 
-            this.body.unbind('mousemove touchmove', this.mousemoveHandler);
-            this.body.unbind('mouseup touchend', this.mouseupHandler);
+            this.body.off('mousemove touchmove', this.mousemoveHandler);
+            this.body.off('mouseup touchend', this.mouseupHandler);
 
             this.draggableElement = {};
         },
diff --git a/app/code/Magento/Ui/view/base/web/js/modal/alert.js b/app/code/Magento/Ui/view/base/web/js/modal/alert.js
@@ -44,7 +44,7 @@ define([
          */
         closeModal: function () {
             this.options.actions.always();
-            this.element.bind('alertclosed', _.bind(this._remove, this));
+            this.element.on('alertclosed', _.bind(this._remove, this));
 
             return this._super();
         }
diff --git a/app/code/Magento/Ui/view/base/web/js/modal/confirm.js b/app/code/Magento/Ui/view/base/web/js/modal/confirm.js
@@ -95,7 +95,7 @@ define([
                 this.options.actions.cancel(event);
             }
             this.options.actions.always(event);
-            this.element.bind('confirmclosed', _.bind(this._remove, this));
+            this.element.on('confirmclosed', _.bind(this._remove, this));
 
             return this._super();
         }
diff --git a/app/code/Magento/Ui/view/base/web/js/modal/modal.js b/app/code/Magento/Ui/view/base/web/js/modal/modal.js
@@ -276,18 +276,18 @@ define([
          * Set events listener when modal is opened.
          */
         _setKeyListener: function () {
-            this.modal.find(this.options.focusableStart).bind('focusin', this._tabSwitcher);
-            this.modal.find(this.options.focusableEnd).bind('focusin', this._tabSwitcher);
-            this.modal.bind('keydown', this.keyEventSwitcher);
+            this.modal.find(this.options.focusableStart).on('focusin', this._tabSwitcher);
+            this.modal.find(this.options.focusableEnd).on('focusin', this._tabSwitcher);
+            this.modal.on('keydown', this.keyEventSwitcher);
         },
 
         /**
          * Remove events listener when modal is closed.
          */
         _removeKeyListener: function () {
-            this.modal.find(this.options.focusableStart).unbind('focusin', this._tabSwitcher);
-            this.modal.find(this.options.focusableEnd).unbind('focusin', this._tabSwitcher);
-            this.modal.unbind('keydown', this.keyEventSwitcher);
+            this.modal.find(this.options.focusableStart).off('focusin', this._tabSwitcher);
+            this.modal.find(this.options.focusableEnd).off('focusin', this._tabSwitcher);
+            this.modal.off('keydown', this.keyEventSwitcher);
         },
 
         /**
@@ -441,15 +441,15 @@ define([
             }
             events = $._data(this.overlay.get(0), 'events');
             events ? this.prevOverlayHandler = events.click[0].handler : false;
-            this.options.clickableOverlay ? this.overlay.unbind().on('click', outerClickHandler) : false;
+            this.options.clickableOverlay ? this.overlay.off().on('click', outerClickHandler) : false;
         },
 
         /**
          * Destroy overlay.
          */
         _destroyOverlay: function () {
             if (this._getVisibleCount()) {
-                this.overlay.unbind().on('click', this.prevOverlayHandler);
+                this.overlay.off().on('click', this.prevOverlayHandler);
             } else {
                 $(this.options.appendTo).removeClass(this.options.parentModalClass);
                 this.overlay.remove();
diff --git a/app/code/Magento/Ui/view/base/web/js/modal/prompt.js b/app/code/Magento/Ui/view/base/web/js/modal/prompt.js
@@ -206,7 +206,7 @@ define([
             }
 
             this.options.actions.always();
-            this.element.bind('promptclosed', _.bind(this._remove, this));
+            this.element.on('promptclosed', _.bind(this._remove, this));
 
             return this._super();
         }
diff --git a/lib/web/mage/backend/menu.js b/lib/web/mage/backend/menu.js
@@ -49,7 +49,7 @@ define([
                 })
                 // need to catch all clicks on disabled menu
                 // not possible through _on
-                .bind('click' + this.eventNamespace, $.proxy(function (event) {
+                .on('click' + this.eventNamespace, $.proxy(function (event) {
                     if (this.options.disabled) {
                         event.preventDefault();
                     }
diff --git a/lib/web/mage/gallery/gallery.js b/lib/web/mage/gallery/gallery.js
@@ -192,8 +192,8 @@ define([
                 settings.closeIcon.show();
                 settings.focusableStart.attr('tabindex', '0');
                 settings.focusableEnd.attr('tabindex', '0');
-                settings.focusableStart.bind('focusin', self._focusSwitcher);
-                settings.focusableEnd.bind('focusin', self._focusSwitcher);
+                settings.focusableStart.on('focusin', self._focusSwitcher);
+                settings.focusableEnd.on('focusin', self._focusSwitcher);
                 settings.api.updateOptions(settings.defaultConfig.options, true);
                 settings.api.updateOptions(settings.fullscreenConfig, true);
 
@@ -208,8 +208,8 @@ define([
                 settings.focusableStart.attr('tabindex', '-1');
                 settings.focusableEnd.attr('tabindex', '-1');
                 settings.api.updateOptions(settings.defaultConfig.options, true);
-                settings.focusableStart.unbind('focusin', this._focusSwitcher);
-                settings.focusableEnd.unbind('focusin', this._focusSwitcher);
+                settings.focusableStart.off('focusin', this._focusSwitcher);
+                settings.focusableEnd.off('focusin', this._focusSwitcher);
                 settings.closeIcon.hide();
 
                 if (!_.isEqual(settings.activeBreakpoint, {}) && settings.breakpoints) {
diff --git a/lib/web/mage/list.js b/lib/web/mage/list.js
@@ -42,7 +42,7 @@ define([
             this.element
                 .addClass('list-widget');
 
-            addButton.bind('click', $.proxy(this.handleAdd, this));
+            addButton.on('click', $.proxy(this.handleAdd, this));
 
             //handle remove
             destination.on('click', this.options.removeButton, $.proxy(this.removeItem, this));
diff --git a/lib/web/magnifier/magnify.js b/lib/web/magnifier/magnify.js
@@ -652,7 +652,7 @@ define([
                     }
                 });
             } else {
-                $image.unbind('dblclick');
+                $image.off('dblclick');
                 $image.dblclick(dblClickHandler);
             }
 
@@ -746,7 +746,7 @@ define([
             });
 
             if (keyboardNavigation) {
-                $(document).unbind('keydown', keyboardNavigation);
+                $(document).off('keydown', keyboardNavigation);
             }
 
             /**

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ define([`
`44`	`44`	`*/`
`45`	`45`	`closeModal: function () {`
`46`	`46`	`this.options.actions.always();`
`47`		`- this.element.bind('alertclosed', _.bind(this._remove, this));`
	`47`	`+ this.element.on('alertclosed', _.bind(this._remove, this));`
`48`	`48`
`49`	`49`	`return this._super();`
`50`	`50`	`}`
Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ define([`
`95`	`95`	`this.options.actions.cancel(event);`
`96`	`96`	`}`
`97`	`97`	`this.options.actions.always(event);`
`98`		`- this.element.bind('confirmclosed', _.bind(this._remove, this));`
	`98`	`+ this.element.on('confirmclosed', _.bind(this._remove, this));`
`99`	`99`
`100`	`100`	`return this._super();`
`101`	`101`	`}`
Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,7 @@ define([`
`206`	`206`	`}`
`207`	`207`
`208`	`208`	`this.options.actions.always();`
`209`		`- this.element.bind('promptclosed', _.bind(this._remove, this));`
	`209`	`+ this.element.on('promptclosed', _.bind(this._remove, this));`
`210`	`210`
`211`	`211`	`return this._super();`
`212`	`212`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ define([`
`49`	`49`	`})`
`50`	`50`	`// need to catch all clicks on disabled menu`
`51`	`51`	`// not possible through _on`
`52`		`- .bind('click' + this.eventNamespace, $.proxy(function (event) {`
	`52`	`+ .on('click' + this.eventNamespace, $.proxy(function (event) {`
`53`	`53`	`if (this.options.disabled) {`
`54`	`54`	`event.preventDefault();`
`55`	`55`	`}`