Clean up command classes #3787

New Issue

OVERLORD · 2026-02-05T07:28:30+03:00

OVERLORD commented

2026-02-05 07:28:30 +03:00

Originally created by @ssddanbrown on GitHub (May 7, 2023).

Update our command classes to follow a common convention. Including:

Update to inject via the handle method (Advised approach in Laravel docs).
Standardise how response codes are returned (And ensure they are) (int vs SymfonyCommand::[SUCCESS/FAILURE]).
- Laravel internal commands appear to just use ints, is probably the cleaner option.

Originally created by @ssddanbrown on GitHub (May 7, 2023). Update our command classes to follow a common convention. Including: - Update to inject via the `handle` method (Advised approach in Laravel docs). - Standardise how response codes are returned (And ensure they are) (int vs `SymfonyCommand::[SUCCESS/FAILURE]`). - Laravel internal commands appear to just use ints, is probably the cleaner option.

OVERLORD added the 🔧 Maintenance 🏭 Back-End labels 2026-02-05 07:28:30 +03:00

OVERLORD closed this issue

2026-02-05 07:28:32 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#3787