notice: Undefined variable: output in /*****/*****/public_html/d6dev/sites/all/modules/page_title/page_title.module on line 38.

The above is presented on the home page after installation and on the administer screen twice.

This is the Feb 9 6.x-2.x-dev release.

Comments

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

Cheers - I can see why this would happen. I need to make my error outputting more verbose when developing ;-) Sounds like you have E_ALL enabled which is why its producing "notice" level errors. Notice errors are (usually) completely safe to ignore. In this case, PHP is complaining that we're returning $output from hook_help but we've not defined what $output IS for this page...

vm’s picture

vm commented

E_All is indeed enabled and I know I can suppress the notices however, Drupal core is pursuing E_ALL compatibility and contrib modules should want to pursue as well.

reference: http://drupal.org/node/189454 though there were deeper discussions about removing the E_NOTICE flags from core.

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

Yup ;-) Hence me saying...

I need to make my error outputting more verbose when developing ;-)

nicholasthompson’s picture

nicholasthompson
Primary language English
commented
Title: notice: Undefined variable: » notice: Undefined variable: output on line 38 of page_title.module
Priority: Normal » Minor
Status: Active » Needs work
suryaprakash’s picture

suryaprakash commented
Assigned: Unassigned » suryaprakash

I will solve

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

I've been doing work on the module (6.x-2.x-dev) with E_ALL on my dev box running 5.2.9 and I'm not getting errors... Is this fixed? Can you produce any errors?

nicholasthompson’s picture

nicholasthompson
Primary language English
commented
Title: notice: Undefined variable: output on line 38 of page_title.module » notice: Undefined variable: output on line 38 of page_title.module (E_ALL compatibility)

changing the title for future reference...

vm’s picture

vm commented

I will retest and report.

nicholasthompson’s picture

nicholasthompson
Primary language English
commented
Status: Needs work » Fixed

I have been running E_ALL recently and haven't seen any errors.

Status: Fixed » Closed (fixed)

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