Closed (fixed)
Project:
Ubercart Addresses
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Reporter:
Created:
25 Jul 2008 at 20:59 UTC
Updated:
29 Aug 2008 at 03:02 UTC
Is there a way (or a setting) or something that could be added so that users can create accounts without having to add their address info right away? Of course they need it to check out, but not necessarily to browse the site as an authenticated user.
Thanks for a great module!
Comments
Comment #1
freixas commentedIt seems like this would be fairly easy. The whole system had to be designed to work with and without addresses, so it's ready that way. I always wonder about adding more and more options, though. Does anyone else need this?
If you're not a programmer and need a quick fix, look in uc_addresses.module for "function uc_addresses_user(". The scan down for "case 'register':". Delete from "'case 'register':" to "return $form" (seven lines). That should do it.
If you're a programmer, you could always submit a patch.
If anyone else wants this, let me know.
Comment #2
goodeit commentedThat's true about options... too many and you risk confusing people and cluttering the interface. And while I am a programmer (and would love to be making patches for small bugs that I find), I have not ever found how to make patches easily in windows (e.g. without cygwin).
Thanks for the help!
EDIT: of course right after posting this, my download of UnixUtils actually goes through this time, so I'm off to play with it's diff program...
Comment #3
freixas commentedTry WinMerge (free software). Under the Tools menu it has an option called Generate Patch...
Comment #4
freixas commentedOk, this was so easy I added it. If I did the check-in properly, it should be available in the next 5.x-1.x-dev build.
I did only some minimal testing. Could you let me know if you run into any problems when users register without entering an address?
Comment #5
impudicus commentedFirstly, thanks for your work on this module. User profiles addresses is greatly needed.
Not wanting to have mandatory address input as part of the user registration, I have downloaded and installed the 5.x-1.x-dev version.
Here is the problem:
At the checkout page, none of the mandatory fields are no longer mandatory.
If I choose to manually enter my address information and neglect to enter any required fields the order is still processed and this less-than-complete address is sent to my payment gateway and added to my profile.
Any means of correcting this ?
Thanks for your time and efforts
-Imp
Comment #6
impudicus commentedComment #7
freixas commentedI reproduced the problem. Bizarre! Definitely needs to be fixed.
Comment #8
freixas commentedThanks for spotting this! This bug has been present since May and no one (including myself) noticed!
It was not related to the change I made for you. The bug occurred when the delivery or billing address was filled in automatically. The code I used re-created the entire form field (including all attributes such as required/not required) rather than just overriding the field's value. Unfortunately, I set the #required attribute to FALSE.
I've changed the code to only change the #default_value.
Please try it out and report back. You can check this out from CVS or wait a day for the dev version to get updated.
Comment #9
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.