Skip to content

Conversation

@sallyx
Copy link
Contributor

@sallyx sallyx commented Feb 21, 2018

  • bug fix? no
  • new feature? yes
  • BC break? yes

After this change result from \Tracy\Dumper::dump() will reflect the return value from __debugInfo() magic method (if defined).

As this might change output, I consider it as a BC break.

@sallyx sallyx force-pushed the add-support-for-debuginfo-method branch from 903349d to 50eb08b Compare February 21, 2018 12:14
@JanTvrdik
Copy link
Contributor

I'm strongly against using __debugInfo() by default because it hides the actual object state.

@JanTvrdik
Copy link
Contributor

Also see #135, you can use

Tracy\Dumper::$objectExporters[NULL] = function ($obj) {
    return method_exists($obj, '__debugInfo') ? $obj->__debugInfo() : (array) $obj;
};

@sallyx sallyx force-pushed the add-support-for-debuginfo-method branch from 50eb08b to 23c54f5 Compare February 21, 2018 12:33
@sallyx
Copy link
Contributor Author

sallyx commented Feb 21, 2018

Yes, but that's exactly what is __debugInfo() for :-)

I missed that PR. Anyway, thanks for the code example ;-)

@sallyx sallyx force-pushed the add-support-for-debuginfo-method branch from 23c54f5 to a0ece90 Compare February 21, 2018 12:48
@milo
Copy link
Member

milo commented Feb 21, 2018

@sallyx I trust to my code, so I would accept __debugInfo() on my classes. But when I work with 3rd party code, I really want to see the real content of the object and not the trimmed one.

@sallyx
Copy link
Contributor Author

sallyx commented Feb 21, 2018

Ok, I could add option [ Dumper::DEBUGINFO => FALSE ] like it was in #135 ...

@dg dg force-pushed the master branch 4 times, most recently from d13a9ce to 7f24887 Compare February 25, 2018 23:58
@dg dg force-pushed the master branch 8 times, most recently from 427177a to 7ac33a2 Compare March 26, 2018 11:48
@dg dg closed this in 1e0f246 Apr 2, 2018
dg pushed a commit that referenced this pull request Apr 2, 2018
dg pushed a commit that referenced this pull request Apr 3, 2018
dg pushed a commit that referenced this pull request Apr 6, 2018
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.

3 participants