- Issue created by @Liam Morland
- Assigned to samir_shukla
- Status changed to Needs review
12 months 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
12 months 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
11 months ago 7:30am 9 January 2024 - Status changed to Needs work
11 months 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
11 months ago 5:40am 10 January 2024 - Status changed to Needs work
11 months 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
7 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
7 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
7 months ago 2:37pm 14 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.