Skip to content

Commit

Permalink
Merge pull request #8003 from kenjis/fix-Config-Kint
Browse files Browse the repository at this point in the history
fix: error on `Config\Kint` with Config Caching
  • Loading branch information
kenjis authored Oct 11, 2023
2 parents b6684e6 + ff2c6b6 commit a8f589d
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
3 changes: 1 addition & 2 deletions app/Config/Kint.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@

namespace Config;

use CodeIgniter\Config\BaseConfig;
use Kint\Parser\ConstructablePluginInterface;
use Kint\Renderer\AbstractRenderer;
use Kint\Renderer\Rich\TabPluginInterface;
Expand All @@ -18,7 +17,7 @@
*
* @see https://kint-php.github.io/kint/ for details on these settings.
*/
class Kint extends BaseConfig
class Kint
{
/*
|--------------------------------------------------------------------------
Expand Down
2 changes: 1 addition & 1 deletion system/CodeIgniter.php
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ private function autoloadKint(): void

private function configureKint(): void
{
$config = config(KintConfig::class);
$config = new KintConfig();

Kint::$depth_limit = $config->maxDepth;
Kint::$display_called_from = $config->displayCalledFrom;
Expand Down
3 changes: 3 additions & 0 deletions user_guide_src/source/installation/upgrade_450.rst
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,9 @@ Config
- app/Config/Feature.php
- ``Config\Feature::$multipleFilters`` has been removed, because now
:ref:`multiple-filters` are always enabled.
- app/Config/Kint.php
- It no longer extends ``BaseConfig`` because enabling
:ref:`factories-config-caching` could cause errors.

All Changes
===========
Expand Down

0 comments on commit a8f589d

Please sign in to comment.