Strict warning: Only variables should be passed by reference in wysiwyg_wymeditor_settings() (line 176 of sites/all/modules/wysiwyg/editors/wymeditor.inc).
The messages appears on WYMeditor profile edit page.

CommentFileSizeAuthor
#5 wysiwyg.1531896.5.patch1.48 KBtwod
#1 quickfix-1531896-0.patch711 byteschi

Comments

chi’s picture

chi commented
Status: Active » Needs review
StatusFileSize
new quickfix-1531896-0.patch711 bytes
twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Needs review » Fixed

Thanks for the patch!
Tested and committed to all main branches.
I'm sorry to have kept this change change waiting.
The -dev snapshots will be updated within 12 hours and this fix will be part of the next releases.

Status: Fixed » Closed (fixed)

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

Emmanuel_S’s picture

Emmanuel_S commented
Status: Closed (fixed) » Patch (to be ported)

I get the same issue in all\modules\wysiwyg\editors\openwysiwyg.inc line 103.
The patch at #1 is working too :
- $settings['stylesheet'] = reset(wysiwyg_get_css());
+ $css = wysiwyg_get_css();
+ $settings['stylesheet'] = reset($css);

Please make correction in the dev version.
Thanks !

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new wysiwyg.1531896.5.patch1.48 KB

Found one more. Tried searching for array_* functions that also require a reference, but found no issues.
If anyone is able to find more, it'd be nice to get them all in one patch.

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Needs review » Fixed

Committed to all main branches.
The -dev snapshots will be updated within 12 hours and this fix will be part of the next releases.

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

grammar