Skip to content

docs: correct what make:migration --session reads from Config\Session - #10549

Merged
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:session-migration-docs
Sep 13, 2026
Merged

docs: correct what make:migration --session reads from Config\Session#10549
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:session-migration-docs

Conversation

@paulbalandan

Copy link
Copy Markdown
Member

Description

Discovered while doing #10545 .

The sessions guide says make:migration --session "will take the $savePath and $matchIP settings into account". That was true when the command was added in 094f675 (2017), but fc9add0 (2018) dropped the $savePath lookup in favor of a plain ci_sessions default, and the sentence was never updated. Today the command reads only $matchIP. The table name and database group come from --table and --dbgroup.

The paragraph now states that, shows both options, and notes that their values should match $savePath and $DBGroup.

Checklist:

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

@github-actions github-actions Bot added the documentation Pull requests for documentation only label Sep 12, 2026
@paulbalandan
paulbalandan merged commit 64375f1 into codeigniter4:develop Sep 13, 2026
7 checks passed
@paulbalandan
paulbalandan deleted the session-migration-docs branch September 13, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Pull requests for documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants