CacheAdminA_trunk2010/vhTest/Fake/APPS/EDI

Bestel

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 29 Jun 16
Commits this week: 0
Total Lines of Code (LoC): 199
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.
Thursday 27 Apr 2017
1:38 pm

Jo Claes

1:38 pm

Jo Claes

Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen. Geen testen ...

Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen.
Geen testen ==> needs resolution -> De testen zitten dus in de svn-repo

Wednesday 26 Apr 2017
3:39 pm

Martijn Vierstraete

  • More
  • CR-188
  • finished reviewing
3:39 pm

Martijn Vierstraete

Niet alle commits van ICT242 zijn opgenomen in deze review *Geen testen ==> needs resolution http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif *Ik zie g...
  • Niet alle commits van ICT242 zijn opgenomen in deze review
  • Geen testen ==> needs resolution
  • Ik zie geen inhoudelijke wijzigingen, enkel code generatie aanpassingen (bv. package name, cdata)
Thursday 20 Apr 2017
4:01 pm

Jo Claes

4:00 pm

Jo Claes

[ICT242] - Synchronizer terug werkend krijgen
[ICT242] - Synchronizer terug werkend krijgen
Wednesday 22 Feb 2017
4:34 pm

Tom Vermeulen

4:02 pm

Jo Claes

  • More
  • CR-87
  • finished reviewing
12:04 pm

Tom Vermeulen

  • More
  • CR-87
  • started review
[UST2214] DKG EDI: informeer NBR in geval van nieuw EDI-order
[UST2214] DKG EDI: informeer NBR in geval van nieuw EDI-order