CacheAdminA_trunk2010/vhUnitTest/AXimpl/Dynamics/CustomerIn/Converter/LeverAdresUtils

ZetLeverAdressen

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 12 Feb 18
Commits this week: 0
Total Lines of Code (LoC): 586
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
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?)
12:21 pm

Tommy Hebb

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

- UniekId ipv KlantAdresNr meegeven aan VerwijderViaID

Options
    • -1
    • +2
    ./Test.cls.xml
  2. … 1 more file in changeset.
11:46 am

Tommy Hebb

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

- Opkuisen BestaandLeverAdres op oude L-nr vooraleer op L1 te zetten

Options
    • -0
    • +1
    ./Test.cls.xml
  2. … 1 more file in changeset.
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:
6:47 pm

Tommy Hebb

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

- KlantId als klasse-parameter

Options
    • -5
    • +9
    ./Test.cls.xml
6:43 pm

Tommy Hebb

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

- Verduidelijking in naamgeving vraiabelen => NieuwPrimaryDlvAddrAdres en BestaandPrimaryDlvAddrAdres en AnderBestaandAdres

- Nog een vergeten assertje

Options
    • -24
    • +25
    ./Test.cls.xml
6:36 pm

Tommy Hebb

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

- Adres is PrimaryDlvAddr en is bekend in cache maar staat niet op L 1 en ander adres staat op L 1 => ander adres verzetten naar nieuw KlantAdresNr en PrimaryDlvAddr verzetten naar L 1

Options
    • -12
    • +55
    ./Test.cls.xml
5:45 pm

Tommy Hebb

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

- Adres is PrimaryDlvAddr en is bekend in cache en staat op L 1 => Bewaar

Options
    • -2
    • +36
    ./Test.cls.xml
4:59 pm

Tommy Hebb

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

- Adres is PrimaryDlvAddr en is nog onbekend in cache en L 1 is bezet => L 1 veplaatsen naar nieuw KlantAdresNr en this adres op L 1 zetten

Options
    • -3
    • +50
    ./Test.cls.xml
  2. … 1 more file in changeset.
4:20 pm

Tommy Hebb

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

- Adres is PrimaryDlvAddr en is nog onbekend in cache en L 1 is nog vrij

Options
    • -1
    • +36
    ./Test.cls.xml
  2. … 1 more file in changeset.
Thursday 15 Nov 2018
4:55 pm

Tommy Hebb

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

- LeverAdresUtils => ZetLeverAdressen => Voorafgaandelijke refactor ter bevordering van leesbaarheid

- DataMKlantNative => Afzondering GeefVolgendVrijKlantAdresNr => nodig voor verdere implementatie van deze story => om onterecht op L 1 staand adres te kunnen verzetten en daarna bewaar op repo te doen

Options
    • -1
    • +1
    ./Test.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
    • -2
    • +5
    ./Test.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!

11:33 am

Tommy Hebb

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

- Aanpassen testje => VerwachtMethodCall met ExactAantalKeer geeft meer duidelijkheid en zekerheid

Options
    • -4
    • +4
    ./Test.cls.xml
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:
10:17 am

Tommy Hebb

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

- Cache Leveradressen na Syncen => VerwijderViaID na GeefRecentsteUniekIdVolgensAxLeveradresId indien BestaatAXLeverAdresIdMeermaalsViaAXLeverAdresIdEnKlantNummer

Options
    • -16
    • +67
    ./Test.cls.xml
  2. … 1 more file 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?

11:04 am

Tommy Hebb

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

11:01 am

Tommy Hebb

KlantId of KlantNummer?

KlantId of KlantNummer?