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.

CommentFileSizeAuthor
patch_41.txt885 bytesastroboy

Comments

vhmauery’s picture

thanks 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.

astroboy’s picture

I'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.

vhmauery’s picture

Okay, 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:

  1. install drupal, get database set up
  2. go to admin/build/modules and enable:
    • acidfree
    • image
    • video
    • video_image
    • video_upload
    • views
  3. go to admin/build/block
    • enable 'Add Acidfree items quick links' block
  4. go to admin/settings/file-system
    • set the 'File System Path' to something that is writeable by apache user
  5. go to admin/content/video/image
    • select 'Auto thumbnail for videos'
    • provide proper path for mplayer at 'mplayer executable path'
    • uncheck 'Publish the video thumbnails'
  6. run the two following sql commands:
    • update system set weight=1 where name='video_image';
    • truncate cache;

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.

vhmauery’s picture

Status: Needs review » Fixed
vhmauery’s picture

Status: Fixed » Closed (fixed)