Merge pull request #827 from xwp/bugfix/issue-308

Luke Carbis · Luke Carbis · commit 3d5140caab87 · 2016-03-11T09:23:24.000+10:00
Logging changes of settings in Customizer
diff --git a/connectors/class-connector-settings.php b/connectors/class-connector-settings.php
@@ -541,7 +541,7 @@ function( $rule ) use ( $submenu, $record ) {
 	}
 
 	/**
-	 * Trigger this connector from WP CLI, only for known Settings
+	 * Trigger this connector from WP CLI or the Customizer, only for known Settings
 	 *
 	 * @action update_option
 	 *
@@ -550,7 +550,7 @@ function( $rule ) use ( $submenu, $record ) {
 	 * @param mixed $value
 	 */
 	public function callback_update_option( $option, $value, $old_value ) {
-		if ( defined( '\WP_CLI' ) && \WP_CLI && array_key_exists( $option, $this->labels ) ) {
+		if ( ( defined( '\WP_CLI' ) && \WP_CLI || did_action( 'customize_save' ) ) && array_key_exists( $option, $this->labels ) ) {
 			$this->callback_updated_option( $option, $value, $old_value );
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -541,7 +541,7 @@ function( $rule ) use ( $submenu, $record ) {`
`541`	`541`	`}`
`542`	`542`
`543`	`543`	`/**`
`544`		`- * Trigger this connector from WP CLI, only for known Settings`
	`544`	`+ * Trigger this connector from WP CLI or the Customizer, only for known Settings`
`545`	`545`	`*`
`546`	`546`	`* @action update_option`
`547`	`547`	`*`
`@@ -550,7 +550,7 @@ function( $rule ) use ( $submenu, $record ) {`
`550`	`550`	`* @param mixed $value`
`551`	`551`	`*/`
`552`	`552`	`public function callback_update_option( $option, $value, $old_value ) {`
`553`		`- if ( defined( '\WP_CLI' ) && \WP_CLI && array_key_exists( $option, $this->labels ) ) {`
	`553`	`+ if ( ( defined( '\WP_CLI' ) && \WP_CLI \|\| did_action( 'customize_save' ) ) && array_key_exists( $option, $this->labels ) ) {`
`554`	`554`	`$this->callback_updated_option( $option, $value, $old_value );`
`555`	`555`	`}`
`556`	`556`	`}`