When determining whether the user has access to previous checkout pages this module uses a simple 'every page before this page is allowed' algorithm rather than the more complex algorithm used in commerce_checkout_page_access which checks that there is a 'previous' or 'next' button chain between this page and the requested page. A patch to change
if ($visited){
to
if (commerce_checkout_access_page($page_id, $order)){
in theme_commerce_checkout_progress_list would use the commerce function. At the moment I override the theme function to get this functionality as I use a complex checkout process.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nvahalik’s picture

Title: Previous Page Access » Use commerce_checkout_page_access() to determine whether previous pages are accessible
Issue summary: View changes
Issue tags: +sprint
jcisio’s picture

Status: Active » Needs review
FileSize
1.13 KB

  • jcisio committed bde6d00 on 7.x-1.x
    Issue #1699610 by jcisio, nvahalik: Use commerce_checkout_page_access()...
jcisio’s picture

Status: Needs review » Fixed

Committed and pushed #2.

Status: Fixed » Closed (fixed)

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

jcisio’s picture

Version: 7.x-1.2 » 7.x-1.x-dev
Status: Closed (fixed) » Needs review
FileSize
883 bytes

The last patch has a problem with the cart page. This patch fixes it.

jcisio’s picture

Status: Needs review » Fixed

Committed and pushed.

  • jcisio committed 8d51251 on 7.x-1.x
    Issue #1699610 by jcisio: Use commerce_checkout_page_access() to...

Status: Fixed » Closed (fixed)

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