Drupal 7 en UberCart 3 uc_get_address overschrijven

[Afvalzak]

Voor een Drupal + UberCart project wil ik een aantal wijzigingen aanbrengen in de UberCart module.
Deze slaat op dit moment namelijk alle adresgegevens op in de ordertabellen (uc_orders) waardoor deze gegevens dus per order opgeslagen worden en telkens als een klant besteld zal de gehele uc_orders tabel doorzocht moeten worden naar eerdere orders.
Dit leek mij iets teveel van het goede dus heb ik een tabel aangemaakt uc_addresses waarin de adressen komen te staan en deze kunnen vervolgens gekoppeld worden via de order.

Nu heeft UberCart al een methode uc_get_addresses() welke alle addressen uit de ordertabel haalt en deze vervolgens in een combobox toont om de klant een keuze te maken (hij kan immers ook een nieuw adres toevoegen). Deze methode wil ik overschrijven in mijn eigen module, deze module gebruik ik ook voor andere tweaks aan de formulieren en werkt in principe prima.

Na een tijdje lezen/googlen heb ik met wat informatie over 'theme functions' de methode overschreven te krijgen. Echter wil dit niet lukken.

Mijn module heet: uc_fujicolor_store
Als ik een methode uc_fujicolor_store_get_addresses aan maak met hierin een 'echo' gebeurt er niets. OOk niet als ik de cache heb leeggehaald etc.

Ik heb dit ook op het UC forum geplaatst maar ik heb het gevoel dat ik daar weinig response ga krijgen (omdat het module gedeelte een beetje stil is )

Hebben jullie misschien ideeën / tips waardoor ik weer een stapje verder kan?

[Creepy]

Dus dan past iemand z'n adres aan en dan werkt dat ook door op z'n oude orders? Ik hoop dat ik je verkeerd begrijp..

[Afvalzak]

Nee, dat is niet de bedoeling de adresgegevens blijven nog in de order opgeslagen en getoond op het moment dat de order opgevraagd wordt.;)
Het gaat mij er meer om dat het zoeken van de adressen sneller gaat etc.

[kwaakvaak_v2]

volgens mij wordt die functie niet met een hook aangeroepen.

zie http://api.ubercart.me/ap...nction/uc_get_addresses/7

[Afvalzak]

kwaakvaak_v2 schreef op maandag 19 maart 2012 @ 11:47:
volgens mij wordt die functie niet met een hook aangeroepen.

zie http://api.ubercart.me/ap...nction/uc_get_addresses/7

Nee, daar was ik al bang voor, dat betekend dus dat het niet mogelijk is of is er een andere manier om een methode te overschrijven?
Zonder dat updaten van Drupal/UberCart problemen oplevert?

[RobIII]

Doe je 3 miljoen orders per maand? Nee? Lekker zo laten. "De hele orders tabel doorzoeken" valt dan heus wel mee en die energie kun je beter besteden aan andere zaken. Controleer liever of er een juiste index aanwezig is en zo ja, klaar.

Tenzij 't een bewezen (lees: meten == weten) bottleneck is in je webshop heb je hier niets te winnen met moeilijk doen met aparte tabellen, hooks en weet-ik-wat. En ik acht de kans dat daar je bottleneck zit nihil.

[ Voor 48% gewijzigd door RobIII op 19-03-2012 12:48 ]

[Afvalzak]

RobIII schreef op maandag 19 maart 2012 @ 12:42:
Doe je 3 miljoen orders per maand? Nee? Lekker zo laten. "De hele orders tabel doorzoeken" valt dan heus wel mee en die energie kun je beter besteden aan andere zaken. Controleer liever of er een juiste index aanweig is en zo ja, klaar.

Als dat het enige was ben ik het met je eens, maar dit is een verlengde van een andere functionaliteit, de klant wil namelijk zelf gebruikers aanmaken met adresgegevens, zodat de gebruiker dit zelf niet hoeft te doen en de gegevens overeen komen met ons interne order systeem.

Hierdoor is het volgens mij wel gewenst om een adrestabel te gebruiken i.p.v. de ordertabel.
Er zou immers altijd een order van de klant aangemaakt moeten worden om de adresgegevens op te slaan en dat lijkt mij een beetje krom.

[RobIII]

afvalzak schreef op maandag 19 maart 2012 @ 12:48:
maar dit is een verlengde van een andere functionaliteit, de klant wil namelijk zelf gebruikers aanmaken met adresgegevens, zodat de gebruiker dit zelf niet hoeft te doen en de gegevens overeen komen met ons interne order systeem.

Hierdoor is het volgens mij wel gewenst om een adrestabel te gebruiken i.p.v. de ordertabel.

Ja, maar nu fluit je dus een heel ander wijsje dan je in je topicstart deed Het is wel handig als je in een topicstart alle relevante informatie vermeldt en niet het halve probleem

[ Voor 13% gewijzigd door RobIII op 19-03-2012 12:50 ]

[Afvalzak]

RobIII schreef op maandag 19 maart 2012 @ 12:49:
[...]

Ja, maar nu fluit je dus een heel ander wijsje dan je in je topicstart deed Het is wel handig als je in een topicstart alle relevante informatie vermeldt en niet het halve probleem

Daar heb je gelijk in Maar ik wilde het probleem ook niet ingewikkelder doen lijken dan het is, aangezien ik 'simpelweg' (althans zo klinkt het ) een niet-hook functie overschrijven in Drupal.

[kwaakvaak_v2]

Dan moet de patch denk ik op een andere plek inbouwen.

Kijk eens naar http://drupal.org/project/uc_addresses

[Afvalzak]

Dankje! Had die module nog niet gezien, daar moet inderdaad wel wat bruikbaars in zitten.

-- edit:

Die module voegt al een hoop functionaliteit toe die ik zelf wilde gaan maken Volgende keer toch beter zoeken.

Het is zelfs zo dat een hoop functionaliteit uitgeschakeld kan worden in die module omdat ik het niet nodig heb.

Nogmaals bedankt, ik kan weer rustig verder rommelen met Drupal en friends

[ Voor 64% gewijzigd door Afvalzak op 19-03-2012 15:19 ]