Add website fields to project meta box
Adds a 'This project is a website' toggle to the Repository meta box. When checked, site fields (URL, platform, optional screenshot) replace the git-repo fields; when unchecked, repo fields are shown and site fields are cleared. Existing posts default to repo mode. - admin/metabox.php: checkbox, site URL/platform/screenshot inputs with media-library picker, JS toggle, save handler that switches modes - includes/helper-functions.php: projects_portfolio_is_website / get _site_url / get_site_platform / get_site_screenshot_url helpers - templates/single-projects.php: website-mode rendering (Visit Site button, platform row, screenshot) - tests/test-metabox.php: website-mode persistence + mode-switch tests - tests/wp-stubs.php: sanitize_text_field, absint stubs
This commit is contained in:
@@ -142,4 +142,115 @@ class Metabox_Test extends \PHPUnit\Framework\TestCase {
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_github_url', $existing );
|
||||
$this->assertContains( '_projects_portfolio_github_url', $deleted );
|
||||
}
|
||||
|
||||
public function test_website_mode_persists_website_fields_and_clears_repo_fields(): void {
|
||||
$existing = [
|
||||
'_projects_portfolio_provider' => 'github',
|
||||
'_projects_portfolio_repo_url' => 'https://github.com/owner/repo',
|
||||
'_projects_portfolio_github_url' => 'https://github.com/owner/repo',
|
||||
];
|
||||
$deleted = [];
|
||||
\Brain\Monkey\Functions\stubs( [
|
||||
'get_post_meta' => function ( $post_id, $key, $single = false ) use ( &$existing ) {
|
||||
return $existing[ $key ] ?? '';
|
||||
},
|
||||
'update_post_meta' => function ( $post_id, $key, $value ) use ( &$existing ) {
|
||||
$existing[ $key ] = $value;
|
||||
return true;
|
||||
},
|
||||
'delete_post_meta' => function ( $post_id, $key ) use ( &$existing, &$deleted ) {
|
||||
unset( $existing[ $key ] );
|
||||
$deleted[] = $key;
|
||||
return true;
|
||||
},
|
||||
'wp_verify_nonce' => function () { return true; },
|
||||
'wp_nonce_field' => function () { /* noop */ },
|
||||
] );
|
||||
|
||||
$_POST['projects_portfolio_meta_box_nonce'] = 'nonce';
|
||||
$_POST['projects_portfolio_is_website'] = '1';
|
||||
$_POST['projects_portfolio_site_url'] = 'https://example.com';
|
||||
$_POST['projects_portfolio_site_platform'] = 'WordPress';
|
||||
$_POST['projects_portfolio_site_screenshot_id'] = '42';
|
||||
|
||||
projects_portfolio_save_meta_box( 11 );
|
||||
|
||||
$this->assertSame( '1', $existing['_projects_portfolio_is_website'] );
|
||||
$this->assertSame( 'https://example.com', $existing['_projects_portfolio_site_url'] );
|
||||
$this->assertSame( 'WordPress', $existing['_projects_portfolio_site_platform'] );
|
||||
$this->assertSame( 42, $existing['_projects_portfolio_site_screenshot_id'] );
|
||||
|
||||
// Repo-mode fields should be cleared when switching to website mode.
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_provider', $existing );
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_repo_url', $existing );
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_github_url', $existing );
|
||||
$this->assertContains( '_projects_portfolio_provider', $deleted );
|
||||
$this->assertContains( '_projects_portfolio_repo_url', $deleted );
|
||||
$this->assertContains( '_projects_portfolio_github_url', $deleted );
|
||||
}
|
||||
|
||||
public function test_unchecked_website_checkbox_clears_website_fields(): void {
|
||||
$existing = [
|
||||
'_projects_portfolio_is_website' => '1',
|
||||
'_projects_portfolio_site_url' => 'https://example.com',
|
||||
'_projects_portfolio_site_platform' => 'WordPress',
|
||||
'_projects_portfolio_site_screenshot_id' => 42,
|
||||
];
|
||||
$deleted = [];
|
||||
\Brain\Monkey\Functions\stubs( [
|
||||
'get_post_meta' => function ( $post_id, $key, $single = false ) use ( &$existing ) {
|
||||
return $existing[ $key ] ?? '';
|
||||
},
|
||||
'update_post_meta' => function ( $post_id, $key, $value ) use ( &$existing ) {
|
||||
$existing[ $key ] = $value;
|
||||
return true;
|
||||
},
|
||||
'delete_post_meta' => function ( $post_id, $key ) use ( &$existing, &$deleted ) {
|
||||
unset( $existing[ $key ] );
|
||||
$deleted[] = $key;
|
||||
return true;
|
||||
},
|
||||
'wp_verify_nonce' => function () { return true; },
|
||||
'wp_nonce_field' => function () { /* noop */ },
|
||||
] );
|
||||
|
||||
$_POST['projects_portfolio_meta_box_nonce'] = 'nonce';
|
||||
// is_website is absent — repo mode is implied.
|
||||
|
||||
projects_portfolio_save_meta_box( 13 );
|
||||
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_is_website', $existing );
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_site_url', $existing );
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_site_platform', $existing );
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_site_screenshot_id', $existing );
|
||||
$this->assertContains( '_projects_portfolio_is_website', $deleted );
|
||||
}
|
||||
|
||||
public function test_existing_repo_post_without_checkbox_keeps_repo_fields(): void {
|
||||
$existing = [
|
||||
'_projects_portfolio_provider' => 'github',
|
||||
'_projects_portfolio_repo_url' => 'https://github.com/owner/repo',
|
||||
];
|
||||
\Brain\Monkey\Functions\stubs( [
|
||||
'get_post_meta' => function ( $post_id, $key, $single = false ) use ( &$existing ) {
|
||||
return $existing[ $key ] ?? '';
|
||||
},
|
||||
'update_post_meta' => function ( $post_id, $key, $value ) use ( &$existing ) {
|
||||
$existing[ $key ] = $value;
|
||||
return true;
|
||||
},
|
||||
'delete_post_meta' => function () { return true; },
|
||||
'wp_verify_nonce' => function () { return true; },
|
||||
'wp_nonce_field' => function () { /* noop */ },
|
||||
] );
|
||||
|
||||
$_POST['projects_portfolio_meta_box_nonce'] = 'nonce';
|
||||
// is_website not present in $_POST — should be treated as unchecked.
|
||||
|
||||
projects_portfolio_save_meta_box( 17 );
|
||||
|
||||
$this->assertSame( 'github', $existing['_projects_portfolio_provider'] );
|
||||
$this->assertSame( 'https://github.com/owner/repo', $existing['_projects_portfolio_repo_url'] );
|
||||
$this->assertArrayNotHasKey( '_projects_portfolio_is_website', $existing );
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user