Skip to content

Commit

Permalink
Merge pull request #8286 from kenjis/remove-Controller-loadHelpers
Browse files Browse the repository at this point in the history
refactor: remove deprecated Controller::loadHelpers()
  • Loading branch information
kenjis authored Dec 7, 2023
2 parents 1081b19 + 370becc commit 74b4ac5
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 23 deletions.
5 changes: 0 additions & 5 deletions phpstan-baseline.php
Original file line number Diff line number Diff line change
Expand Up @@ -816,11 +816,6 @@
'count' => 4,
'path' => __DIR__ . '/system/Config/Services.php',
];
$ignoreErrors[] = [
'message' => '#^Construct empty\\(\\) is not allowed\\. Use more strict comparison\\.$#',
'count' => 1,
'path' => __DIR__ . '/system/Controller.php',
];
$ignoreErrors[] = [
'message' => '#^Only booleans are allowed in a negated boolean, array given\\.$#',
'count' => 1,
Expand Down
18 changes: 0 additions & 18 deletions system/Controller.php
Original file line number Diff line number Diff line change
Expand Up @@ -123,24 +123,6 @@ protected function cachePage(int $time)
Services::responsecache()->setTtl($time);
}

/**
* Handles "auto-loading" helper files.
*
* @deprecated Use `helper` function instead of using this method.
*
* @codeCoverageIgnore
*
* @return void
*/
protected function loadHelpers()
{
if (empty($this->helpers)) {
return;
}

helper($this->helpers);
}

/**
* A shortcut to performing validation on Request data.
*
Expand Down
1 change: 1 addition & 0 deletions user_guide_src/source/changelogs/v4.5.0.rst
Original file line number Diff line number Diff line change
Expand Up @@ -211,6 +211,7 @@ Others
------

- **Config:** The deprecated ``CodeIgniter\Config\Config`` class has been removed.
- **Controller:** The deprecated ``Controller::loadHelpers()`` method has been removed.

Enhancements
************
Expand Down

0 comments on commit 74b4ac5

Please sign in to comment.