New fields get added with the content_add_more_js() function (callback is content/js_add_more); filefield_js() (callback is filefield/ahah) is used to handle file uploads. Use this workflow instead of the custom one we currently have. Also check out content_multigroup_add_more_js() (callback is content_multigroup/js_add_more)

CommentFileSizeAuthor
#3 imagefield_zip-1281916-3.patch3.29 KBmikeytown2
#2 imagefield_zip-1281916-2.patch16.07 KBmikeytown2
#1 imagefield_zip-1281916-1.patch10.84 KBmikeytown2

Comments

mikeytown2’s picture

mikeytown2 commented
Issue summary: View changes

Updated issue summary.

mikeytown2’s picture

mikeytown2 commented
StatusFileSize
new imagefield_zip-1281916-1.patch10.84 KB

First step is to get this working 50% of the time... patch here does that & its been committed.

mikeytown2’s picture

mikeytown2 commented
Status: Active » Fixed
StatusFileSize
new imagefield_zip-1281916-2.patch16.07 KB

100% with this patch

mikeytown2’s picture

mikeytown2 commented
StatusFileSize
new imagefield_zip-1281916-3.patch3.29 KB

One more patch was just committed.

Status: Fixed » Closed (fixed)

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

barteque’s picture

barteque commented

Hi !
Do I need to apply all of those 3 patches or maybe there is any complete version of the module with those patches?

mikeytown2’s picture

mikeytown2 commented

6.x-1.2 or 6.x-1.x-dev is what you're looking for.
http://drupal.org/project/imagefield_zip#downloads

If you look at the commit logs these patches were committed a long time ago
http://drupalcode.org/project/imagefield_zip.git/shortlog
Look near the bottom of the page for #1281916 (this issues node id). Since that time, the module has undergone a lot of changes. If you look at the comments in this issue I state that the patches have been committed.

Using git blame and some investigative research http://drupalcode.org/project/imagefield_zip.git/blame/HEAD:/imagefield_... I found the issue that refactors the code from this patch #556704: Submit twice or die

@barteque
What makes you think you need to apply these patches?

mikeytown2’s picture

mikeytown2 commented
Issue summary: View changes

Updated issue summary.