Skip to content

Use keysToHide config for the DIC panel and debugger too#609

Closed
spaze wants to merge 12 commits intonette:masterfrom
spaze:spaze/keystohide-config-for-panel-debugger
Closed

Use keysToHide config for the DIC panel and debugger too#609
spaze wants to merge 12 commits intonette:masterfrom
spaze:spaze/keystohide-config-for-panel-debugger

Conversation

@spaze
Copy link
Contributor

@spaze spaze commented Dec 22, 2025

  • bug fix / new feature? new/possibly both
  • BC break? no
  • doc PR: will create one, I promise

Set tracy.keysToHide config for Debugger, DIC panel and bdump as well, not just for BlueScreen.

Will require nette/di#327 because the ContainerPanel::$keysToHide property is added there, I'm not sure how to express it in composer.json (with conflict?)

I could also add a condition like:

if (property_exists(Tracy\Debugger::class, 'keysToHide')) array_push(Nette\Bridges\DITracy\ContainerPanel::$keysToHide, ...$keysToHide)

to the generated code for keysToHide in $tbl, to avoid updating or adding any composer version constraints, which I think would be the best thing to do. Let me know what you think.

Close #608

@dg
Copy link
Member

dg commented Feb 8, 2026

Thanks, merged, but the inserting it into the DIC must be done (somehow) on the nette/di side…

@spaze
Copy link
Contributor Author

spaze commented Feb 14, 2026

Thanks! I have updated nette/di#328 which now uses the Bluescreen property to pass keysToHide to Dumper. I hope I got it right but if not please let me know.

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.

tracy.keysToHide config in Nette config file not used for dump()/bdump()

2 participants