Hello,

When I uncheck 'Allow anonymous checkout' in Account checkout pane. Anonymous user do not see 'Checkout as guest' button.

But when user is logged in and make order from start 'Continue to next step' button is not accessible. Pane only shows cancel link.

Thanks,
Rahul Lamkhade

Comments

rahullamkhade created an issue. See original summary.

legolasbo’s picture

Parent issue: » #2580227: [META] 2.0 release

Adding this to the roadmap for 7.x-2.x

laryn’s picture

StatusFileSize
new581 bytes

This is my fix for the issue. Basically logged in users got stranded on the "Account info" pane -- I just got rid of the "Account info" pane if a user is logged in. Maybe helpful to someone else.

kcannick’s picture

Wow... This should be committed. Customers were unable to check out for 4 days and I spent an extra day thinking I misconfigured something.

arnaud.dabouis’s picture

This patch also removes ability to configure the checkout steps.

legolasbo’s picture

Status: Active » Needs work

This patch is not the correct solution. A better solution 'to get rid of the account pane' would be to just return an empty form if the user is logged in. Still better would be to just show the account details instead of a form.

legolasbo’s picture

Issue tags: +Needs tests

Also, I would like to see automated tests for this.

arnaud.dabouis’s picture

StatusFileSize
new1.5 KB

This patch fixes the issue. Tell me if it breaks anything.

arnaud.dabouis’s picture

Status: Needs work » Needs review
legolasbo’s picture

Status: Needs review » Needs work

Doesn't break any existing tests, but the new functionality is not covered by the existing tests. Please add test coverage for this.

caschbre’s picture

Question... is there a reason to even show this checkout page if the user is already logged in and not simply redirect them to the next checkout page?

spidersilk’s picture

Dealing with this issue too. Caschbre's point is a good one - why even have the account info page, rather than taking them directly to the checkout process as you'd expect?

spidersilk’s picture

The patch in #8 worked for me!

jaguero’s picture

#8 works, but hides the configuration form in admin area.

Old:
if (user_is_logged_in()) {

New:
if (user_is_logged_in() && !path_is_admin(current_path())) {

mrwconnected’s picture

Agree, this is not a great experience for the user. Why not just direct them to checkout?

matthiasm11’s picture

StatusFileSize
new1.53 KB
new105.67 KB

I think we should not redirect the user, since we are going through the checkout progress step by step.
The account information page confirms to the user which account he is using at the moment.

account-information

Updatet the patch from #8 with the comment from #14.

matthiasm11’s picture

Status: Needs work » Needs review
solideogloria’s picture

Priority: Critical » Major

"Critical" means data is corrupted. Changing to Major.