avoid using of devel functions if it is not present

vectoroc - October 26, 2009 - 03:40
Project:Drupal for Facebook
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Dave Cohen
Status:closed
Description

I mean for example - fb_user.module line 202

watchdog('fb_user', t("Created new ... %app" . dprint_r(array(...), 1),
array('!username' => theme('username', $user), '%app' => $fb_app->label)));

replace with

watchdog('fb_user', t("Created new ... %app!debug_info",
array('!username' => theme('username', $user),
'!debug_info' => (function_exists('dprint_r') ? dprint_r(array('session_name' => session_name(), 'session_id' => session_id()), 1) : ''),
'%app' => $fb_app->label)));

#1

Dave Cohen - October 26, 2009 - 18:20
Category:support request» bug report
Assigned to:Anonymous» Dave Cohen

That got in there by accident.

Keep submmitting issues if you find more calls like that. Thanks.

#2

Dave Cohen - October 27, 2009 - 20:40
Status:active» fixed

Removed the dprint_r() calls.

#3

System Message - November 10, 2009 - 20:50
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.