Changeset 38175

Timestamp:

07/31/2016 06:56:57 PM (8 years ago)

Author:

jorbin

Message:

Upgrade/Install: Trigger additional JS events in shiny updates

Events for updating exist, but they lack context. This adds args so that plugins can detec t what plugin/theme is being installed. Additionally, events for bulk actions, deleting and that and install is starting didn't exist, so this adds them.

Fixes #37512.
Props DavidAnderson, and ocean90, swissspidy for review.

File:

• trunk/src/wp-admin/js/updates.js (modified) (7 diffs)

trunk/src/wp-admin/js/updates.js

@@ -370 +370 @@
             .text( wp.updates.l10n.updating );
 
-        $document.trigger( 'wp-plugin-updating' );
+        $document.trigger( 'wp-plugin-updating', args );
 
         return wp.updates.ajax( 'update-plugin', args );
@@ -526 +526 @@
         $card.removeClass( 'plugin-card-install-failed' ).find( '.notice.notice-error' ).remove();
 
+        $document.trigger( 'wp-plugin-installing', args );
+        
         return wp.updates.ajax( 'install-plugin', args );
     };
@@ -716 +718 @@
         wp.a11y.speak( wp.updates.l10n.deleting, 'polite' );
 
+        $document.trigger( 'wp-plugin-deleting', args );
+        
         return wp.updates.ajax( 'delete-plugin', args );
     };
@@ -906 +910 @@
         $notice.text( wp.updates.l10n.updating );
 
-        $document.trigger( 'wp-theme-updating' );
+        $document.trigger( 'wp-theme-updating', args );
 
         return wp.updates.ajax( 'update-theme', args );
@@ -1039 +1043 @@
         $( '.install-theme-info, [data-slug="' + args.slug + '"]' ).removeClass( 'theme-install-failed' ).find( '.notice.notice-error' ).remove();
 
+        $document.trigger( 'wp-theme-installing', args );
+        
         return wp.updates.ajax( 'install-theme', args );
     };
@@ -1168 +1174 @@
         $( '.theme-info .update-message' ).remove();
 
+        $document.trigger( 'wp-theme-deleting', args );
+        
         return wp.updates.ajax( 'delete-theme', args );
     };
@@ -1974 +1982 @@
             $bulkActionForm.find( '.manage-column [type="checkbox"]' ).prop( 'checked', false );
 
+            $document.trigger( 'wp-' + type + '-bulk-' + bulkAction, itemsSelected );
+            
             // Find all the checkboxes which have been checked.
             itemsSelected.each( function( index, element ) {