On a settled order (confirmed, completed or cancelled) the order summary still shows a "Change seats" link on each seated event. Re-seating only moves a visitor's held seats through their basket, so the link is a dead end once the order has left it, and on an order page it acts on the current visitor's own cart rather than the order shown.

The summary's event alter hook received only the slot, so seating could not tell what state the order was in. This passes the transaction to hook_yoyaku_summary_event_alter() too, and yoyaku_seating offers the link only while the order is pending. Kernel coverage checks the link shows for a pending basket and is gone once the order is completed.

Follow-up to #3612364.

Issue fork yoyaku-3612555

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

mably created an issue. See original summary.

mably opened merge request !70

  • mably committed ecbb66bd on 1.x 
    fix: #3612555 Hide the 'Change seats' link on a settled order: re-...
mably’s picture

mably commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.