- Issue created by @Liam Morland
- Assigned to samir_shukla
- Status changed to Needs review
over 1 year ago 5:23am 8 January 2024 - 🇮🇳India samir_shukla bareilly
Hi, I have created patch which replace dpm with print_r. There are also many other alternatives like ksm, var_dump, kint etc.
- Issue was unassigned.
- Status changed to Needs work
over 1 year ago 2:40pm 8 January 2024 - 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
dpm()makes a neat output into the Drupal UI. A replacement would need to do a similar thing. - 🇮🇳India samir_shukla bareilly
Hi, created the new patch for the issue. Please review.
- Status changed to Needs review
over 1 year ago 7:30am 9 January 2024 - Status changed to Needs work
over 1 year ago 2:39pm 9 January 2024 - 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
ksm()will raise the same coding standards warning asdpm(). - 🇮🇳India samir_shukla bareilly
Hi, if we are restricted to use debug statement due to phpcs, then we can use something like messenger service. Here is the patch for the same. Please review.
- Status changed to Needs review
over 1 year ago 5:40am 10 January 2024 - Status changed to Needs work
over 1 year ago 2:59pm 10 January 2024 - 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
The first param to
::addMessage()needs to be a string orMarkupInterface;$revision_arrayis an array. - Status changed to Needs review
11 months ago 10:09pm 13 May 2024 -
smulvih2 →
committed c2aab33e on 1.1.x
Issue #3412589: Remove use of dpm()
-
smulvih2 →
committed c2aab33e on 1.1.x
- Status changed to RTBC
11 months ago 2:26pm 14 May 2024 - 🇨🇦Canada smulvih2 Canada 🍁
Fixed the merge conflict and pushed this change to 1.1.x
- Status changed to Fixed
11 months ago 2:37pm 14 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.