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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chi’s picture

Status: Active » Needs review
FileSize
711 bytes
TwoD’s picture

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

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

Status: Patch (to be ported) » Needs review
FileSize
1.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

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

Issue summary: View changes

grammar