Problem/Motivation

By introducing a second kint dumper instead of overriding the default one, we introduce a lot of issues/unwanted behaviour:

The only feature we lose is having the possibility to invoke kint without showing extras, but I don't believe that use case outweighs the benefits. Please correct me if I'm wrong.

Proposed resolution

Override the default kint dumper instead of providing a separate one.

Remaining tasks

  • Override the default kint dumper
  • Provide an upgrade path from previous versions of the module
  • Update the docs, since configuration will no longer be necessary
CommentFileSizeAuthor
#3 3309491-3.patch2.47 KBdieterholvoet
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

DieterHolvoet created an issue. See original summary.

dieterholvoet’s picture

Status: Active » Needs review
StatusFileSize
new2.47 KB

I implemented this, feedback is welcome. Make sure to run database updates after applying the patch, so the devel config is automatically updated. Here's a patch to use for testing in your projects.

  • DieterHolvoet committed d32a5c7 on 1.x
    Issue #3309491 by DieterHolvoet: Override the default kint dumper
    

dieterholvoet’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

sergiur’s picture

The main module page still says the following under Configuration:

Enable this module and select the Kint Extended variables dumper in the Devel settings (/admin/config/development/devel).

Could this also be removed or replaced with something along the lines of "Enable the module as usual. This will replace the default Kint dumper and should work without any other configuration."

dieterholvoet’s picture

Missed that one. Changed the project description, thanks.