This module provides additional conditions for webform conditional functionality. It provides such new conditions as
'amount of checked items is less\more than'
'amount of checked items is'
'amount of checked items is 1 and it is'
https://www.drupal.org/sandbox/_kurt_/2599682
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/_KurT_/2599682.git
Webform have ability to build multistep forms and use conditions to show\hide fields according to previous user answers. I worked on such multi step form and found that it is hard to implement some not simple conditions and it will be much easier to do it if we will have more conditions.
I think this new conditions should be in a separate module because webform already have enough conditions to meet most of needs, but if you want something extraordinary - use additional module.
My reviews:
https://www.drupal.org/node/2607484#comment-10590188
https://www.drupal.org/node/2592703#comment-10605708
https://www.drupal.org/node/2598572#comment-10623258
Comment | File | Size | Author |
---|---|---|---|
#10 | http git.drupal.org sandbox _KurT_ 2599682.git pareview.sh_.png | 397.76 KB | cbuvaneswaran |
Comments
Comment #2
PA robot CreditAttribution: PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandbox_KurT_2599682git
Fixed the git clone URL in the issue summary for non-maintainer users.
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 #3
_KurT_ CreditAttribution: _KurT_ at FFW commentedComment #4
PA robot CreditAttribution: PA robot commentedFixed the git clone URL in the issue summary for non-maintainer users.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #5
_KurT_ CreditAttribution: _KurT_ at FFW commentedComment #6
_KurT_ CreditAttribution: _KurT_ at FFW commentedComment #7
_KurT_ CreditAttribution: _KurT_ at FFW commentedComment #8
_KurT_ CreditAttribution: _KurT_ at FFW commentedComment #9
_KurT_ CreditAttribution: _KurT_ at FFW commentedComment #10
cbuvaneswaran CreditAttribution: cbuvaneswaran as a volunteer commentedhttp://pareview.sh/pareview/httpgitdrupalorgsandboxkurt2599682git
PFA, Pareview report has lot of errors in webform_extra.js file. Please first fix these issues.
Thanks.
Comment #11
_KurT_ CreditAttribution: _KurT_ at FFW commentedFixed.
Comment #12
klausiComment #13
klausiproject page at https://www.drupal.org/sandbox/_kurt_/2599682 is empty, could you describe your module according to https://www.drupal.org/node/997024 ?
Comment #14
_KurT_ CreditAttribution: _KurT_ at FFW commentedDone.
Comment #15
klausiReview of the 7.x-1.x branch (commit 642af26):
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
manual review:
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 mpdonadio as he might have time to take a final look at this.
Comment #16
mpdonadioAutomated Review
Review of the 7.x-1.x branch (commit 642af26):
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
Manual Review
Typo in description in .info file
I think you can just implement HOOK_form_webform_client_form_alter() to target webforms, instead of a generic form alter
and checking the ID.
Your JS has an empty .ready(). Remove this eventualy (and in Drupal it should be an attach behavior anyway).
Nothing jumped out at me to block this application.
Comment #17
mpdonadioThanks for your contribution, _KurT_!
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.