Reminder that we have not fixed the Undefined variable: rows in scheduler_list(), line 429 of scheduler.module.

It happens when you try to veiw the new tab 'Scheduled' in 'My account'. I am guessing it's caused when you do not have any scheduled contents. I noticed this in 1.8 when switching to error level E_ALL. 7.x seems to be ok - maybe we fixed it but did not port back to 6.x?

Jonathan

CommentFileSizeAuthor
#1 1981900_1.initialise_rows_array.d6.patch368 bytesjonathan1055

Comments

jonathan1055’s picture

jonathan1055 commented
Status: Active » Needs review
StatusFileSize
new 1981900_1.initialise_rows_array.d6.patch368 bytes

Here's a patch to scheduler_list() that fixes it.

Jonathan

rickmanelius’s picture

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

Technically this is a duplicate of #1040176: Notice on scheduled content list, table sort, use D7's abstraction layer on the content list query, add back pager, but there is so much thrown into that issue that I like the cleaner approach of this simple patch.

Looks good... setting to RBTC.

rickmanelius’s picture

rickmanelius commented
Status: Reviewed & tested by the community » Fixed

Committed
http://drupalcode.org/project/scheduler.git/commitdiff/82035e4?hp=b21597...

jonathan1055’s picture

jonathan1055 commented

Thanks Rick

Status: Fixed » Closed (fixed)

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