This patch removes dpm() calls, and gets the statistics and blacklist routes working by changing _content to _controller.

CommentFileSizeAuthor
mollom-remove-dpm.patch2.29 KBjhedstrom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eshta’s picture

eshta CreditAttribution: eshta commented
Status: Needs review » Reviewed & tested by the community

Thanks - yes obviously we are still way in dev mode with all those dpm calls.... Happy to take the help on the controller fixes though.

  • eshta committed c122f69 on 8.x-1.x authored by jhedstrom 
    Issue #2449083 by jhedstrom: Removes dpm calls/fixes statistics page
eshta’s picture

eshta CreditAttribution: eshta commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Status: Closed (fixed) » Needs work

The last submitted patch, mollom-remove-dpm.patch, failed testing.

Status: Needs work » Needs review

isntall queued mollom-remove-dpm.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, mollom-remove-dpm.patch, failed testing.

eshta’s picture

eshta CreditAttribution: eshta commented
Status: Needs work » Closed (fixed)