Changeset 16606 for trunk/wp-admin/network/site-themes.php

Timestamp:

11/29/2010 07:24:54 AM (14 years ago)

Author:

PeteMall

Message:

Update and error messages for site-themes. See #14897

File:

• trunk/wp-admin/network/site-themes.php (modified) (5 diffs)

trunk/wp-admin/network/site-themes.php

@@ -44 +44 @@
         case 'enable':
             $theme = $_GET['theme'];
+            $update = 'enabled';
             if ( !$allowed_themes )
                 $allowed_themes = array( $theme => true );
@@ -51 +52 @@
         case 'disable':
             $theme = $_GET['theme'];
+            $update = 'disabled';
             if ( !$allowed_themes )
                 $allowed_themes = array();
@@ -57 +59 @@
             break;
         case 'enable-selected':
-            $themes = isset( $_POST['checked'] ) ? (array) $_POST['checked'] : array();
-            if ( empty($themes) ) {
-                restore_current_blog();
-                wp_redirect( wp_get_referer() );
-                exit;
-            }                       
-            foreach( (array) $themes as $theme )
-                $allowed_themes[ $theme ] = true;
+            if ( isset( $_POST['checked'] ) ) {
+                $update = 'enable';
+                $themes = (array) $_POST['checked'];
+                foreach( (array) $themes as $theme )
+                    $allowed_themes[ $theme ] = true;
+            } else {
+                $update = 'error';
+            }
             break;
         case 'disable-selected':
-            $themes = isset( $_POST['checked'] ) ? (array) $_POST['checked'] : array();
-            if ( empty($themes) ) {
-                restore_current_blog();
-                wp_redirect( wp_get_referer() );
-                exit;
-            }                       
-            foreach( (array) $themes as $theme )
-                unset( $allowed_themes[ $theme ] );
+            if ( isset( $_POST['checked'] ) ) {
+                $update = 'disable';
+                $themes = (array) $_POST['checked'];
+                foreach( (array) $themes as $theme )
+                    unset( $allowed_themes[ $theme ] );
+            } else {
+                $update = 'error';
+            }
             break;
     }
@@ -81 +83 @@
     restore_current_blog();
     
-    wp_redirect( wp_get_referer() ); // @todo add_query_arg for update message
+    wp_redirect( add_query_arg( 'update', $update, wp_get_referer() ) );
     exit;   
 }
@@ -111 +113 @@
 }
 ?>
-</h3>
+</h3><?php
+
+if ( isset( $_GET['update'] ) ) {
+    switch ( $_GET['update'] ) {
+    case 'enabled':
+        echo '<div id="message" class="updated"><p>' . __( 'Theme enabled.' ) . '</p></div>';
+        break;
+    case 'disabled':
+        echo '<div id="message" class="updated"><p>' . __( 'Theme disabled.' ) . '</p></div>';
+        break;
+    case 'error':
+        echo '<div id="message" class="error"><p>' . __( 'No theme selected.' ) . '</p></div>';
+        break;
+    }
+} ?>
+
 <p><?php _e( 'Network enabled themes are not shown on this screen.' ) ?></p>