Skip to content

Add commands for managing signups on multisite #489

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
swissspidy merged 17 commits into from
May 4, 2024
Merged

Add commands for managing signups on multisite #489

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
f852acf
Add signup command class
ernilambar Mar 15, 2024
660fff5
Update command docs
ernilambar Mar 18, 2024
986adb9
Update success and error messages
ernilambar Mar 19, 2024
5aee1c4
Update readme based on new success messages
ernilambar Mar 19, 2024
e3bfbbc
Update delete method to user callback function
ernilambar Mar 19, 2024
75b5b55
Accept multiple signups in activate command
ernilambar Mar 19, 2024
d5592a6
Update arguments description
ernilambar Mar 19, 2024
a02a3e6
Move signup under user command
ernilambar Apr 8, 2024
32b75d4
Remove Signup fetcher class
ernilambar Apr 9, 2024
ef4f8dc
Update wp-cli version
ernilambar Apr 26, 2024
778ca3e
Revert cli version
ernilambar Apr 26, 2024
adde1d7
Use fetcher from main library
ernilambar Apr 26, 2024
b266645
Use batch results helper for signup activate
ernilambar Apr 26, 2024
0d2e74a
Implement --all in signup delete command
ernilambar Apr 26, 2024
d0d5a84
Use log method instead of success
ernilambar Apr 27, 2024
5fd3657
Fix indenting issue in feature test file
ernilambar Apr 29, 2024
889336b
Introduce --per_page argument in signup list command
ernilambar Apr 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update delete method to user callback function
  • Loading branch information
@ernilambar
ernilambar committed Apr 26, 2024
commit e3bfbbc79346028633b5ff160382636a384b416c
27 changes: 20 additions & 7 deletions src/Signup_Command.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -228,12 +228,12 @@ public function activate( $args, $assoc_args ) {
}

/**
* Deletes a signup.
* Deletes one or more signups.
*
* ## OPTIONS
*
* <signup>
* : Signup ID, user login, user email, or activation key.
* <signup>...
* : Signup ID, user login, user email, or activation key of the signup(s) to delete.
Comment thread
ernilambar marked this conversation as resolved.
Outdated
*
* ## EXAMPLES
*
Expand All @@ -244,16 +244,29 @@ public function activate( $args, $assoc_args ) {
* @package wp-cli
*/
public function delete( $args, $assoc_args ) {
Comment thread
ernilambar marked this conversation as resolved.
$signups = $this->fetcher->get_many( $args );

parent::_delete( $signups, $assoc_args, [ $this, 'delete_callback' ] );
}

/**
* Callback used to delete a signup.
*
* @param $signup
* @param $assoc_args
* @return array
*/
protected function delete_callback( $signup, $assoc_args ) {
global $wpdb;

$signup = $this->fetcher->get_check( $args[0] );
$signup_id = $signup->signup_id;

$result = $wpdb->delete( $wpdb->signups, array( 'signup_id' => $signup->signup_id ), array( '%d' ) );
$result = $wpdb->delete( $wpdb->signups, array( 'signup_id' => $signup_id ), array( '%d' ) );
Comment thread
danielbachhuber marked this conversation as resolved.

if ( $result ) {
WP_CLI::success( "Signup {$signup->signup_id} deleted." );
return [ 'success', "Signup {$signup_id} deleted." ];
} else {
WP_CLI::error( "Failed deleting signup {$signup->signup_id}." );
return [ 'error', "Failed deleting signup {$signup_id}." ];
}
}
}