When clearing the decorator cache (or something like that), I get the following error:

Strict warning: Only variables should be passed by reference in _decorator_file_cache() (line 250 of /.../sites/all/modules/decorator/decorator.module).

Since it's a strict warning, it's not a big deal, but it should be easy to get rid of that error. I'm working on a patch right now and will post when it's ready.

CommentFileSizeAuthor
#1 decorator-strict_passed_by_reference_warning-2374971-1.patch782 bytescuteapoot

Comments

cuteapoot’s picture

cuteapoot commented
StatusFileSize
new decorator-strict_passed_by_reference_warning-2374971-1.patch782 bytes

Documentation for file_prepare_directory() shows the first parameter as &$directory, so I just re-used that variable name and assigned it prior to calling file_prepare_directory(). No more strict warnings!

kebz’s picture

kebz commented

I just installed this module and received these warnings

Strict warning: Declaration of flot_handler_field_datapoint::pre_render() should be compatible with views_handler_field::pre_render(&$values) in _registry_check_code() (line 3179 of /.../../.../.../.../.../includes/bootstrap.inc).

Warning: class_exists() expects parameter 1 to be string, array given in views_decorator_build_handler_decorators() (line 21 of /.../../.../.../.../.../sites/all/modules/views_decorator/views_decorator.views.inc).

Strict warning: Only variables should be passed by reference in _decorator_file_cache() (line 250 of /.../../.../.../.../.../sites/all/modules/decorator/decorator.module).

smitty’s picture

smitty commented
Status: Active » Reviewed & tested by the community

I got this strict warning when clearing the cache since I replaced php 5.3 with php 5.4.

After applying this patch the warning has gone. Well done! Thank you!

The two other warnings, that were mentioned by Kebz seem to be another problem, located in the Views Decorator Module. I have to struggle with those warnings too but I will post my issue over there