CacheAdminA_trunk2010/vhTest/Fake/APPS/EC/Winkelkar/impl

MaatwerkLijnBuilder.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 14 Jan 19
Commits this week: 0
Total Lines of Code (LoC): 86
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 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
    • -2
    • +1
    ./MaatwerkLijnBuilder.cls.xml
  2. … 11 more files in changeset.
Wednesday 07 Nov 2018
10:19 am

Jo Willems

Monday 05 Nov 2018
10:09 am

Martijn Vierstraete

10:09 am

Martijn Vierstraete

Naamgeving stub ipv mock

Naamgeving stub ipv mock

10:07 am

Martijn Vierstraete

Is deze test overbodig?

Is deze test overbodig?

10:06 am

Martijn Vierstraete

casing

casing

10:01 am

Martijn Vierstraete

Casing! (zeker bij nieuwe klasses)

Casing! (zeker bij nieuwe klasses)

Friday 26 Oct 2018
4:19 pm

Martijn Vierstraete

Bij nieuwe code mag je api's newen

Bij nieuwe code mag je api's newen

Tuesday 23 Oct 2018
11:34 am

Jo Willems

TECH.Interface is niet genoeg om een mock te laten genereren.... Open / closed principe : extend,not modify inherited behaviour...

TECH.Interface is niet genoeg om een mock te laten genereren....
Open / closed principe : extend,not modify inherited behaviour...