CacheAdminA_trunk2010/vhUnitTest/APPS/EC/Winkelkar/impl/WinkelkarService/VoegToeMaatwerkEnGeefWinkelkarTerug

TAX

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 31 Oct 16
Commits this week: 0
Total Lines of Code (LoC): 107
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
Wednesday 12 Jun 2019
1:03 pm

Tom Vermeulen

Tuesday 11 Jun 2019
10:45 am

Kurt Callebaut

8:19 am

Tom Vermeulen

[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)
[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)
Thursday 06 Jun 2019
4:23 pm

Tom Vermeulen

[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)

- alles ineens verwijderd

Options
    • -4
    • +4
    ./TestVolumeAandeel.cls.xml
  2. … 15 more files in changeset.
Monday 28 Jan 2019
10:21 am

Tom Vermeulen

Thursday 24 Jan 2019
11:23 am

Tom Vermeulen

ja, maar code vliegt er over twee weken toch weer uit

ja, maar code vliegt er over twee weken toch weer uit

11:22 am

Tom Vermeulen

nope, code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

10:08 am

Kurt Callebaut

10:07 am

Kurt Callebaut

Eén instantie maken van legacypartij lijkt me voldoende.

Eén instantie maken van legacypartij lijkt me voldoende.

10:06 am

Kurt Callebaut

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

10:01 am

Kurt Callebaut

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

9:51 am

Kurt Callebaut

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvr...

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvragen van klantsessiedata.

9:48 am

Kurt Callebaut

Duplicate testcode op klantid na ?

Duplicate testcode op klantid na ?

8:56 am

Kurt Callebaut

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

8:56 am

Kurt Callebaut

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

8:54 am

Kurt Callebaut

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevest...

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevestigingschroevenVoorKlantID().

8:48 am

Kurt Callebaut

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?

8:43 am

Kurt Callebaut

equals() ?

equals() ?

8:42 am

Kurt Callebaut

stringutils.equals ?

stringutils.equals ?

8:42 am

Kurt Callebaut

2 ifs samenvoegen om duplicate code te vermijden.

2 ifs samenvoegen om duplicate code te vermijden.

8:39 am

Kurt Callebaut

ik ben niet helemaal mee in het verhaal. Hoe noemen we een klantid met prefix hier? In deze klasse gebruiken we || voor partij id, terwijl deze methods met underscore werken. Horen deze methods hie...

ik ben niet helemaal mee in het verhaal. Hoe noemen we een klantid met prefix hier? In deze klasse gebruiken we || voor partij id, terwijl deze methods met underscore werken. Horen deze methods hier thuis ? en zoja. Als dit verschil is tussen partijid en legacypartijId, dan zou ik de method hernoemen naar bv GeefKlanIdVanLegacyPartijID() en GeefLegacyPartijIDVanKlantID()

8:31 am

Kurt Callebaut

injecteren zodat er kan uitgemocked worden ?

injecteren zodat er kan uitgemocked worden ?

8:02 am

Kurt Callebaut

Mogen deze 2 ifkes één statement worden die korting 35 weergeeft ?

Mogen deze 2 ifkes één statement worden die korting 35 weergeeft ?

7:58 am

Kurt Callebaut

oei oei lange lijn valt buiten scherm. misschien een private method maken IsAlpnach() ? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

oei oei lange lijn valt buiten scherm. misschien een private method maken IsAlpnach() ?

Thursday 17 Jan 2019
9:59 am

Tom Vermeulen

[UST4131]EDI: MP: SFS: Alpnach nummer wijzigingen.
[UST4131]EDI: MP: SFS: Alpnach nummer wijzigingen.
Monday 14 Jan 2019
3:29 pm

Tom Vermeulen

[UST4131]EDI: MP: SFS: Alpnach nummer wijzigingen.

- String SFS_83723 vervangen door ##class(DOM.VKP.enu.SfsKlant).AlpnachOud()

Options
    • -4
    • +4
    ./TestVolumeAandeel.cls.xml
  2. … 11 more files in changeset.
Wednesday 15 Nov 2017
10:16 am

Kurt Callebaut

Friday 10 Nov 2017
11:51 am

Kurt Callebaut

Deze code was dus niet correct...http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif is intussen gewijzigd. Later misschien nog te refactoren.

Deze code was dus niet correct... is intussen gewijzigd. Later misschien nog te refactoren.

Wednesday 11 Oct 2017
8:01 am

Kurt Callebaut

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

Monday 09 Oct 2017
1:30 pm

Jo Claes

  • More
  • CR-580
  • finished reviewing