CacheAdminA_trunk2010/vhTest/Mock/APPS/common

KlantInstellingenService.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 6
Last Commit: 09 Nov 21
Commits this week: 0
Total Lines of Code (LoC): 128
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 29 Nov 2021
9:39 am

Tommy Hebb

9:10 am

Wim Vermeulen

Tuesday 23 Nov 2021
10:40 am

Tommy Hebb

[ICT-2581] [rvWV] HX: TAX: Movento-koppeling: Aanpassing ProductCreatie volgens KoppelingType:
[ICT-2581] [rvWV] HX: TAX: Movento-koppeling: Aanpassing ProductCreatie volgens KoppelingType:
Tuesday 09 Nov 2021
10:39 am

Tommy Hebb

[ICT-2581] [rvWV] HX: TAX: Movento-koppeling: Aanpassing ProductCreatie volgens KoppelingType:

- HeeftFrontHoogteVerstelling niet meer via KlantInstellingenService => mag dus weer weg => Is ondertussen volgens KoppelingType = Movento

Options
    • -7
    • +0
    ./KlantInstellingenService.cls.xml
  2. … 2 more files in changeset.
Tuesday 05 Oct 2021
12:49 pm

Tommy Hebb

12:21 pm

Tom Vermeulen

11:09 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
Thursday 23 Sep 2021
4:17 pm

Tommy Hebb

Zoals besproken blijkbaar ongewild in deze review terechtgekomen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zoals besproken blijkbaar ongewild in deze review terechtgekomen

4:06 pm

Tom Vermeulen

is deze wel nodig?

is deze wel nodig?

Thursday 12 Aug 2021
4:14 pm

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Uitbreiding Klant-enum met 3626 OlBeslag => teruggedraaid, want toch niet nodig

- Uitbreiding KlantInstellingenService met MagKlantTAORLadeKleurOudZwartBestellen => Enkel klant OlBeslag geeft true => teruggedraaid, want toch niet nodig

Options
    • -7
    • +0
    ./KlantInstellingenService.cls.xml
  2. … 5 more files in changeset.
Tuesday 10 Aug 2021
5:20 pm

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Uitbreiding Klant-enum met 3626 OlBeslag

- Uitbreiding FreesProgramma-enum met X GeenBewerking

- Uitbreiding KlantInstellingenService met MagKlantTAORLadeKleurOudZwartBestellen => Enkel klant OlBeslag geeft true

- Uitbreiding KantenbandData met LadeKleur => in latere commit nodig voor KantenbandBepaler

Options
    • -0
    • +7
    ./KlantInstellingenService.cls.xml
  2. … 9 more files in changeset.
Tuesday 16 Mar 2021
4:47 pm

Tommy Hebb

4:46 pm

Tom Vermeulen

Thursday 11 Mar 2021
4:43 pm

Tommy Hebb

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt. Ik dacht dat een OnBeforeOne...

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt.
Ik dacht dat een OnBeforeOneTest bedoeld is voor als er een gevaar is dat hij door eerdere UTs zou kunnen bevuild worden, wat hier niet het geval is.

3:13 pm

Tommy Hebb

Bwa, is een beetje zo gegroeid door refactoren... Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests? Daarna de afweging: Waarom dan nog de call on...

Bwa, is een beetje zo gegroeid door refactoren...
Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests?
Daarna de afweging: Waarom dan nog de call onder test speciaal gaan afzonderen? En bovendien: Waarom beschikken we anders over een $$$AssertTrue en $$$AssertFalse?
Het was bovendien daardoor ook niet meer nodig om een betekenisvolle, extra lokale variabele te voorzien in de UnitTesten (VerwachtInOpstartfase), dus ook al een regel minder per UT, zonder aan leesbaarheid in te boeten (eerder het tegendeel).
Doe de nieuwe versie eens open in studio en zie eens hoe leesbaar die is
Misschien moeten we dan eerder (mettertijd, als die klassen eens onder change komen) die van LBX en TBX aanpassen naar dit model?

