Imagefield Import Archive adds additional functionality to the Imagefield Import module which allows users to upload .zip files containing images. The correct import directory is retrieved from the database, and the images are extracted into it. The user is then directed to the standard Imagefield Import page where their images are ready to be imported. Basically, this module prevents users having to install and configure and FTP client and manually upload their images into the import directory.

Link to project page: http://drupal.org/sandbox/opdavies/1165110

Comments

opdavies’s picture

It's been almost 4 weeks since I opened this post - when will there be an update made?

The module has been used in the sandbox state by users, and it's been verified as working. See http://drupal.org/node/267241#comment-4502704.

Many thanks,

Oliver

ccardea’s picture

Status: Needs review » Needs work

There is a module called Image Import Zip which does this, which is a Drupal 5.x module and is listed as no further development. I suggest that you use the procedure discussed at http://drupal.org/node/251466 to take over this project. Image Import Zip was discussed at http://drupal.org/node/267241 #10.

Looking at your code, my biggest concern is that the php zip extension might not be present on the host where the module is installed. I think your code should attempt to deal with this somehow, maybe with a try catch block. I noticed that Image Import Zip warned his users about this on the project page.

Also on line 18 of you .module file, you use 'imagefield_import_form as a page callback, which is not present in your code. I did not really look much farther into your code. If you follow the abandoned project procedure, you won't need a code review.

opdavies’s picture

Hi ccardea,

Thanks for your reply.

I see that Image Import Zip was an extension for Image Import (part of the Image module), whereas my module is an extension of Imagefield Import module. Does this make a difference if I take over an abandoned module?

By 'try catch block', I'm assuming that you mean something that checks that the zip extension is installed before allowing the module to be enabled?

I'll take a look at that page callback. :)

opdavies’s picture

Status: Needs work » Needs review
ccardea’s picture

Status: Needs review » Needs work

A try - catch block is a means of exception handling, such that your code would handle any exception thrown by the extension not being present, instead of your users seeing a PHP error message. I don't know of a way to check to see if the extension is present, but if you could find a way, that would be even better.

There shouldn't be any problem with taking over Image Import Zip. You should check with the webmasters. If there's a problem, you can always re-open your application.

klausi’s picture

Status: Needs work » Closed (won't fix)

No activity in several months. Reopen and set the status to "needs review" if you are still pursuing this application.