Hi there,
Any chance of a D7 port of this module happening at any stage? I'm willing to assist if needed.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | multifileD7.patch | 1.38 KB | elgordogrande |
| #5 | webform_multifile_d7_hooks.patch | 4.12 KB | Anonymous (not verified) |
| #4 | webform_multifile_d7.patch | 684 bytes | Anonymous (not verified) |
| #2 | webform_multifile.zip | 94.87 KB | vzblk |
Comments
Comment #1
jrsinclair commentedComment #2
vzblk commentedI ported it to Drupal 7 for my project and it works but not good tested.
Comment #3
berryintlonline commentedAny chance that this module will be back ported to D6?
Comment #4
Anonymous (not verified) commentedI patched the "Required" validation. You just forgot to use "_fids", instead you left it "_fid".
Comment #5
Anonymous (not verified) commentedAdditional hooks are needed to handle download and private files and needed modification of delete upload file. See attached patch.
Comment #6
attiks commentedFYI: I created a full project for this, if any one wants to help feel free: http://drupal.org/project/webform_multifile
Comment #7
Ludwig commentedI applied the patch (and the other patch for D7) but I get this error
This occurs if I select Analysis in the Results
I'm not a php dude but I'm pretty sure I applied the patches correctly)
Sorry to be a pain but do you have any clues?
ENVIRONMENT NOTE: MacOS X (MAMP) and I am running SQLite
Comment #8
Ludwig commentedI'm also getting this error:
Notice: Undefined variable: suffix in theme_webform_render_multifile() (line 407 of /Applications/MAMP/htdocs/owlbooks/sites/all/modules/webform_multifile/multifile.inc).Comment #9
blasthaus commentedto fix the suffix error, just add
$suffix = '';around line 406 of multifile.inc
Comment #10
jelle_sThere is a drupal 7 dev version available for webform multifile at Webform Multiple File Upload's project page (will probably show op on the page within 12 hours or so, already available directly from git...)
Comment #12
elgordogrande commentedI had a few problems with the multifile.inc in the D7 port and made some changes. I'm uploading a patch for you to review. Please let me know your thoughts. It's my first patch and my first contribution, so I would appreciate any feedback.
Comment #13
attiks commented@gmakely: we createda full project at http://drupal.org/project/webform_multifile can you test your patch against the dev version (git clone --branch 7.x-1.x attiks@git.drupal.org:project/webform_multifile.git)?
PS: Your patch looks strange, did you use git diff?