Ticket #17884: 17884.4.diff

tests/phpunit/tests/basic.php

@@ -88 +88 @@
 EOF;
                 $this->assertEquals($expected, mask_input_value($in));
         }
+
+        function test_setting_nonexistent_arrays() {
+                $page = 1;
+                $field = 'settings';
+
+                error_reporting( -1 );
+
+                $empty_array[$page][$field] = 'foo';
+
+                unset( $empty_array[$page]['bar']['baz'] );
+        }
 }

src/wp-admin/includes/template.php

@@ -1072 +1072 @@
                 $page = 'reading';
         }
 
-        if ( !isset($wp_settings_sections) )
-                $wp_settings_sections = array();
-        if ( !isset($wp_settings_sections[$page]) )
-                $wp_settings_sections[$page] = array();
-        if ( !isset($wp_settings_sections[$page][$id]) )
-                $wp_settings_sections[$page][$id] = array();
-
         $wp_settings_sections[$page][$id] = array('id' => $id, 'title' => $title, 'callback' => $callback);
 }
 
@@ -1117 +1110 @@
                 $page = 'reading';
         }
 
-        if ( !isset($wp_settings_fields) )
-                $wp_settings_fields = array();
-        if ( !isset($wp_settings_fields[$page]) )
-                $wp_settings_fields[$page] = array();
-        if ( !isset($wp_settings_fields[$page][$section]) )
-                $wp_settings_fields[$page][$section] = array();
-
         $wp_settings_fields[$page][$section][$id] = array('id' => $id, 'title' => $title, 'callback' => $callback, 'args' => $args);
 }
 
@@ -1143 +1129 @@
 function do_settings_sections( $page ) {
         global $wp_settings_sections, $wp_settings_fields;
 
-        if ( ! isset( $wp_settings_sections ) || !isset( $wp_settings_sections[$page] ) )
+        if ( ! isset( $wp_settings_sections[$page] ) )
                 return;
 
         foreach ( (array) $wp_settings_sections[$page] as $section ) {
@@ -1178 +1164 @@
 function do_settings_fields($page, $section) {
         global $wp_settings_fields;
 
-        if ( !isset($wp_settings_fields) || !isset($wp_settings_fields[$page]) || !isset($wp_settings_fields[$page][$section]) )
+        if ( ! isset( $wp_settings_fields[$page][$section] ) )
                 return;
 
         foreach ( (array) $wp_settings_fields[$page][$section] as $field ) {
@@ -1219 +1205 @@
 function add_settings_error( $setting, $code, $message, $type = 'error' ) {
         global $wp_settings_errors;
 
-        if ( !isset($wp_settings_errors) )
-                $wp_settings_errors = array();
-
         $new_error = array(
                 'setting' => $setting,
                 'code' => $code,