CacheAdminA_trunk2010/vhTest/Fake/BL

Legacy

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 20 Dec 17
Commits this week: 0
Total Lines of Code (LoC): 41
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
Friday 27 Sep 2019
8:16 am

Kurt Callebaut

7:54 am

Kurt Callebaut

De stat juist, Heb ik bewust gewijzigd. Anders krijgen we standaard het gedrag dat spoelbak en smalle lade altijd in spoelbakgroep terecht komen en default gedrag is dat dit niet gebeurd. daarom he...

De stat juist, Heb ik bewust gewijzigd. Anders krijgen we standaard het gedrag dat spoelbak en smalle lade altijd in spoelbakgroep terecht komen en default gedrag is dat dit niet gebeurd. daarom heb ik ook de altijdsplitser moeten toevoegen.
maw indien true, faalde de bestaande testen op gewone aantallen als er geen splitser meegegeven wordt..

(Ik vind het nog steeds een gevaarlijk iets om te beslissen dat de groepering per kenmerk vervangen wordt door groepering op productiegroep, terwijl het in dit geval toevallig hetzelfde is.)

Thursday 26 Sep 2019
2:12 pm

Wim Vermeulen

2:11 pm

Wim Vermeulen

Kritische vraag (misschien om in de groep te gooien) : Is dit een project-setting? --> volgens mij enkel bedoeld om iets nieuw te kunnen "infaseren". Nadien wordt een project-setting opgekuist (in ...

Kritische vraag (misschien om in de groep te gooien) :
Is dit een project-setting? --> volgens mij enkel bedoeld om iets nieuw te kunnen "infaseren". Nadien wordt een project-setting opgekuist (in theorie althans )
Is dit dan eerder een gewone ConfigItem? --> is eigenlijk bedoeld om onderscheid te kunnen maken tussen de verschillende omgevingen (bvb. andere waarde op PROD dan op DEV)
Is het voldoende om deze gewoon als Class Parameter te definieren? --> volgens mij wel. Dit is een single-responsibility klasse, dus een wijziging van de waarde committen en mergen, kan supersnel.

Willen we dit toch wegtrekken uit onze code, dan misschien toch een aparte data-structuur voor maken. Functioneel is config-item hiervoor ook wel geschikt, maar ik vraag me af of we den boel dan niet te veel gaan vervuilen?

1:10 pm

Wim Vermeulen

Ook testmethod maken met VerwachtMethodCall("IsNodigTeSplitsen" --> dan Return False

Ook testmethod maken met
VerwachtMethodCall("IsNodigTeSplitsen" --> dan Return False

12:29 pm

Wim Vermeulen

ToeleveringSplitserMock.Verifieer()

ToeleveringSplitserMock.Verifieer()

12:07 pm

Wim Vermeulen

AltijdSplitser kan best in de constructor, want nu krijg je een nieuwe instantie bij iedere "VoegToe"-call.

AltijdSplitser kan best in de constructor, want nu krijg je een nieuwe instantie bij iedere "VoegToe"-call.

11:38 am

Wim Vermeulen

Quit $$$True (want ik zie dat de UT-resultaten gewijzigd zijn, en dat was niet de bedoeling http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

Quit $$$True
(want ik zie dat de UT-resultaten gewijzigd zijn, en dat was niet de bedoeling )

11:12 am

Wim Vermeulen

Deze klasse mag weg, indien geen Config-item gebruikt wordt. (zie opmerking bij ToeleveringSplitser)

Deze klasse mag weg, indien geen Config-item gebruikt wordt. (zie opmerking bij ToeleveringSplitser)

10:53 am

Wim Vermeulen

Deze klasse is niet meer nodig, denk.

Deze klasse is niet meer nodig, denk.

10:53 am

Wim Vermeulen

Deze klasse is niet meer nodig, denk.

Deze klasse is niet meer nodig, denk.

10:51 am

Wim Vermeulen

Klein detail : deze lijn onder de ..IngegevenKenmerkenTypeApi plaatsen (volgorde zelfde als de args erboven)

Klein detail : deze lijn onder de ..IngegevenKenmerkenTypeApi plaatsen (volgorde zelfde als de args erboven)

Monday 23 Sep 2019
4:24 pm

KurtC

[ICT-817] Productiegroep Specials PM voor meer dan 10 LBX smalle lades

- move to APPS.Halux.common

Options
    • -19
    • +0
    ./LadeVoorBewerkingsCenterBepaler.cls.xml
  2. … 3 more files in changeset.
3:44 pm

KurtC

[ICT-817] Productiegroep Specials PM voor meer dan 10 LBX smalle lades

- Verder uitwerken. (single responsibility)

Options
    • -0
    • +19
    ./LadeVoorBewerkingsCenterBepaler.cls.xml
  2. … 8 more files in changeset.
Thursday 19 Sep 2019
3:17 pm

Kurt Callebaut

ICT-817: Productiegroep Specials PM voor meer dan 10 LBX smalle lades
ICT-817: Productiegroep Specials PM voor meer dan 10 LBX smalle lades
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
    • -26
    • +0
    ./TRANSP/OrderOpBonBepaler.cls.xml
  2. … 4 more files in changeset.
8:56 am

KurtC

[ICT-796] Remboursbedrag dubbel aangerekend.
Options
    • -0
    • +25
    ./TRANSP/LeverbonService.cls.xml
  2. … 9 more files in changeset.
Friday 26 Apr 2019
2:09 pm

Tom Vermeulen

Reverse merged revision(s) 56420 from trunk2010:

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

revert van de revert omdat mijn wijzigingen wel goed waren, maar de rest van de ventjes niet :)

Options
    • -2
    • +1
    ./FLOWTOE.cls.xml
  2. … 65 more files in changeset.
1:53 pm

Tom Vermeulen

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

Options
    • -1
    • +2
    ./FLOWTOE.cls.xml
  2. … 65 more files in changeset.
11:47 am

Tom Vermeulen

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
Options
    • -2
    • +1
    ./FLOWTOE.cls.xml
  2. … 65 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
    • -2
    • +9
    ./TRANSP/OrderOpBonBepaler.cls.xml
  2. … 7 more files in changeset.
Thursday 28 Feb 2019
8:57 am

Kurt Callebaut