Make sure publish_customize_snapshots cap is checked

westonruter · westonruter · commit f5afffaa22c6 · 2016-08-22T15:29:12.000-07:00
diff --git a/php/class-post-type.php b/php/class-post-type.php
@@ -588,7 +588,6 @@ public function filter_user_has_cap( $allcaps, $caps ) {
 			if ( ! current_user_can( 'customize_publish' ) || empty( $allcaps['customize_publish'] ) ) {
 				$allcaps[ $post_type_obj->cap->publish_posts ] = false;
 			}
-
 			if ( ! current_user_can( 'edit_others_posts' ) ) {
 				$allcaps[ $post_type_obj->cap->edit_others_posts ] = false;
 			}
diff --git a/tests/php/test-class-ajax-customize-snapshot-manager.php b/tests/php/test-class-ajax-customize-snapshot-manager.php
@@ -430,6 +430,7 @@ function test_ajax_update_snapshot_schedule() {
 		$setting_key = 'anyonecanedit';
 		$tomorrow = date( 'Y-m-d H:i:s', time() + 86400 );
 		$this->set_current_user( 'administrator' );
+		$this->assertTrue( current_user_can( 'publish_customize_snapshots' ) );
 		$this->set_input_vars( array(
 			'action' => Customize_Snapshot_Manager::AJAX_ACTION,
 			'nonce' => wp_create_nonce( Customize_Snapshot_Manager::AJAX_ACTION ),
@@ -483,12 +484,12 @@ function test_ajax_update_snapshot_ok_for_draft_and_pending_but_not_future() {
 		}, 10, 3 );
 		$tomorrow = date( 'Y-m-d H:i:s', time() + 86400 );
 		$this->set_current_user( 'contributor' );
+		$this->assertFalse( current_user_can( 'publish_customize_snapshots' ) );
 		$post_vars = array(
 			'action' => Customize_Snapshot_Manager::AJAX_ACTION,
 			'nonce' => wp_create_nonce( Customize_Snapshot_Manager::AJAX_ACTION ),
 			'customize_snapshot_uuid' => self::UUID,
 			'customized' => wp_json_encode( array( $setting_key => 'Hello' ) ),
-			'status' => 'draft',
 			'publish_date' => $tomorrow, // Tomorrow.
 		);
 
@@ -497,7 +498,7 @@ function test_ajax_update_snapshot_ok_for_draft_and_pending_but_not_future() {
 		$this->add_setting();
 
 		// Draft pass.
-		$post_vars['status'] = 'pending';
+		$post_vars['status'] = 'draft';
 		$this->set_input_vars( $post_vars );
 		$this->make_ajax_call( Customize_Snapshot_Manager::AJAX_ACTION );
 		$response = json_decode( $this->_last_response, true );

Original file line number	Diff line number	Diff line change
`@@ -588,7 +588,6 @@ public function filter_user_has_cap( $allcaps, $caps ) {`
`588`	`588`	`if ( ! current_user_can( 'customize_publish' ) \|\| empty( $allcaps['customize_publish'] ) ) {`
`589`	`589`	`$allcaps[ $post_type_obj->cap->publish_posts ] = false;`
`590`	`590`	`}`
`591`		`-`
`592`	`591`	`if ( ! current_user_can( 'edit_others_posts' ) ) {`
`593`	`592`	`$allcaps[ $post_type_obj->cap->edit_others_posts ] = false;`
`594`	`593`	`}`