I get this strict warning... everything seems to be working fine though...
Strict warning: Only variables should be passed by reference in multiblock_general() (line 131 of C:\xampp\htdocs\quoteicon\sites\all\modules\multiblock\multiblock.module).

CommentFileSizeAuthor
#1 strict-warning-1704146.patch695 bytesh3rj4n
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

h3rj4n’s picture

h3rj4n CreditAttribution: h3rj4n commented
Status: Active » Needs review
FileSize
strict-warning-1704146.patch695 bytes

Created a fix for this issue.

Patch included.

deruitern’s picture

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

Reviewed and tested patch. Verified that it does resolve the PHP warning.

VM’s picture

VM CreditAttribution: VM commented
Version: 7.x-1.1 » 7.x-1.x-dev

shifting to 7.x-1.x-dev as that is where the patches would be applied.

kingfisher64’s picture

kingfisher64 CreditAttribution: kingfisher64 commented

Can this be committed to dev so other users are not greeting on enabling the mod with this message. Thank you.

kevinsiji’s picture

kevinsiji CreditAttribution: kevinsiji commented

Patch @ #1 fix the strict warning for me.

Thanks.

kingfisher64’s picture

kingfisher64 CreditAttribution: kingfisher64 commented

Yep #1 works, lovely jubbly.

rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented
Issue summary: View changes

It works also for me
Please commit it to dev

kitzinger’s picture

kitzinger CreditAttribution: kitzinger commented

Yep, it works, please hurry up and commit it to dev after 3 years.

  • intrafusion committed 8842c5e on 7.x-1.x authored by h3rj4n 
    Issue #1704146 by h3rj4n: Strict warning: Only variables should be...
intrafusion’s picture

intrafusion
Primary language English
Location Edinburgh, UK
CreditAttribution: intrafusion commented
Status: Reviewed & tested by the community » Fixed

This patch has been applied and a new release has been rolled

kitzinger’s picture

kitzinger CreditAttribution: kitzinger commented

@intrafusion - WOoOOOOOOHOOOOO !!! Thank you!

  • intrafusion committed 8842c5e on 8.x-1.x authored by h3rj4n 
    Issue #1704146 by h3rj4n: Strict warning: Only variables should be...

Status: Fixed » Closed (fixed)

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