8:57 am

Wim Vermeulen

Wednesday 10 Mar 2021
7:51 pm

Wim Vermeulen

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval. Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (pro...

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval.
Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (productieSequentie) toe te voegen.
Na al je inspanningen, keur ik deze oplossing goed

7:29 pm

Wim Vermeulen

Ik begrijp niet echt waarom je deze "Assert..." method hebt weggewerkt. Dit is immers een algemeen gebruikt principe, o.a. om aan te geven dat de test-methods in deze klasse dezelfde logica oproepe...

Ik begrijp niet echt waarom je deze "Assert..." method hebt weggewerkt.
Dit is immers een algemeen gebruikt principe, o.a. om aan te geven dat de test-methods in deze klasse dezelfde logica oproepen. Hoe eenvoudig deze ook is.

En voor zover consistentie een argument i : de implementatie bij TAX is nu minder consistent met die van LBX en TBX.

Ik vind dit geen drama, hoor. Als je wilt kunnen we hierover van gedachten wisselen

7:17 pm

Wim Vermeulen

Kleine suggestie : OpstartfaseBepaler voor iedere test(method) newen, dat is ietsje properder --> dus in de OnBeforeOneTest() is beter. Let op: parameter (aTestCase) !

Kleine suggestie :
OpstartfaseBepaler voor iedere test(method) newen, dat is ietsje properder --> dus in de OnBeforeOneTest() is beter.
Let op: parameter (aTestCase) !

Monday 01 Mar 2021
9:55 am

Tom Vermeulen

ineens een Set van maken, lijn wordt korter en leesbaarder

ineens een Set van maken, lijn wordt korter en leesbaarder

Friday 26 Feb 2021
3:00 pm

Tommy Hebb

[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing...
Monday 22 Feb 2021
4:28 pm

Tommy Hebb

[ICT-2144] [rvWV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing (frontverstelmogelijkheid):

- Uitgebreid met MagKlantTAORFrontHoogteVerstellingBestellen => Voorlopig default False

Options
    • -0
    • +7
    ./KlantInstellingenService.cls.xml
  2. … 2 more files in changeset.
Wednesday 15 Jan 2020
1:35 pm

Laura Landuyt

Wednesday 04 Dec 2019
8:09 am

Kurt Callebaut

7:44 am

Laura Landuyt

[ICT-998] [+KCA] MAG: Leverbon: VHOSS nummer vermelden voor Group Nivelles; procom voor Vanhout.Pro
[ICT-998] [+KCA] MAG: Leverbon: VHOSS nummer vermelden voor Group Nivelles; procom voor Vanhout.Pro
Friday 22 Nov 2019
10:25 am

LauraL

[ICT-998] MAG: Leverbon: VHOSS nummer vermelden voor Group Nivelles; procom voor Vanhout.Pro

/KlantInstellingenService (+ Mock en impl): Add Method MoetVhossNummerOpLeveringBon

Options
    • -2
    • +8
    ./KlantInstellingenService.cls.xml
  2. … 2 more files in changeset.
Wednesday 01 Mar 2017
8:04 am

Kurt Callebaut

Monday 27 Feb 2017
1:07 pm

Jo Claes

  • More
  • CR-99
  • finished reviewing
Thursday 23 Feb 2017
2:34 pm

Jo Claes

Testen zouden eigenlijk het aantal moeten kunnen uitmocken zodat de logica van de code kan getest worden onafhankelijk van het aantal. Anders zal er bij iedere configuratie wijzigingen test-resulta...

Testen zouden eigenlijk het aantal moeten kunnen uitmocken zodat de logica van de code kan getest worden onafhankelijk van het aantal. Anders zal er bij iedere configuratie wijzigingen test-resultaten wijzigen

2:31 pm

Jo Claes

geen Initialen in deploy stories http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/biggrin.gif

geen Initialen in deploy stories