Environment is Windows / IIS6 / PHP5.1.4 / Drupal-CVS HEAD of october 30.
when modifying an image page, you get the message :
warning: array_merge() [function.array-merge]: Argument #2 is not an array in C:\your-path-to\drupal\modules\acidfree\class_image.inc on line 224.
it's because array_merge gets called with $form['image']['#attributes'], that does not always have a value.
I added a check for existence of the value, and adjusted the value accordingly.
| Comment | File | Size | Author |
|---|---|---|---|
| patch_41.txt | 885 bytes | astroboy |
Comments
Comment #1
vhmauery commentedthanks for finding this. If you are willing and brave to try out the acidfree 5.0 alpha (which is the CVS HEAD) I might as well make use of you. :)
You might notice that there are several things missing -- I still need to figure out how to automatically import the 2 views that are used. Installation and upgrade still doesn't work. Not much is automatic right now. You might have to add some patches to get the video module working like I have envisioned (they haven't been committed yet, or even reviewed.)
These two video issues need to be reviewed:
http://drupal.org/node/82396 - handle remote/local video types differently
http://drupal.org/node/82954 - automatic thumbnailing for videos
I could seriously use your help with any 5.0 testing you can do. This is fixed on my test box, but I haven't committed it yet. I will try to do more regular updates (maybe daily) so it is easier for people to collaborate with me.
Comment #2
astroboy commentedI'd do that with pleasure, I'm already checking out drupal contrib daily, and I've yet to update it today.
Tell me what you would like me to do, and I'd be glad to help.
Comment #3
vhmauery commentedOkay, I just committed a whole lot of changes to acidfree. I can't remember if this one got fixed or not. But a lot of other problems did.
It looks awful with the default drupal theme. Sorry. I might have to come up with a new default acidfree theme.
But here are the steps to get it to work:
After all that, you should be able to add images, videos, albums and do mass import. Album contents should work as well. Upgrading from 4.7 does NOT work yet.
Comment #4
vhmauery commentedComment #5
vhmauery commented