Hi,
uc_cart_form_alter() should be aware of languages prefix, this is causing such issues #581116: Prefix and login, add a user and http://drupal.org/node/561102.

Here comes a small patch which fixes the problem. Please test.

Thank you!
Jérémy

CommentFileSizeAuthor
language_negotiation.diff1.32 KBjchatard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nicoloconte’s picture

Ok I apply it!
Now everything is working, thank you very much!

Nicolo'

selinav’s picture

Oh great, thanks a lot, It's working!

mairav’s picture

Thanks! It's fixed now!!! Do you know if this will be applyed to next ubercart release? do you know when will this be?
Thanks again!

rszrama’s picture

Priority: Critical » Normal

We can target it for the 2.0 release if we can get it reviewed and tested in the next two weeks. What must be done to test this patch?

mairav’s picture

I don't know what do you mean, or how do you want to test this, but for me the patch is working.

The failure happens when you have a multilanguage site with prefix. For example, my default language is spanish and english is the second language that is accessed by entering to www.thedomain.com/en

When you try to create a new account for an user in the second language, the "create new account" button has double "en" and tries to take you to /en/en/user/register givving a 404 error.

This happens in sites with ubercart, but another user has this problem without ubercart. Maybe another module is affecting the register form.

I hope you can add this to the next release.

longwave’s picture

uc_cart_form_alter() has already been rewritten since rc6 in #561210: uc_cart_form_alter() is ugly and broken, so this patch no longer applies. Please re-test with rc7, where this issue should be fixed.

mairav’s picture

Yes, you are right, with the rc7 this problem is solved!!! Thank you very much!!!!!

Island Usurper’s picture

Status: Needs review » Closed (duplicate)

Good to hear that it works fine now.