Database as optional parameter for set_credentials #25

Merged
vlw merged 1 commit from refs/pull/25/head into master 2026-02-12 08:48:38 +01:00
vlw commented 2026-02-12 08:46:55 +01:00 (Migrated from codeberg.org)

This is another follow-up PR for #22 which fixes a deprecation issue about defining optional parameters that are then called as named.

Database::set_credentials("db");
vlw\Scaffold\Database::set_credentials(): Optional parameter $host declared before required parameter $database is implicitly treated as a required parameter
This is another follow-up PR for #22 which fixes a deprecation issue about defining optional parameters that are then called as named. ```php Database::set_credentials("db"); ``` ``` vlw\Scaffold\Database::set_credentials(): Optional parameter $host declared before required parameter $database is implicitly treated as a required parameter ```
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
vlw/scaffold!25
No description provided.