Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
Comment | File | Size | Author |
---|---|---|---|
language_negotiation.diff | 1.32 KB | jchatard | |
Comments
Comment #1
nicoloconte CreditAttribution: nicoloconte commentedOk I apply it!
Now everything is working, thank you very much!
Nicolo'
Comment #2
selinav CreditAttribution: selinav commentedOh great, thanks a lot, It's working!
Comment #3
mairav CreditAttribution: mairav commentedThanks! 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!
Comment #4
rszrama CreditAttribution: rszrama commentedWe 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?
Comment #5
mairav CreditAttribution: mairav commentedI 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.
Comment #6
longwaveuc_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.
Comment #7
mairav CreditAttribution: mairav commentedYes, you are right, with the rc7 this problem is solved!!! Thank you very much!!!!!
Comment #8
Island Usurper CreditAttribution: Island Usurper commentedGood to hear that it works fine now.