The webform random fieldset component works but the fields inside are not randomized. The attached patch addresses this issue. Please review when you get a chance. Limitation: This works on the top level only. In other words, you cannot nest random fieldsets within other random fieldsets. The nested fieldset fields will not be randomized. Also, the attached webform_random_fieldset submodule patch needs committed in the repo.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | webform_random_coding_standards-2906065.patch | 10.42 KB | uberhacker |
Comments
Comment #2
uberhacker commentedComment #4
uberhacker commentedComment #5
biigniick commentedthanks
Comment #6
biigniick commentedthe new dev version should have new functionality. give it a look. thanks in advance.
- nick
Comment #7
uberhacker commentedI tested the module and it works great. Both top-level and fieldset-level elements are randomized. Attached is a patch to correct issues with Drupal coding standards and best practices. I fixed some grammar issues as well. You might also consider adding a hook_help function to assist new users with configuration. Some Simpletests wouldn't be a bad idea either. Disregard this patch and use #8 below instead.
Comment #8
uberhacker commentedI removed the pointless db_like function that wasn't working and replaced it with something that actually works when you uninstall the modules.
Comment #9
biigniick commentedgreat work, @uberhacker!
everything looks good to me.
Comment #10
biigniick commented