Summary

This has been a request by Vincenzo to become the module maintainer of
http://drupal.org/project/imceimage

Now donquixote asks the same (see below), as Vincenzo has moved to filefield sources and is no longer interested in this module.

New request by donquixote

I (donquixote) believe in the technical approach of this module, as a real alternative to filefield sources, but I don't want to rely on an unmaintained module.

Things I plan to do:
- Improve code quality (I already did this locally, could provide a patch, if this is really desired).
- Fix any problems I run into.
- Be responsive on the issue queue.
- Slowly work through older issues.
- Allow a co-maintainer to jump on the boat, if someone is interested.
- Recent work will go into a new branch (6.x-2.x).
- Some day, produce a D7 version (but preferably if someone else is going to help)

There are no particular bugs I want to fix (yet), I just don't want this module to be abandoned.

Original message by Vincenzo

Hello.
I've been working on this module for a project I am working on. Given my requirements, it was quite incomplete (missing Title attribute support) and/or buggy from a usability point of view. Plus, there were some other bugs (the _empty hook wasn't working properly, causing the fields instances in the edit form to indefinitely increase in number every time you deleted an image from a content).

I am sorry I am not able to provide different patches for each one of things I mention in the title, but I've made the changes all at once and this is the result.

I'd be glad if you could review the code and let me know whether you reckon it can be integrated or not in the main branch.

Next things I'd like to implement: settings to decide whether or not to show Title and Alt fields in the Edit forms, and to decide if they should be a required part of the field or not.

Please, bear in mind that before you apply the patch, you are required to re-organise the locations of the files within the module directory, as follows

.
./imceimage.info
./imceimage.module
./includes
./includes/images
./includes/images/1x1.png
./includes/images/add-image.jpg
./includes/images/warning.png
./includes/js
./includes/js/imceimage.js
./LICENSE.txt
./translations
./translations/es.po
./translations/hu.po

Also, the Tarball attached includes a new .install file for the module as well.

Thanks,

Vincenzo

CommentFileSizeAuthor
#1 imceimage-patch.tar_.gz10 KBVincenzo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Vincenzo’s picture

FileSize
10 KB
Vincenzo’s picture

Title: Support for title other than alt text, improved usability, various bugfixes » Offering to maintain Imce CCK Image
Category: feature » support

Changing this to a support request for taking over the maintenance of this module.
I've had a private email exchange with the original author of the module and he told me he's not developing the module anymore and he was willing to give me access to the module development. However, I know this must be done publicly, so here we go.

In the previous comment I've also attached a patch which proves that I am actually working on the module.

Cheers,

Vincenzo

panis’s picture

agreed.. with transfer of maintenance over to Vincenzo

Vincenzo’s picture

Thanks panis. Appreciated.

apaderno’s picture

Assigned: Unassigned » apaderno
Status: Needs review » Fixed

I approved Vincenzo's application, and passed him the ownership of the project.

A note about LICENSE.txt: license files are not allowed to be committed in CVS, as all the committed code is under the same license of Drupal.

Vincenzo’s picture

Great. Thanks for the advice about licence files.

Status: Fixed » Closed (fixed)

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

donquixote’s picture

Status: Closed (fixed) » Needs work

Hi Vicenzo,
are you still maintaining the module, and working on it?
I see your commit went into the dev branch, but the latest official release is quite a while ago.
If you are not sure about your commit, you could make this a new branch (6.x-2.x), so people can still choose to use the old branch if they want.

I would offer my help as a co-maintainer, if you are interested.
See #935518-4: Why Deprecated? Please search for new maintainer.

Vincenzo’s picture

Status: Needs work » Needs review

Hello.

I am happy for donquixote to take full maintainership of the imceimage module, as I am no longer interested in developing it. Please, go ahead with this as per standard procedure.

Cheers,

V

donquixote’s picture

Project: Imce CCK Image » Drupal.org site moderators
Version: 6.x-1.x-dev »
Component: Code » Project ownership
Assigned: apaderno » Unassigned

Moving this to webmasters queue, as suggested on
http://drupal.org/node/251466

donquixote’s picture

Project: Drupal.org site moderators » Imce CCK Image
Issue summary: View changes

Explain new maintainer request

apaderno’s picture

Version: » 6.x-1.x-dev
Component: Project ownership » Code
Assigned: Unassigned » apaderno
apaderno’s picture

Status: Needs review » Fixed

I passed the ownership of the module to donquixote.

Remember to change the project page, as the project is reported to be abandoned, and deprecated.

donquixote’s picture

Thanks!

donquixote’s picture

Status: Fixed » Needs work

Oh, very nice.
It appears that I still don't have push access on git yet.
Something I need to do?

$ git push origin 6.x-2.x
donquixote@git.drupal.org's password: 
fatal: remote error: Permission denied when accessing 'imceimage' as user 'donquixote'
donquixote’s picture

Project: Imce CCK Image » Drupal.org site moderators
Version: 6.x-1.x-dev »
Component: Code » Project ownership

Let the webmasters fix this..

donquixote’s picture

Project: Drupal.org site moderators » Imce CCK Image
Version: » 6.x-1.x-dev
Component: Project ownership » Miscellaneous
Status: Needs work » Closed (fixed)

git clone again, now I have access.
Closing it.

apaderno’s picture

Status: Closed (fixed) » Fixed

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

sub-headline "New request by donquixote"