Project:Web Links
Version:master
Component:Code
Category:bug report
Priority:normal
Assigned:NancyDru
Status:closed (fixed)

Issue Summary

There are some devel debugging lines (dsm) in hook_block that fail and make the Admin screen go white.

AttachmentSize
weblinks29.patch715 bytes

Comments

#1

You can remove them if you want, but I have a better patch coming...

#2

Title:Debugging code causes Admin to WSOD» Block problems
Priority:critical» normal

In addition to the above, the fix for http://drupal.org/node/131416 got changed and did not work. Also, it was only to the "recent" block. This patch extends that setting to all blocks (including even the upcoming "popular" block).

Note the code has been refactored to improve readability and extensibility.

The query code has a small change that opens a nice new feature ... coming soon.

AttachmentSize
weblinks29.patch 5.88 KB

#3

Stop me, I'm out of control!!! In a few minutes, I'll throw in the "most popular" block while I'm, here.

#4

ops I put debuging code into cvs. my mistake. Sorry.

#5

Hopefully, I'll leave this alone for a little while... All the above, plus the "Most popular" request.

AttachmentSize
weblinks29.patch 7.89 KB

#6

Status:needs review» patch (to be ported)

committed to 6.x

#7

Status:patch (to be ported)» fixed

Backported to 5. Commited to CVS.

#8

Status:fixed» closed (fixed)

Since I reported it and no one else seemed to notice, let's clear up the issue.

#9

No one noticed as the dsm code went in after alpha9 was released so that debuging code only appeared in CVS.

Thanks
Robert