Migrates imceimage cck fields to filefield cck fields. All referenced files will be copied to the drupal files directory, in a specified subdirectory.

See http://drupal.org/node/924400

You should prevent any content editing while this process is running.

You should NOT trust this module will migrate your data perfectly. If it fails you must restore your site from a backup. Test on a copy! Make backups!

Usage:

1. Install module in the normal way.
2. Grant "administer imceimage_2_filefield" permissions - /admin/user/permissions
3. Configure and run - admin/settings/imceimage_2_filefield

Comments

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Active » Postponed (maintainer needs more info)

Hello, tbfisher. You need to create a sandbox project, and report here its link.

brianfisher’s picture

brianfisher commented

thanks kiamlaluno,

http://drupal.org/sandbox/tbfisher/1078110

brianfisher’s picture

brianfisher commented
Status: Postponed (maintainer needs more info) » Active
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review

Setting to needs review

ralt’s picture

ralt commented
Status: Needs review » Needs work

Hello,

Here is a quick review :

  • Rename your README file into README.txt
  • Your .info file is definitely not completed. I don't even know if your module is for d6 or d7.

I wanted to review your whole module, but since I don't know which drupal version this module is for, I can't review your code.

Before you come back, have you tested your module with the Coder module?

brianfisher’s picture

brianfisher commented
Status: Needs work » Active

Thanks Ralt, I completed your suggestions.

ralt’s picture

ralt commented
Status: Active » Needs review
ccardea’s picture

ccardea commented
Component: new project application » module
Assigned: Unassigned » ccardea
Status: Needs review » Reviewed & tested by the community

I conducted the Code Review for this project.

Licensing: No problem.
Module Duplication: No problem.
Overall Understanding of Drupal API's: Makes good and appropriate use of batch, sql, & content API's. Did a good job of form validation, used error checking throughout.

Security:
CSRF - uses Forms API
SQL injection : no vulnerabilities found
XSS: no vulnerabilities found

Coding Style: A couple of items noted
Documentation: Needs improvement

See Sandbox Issue Queue

This is a utility module for imce cck image which has been abandoned in favor of the recommended Filefield Sources module. This project migrates imce image field data to filefield (not Filefield Sources). The project author and a few users report that it works without problems, and the code looks solid. Note that this project has been in the queue since March.

ccardea’s picture

ccardea commented
Status: Reviewed & tested by the community » Needs work

I haven't changed my opinion of this project, but since the imce image module is abandoned, I think we should explore the possibility of taking over maintainership of that project and add this to it.

BMorse’s picture

BMorse commented

Is there some advantage to imceimagefield over filefield that would prompt you to revive that module?

brianfisher’s picture

brianfisher commented

see http://drupal.org/node/1005548

I am not interested in maintaining imceimage.

ccardea’s picture

ccardea commented
Status: Needs work » Reviewed & tested by the community

Taking over an abandoned project is another way of getting full project status. Not to belabor the point, I think you did a good job on this project and I'm going to recommend full project status for you.

brianfisher’s picture

brianfisher commented

thanks ccardea. I've resolved the issues you added to the queue.

Anonymous’s picture

Anonymous (not verified) commented

I am a user of imceimage and I have been wanting to migrate to the filefield module in conjunction with filefield_sources. I ran this module in two environments today and everything worked perfectly.

Thanks for your hard work, I'm going to run this module on every site where I'm using imceimage so I have an upgrade path to D7.

sreynen’s picture

sreynen commented
Status: Reviewed & tested by the community » Needs work

Sorry, I'm moving this back to "needs work". I opened/reopened a couple issues in the project queue. If it was just the coding standards, I wouldn't be moving it back, but the other issue is a little more serious.

brianfisher’s picture

brianfisher commented

all issues have been resolved

sreynen’s picture

sreynen commented
Status: Needs work » Fixed

Hi tbfisher,

Thanks for your contribution and welcome to the community of project contributors on drupal.org.

I've granted you the git vetted user role which will let you promote this to a full project and also create new projects as either sandbox or "full" projects depending on which you feel is best.

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. As someone who has recently completed an application, your input would be especially useful in the code review group as we work to improve this process.

brianfisher’s picture

brianfisher commented

thanks sreynen.

this has been promoted to a full project, and can be downloaded here: http://drupal.org/project/imceimage_2_filefield

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.