Skip to content

fix: Remove the leftover $query_builder argument and docs from the Loader. - #80

Merged
pocketarc merged 1 commit into
developfrom
fix/loader-database-args
Nov 1, 2026
Merged

pocketarc merged 1 commit into
developfrom
fix/loader-database-args

Conversation

@pocketarc

@pocketarc pocketarc commented Nov 1, 2026 •

Copy link
Copy Markdown
Owner

Loading a model with a database connection no longer passes an unused third argument to $this->load->database(). No change to existing behavior, since PHP was already ignoring the extra argument.

Fixes #76.

@pocketarc
pocketarc merged commit e7fafb6 into develop Nov 1, 2026
186 checks passed
@pocketarc
pocketarc deleted the fix/loader-database-args branch November 1, 2026 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] CI_Loader::model() calls $this->database() with 3 parameters, but method only accepts 2

1 participant