Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Follow-up of #2795567: Use Symfony's VarDumper for easier test debugging with dump().
AssertLegacyTrait::verbose is leftover of Simpletest where it was dumping some content in the Simpletest result UI, but in PHPUnit it is actually doing nothing. After #2795567: Use Symfony's VarDumper for easier test debugging with dump(), dump()
can be used in cases where it's necessary to debug output to the CLI results.
Steps to reproduce
Deprecate the method and remove its usage.
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|
Issue fork drupal-3193163
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:
- 3193163-deprecate-assertlegacytraitverbose-and changes, plain diff MR !304
Comments
Comment #2
mondrakeComment #3
mondrakeComment #4
mondrakeParent was committed
Comment #6
mondrakeComment #7
longwaveWe should probably close #2875038: AssertLegacyTrait::verbose() doesn't do anything as won't fix or duplicate of this.
Comment #8
mondrake#7 aw I forgot about that one. I guess we should credit here those working on that one before closing it.
Comment #9
mondrakeAdded CR.
In the process, also
AssertMailTrait::verboseEmail()
, that is callingverbose()
but has no use in Core any more, was deprecated too.Comment #10
longwaveTwo now-incorrect docblocks but otherwise this is looking ready to go.
Looks possible that we can now deprecate executeView() itself and just call it on the view object directly, but definitely out of scope for here.
Comment #11
mondrakeThanks @longwave
Comment #12
longwaveLGTM.
@core committers please see #8 and credit contributors from the linked issue.
Comment #13
alexpottCommitted 33700d0 and pushed to 9.2.x. Thanks!
This is really nice tidy up.