CacheAdminA_trunk2010/BL/Legacy/TRANSP

OrderOpBonBepaler.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 20 Aug 19
Commits this week: 0
Total Lines of Code (LoC): 0
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 27 Aug 2019
2:58 pm

Kurt Callebaut

2:58 pm

Kurt Callebaut

terecht, zal er volgende keer rekening mee houden.

terecht, zal er volgende keer rekening mee houden.

1:22 pm

Martijn Vierstraete

1:21 pm

Martijn Vierstraete

Dit is moelijk te reviewen. Is dit enkel een naamwijziging? Zoja SVN replace Zonee, naamwijziging apart committen

Dit is moelijk te reviewen. Is dit enkel een naamwijziging?
Zoja SVN replace
Zonee, naamwijziging apart committen

Wednesday 21 Aug 2019
9:40 am

Kurt Callebaut

ICT-796: Remboursbedrag dubbel aangerekend
ICT-796: Remboursbedrag dubbel aangerekend
Tuesday 20 Aug 2019
8:56 am

KurtC

[ICT-796] Remboursbedrag dubbel aangerekend.

- oude classen verwijderd

Options
    • -67
    • +0
    ./OrderOpBonBepaler.cls.xml
  2. … 4 more files in changeset.
Wednesday 13 Mar 2019
10:45 am

Kurt Callebaut

Tuesday 05 Mar 2019
3:25 pm

Martijn Vierstraete

3:02 pm

Kurt Callebaut

[DEF738] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)
[DEF738] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)
12:09 pm

KurtC

[DEF738] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)

bij opsplitsen van orders bij orgalux - niet orgalux is geen rekening gehouden met rembours op bon waardoor er mogelijks rembours meerdere malen op bon gezet wordt. (dubbel)

Deze code zet rembours van een order op de eerst aangemaakt bon / transport.

Zoeken via KUL ipv ORDW.

Options
    • -28
    • +14
    ./OrderOpBonBepaler.cls.xml
  2. … 7 more files in changeset.
Thursday 28 Feb 2019
8:57 am

Kurt Callebaut

8:49 am

Kurt Callebaut

ok. Maak me hier wel de bedenking omdat dit momenteel overbodig is in deze code. Als het preventief is, dan zouden we dit voor alle methods moeten doorvoeren. En ik kan me voorstellen dat in sommig...

ok. Maak me hier wel de bedenking omdat dit momenteel overbodig is in deze code. Als het preventief is, dan zouden we dit voor alle methods moeten doorvoeren. En ik kan me voorstellen dat in sommige gevallen je liever een error hebt, dan een lege waarde waardoor de data kan foutlopen ?

8:46 am

KurtC

[DEF738r] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)

bij opsplitsen van orders bij orgalux - niet orgalux is geen rekening gehouden met rembours op bon waardoor er mogelijks rembours meerdere malen op bon gezet wordt. (dubbel)

Deze code zet rembours van een order op de eerst aangemaakt bon / transport.

Options
    • -2
    • +2
    ./OrderOpBonBepaler.cls.xml
8:44 am

Kurt Callebaut

Is goed ga ik aanpassen. Het idee hier was duidelijk te maken dat het over oude globals gaat -> zie bijbel omschrijvingen. Het zoeken in code naar "vervref" zal dan iets lastiger worden.

Is goed ga ik aanpassen. Het idee hier was duidelijk te maken dat het over oude globals gaat -> zie bijbel omschrijvingen. Het zoeken in code naar "vervref" zal dan iets lastiger worden.

8:41 am

Kurt Callebaut

De oorspronkelijke code die deze class gebruikt bevat BonLijn.GeefOrderID() en ik hou het hier graag consistent. Alle andere code refactoren naar Id lijkt me ook geen goed plan.

De oorspronkelijke code die deze class gebruikt bevat BonLijn.GeefOrderID() en ik hou het hier graag consistent. Alle andere code refactoren naar Id lijkt me ook geen goed plan.

8:39 am

Kurt Callebaut

Dat zou es verder uitgespit moeten worden. De tijdelijke globals worden lokaal opgezet en als ze naar onafteralltests gaan, dan blijven er variabelen open staan. Dus het lijkt erop dat dan ook de o...

Dat zou es verder uitgespit moeten worden. De tijdelijke globals worden lokaal opgezet en als ze naar onafteralltests gaan, dan blijven er variabelen open staan. Dus het lijkt erop dat dan ook de opgezetten data (en niet alleen de globalnaam als string) naar de onbeforetest zou moeten, maar die zijn test afhankelijk.

Friday 22 Feb 2019
3:14 pm

Martijn Vierstraete

3:14 pm

Martijn Vierstraete

Misschien kan je deze method ook afzonderen in een bepaler, dan kun je in de test duidelijk maken dat de HeeftOrderlijnenOpTransport afhankelijk zijn van het aanwezig zijn van een vervoersreferenti...

Misschien kan je deze method ook afzonderen in een bepaler, dan kun je in de test duidelijk maken dat de HeeftOrderlijnenOpTransport afhankelijk zijn van het aanwezig zijn van een vervoersreferentie en moet je KUL niet "mocken" in je test

3:10 pm

Martijn Vierstraete

VervRef mag voluit geschreven worden (leest gemakkelijker)

VervRef mag voluit geschreven worden (leest gemakkelijker)

3:09 pm

Martijn Vierstraete

OrderId

OrderId

3:07 pm

Martijn Vierstraete

optionele constructor parameters defaulten met ""

optionele constructor parameters defaulten met ""

Thursday 21 Feb 2019
4:24 pm

Martijn Vierstraete

Is het killen van de globals nodig? Zoja, dan zou ik in onAfterTest doen (en aanmaken in onBeforeTest), zodat deze altijd goed opgekuist worden.

Is het killen van de globals nodig? Zoja, dan zou ik in onAfterTest doen (en aanmaken in onBeforeTest), zodat deze altijd goed opgekuist worden.

4:03 pm

Kurt Callebaut

[DEF738] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)
[DEF738] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)
1:41 pm

KurtC

[DEF738] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)

bij opsplitsen van orders bij orgalux - niet orgalux is geen rekening gehouden met rembours op bon waardoor er mogelijks rembours meerdere malen op bon gezet wordt. (dubbel)

Deze code zet rembours van een order op de eerst aangemaakt bon / transport.

Options
    • -5
    • +29
    ./OrderOpBonBepaler.cls.xml
  2. … 3 more files in changeset.
11:39 am

KurtC

[DEF738] MAG: Transport: Remboursbedrag bij ORGALUX + niet-ORGALUX (niet dubbel tellen)

bij opsplitsen van orders bij orgalux - niet orgalux is geen rekening gehouden met rembours op bon waardoor er mogelijks rembours meerdere malen op bon gezet wordt. (dubbel)

Deze code zet rembours van een order op de eerst aangemaakt bon / transport.

Options
    • -0
    • +57
    ./OrderOpBonBepaler.cls.xml
  2. … 5 more files in changeset.