Strict warning: Only variables should be passed by reference in resp_img_field_attach_view_alter() [#1813364]

Comment	File	Size	Author
#1	resp_img_strict_warning_resolve-1813364.patch	712 bytes	jaffaralia

Comment #1

jaffaralia commented 15 October 2012 at 17:07

Status:

Active

» Needs review

Status	File	Size
new	resp_img_strict_warning_resolve-1813364.patch	712 bytes

Here's a patch the reset($breakpoint_styles['mapping']) function had been assigned to a variable $reset and it has been send to reset($reset) function after that warning message disappear. When tested it worked for me.

Log in or register to post comments

Comment #2

luksak

he/him

commented 15 October 2012 at 21:27

Status:

Needs review

» Needs work

Thank you! The warnings are gone. Now I am getting this error:

Notice: Undefined index: uri in theme_picture() (line 374 of /sites/all/modules/resp_img/resp_img.module).

The images are still not showing...

Log in or register to post comments

Comment #3

attiks commented 16 October 2012 at 14:43

Assigned:

Unassigned

» jelle_s

Log in or register to post comments

Comment #4

jaffaralia commented 17 October 2012 at 07:09

I can't replicate this notice can you explain how you configured the breakpoints and reponsive image and style.

Log in or register to post comments

Comment #5

attiks commented 17 October 2012 at 08:37

Status:

Needs work

» Needs review

#2 Looks to me this is unrelated to this patch, as suggested in #4 can you check you're settings? If the problem still exists can you create a new issue, so we can use this one to keep track of the patch.