Return schedule as array of objects / arrays

[turtle0x1]

Description
Its cool that you can run vendor/bin/crunz s:l but it would be really good if we could see these jobs displayed on a web interface.

Unfortunately I'm not quite sure how to approach this issue so im requesting it but cant produce a PR (which I know is rude sorry).

Happy to give it a shot but will need some pointers 😄

[PabloKowalczyk]

Will JSON output be ok?

[turtle0x1]

JSON would work better if not possible to call pragmatically withoutout opening a shell process.

Ideally what Im trying todo is something like;

$crunz = new Crunz();
$schedules = $crunz->getSchedule();
// [ Task(), Task(), Task() ]

[PabloKowalczyk]

JSON would work better if not possible to call pragmatically withoutout opening a shell process.

Currently it won't be easy, because there is no place where developer can hook after collecting all tasks.
Will --format option on schedule:list help?

[turtle0x1]

Yeah because you can do some hacky stuff like

$results = json_decode(shell_exec("vendor/bin/crunz s:l --format json"))

it might be worth thinking about the case where an exception / error happens as well and what will be out put on that case!

If the errors aren't JSON you'd have todo something like;

$results = json_decode(shell_exec("vendor/bin/crunz s:l --format json"))
// Might not have access to JSON_THROW_EXCEPTION
if(json_last_error()){
     // If when crunz errors the exception isn't JSON we wont be able to parse a real error message
     throw new \Exception("Something went wrong in the crunz script")
}

[PabloKowalczyk]

Ok, give me few days :)

it might be worth thinking about the case where an exception / error happens as well and what will be out put on that case!

In that case you should rely on process exit code or use symfony/process and mustRun (https://symfony.com/doc/current/components/process.html).

[PabloKowalczyk]

Ok, check PR #31.

[turtle0x1]

Awesome, thank you 😄 Will try to give this a test over the next couple of days!

[PabloKowalczyk]

Fixed by #31.