Extention for Webform module - Webform Multiple File. Module provides new webform component, that allows users to upload multiple files. I know, that exist Webform Multiple File Upload, but the goal of my module - it works in Drupal way without JS and external libraries. Design of my module similar to file field widget, that allows correct validation and file processing.
Link to the project: https://drupal.org/sandbox/chalk/2151949
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/Chalk/2151949.git webform_multiple_file
I know that my module requires applying patch for Webform module, but I have already create feature request #2153723: Form_state values in _webform_render_component implementations to solve this task.
Reviews of other projects:
https://drupal.org/comment/8394291#comment-8394291
https://drupal.org/comment/8394437#comment-8394437
https://drupal.org/comment/8394545#comment-8394545
Comments
Comment #1
PA robot CreditAttribution: PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxChalk2151949git
We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #2
Chalk CreditAttribution: Chalk commentedComment #3
Chalk CreditAttribution: Chalk commentedI've fixed all syntax errors in code. See please - http://pareview.sh/pareview/httpgitdrupalorgsandboxchalk2151949git-7x-1x
About function naming: it is not possible to rename these functions - see webform_component_invoke() in webform.module file.
About form_state: in case of my module should be 'input' key.
Comment #4
Chalk CreditAttribution: Chalk commentedComment #5
Chalk CreditAttribution: Chalk commentedComment #6
aroq CreditAttribution: aroq commentedCopy-pasted function comments, like:
/**
* Implements _webform_theme_component().
*/
function _webform_theme_multiple_file() {
/**
* Implements _webform_edit_component().
*/
function _webform_edit_multiple_file($component) {
and other functions.
Comment #7
aroq CreditAttribution: aroq commentedComment #8
Chalk CreditAttribution: Chalk commented@aroq I wrote comments for functions like it was in webform module.
But OK - comments were changed according to your remark.
Comment #9
Chalk CreditAttribution: Chalk commentedComment #10
amreana CreditAttribution: amreana commentedManual review:
Upon uploading a file i get this error:
Warning: Illegal string offset 'fid' in webform_multiple_file_managed_file_submit() (line 531 of /var/www/mysite/sites/all/modules/webform_multiple_file/webform_multiple_file.webform.inc).
Comment #11
Chalk CreditAttribution: Chalk commented@amreana
Describe please your modules environment, version of Drupal and Webform module. Did you apply patch? I haven't seen this warning before. Module successfully works at more than 10 different sites
Comment #12
Chalk CreditAttribution: Chalk commentedThere is no answer from @amreana
Comment #13
Nitesh Sethia CreditAttribution: Nitesh Sethia commented@Chalk
I have tested your module and found that the code does not follow drupal standards.
All functions should be prefixed with your module name to avoid name clashes.
For your convenience, have a look at the issues pointed out in the link.
http://pareview.sh/pareview/httpgitdrupalorgsandboxchalk2151949git
Comment #14
Nitesh Sethia CreditAttribution: Nitesh Sethia commentedComment #15
Chalk CreditAttribution: Chalk commented@Nitesh Sethia
I have written in https://drupal.org/comment/8260925#comment-8260925 that function naming is defined by Webform module
Comment #16
Chalk CreditAttribution: Chalk commentedComment #17
Chalk CreditAttribution: Chalk commentedReview bonus
Comment #18
klausimanual review:
But otherwise looks RTBC to me. Removing review bonus tag, you can add it again if you have done another 3 reviews of other projects.
Assigning to patrickd as he might have time to take a final look at this.
Comment #19
Chalk CreditAttribution: Chalk commented@klausi, thanks for review!
Comment #20
klausino objections for more than a week, so ...
Thanks for your contribution, Chalk!
I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!
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. I encourage you to learn more about that process and join the group of reviewers.
Thanks to the dedicated reviewer(s) as well.