Posted by Vin7 on October 12, 2009 at 10:15pm
4 followers
Jump to:
| Project: | Image |
| Version: | 6.x-1.0-beta3 |
| Component: | image.module |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Warning notice displays when in admin/settings/image:
warning: Parameter 1 to theme_image_settings_sizes_form() expected to be a reference, value given in /srv/www/htdocs/zus/includes/theme.inc on line 617.
When I switch from reference to actual variable in theme_image_settings_sizes_form function, warning disappears. Was reference intended or not?
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| image.admin_.inc_.patch | 508 bytes | Ignored: Check issue status. | None | None |
Comments
#1
I don't get that warning -- have you got E_ALL on maybe?
But looking at http://api.drupal.org/api/drupal/developer--topics--forms_api.html you're right -- the $form element should be passed in as a value not a reference.
Thanks for the patch :)
For future reference though, don't leave commented-out old code in a patch, and don't add lines crediting yourself. The code would quickly become unreadable with those in.
#2
Automatically closed -- issue fixed for 2 weeks with no activity.
#3
I'm getting this error -> warning: Parameter 1 to theme_image_settings_sizes_form() expected to be a reference, value given in /opt/lampp/htdocs/uberdru/includes/theme.inc on line 617.
on Version 6.14. Can I use the patch to fix it?
#4
6.14 is Drupal core.
Download the latest dev of image.