Skip to content
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.

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

TasksListCommand 'Next Run' incorrect and deprecated warning DateTime #43665

Closed
brbrbr opened this issue Jun 17, 2024 · 3 comments
Closed

TasksListCommand 'Next Run' incorrect and deprecated warning DateTime #43665

brbrbr opened this issue Jun 17, 2024 · 3 comments

Comments

@brbrbr
Copy link

brbrbr commented Jun 17, 2024

Steps to reproduce the issue

Create a scheduled task with Execution Rule on Manual
execute the command line scheduler:list

Expected result

Table with 'Next run' for above task set to Never

Actual result

'Next run' set to 'DUE!'
PHP Deprecated: DateTime::__construct(): Passing null to parameter #1 ($datetime) of type string is deprecated in libraries/src/Date/Date.php

The deprecated warning also shows in the Tasks view when a task crashed.

System information (as much as possible)

joomla 5.1
php 8.3

Additional comments

As a site note, The Captions 'Test Task' and 'Run Test' in the Tasks view is a kind of misleading.

It suggests a kind of dry run, while the tasks are actually executed. So 'Run Task' and 'Run' would be more appropriate.

@alikon
Copy link
Contributor

alikon commented Jul 7, 2024

please test #43750 for the cli part

@Quy
Copy link
Contributor

Quy commented Jul 7, 2024

As a site note, The Captions 'Test Task' and 'Run Test' in the Tasks view is a kind of misleading.

It suggests a kind of dry run, while the tasks are actually executed. So 'Run Task' and 'Run' would be more appropriate.

PR #36725.

@brbrbr
Copy link
Author

brbrbr commented Jul 7, 2024

As a site note, The Captions 'Test Task' and 'Run Test' in the Tasks view is a kind of misleading.
It suggests a kind of dry run, while the tasks are actually executed. So 'Run Task' and 'Run' would be more appropriate.

PR #36725.

That PR seams a bit broken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants