Call to dsm instead of drupal_set_message in views_handler_field_counter

carlosdavis - November 2, 2009 - 23:27
Project:Views
Version:6.x-3.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:matt2000
Status:closed
Description

I noticed that I was getting a dead page on any view using the Global: View Result Counter field.

My error logs had:
PHP Fatal error: Call to undefined function dsm() in /home/klat/web/sites/all/modules/contrib/views/handlers/views_handler_field_counter.inc on line 29

I assume that dsm is misused shorthand for drupal_set_message.

#1

dereine - November 3, 2009 - 00:47
Status:active» needs review

Fast fix.

AttachmentSize
views-counter-fix.patch 854 bytes

#2

matt2000 - November 3, 2009 - 01:03
Assigned to:Anonymous» matt2000

Yup, that's a wrapper from devel module. Looks like debugging that didnt get cleaned up.

This patch removes the unnecessary and problematic line.

AttachmentSize
621532_views_handler_field_counter.patch 871 bytes

#3

vacilando - November 5, 2009 - 11:13
Status:needs review» reviewed & tested by the community

Reviewed, the #2 patch fixes the issue.

#4

ermannob - November 9, 2009 - 12:56

I can confirm both patches are good.

#5

dereine - November 9, 2009 - 13:21

Ah by the way, this was fixed in the dev version of views 2.

#6

dagmar - November 9, 2009 - 14:55
Status:reviewed & tested by the community» fixed

We should mark this as fixed, isn't?

#7

dereine - November 9, 2009 - 15:26
Version:6.x-2.7» 6.x-3.x-dev
Status:fixed» needs review

The patch wasn't applied to views-3 yet.
http://drupal.org/files/issues/views-counter-fix.patch

#8

merlinofchaos - November 9, 2009 - 21:30
Status:needs review» fixed

Fixed in 3.x as well.

#9

System Message - November 23, 2009 - 21:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.