WD php: Notice: Undefined variable: views in signup_views_default_views() (line 28 of [error]
/var/www/globalnet/dscadev/trunk/docroot/sites/all/modules/contrib/signup/views/signup.views_default.inc).

--

The $views variable that's returned is never defined, which means that function, at times, tries to return an undefined variable.

CommentFileSizeAuthor
#3 fix-undefined-var-1266764-3.patch585 bytes30equals
signup_undefined-variable-default-views.patch610 bytesmstef
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mstef’s picture

mstef CreditAttribution: mstef commented
Status: Active » Needs review
geert’s picture

geert CreditAttribution: geert commented

The same happens in D7.8 using signup DEV of 29/06 (most recent version available)

Notice: Undefined variable: views in signup_views_default_views() (line 25 of d7test/drupal/sites/all/modules/signup/views/signup.views_default.inc).

30equals’s picture

30equals CreditAttribution: 30equals commented
FileSize
fix-undefined-var-1266764-3.patch585 bytes

Hey, i got the same error on a Drupal 7 install , and i created a proper git diff patch for the 7.x-1.x branch

30equals’s picture

30equals CreditAttribution: 30equals commented
Version: 6.x-1.0 » 7.x-1.x-dev

switching the version as it still appears in the D7 version as well.

30equals’s picture

30equals CreditAttribution: 30equals commented

mark as needs review

sgabe’s picture

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

The patch in #3 applies fine and eliminates the notice.

sgabe’s picture

sgabe CreditAttribution: sgabe commented
Component: Code » Views integration

Changing component.

sgabe’s picture

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

Committed, thanks!

Status: Fixed » Closed (fixed)

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