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

Timestamp:

11/29/2010 07:24:54 AM (16 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>