Currently when the progress is set to make the titles as links the only items which are links are to the left of the current page. It is in fact valid to link to any page which is less than the current order status. If my checkout process has 5 steps and I have made it to step 4, if I click on the link for step 2 then I can't directly get back to step 4, which I should be able to.

Patch to follow.

Comments

applicity_sam’s picture

Status:Active» Needs review
StatusFileSize
new3.8 KB

The attached patch makes the change in behaviour described. I have added a configuration option so that it will not change the behaviour for existing implementations. You will need to check the box in the module configuration in order for the new behaviour to be seen.

andrewbelcher’s picture

Issue summary:View changes
StatusFileSize
new1.3 KB

Rather than duplicating code, which could lead to similar issues, we switch to using commerce_checkout_page_access() which is specifically for this?

Patch attached.