Changeset 19579 for trunk/wp-admin/network/themes.php

Timestamp:

12/10/2011 06:26:48 PM (15 years ago)

Author:

ryan

Message:

Best practice, use wp_safe_redirect() when dealing with referrers. Props nacin.

File:

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

trunk/wp-admin/network/themes.php

@@ -48 +48 @@
                         unset( $allowed_themes[ $_GET['theme'] ] );
                         update_site_option( 'allowedthemes', $allowed_themes );
-                        wp_redirect( add_query_arg( 'disabled', '1', $referer ) );
+                        wp_safe_redirect( add_query_arg( 'disabled', '1', $referer ) );
                         exit;
                         break;
@@ -55 +55 @@
                         $themes = isset( $_POST['checked'] ) ? (array) $_POST['checked'] : array();
                         if ( empty($themes) ) {
-                                wp_redirect( add_query_arg( 'error', 'none', $referer ) );
+                                wp_safe_redirect( add_query_arg( 'error', 'none', $referer ) );
                                 exit;
                         }
@@ -61 +61 @@
                                 $allowed_themes[ $theme ] = true;
                         update_site_option( 'allowedthemes', $allowed_themes );
-                        wp_redirect( add_query_arg( 'enabled', count( $themes ), $referer ) );
+                        wp_safe_redirect( add_query_arg( 'enabled', count( $themes ), $referer ) );
                         exit;
                         break;
@@ -68 +68 @@
                         $themes = isset( $_POST['checked'] ) ? (array) $_POST['checked'] : array();
                         if ( empty($themes) ) {
-                                wp_redirect( add_query_arg( 'error', 'none', $referer ) );
+                                wp_safe_redirect( add_query_arg( 'error', 'none', $referer ) );
                                 exit;
                         }
@@ -74 +74 @@
                                 unset( $allowed_themes[ $theme ] );
                         update_site_option( 'allowedthemes', $allowed_themes );
-                        wp_redirect( add_query_arg( 'disabled', count( $themes ), $referer ) );
+                        wp_safe_redirect( add_query_arg( 'disabled', count( $themes ), $referer ) );
                         exit;
                         break;
@@ -118 +118 @@
 
                         if ( empty( $themes ) ) {
-                                wp_redirect( add_query_arg( 'error', 'none', $referer ) );
+                                wp_safe_redirect( add_query_arg( 'error', 'none', $referer ) );
                                 exit;
                         }
@@ -135 +135 @@
 
                         if ( empty( $themes ) ) {
-                                wp_redirect( add_query_arg( 'error', 'main', $referer ) );
+                                wp_safe_redirect( add_query_arg( 'error', 'main', $referer ) );
                                 exit;
                         }