Not being redirected to checkout after having logged in

What kind of account creation are you using ? Do you verify the user's email address or not ?

I have enough info, thanks... :-)

Now it's just a matter of finding the time to work on this...

Instead of messing with the session, why not just append a destination in your drupal_goto call:

/**
 * Implements hook_commerce_checkout_router()
 */
function commerce_checkout_redirect_commerce_checkout_router($order, $checkout_page) {
  global $user;
  
  if (!$user->uid) {
    drupal_set_message(t('You need to login or create a new account in order to validate your order'), 'status');
    drupal_goto('checkout/'.$order->order_id.'/user/register', array('query' => array('destination' => 'checkout')));
  }
}

@manatwo, your solution works perfectly if the user decides to register. If he logs in instead, the extra click on the "log in" tab loses the redirection fragment in the URL.

@manatwo Also note that this solution will not work in case an email is sent out to the user to verify his email address. This is why I used the session instead of the destination parameter.

Concerning this issue, it should be fixed with this commit:
http://drupalcode.org/project/commerce_checkout_redirect.git/commit/4e8f29d

Please test it and, if it works for you, change the issue to "reviewed & tested by the community". Thanks !

The problem is that user is not redirected when user email verification is enabled.

Here is my though, when have a user email verification is required redirect user to the checkout login form otherwise continue the checkout process.

Thanks a lot for your patch.
It's run well on my site.

Now i need one more functionality to redirect to "checkout/%/user/register/no-email" path.
i have add an menu item below

  $items['checkout/%/user/register/no-email'] = array(
    'title' => 'Create new account without an email',
    'page callback' => 'drupal_get_form',
    'page arguments' => array('user_register_form'),
    'access callback' => 'user_register_access',
    'type' => MENU_LOCAL_TASK,
  );

I have modified my registered form also, use

function mymodule_form_alter(&$form, $form_state, $form_id) {
	switch ($form_id) {
		case 'user_register_form':
			if (arg(0) == 'checkout' && arg(2) == 'user' && arg(3) == 'register' && arg(4) == 'no-email') {
			//....
			}
    	break;
	}
}

my problem is, what the function that i have to modified so i can redirect to path above ?
Any help will be very appreciated.

the patch from #11 was included in the patch from #1677106-5: Add user password recovery support..
it seems there's conflict applying both patches and both are needed for Commerce Kickstart.

Committed.

Confused as to what is committed. With latest Dev version dated Dec 19, the user is directed to home page after registering for a new account, NOT to checkout page as is the functionality when a current user logs in.

However when I enable Commerce Kickstart Checkout Redirect, the user is directed to checkout after registering for an account. Are both modules required for this or is the goal to have this functionality with only one module required?

This isn`t fixed yet? Can`t get this to work. When activating Checkout Redirect and Commerce Kickstart Redirect the notification Mail for registration is send twice times system message also shows up twice. Without the Commerce Kickstart Redirect Module activated I´m not redirected as nelslynn said. Need Help!

Reactivating this.

Upon new account creation with required email verification; user is directed to home page upon logging in not to checkout.

Please advise.

Thanks

The attached patch allows to perfom the redirect after the user has used the one time login and saved its password.

Oops, wrong version (actually the patch is rolled against the 2.x-dev branch).

There is a little bit of confusion here on what patch to apply. Moreover, I think the downloaded file commerce_checkout_redirect.module for the release 7.x-2.0-beta1 is not complete. It is missing many functions mentioned here. Anyway, I confirm that by applying patch #19 everything works now for me.

To clear up the confusion over which patch to apply, it's plach's

commerce_checkout_redirect-verify_email-1275330-19.patch

against 7.x-2.0-beta1. And use git to apply it.

There is the commit for the @plach solution (#19 patch) : http://drupalcode.org/project/commerce_checkout_redirect.git/commit/552d84f

so think we can close this issue.

I have updated Commerce Checkout Redirect to 7.x-2.0-beta1+7-dev (2014-Feb-28) but still no redirection takes place after having logged in or registered.

In dblog, the following PHP notice is logged after having logged in:

 Undefined index: #order in commerce_checkout_redirect_redirect_anonymous_submit() (line 196 of /etc/drupal7/all/modules/commerce_checkout_redirect/commerce_checkout_redirect.module).

The attached patch fixes the problem. It was introduced by commit 859506 in attempting to fix #1688400: Allow registration to be optional.

The #27 patch works for existing users that login. They will be redirected to their cart after having logged in.

However, it does still not work for new users. They will NOT be redirected to their cart after having registered. Using $GLOBALS['user']->uid instead of $form_state['uid'] for getting the user's UID avoids spitting a PHP notice on registering, so that should be better used, but it still doesn't redirect.

New patch as per comment #28. I think for the redirection problem on new registrations it would be easier to open an own issue.

indeed. it seems the issue takes place in Commerce Kickstart, but not in a clean Drupal Commerce.

The latest patch (#29) has the solution (which i removed within previous commit).
So RTBC.

#29 solution commited.
@roball: thank you Sir

Thank you very much vasike for committing! Now the most important feature of this module is working again in the latest 7.x-2.x-dev. FYI, I do not use Kickstart. The problem happened in a clean Drupal core installation.