oa_appearance_get_space_banner() returns an associative array (let's call it $return) which includes $return['image'] which is meant to be an associative array that you can use like theme('image', $return['image']); or theme('image_style', $return['image']) (or, at least that was my intention - I originally wrote that function ;-)).
In any case, it get's it slight wrong having $return['image']['attributes']['class'] as a string rather than an array.
The fix is simple - I'll upload a patch in a moment.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | oa_appearance-banner-image-class-2413891-1.patch | 501 bytes | dsnopek |
Comments
Comment #1
dsnopekPatch is attached!
Comment #2
mpotter commentedFrom looking at the code, drupal_attributes does an (array) typecast so appears to accept both an array as well as a single string value.
However, for sake of better code, I have applied the patch. But you missed a similar case later in that same function where it was setting 'class' to a string value based on a conditional, so I updated that also.
Committed to 5e46b7f of oa_appearance.
Comment #3
dsnopekAh, good catch! Thanks!