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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

Hi @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?

quicksketch’s picture

Oh, 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.

Squiggles’s picture

Thanks, @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. :)

quicksketch’s picture

Hey 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+).

quicksketch’s picture

Title: Conditionals not deleting? » Conditionals not deleting when removed in RC2

  • Commit 060c211 on 7.x-4.x by quicksketch:
    Issue #2246863: Conditionals not entirely deleting when removed.
    
Squiggles’s picture

Got it. Will get back to you with my feedback. Thanks!

Squiggles’s picture

Confirming that it works. Thanks! :)

fenstrat’s picture

Version: 7.x-4.0-rc2 » 8.x-4.x-dev
Assigned: Unassigned » fenstrat
Status: Active » Patch (to be ported)

Needs porting to 8.x-4.x.

fenstrat’s picture

Version: 8.x-4.x-dev » 7.x-4.0-rc2
Assigned: fenstrat » Unassigned
Status: Patch (to be ported) » Fixed

Committed and pushed 060c211 to 8.x-4.x.

Thanks for the feedback @Squiggles, marking as Fixed.

  • Commit 59a2f21 on 8.x-4.x authored by quicksketch, committed by fenstrat:
    Issue #2246863 by quicksketch: Conditionals not entirely deleting when...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.