Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
I was testing the conditionals section of the webform, but I can't seem to delete conditionals that I posted. Is this part of its behaviour? Thanks.
Comment | File | Size | Author |
---|---|---|---|
#4 | webform_remove_conditional-2246863.patch | 3.07 KB | quicksketch |
Comments
Comment #1
quicksketchHi @Squiggles, the new RC2 release just came out an hour ago, are you sure you're already using the latest version? If you just upgraded, make sure you clear you caches. If you're already using the latest version, and clearing caches doesn't help, do you get JavaScript errors in your browser's console when clicking the "-" button to remove a condition?
Comment #2
quicksketchOh, now I think I see what you're describing. I can remove conditionals using the "-" button but then when I save, the conditionals come back. :(
This is definitely not expected behavior. I'll see if I can dig up what the problem is and we'll need to make a new followup release.
Comment #3
Squiggles CreditAttribution: Squiggles commentedThanks, @quicksketch. I was experimenting on this feature after getting RC2 installed. I came across this problem as well under Beta 3. I wanted to upgrade, but I needed to see first if it would break my current installation (don't want to redo 500+ form fields for my work :( ). Anyway, thanks for the good work. :)
Comment #4
quicksketchHey Squiggles, if you get the chance, could you try out this patch (or download the latest 4.x-dev)? It should fix the problem with removing conditionals, which was a side-effect of some performance-improvements we were trying in #2088249: Adding new conditions becomes slow after a large number of conditions (50+).
Comment #5
quicksketchComment #7
Squiggles CreditAttribution: Squiggles commentedGot it. Will get back to you with my feedback. Thanks!
Comment #8
Squiggles CreditAttribution: Squiggles commentedConfirming that it works. Thanks! :)
Comment #9
fenstratNeeds porting to 8.x-4.x.
Comment #10
fenstratCommitted and pushed 060c211 to 8.x-4.x.
Thanks for the feedback @Squiggles, marking as Fixed.