I get a notice with:
Notice: Undefined variable: days w template_preprocess_timefield_formatter() (linia 887 z /home/vidicon/ftp/brodnica/sites/all/modules/timefield/timefield.module).

what is wrong??

CommentFileSizeAuthor
#4 undefined_variable-2813751-4.patch503 bytesawm
#3 undefined_variable-2813751-3.patch508 bytessomatick
#2 undefined_variable-2813751-2.patch501 bytesnilesh.addweb

Comments

kazaa created an issue. See original summary.

nilesh.addweb’s picture

nilesh.addweb commented
Version: 7.x-1.0-alpha2 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new undefined_variable-2813751-2.patch501 bytes

PFA Patch.

somatick’s picture

somatick commented
StatusFileSize
new undefined_variable-2813751-3.patch508 bytes

It seems the notice only occurs if there is a weekly repeating time field with no days selected.

I think it would be better if $days is initialised as an array and if it is initialised in the scope that it is used. Please try this patch

awm’s picture

awm commented
StatusFileSize
new undefined_variable-2813751-4.patch503 bytes

the patch above did not apply cleanly on 7.x-1.x attaching a clean patch

  • awm committed f91b3d4 on 7.x-1.x 
    Issue #2813751 by somatick, darshi.addweb, awm: Undefined variable
awm’s picture

awm commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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