In the nodequeue.test, there are many function calls using params passed by reference, which cause fatal errors.

CommentFileSizeAuthor
#2 nodequeue_call-time-pass-by-reference-fatal-error-1796502-2.patch6.16 KBalex_sansom
#1 nodequeue_call-time-pass-by-reference-fatal-error-1796502-1.patch7.65 KBkurtzhong
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kurtzhong’s picture

kurtzhong CreditAttribution: kurtzhong commented
Status: Active » Needs review
FileSize
nodequeue_call-time-pass-by-reference-fatal-error-1796502-1.patch7.65 KB
alex_sansom’s picture

alex_sansom CreditAttribution: alex_sansom commented
Issue summary: View changes
FileSize
nodequeue_call-time-pass-by-reference-fatal-error-1796502-2.patch6.16 KB

Updated patch (PHP 5.5/nodequeue 6.x-2.x).

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Reviewed & tested by the community

Applies cleanly, looks fine.

fizk’s picture

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

Committed, thanks!

  • fizk committed ac9a641 on 6.x-2.x authored by alex_sansom 
    Issue #1796502 by kurtzhong, alex_sansom: PHP Fatal error: Call-time...

Status: Fixed » Closed (fixed)

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