If I add a redirect rule using all the defaults then my redirect works, that is my settings are:
<front> - All triggers - All pages - All roles

If instead I select all the checkboxes for the "Redirect upon triggers" setting (which should have the same effect as leaving them all unchecked) then it works as expected.

Comments

mithy’s picture

mithy commented
Status: Active » Closed (works as designed)

If you do not select any triggers it has an effect of *none* selected. You have to specify triggers explicitly.

rooby’s picture

rooby commented
Title: Redirect doesn't work if you leave all settings at default » Don't allow the creation of rules that have no triggers
Status: Closed (works as designed) » Active

That's fine, but in that case the UI needs some tweaking.

When you are on the list page (admin/config/people/login-destination), if you have selected no triggers, it says 'All triggers'. It should say 'No triggers'.

And following on from this, it shouldn't let you save a rule that has no triggers selected, so some extra validation would be ideal. And there could potentially be an an extra bit in the description of that field that says 'You must select at least one trigger'.

mithy’s picture

mithy commented
Assigned: Unassigned » mithy

I do not remember why I designed it that way but I remember there was a reason. Anyway you are right: this is a flow in the design. I will fix it. I think, that selecting no triggers, may in the end have the effect of selecting all of them.

Thanks for bringing that up.

mithy’s picture

mithy commented
Status: Active » Fixed

fixed on dev.

rooby’s picture

rooby commented

Great thanks, I'll try it out.

Status: Fixed » Closed (fixed)

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