Skip to content

refactor: write the Database commands through $this->getOutput() - #10689

Merged
paulbalandan merged 1 commit into
codeigniter4:4.8from
paulbalandan:database-commands-output
Oct 30, 2026
Merged

paulbalandan merged 1 commit into
codeigniter4:4.8from
paulbalandan:database-commands-output

Conversation

@paulbalandan

Copy link
Copy Markdown
Member

Description
Next batch of the command migration. db:create, db:seed, db:table, migrate, migrate:refresh, migrate:rollback and migrate:status now write through $this->getOutput(), and their remaining prompts ask through $this->getPrompt(), so a Prompt override reaches them.

  • Tables are written through Table in place of CLI::table().
  • The private methods of db:table take the output from execute().

No behaviour changes. The existing tests pass unchanged.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@carson-codeigniter4 carson-codeigniter4 Bot added 4.8 PRs that target the `4.8` branch. refactor Pull requests that refactor code labels Oct 30, 2026
@paulbalandan
paulbalandan merged commit e4f6a02 into codeigniter4:4.8 Oct 30, 2026
60 checks passed
@paulbalandan
paulbalandan deleted the database-commands-output branch October 30, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.8 PRs that target the `4.8` branch. refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants