Dev Guys,

from the "lets remove everything that is not eC core" series, i propose to remove eC address module and adopt a better suited module for managing addresses, like Addresses or Location modules. Addresses module might be better suited because it has both "address book" and "single address per user" features.

Advantages:
* Reducing the eC module size
* Remove all country and states lists from store.localisation.inc
* Less trouble

best regards,

massa

Comments

gordon’s picture

gordon commented

Maybe, but we do need to provide a place to store registered customers addresses.

If the chooses are between Location and addresses then I will be picking addresses as I feel location is too big and is sometimes very messy.

I need to look more at addresses but so far I think it is ok. But if we are to do this then we would need to clude the customer api in the addresses module so it would be in there stable release. I don't think it will be a big issue as the customer interface only need the hook_customer_info() in the addresses module and then the rest can be put into an include.

We would have to also write a conversion routine to go from address to addresses.

gordon’s picture

gordon commented

I have looked at addresses more and I like it. I does seem to look at ec_address evolved, and could work pretty much the same way.

The thing that really needs to be done before we could get rid of ec_address the addresses module need some work done on it so that it will have much better support for the customer names.

I am really quite excited about doing this.

brmassa’s picture

brmassa commented
Assigned: Unassigned » brmassa

Gordon,

great! i will start some work on this quite soon.

Tasks:
1* Make Addresses module changes in order to make it easier for external integration
2* Make ec_addresses Include custom names on addresses.
3* hook_update from previous ec_addresses data to Addresses
4* ec_addresses - Addresses module integration itself (checkout api)
5* Removing some ec_store.localisation.inc data that is duplicated on Addresses module.

regards,

massa

gordon’s picture

gordon commented

Just a quick note, addresses will not use the checkout api, but the customer api so that it can allow the ec_customer to get all the information about the customer.

Gordon.

gordon’s picture

gordon commented
Status: Active » Postponed

ATM I like this idea, but I am going to put it on hold ATM and revisit in D7

gordon’s picture

gordon commented
Version: 6.x-4.x-dev » master