This module allows you to import imagecache presets in a easy and simple way.

The imagecache module provide a export feature but don't have a way to import presets. Actually to import presets, an alternative would be features module, but it does a lot more and we felt the need to have a simpler way to do it in some projects.

Easy to Use, just export some preset from imagecache and then paste the output on the imagecache import form.

Requirements: ImageCache Module
Sandbox on Drupal.org: http://drupal.org/sandbox/rutiolma/1460352
Drupal core version: 6.x

Comments

rutiolma’s picture

Status: Active » Needs review
patrickd’s picture

Status: Needs review » Needs work

welcome

unfortunately you've not committed anything yet,
please have a look at the documentation about how to use git on drupal.org

Regards.

rutiolma’s picture

Status: Needs work » Needs review

Thank you patrickd,
I'm new on this like you notice.

Just committed the files.

Cheers.

patrickd’s picture

Status: Needs review » Needs work

Okay, lets start

You are working in the "master" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.
(This has to be fixed before you can switch back to "needs review")

while waiting for an in-depht review of your module you can start out fixing some coding style issues detected by automated tools:
http://ventral.org/pareview/httpgitdrupalorgsandboxrutiolma1460352git

You can also get a review bonus and we will come back to your application sooner.

patrickd’s picture

Assigned: rutiolma » Unassigned

Please don't assign the issue to your self, only the current reviewer should do this.

aangel’s picture

Got a whole bunch of errors (x100) of this type:

warning: unlink(/var/www/vhosts/proequest.com/subdomains/beta/httpdocs/_/imagecache/horse/horse/a152/n1527/kyraphoto.jpg) [function.unlink]: Permission denied in /var/www/vhosts/proequest.com/subdomains/beta/httpdocs/sites/all/modules/contrib/imagecache/imagecache.module on line 785.
warning: rmdir(/var/www/vhosts/proequest.com/subdomains/beta/httpdocs/_/imagecache/horse/horse/a152/n1527) [function.rmdir]: Directory not empty in /var/www/vhosts/proequest.com/subdomains/beta/httpdocs/sites/all/modules/contrib/imagecache/imagecache.module on line 795.

Also, I left the name blank thinking the import would take it from the import snippet but it did not.

However, it did seem to import the preset.

klausi’s picture

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

Closing due to lack of activity. Feel free to reopen if you are still working on this application.