Changeset 12833

Timestamp:

01/11/2021 04:32:53 PM (4 years ago)

Author:

boonebgorges

Message:

Remove calls to deprecated jQuery methods.

Props imath.
Fixes #8422.

Location:

trunk/src

Files:

• bp-activity/js/mentions.js (modified) (1 diff)
• bp-templates/bp-legacy/js/buddypress.js (modified) (14 diffs)
• bp-xprofile/admin/js/admin.js (modified) (7 diffs)

trunk/src/bp-activity/js/mentions.js

@@ -21 +21 @@
      */
     $.fn.bp_mentions = function( options ) {
-        if ( $.isArray( options ) ) {
+        if ( Array.isArray( options ) ) {
             options = { data: options };
         }

trunk/src/bp-templates/bp-legacy/js/buddypress.js

@@ -50 +50 @@
 
     /* Textarea focus */
-    $whats_new.focus( function(){
+    $whats_new.on( 'focus', function() {
         jq( '#whats-new-options' ).slideDown();
 
@@ -81 +81 @@
             }
         }
-    });
+    } );
 
     /* For the "What's New" form, do the following on focusout. */
@@ -125 +125 @@
                 } else {
                     // Checkboxes/dropdown list can have multiple selected value
-                    if ( ! jq.isArray( inputs[ input.name ] ) ) {
+                    if ( ! Array.isArray( inputs[ input.name ] ) ) {
                         inputs[ input.name ] = new Array( inputs[ input.name ], input.value );
                     } else {
@@ -276 +276 @@
 
     /* Activity filter select */
-    jq('#activity-filter-select select').change( function() {
+    jq( '#activity-filter-select select' ).on( 'change', function() {
         var selected_tab = jq( 'div.activity-type-tabs li.selected' ),
             filter = jq(this).val(),
@@ -290 +290 @@
 
         return false;
-    });
+    } );
 
     /* Stream event delegation */
@@ -801 +801 @@
 
     /* Escape Key Press for cancelling comment forms */
-    jq(document).keydown( function(e) {
+    jq(document).on( 'keydown', function(e) {
         e = e || window.event;
         if (e.target) {
@@ -899 +899 @@
 
     /* When the filter select box is changed re-query */
-    jq('li.filter select').change( function() {
+    jq( 'li.filter select' ).on( 'change', function() {
         var el,
             css_id, object, scope, filter, search_terms, template,
@@ -945 +945 @@
 
         return false;
-    });
+    } );
 
     /* All pagination links run through this function */
@@ -1165 +1165 @@
     } );
 
-    jq('#profile-edit-form input:not(:submit), #profile-edit-form textarea, #profile-edit-form select, #signup_form input:not(:submit), #signup_form textarea, #signup_form select').change( function() {
+    jq( '#profile-edit-form input:not(:submit), #profile-edit-form textarea, #profile-edit-form select, #signup_form input:not(:submit), #signup_form textarea, #signup_form select' ).on( 'change', function() {
         var shouldconfirm = true;
 
@@ -1177 +1177 @@
             }
         };
-    });
+    } );
 
     /** Friendship Requests **************************************/
@@ -1420 +1420 @@
 
     /* AJAX send reply functionality */
-    jq('#send_reply_button').click(
+    jq( '#send_reply_button' ).on( 'click',
         function() {
             var order = jq('#messages_order').val() || 'ASC',
@@ -1492 +1492 @@
 
     /* Selecting/Deselecting all messages */
-    jq('#select-all-messages').click(function(event) {
-        if( this.checked ) {
-            jq('.message-check').each(function() {
+    jq( '#select-all-messages' ).on( 'click', function() {
+        if ( this.checked ) {
+            jq( '.message-check' ).each( function() {
                 this.checked = true;
-            });
+            } );
         } else {
-            jq('.message-check').each(function() {
+            jq( '.message-check' ).each( function() {
                 this.checked = false;
-            });
-        }
-    });
+            } );
+        }
+    } );
 
     /* Make sure a 'Bulk Action' is selected before submitting the messages bulk action form */
@@ -1591 +1591 @@
 
     /* Selecting/Deselecting all notifications */
-    jq('#select-all-notifications').click(function(event) {
-        if( this.checked ) {
-            jq('.notification-check').each(function() {
+    jq( '#select-all-notifications' ).on( 'click', function() {
+        if ( this.checked ) {
+            jq( '.notification-check' ).each( function() {
                 this.checked = true;
-            });
+            } );
         } else {
-            jq('.notification-check').each(function() {
+            jq( '.notification-check' ).each( function() {
                 this.checked = false;
-            });
-        }
-    });
+            } );
+        }
+    } );
 
     /* Make sure a 'Bulk Action' is selected before submitting the form */
@@ -1635 +1635 @@
 
     /* Toolbar & wp_list_pages JavaScript IE6 hover class */
-    jq('#wp-admin-bar ul.main-nav li, #nav li').mouseover( function() {
+    jq('#wp-admin-bar ul.main-nav li, #nav li').on( 'mouseover', function() {
         jq(this).addClass('sfhover');
     });
 
-    jq('#wp-admin-bar ul.main-nav li, #nav li').mouseout( function() {
+    jq('#wp-admin-bar ul.main-nav li, #nav li').on( 'mouseout', function() {
         jq(this).removeClass('sfhover');
     });

trunk/src/bp-xprofile/admin/js/admin.js

@@ -172 +172 @@
     }
 
-    titleprompt.click(function(){
+    titleprompt.on( 'click', function(){
         jQuery(this).addClass('screen-reader-text');
         title.focus();
     });
 
-    title.blur(function(){
+    title.on( 'blur', function(){
         if ( '' === this.value ) {
             titleprompt.removeClass('screen-reader-text');
         }
-    }).focus(function(){
+    }).on( 'focus', function(){
         titleprompt.addClass('screen-reader-text');
-    }).keydown(function(e){
+    }).on( 'keydown', function(e){
         titleprompt.addClass('screen-reader-text');
         jQuery(this).unbind(e);
@@ -192 +192 @@
 
     // Set focus in Field Title, if we're on the right page.
-    jQuery( '#bp-xprofile-add-field #title' ).focus();
+    jQuery( '#bp-xprofile-add-field #title' ).trigger( 'focus' );
 
     // Set up the notice that shows when no member types are selected for a field.
@@ -215 +215 @@
 
     // Set up the sort order change actions.
-    jQuery( '[id^="sort_order_"]' ).change(function() {
+    jQuery( '[id^="sort_order_"]' ).on( 'change', function() {
         if ( jQuery( this ).val() !== 'custom' ) {
             destroySortableFieldOptions();
@@ -221 +221 @@
             enableSortableFieldOptions( jQuery('#fieldtype :selected').val() );
         }
-    });
+    } );
 
     // Show object if JS is enabled.
@@ -277 +277 @@
     var $date_format_custom_value = jQuery( '#date-format-custom-value' );
     var $date_format_sample = jQuery( '#date-format-custom-sample' );
-    $date_format.click( function( e ) {
+    $date_format.on( 'click', function( e ) {
         switch ( e.target.value ) {
             case 'elapsed' :
@@ -296 +296 @@
     // Clicking into the custom date format field should select the Custom radio button.
     var $date_format_custom = jQuery( '#date-format-custom' );
-    $date_format_custom_value.focus( function() {
+    $date_format_custom_value.on( 'focus', function() {
         $date_format_custom.prop( 'checked', 'checked' );
     } );
@@ -302 +302 @@
     // Validate custom date field.
     var $date_format_spinner = jQuery( '#date-format-custom-spinner' );
-    $date_format_custom_value.change( function( e ) {
+    $date_format_custom_value.on( 'change', function( e ) {
         $date_format_spinner.addClass( 'is-active' );
         jQuery.post( ajaxurl, {