CacheAdminA_trunk2010/vhTest/Fake/DOM/VKP/impl

LeverAdresRepository.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 15 Mar 21
Commits this week: 0
Total Lines of Code (LoC): 111
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 16 Mar 2021
5:04 pm

Tommy Hebb

4:53 pm

Tom Vermeulen

Monday 15 Mar 2021
5:04 pm

Tommy Hebb

[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

Options
    • -0
    • +7
    ./LeverAdresRepository.cls.xml
  2. … 5 more files in changeset.
Friday 12 Mar 2021
10:36 am

Tom Vermeulen

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)

Thursday 11 Mar 2021
12:35 pm

Tommy Hebb

[ICT-2246] [rvTVE] Correctie loggingniveau betreffende LeverAdressen op Klant en BezwaarGevonden in...
[ICT-2246] [rvTVE] Correctie loggingniveau betreffende LeverAdressen op Klant en BezwaarGevonden in...
Monday 26 Nov 2018
3:47 pm

Tommy Hebb

3:47 pm

Tommy Hebb

VRB583 voor aangemaakt.

VRB583 voor aangemaakt.

3:11 pm

Tom Vermeulen

3:08 pm

Tom Vermeulen

LergacyPartijApi gebruiken

LergacyPartijApi gebruiken

3:07 pm

Tom Vermeulen

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?)
Friday 16 Nov 2018
6:52 pm

Tommy Hebb

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

Tommy Hebb

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

- BestaatViaKlantIdEnKlantAdresNr toegevoegd

Options
    • -0
    • +7
    ./LeverAdresRepository.cls.xml
  2. … 2 more files in changeset.
Thursday 04 Oct 2018
11:21 am

Tommy Hebb

Wednesday 03 Oct 2018
11:35 am

Tom Vermeulen

9:41 am

Tom Vermeulen

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

Tuesday 02 Oct 2018
3:48 pm

Tommy Hebb

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

- Na review => GeefRecentsteUniekIdVolgensAxLeveradresId op LeverAdresService ipv op LeverAdresRepository

Options
    • -7
    • +0
    ./LeverAdresRepository.cls.xml
  2. … 7 more files in changeset.
1:17 pm

Tom Vermeulen

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!

10:42 am

Tommy Hebb

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

Tommy Hebb

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

- BestaatAXLeverAdresIdMeermaalsViaAXLeverAdresIdEnKlantNummer

- GeefRecentsteUniekIdVolgensAxLeveradresId

Options
    • -0
    • +14
    ./LeverAdresRepository.cls.xml
  2. … 2 more files in changeset.
Monday 05 Mar 2018
2:01 pm

Tom Vermeulen

[UST3680R] AxLocationId < 1000 moet ook werken

- reviewopmerkingske

Options
    • -2
    • +1
    ./LeverAdresRepository.cls.xml
  2. … 1 more file in changeset.
Tuesday 27 Feb 2018
12:27 pm

Tommy Hebb

9:10 am

Tom Vermeulen

9:09 am

Tom Vermeulen

properder om te dimmen buiten de while en dan een set in de while te gebruiken. In dat laatste geval wordt het object altijd overschreven i.p.v. een nieuw stuk memory aan te spreken

properder om te dimmen buiten de while en dan een set in de while te gebruiken. In dat laatste geval wordt het object altijd overschreven i.p.v. een nieuw stuk memory aan te spreken

Friday 23 Feb 2018
3:17 pm

Tommy Hebb

[UST3675] VHIP900: KlantSync: LeverAdressen die geen AxLocationId hebben worden niet verwijderd:
[UST3675] VHIP900: KlantSync: LeverAdressen die geen AxLocationId hebben worden niet verwijderd:
10:51 am

Tommy Hebb

[UST3675] VHIP900: KlantSync: LeverAdressen die geen AxLocationId hebben worden niet verwijderd:

- LeverAdresRepository => implementatie VerwijderLeverAdressenZonderAxLocationIdVanKlant

- LeverAdresRepository => Uitbreiding omschrijving exception met Toenaam, want de KlantSync van AX naar Cache zet mogelijks een deel van Naam in Toenaam (indien Naam te kort is voor wat AX aanreikt)

- AdresUtils => UniekId werd nog niet gezet bij het ophalen van een gepersisteerd adres (werkte enkel in memory indien in memory gezet op object)

Options
    • -0
    • +6
    ./LeverAdresRepository.cls.xml
  2. … 4 more files in changeset.
Wednesday 21 Feb 2018
1:46 pm

Tom Vermeulen

1:17 pm

Tom Vermeulen

Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/...

Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn )

Zal het er wel bijsteken voor de volledigheid

11:19 am

Tommy Hebb

11:10 am

Tommy Hebb

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

11:04 am

Tommy Hebb

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?