LeverAdresRepository.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2574] - Telefoon nummer / provincie toevoegen in Leveradres

- bijkomende definities / convertors

- bijgbehorende testen

  1. … 12 more files in changeset.
[ICT-2246] [rvTVE] Correctie loggingniveau betreffende LeverAdressen op Klant en BezwaarGevonden in ProductUpdater:

- Refactor na review => Maakt nu gebruik van LeverAdresRepository met query-functionaliteit en bijhorende TestCase

  1. … 5 more files in changeset.
ik zou onderstaande private method op de DOM.VKP.impl.LeverAdresRepository zetten. Dan kan ie hier proper uitgemocked worden. Die repo bevat ook een hoop testen die gemakkelijk aangepast kunnen wor...

ik zou onderstaande private method op de DOM.VKP.impl.LeverAdresRepository zetten. Dan kan ie hier proper uitgemocked worden. Die repo bevat ook een hoop testen die gemakkelijk aangepast kunnen worden om onderstaand gedrag te testen.

't is wel zo dat die Repository geen pure repo is, maar ook functionaliteit heeft die eigenlijk in een bovenliggende service zouden moeten zitten (maar dat is een ander verhaal)

Misschien kan de onderstaande functionaliteit dan ook vervangen worden door iets als in BestaatViaAXLeverAdresIdEnKlantNummer, maar dan met de query SELECT count(ID) from Derde_Klant.LevAdres where Klant = ? (zonder de Ax-dingen)

[ICT-2246] [rvTVE] Correctie loggingniveau betreffende LeverAdressen op Klant en BezwaarGevonden in...
[ICT-2246] [rvTVE] Correctie loggingniveau betreffende LeverAdressen op Klant en BezwaarGevonden in...
[UST4188] DPT1: Vh4: MP: Du-Pont: rechtstr. transp.: leveradres overnemen op doorbestelde winkelkar/offerte...
[UST4188] DPT1: Vh4: MP: Du-Pont: rechtstr. transp.: leveradres overnemen op doorbestelde winkelkar/offerte...
[UST4188] DPT1: Vh4: MP: Du-Pont: rechtstr. transp.: leveradres overnemen op doorbestelde winkelkar/offerte

- LeveradresRepository uitgebreid want kan niet zoeken met enkel AxLocationId

  1. … 2 more files in changeset.
VRB583 voor aangemaakt.

VRB583 voor aangemaakt.

[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- LegacyPartijApi gebruiken om klantnummer te pakken

  1. … 1 more file in changeset.
LergacyPartijApi gebruiken

LergacyPartijApi gebruiken

2 opmerkingskes: *geen testjes voor deze method *deze werking laat gaten toe, als ik het goe gezien heb -> problemen voor HoutenLaden (met veeeeeeel leveradressen?)

2 opmerkingskes:

  • geen testjes voor deze method
  • deze werking laat gaten toe, als ik het goe gezien heb -> problemen voor HoutenLaden (met veeeeeeel leveradressen?)
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- BestaatViaKlantIdEnKlantAdresNr toegevoegd

  1. … 2 more files in changeset.
ik zou nog kunnen beginnen neuten over de casing van adres/Adres. Het probleem is dat het in feite mer kleine a moet, want het is een repo van Leveradressen. Het is wel zo dat historisch gezien er ...

ik zou nog kunnen beginnen neuten over de casing van adres/Adres. Het probleem is dat het in feite mer kleine a moet, want het is een repo van Leveradressen. Het is wel zo dat historisch gezien er altijd (foutief) met een grote A geschreven werd (zie andere lijntjes in deze klasse). Doe er mee wat ge wilt

[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:

- Na review => GeefRecentsteUniekIdVolgensAxLeveradresId op LeverAdresService ipv op LeverAdresRepository

  1. … 7 more files in changeset.
Eigenlijk hoort dit niet in de repository. Een repository moet enkel bewaren, geven zoeken enz en krijgt/geeft een object van het type waarover de repository het heeft. Dit hoort eerder in een Leve...

Eigenlijk hoort dit niet in de repository. Een repository moet enkel bewaren, geven zoeken enz en krijgt/geeft een object van het type waarover de repository het heeft.
Dit hoort eerder in een LeveradresService. Pas op, ik wil hier niet beweren dat dat momenteel altijd en overal zo te goei gedaan wordt!

[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:
[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:
[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:

- BestaatAXLeverAdresIdMeermaalsViaAXLeverAdresIdEnKlantNummer

- GeefRecentsteUniekIdVolgensAxLeveradresId

  1. … 2 more files in changeset.
en aangepast http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

en aangepast

opmerkingen via slack gedaan

opmerkingen via slack gedaan

[UST3705] VHIP900 KlantSync: UniekId behouden bij updaten van adressen, ipv telkens een nieuw te geven:...
[UST3705] VHIP900 KlantSync: UniekId behouden bij updaten van adressen, ipv telkens een nieuw te geven:...