Skip to content

Commit d1f7fb0

Browse files
committed
task_id and title are required.
1 parent f53f9ad commit d1f7fb0

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

classes/wp-cli/class-task-command.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -300,8 +300,8 @@ private function delete_task( $task_id, $force ) {
300300
*/
301301
public function create( $args, $assoc_args ) {
302302

303-
$task_id = isset( $assoc_args['task_id'] ) ? $assoc_args['task_id'] : 'collaborator-' . uniqid( '', true );
304-
$title = isset( $assoc_args['title'] ) ? $assoc_args['title'] : 'Test task';
303+
$task_id = isset( $assoc_args['task_id'] ) ? $assoc_args['task_id'] : '';
304+
$title = isset( $assoc_args['title'] ) ? $assoc_args['title'] : '';
305305
$description = isset( $assoc_args['description'] ) ? $assoc_args['description'] : 'Test description ';
306306
$points = isset( $assoc_args['points'] ) ? (int) $assoc_args['points'] : 1;
307307
$provider_id = isset( $assoc_args['provider_id'] ) ? $assoc_args['provider_id'] : 'collaborator';
@@ -311,6 +311,11 @@ public function create( $args, $assoc_args ) {
311311
$dismissable = isset( $assoc_args['dismissable'] ) ? $assoc_args['dismissable'] : true;
312312
$snoozable = isset( $assoc_args['snoozable'] ) ? $assoc_args['snoozable'] : true;
313313

314+
if ( empty( $task_id ) || empty( $title ) ) {
315+
\WP_CLI::error( 'task_id and title are required.' ); // @phpstan-ignore-line
316+
return;
317+
}
318+
314319
// We're creating a new task.
315320
\progress_planner()->get_suggested_tasks_db()->add(
316321
[

0 commit comments

Comments
 (0)