Changeset 20645 for trunk/wp-includes/class-wp-customize.php

Timestamp:

04/30/2012 03:46:17 PM (8 years ago)

Author:

koopersmith

Message:

Theme Customizer: Migrate to an ajax-based solution for refreshing the preview and saving. see #20507, #19910.

• Use ajax-based saving, add saving indicator.
• Use ajax-based refreshing instead of form targets.
• Instead of using hidden inputs with prefixed names to track the canonical data, use the values stored in wp.customize. Encode the values as JSON before sending to avoid bugs with ids that contain square brackets (PHP mangles POST values with nested brackets).
• Use wp.customize.Previewer solely for the purpose of previewing; move the postMessage connection with the parent frame and other unrelated code snippets into the 'ready' handler.

File:

• trunk/wp-includes/class-wp-customize.php (modified) (6 diffs)

trunk/wp-includes/class-wp-customize.php

@@ -18 +18 @@
     protected $controls = array();
 
+    protected $customized;
+
+    private $_post_values;
+
     /**
      * Constructor.
@@ -31 +35 @@
         add_action( 'admin_init',   array( $this, 'admin_init' ) );
         add_action( 'wp_loaded',    array( $this, 'wp_loaded' ) );
+
+        add_action( 'wp_ajax_customize_save', array( $this, 'save' ) );
 
         add_action( 'customize_register',                 array( $this, 'register_controls' ) );
@@ -150 +156 @@
 
     /**
+     * Decode the $_POST attribute used to override the WP_Customize_Setting values.
+     *
+     * @since 3.4.0
+     */
+    public function post_value( $setting ) {
+        if ( ! isset( $this->_post_values ) ) {
+            if ( isset( $_POST['customized'] ) )
+                $this->_post_values = json_decode( stripslashes( $_POST['customized'] ), true );
+            else
+                $this->_post_values = false;
+        }
+
+        if ( isset( $this->_post_values[ $setting->id ] ) )
+            return $setting->sanitize( $this->_post_values[ $setting->id ] );
+    }
+
+
+    /**
      * Print javascript settings.
      *
@@ -268 +292 @@
      */
     public function admin_init() {
-        if ( isset( $_REQUEST['save_customize_controls'] ) )
-            $this->save();
-
         if ( ( defined( 'DOING_AJAX' ) && DOING_AJAX ) )
             return;
@@ -298 +319 @@
     public function save() {
         if ( ! $this->is_preview() )
-            return;
-
-        check_admin_referer( 'customize_controls' );
+            die;
+
+        check_ajax_referer( 'customize_controls', 'nonce' );
 
         // Do we have to switch themes?
         if ( $this->get_stylesheet() != $this->original_stylesheet ) {
             if ( ! current_user_can( 'switch_themes' ) )
-                return;
+                die;
 
             // Temporarily stop previewing the theme to allow switch_themes()
@@ -321 +342 @@
 
         add_action( 'admin_notices', array( $this, '_save_feedback' ) );
+
+        die;
     }