Conditionals reordering not working

I've noticed this problem too. Thanks for opening an official issue.

Attached patch fixes reordering not working.

1. Changes the weight feild from #type hidden to weight.
2. Moves weight column to last, because if it's first core's "Hide/Show row weights" doesn't work.
3. Changes the orderBy() clause on conditionals in hook_node_load() to be weight, not rgid.

Thanks @fenstrat! The code went through a few iterations where order was not relevant which is how we ended up in this situation. I'll review the patch when I get a chance.

No worries @quicksketch, look forward to the review.

Update to #2 which used the incorrect $rgid.

I've also noticed form validation "illegal choice detected" errors when there's more than 20 conditions (i.e. more than the default $delta of 10) when deleting many conditions at one time. Unsure of the best way to get around that, but the rest is working well.

I've also noticed form validation "illegal choice detected" errors when there's more than 20 conditions (i.e. more than the default $delta of 10) when deleting many conditions at one time.

That's probably caused by the number of elements influencing the available values in the weight field. If there were once 15 conditionals and one of the conditionals had a weight defined of "-12", the -12 option wouldn't exist any more if you removed 5 conditionals. So Drupal throws an error because the default value was set to something that doesn't exist.

I modified your patch slightly to keep track of the "max" number of conditionals that have been used and always uses that $delta for the weight field. Seems to pass all my testing but another look would be appreciated. In the mean time I've committed this to the 4.x branch. Thanks!

Yep, you're spot on with your reasoning for the "illegal choice detected" @quicksketch. Nice work around. Tested, works well.