Hi
Just installed this module to see how it works. Thanks for your work!
Drupal 5.10 on each page generated by Views module I see warning:
"warning: Missing argument 3 for drupalforfirebug_views_pre_view(), called in /www/ekklesia/sites/all/modules/views/views.module on line 598 and defined in /www/ekklesia/sites/all/modules/drupalforfirebug/drupalforfirebug.module on line 31."

Comments

vladsavitsky’s picture

vladsavitsky commented

Simple replacement remove this warning:
Find (line 31):

function drupalforfirebug_views_pre_view(&$view, &$display_id, &$args) {

Replace with:

function drupalforfirebug_views_pre_view(&$view, &$display_id) {

But don't know is it right solution. Thanks.

lefnire’s picture

lefnire commented

function signature for D6 is hook_views_pre_view(&$view, &$display_id, &$args){
for D5 is hook_views_pre_view($view, &$items){
So the hook should be implemented differently in DFFB 5.x than 6.x, but since their hook only uses $view, the above is a safe interim.

kehan’s picture

kehan commented

when will this get fixed? i'm happy to apply the fix, but if you search google for the error the message pops up on a number of sites being developed around the web (including the drupalforfirebug demo site! - http://drupalforfirebug.chapterthree.com/tests/views).

populist’s picture

populist commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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