Checkout progress indication

Haven't had a chance to give this a test drive yet, but it's on my to-do list. This is definitely desired functionality (Bojhan mentions it here as well), I just hadn't bothered trying to implement it yet.

So, just checked this out, and it's simple enough. One thought I had on the implementation, though, was what if instead of making the markup part of the form array it was implemented as a block that only appeared on checkout pages? This would make it a little more mobile and provide a simple place to add settings, like to control which pages show up in the progress indicator. For example, on my test install the "Payment" step is being shown, but the Payment page is really only an intermediate page for redirected payment methods and wouldn't necessarily be a part of the checkout process for everyone. On the same note, pages that don't have any checkout pages on them are showing up, too.

Last, I'm not sure what version of Commerce you're working against, but your hook_form_FORM_ID_alter() won't work as is (unless something's changed in D7 HEAD I don't know about) because the form ID for each page in the checkout process is unique. I only got it to show by changing that either to specifically reference a particular page's ID or making it a general hook_form_alter() with a broader check.

Also, we probably don't want *all pages* to display in the indicator, so it would probably be a good idea to add a $page->display_indicator property to the page definition.

If a page has $page->display_indicator = FALSE, it should not appear in the progress indicator. If the page is still displayed to the user, the indicator should be shown as if the previous page that is displayed in the progress indicator was active.

This probably needs to be brought up to speed with the latest development after 5 months of inactivity. Additionally, it's probably better served in contrib where it can mature and release faster than core would allow... not every e-commerce site needs a progress tracker after all.

@PieterDC: Nice effort with the checkout progressbar. It is almost what I was searching for.
There is one slight mistake though: you also should change: t($page->title) to t($page['title']) in commerce_checkout_progress_form_block_admin_configure_alter and theme_commerce_checkout_progress_list. :)

PS, $class = ($page_id == $current_page_id) ? 'active' : ''; in theme_commerce_checkout_progress_list works as well.

This is what I need ... but I get a

Notice : Trying to get property of non-object dans commerce_checkout_progress_form_block_admin_configure_alter() (ligne 47 dans /var/www/pbcosmetics/sites/all/modules/commerce_checkout_progress/commerce_checkout_progress.module).

When I want to tell the block, where to be shown there i a list with checkout's and the name "array" (I added the image).

Are you making a "real" module for this ?

Since Ryan has suggested it move to contrib I have created a sandbox and started fixing a few things. I'll promote it to a real project once we get it working with the current version of commerce. Please use the issue queue for any further issues related to commerce_checkout_progress. PieterDC, I've added you as a maintainer so you should have git push access if you want to help out.

Issue Queue: http://drupal.org/project/issues/1291268?categories=All
Project Page: http://drupal.org/sandbox/webmasterkai/1291268

Wow, thanks so much for taking this on, Kai. Good to see you in the queue. : D

Real project now - http://drupal.org/project/commerce_checkout_progress

Excellent. Thanks for following up here with it.