CacheAdminA_trunk2010/vhTest/Fake/APPS

EDI

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 22 Dec 15
Commits this week: 0
Total Lines of Code (LoC): 364
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.
8:57 am

Tom Vermeulen

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

- AanvraagHeaderBepaler uitmocken

Options
  1. ./Bestel/impl/BestelService/PlaatsBestelling
    • -0
    • +19
    ./Bestel/impl/BestelService/PlaatsBestelling/AanvraagHeaderBepaler.cls.xml
  3. … 3 more files in changeset.
Thursday 20 Dec 2018
3:37 pm

Tom Vermeulen

Friday 14 Dec 2018
11:05 am

Wim Vermeulen

10:45 am

Wim Vermeulen

Er zit een probleem in de implementatie van ..ConverterData.Add() --> we zullen dat maandag samen effe bekijken. Nu geen tijd aan spenderen!

Er zit een probleem in de implementatie van ..ConverterData.Add() --> we zullen dat maandag samen effe bekijken. Nu geen tijd aan spenderen!

10:11 am

Wim Vermeulen

Bedenkingen bij naamgeving en impl : Product (Edi.LadeTBX) wordt hier niet gebruikt, dus mag die parameter weg, en is het eigenlijk geen convert, maar gewoon Bereken...() of Bepaal...() en dan kunn...

Bedenkingen bij naamgeving en impl :
Product (Edi.LadeTBX) wordt hier niet gebruikt, dus mag die parameter weg, en is het eigenlijk geen convert, maar gewoon Bereken...() of Bepaal...()
en dan kunnen de 2 private methods samengevoegd worden tot 1.

Thursday 13 Dec 2018
1:47 pm

Tom Vermeulen

[UST4130] VH4: TBX conversie geven problemen bij Mengvorm
[UST4130] VH4: TBX conversie geven problemen bij Mengvorm
Wednesday 12 Dec 2018
3:29 pm

Tom Vermeulen

[UST4130] VH4: TBX conversie geven problemen bij Mengvorm

- EdiNaarLegacy en LegacyNaarEdi bijgewerkt

Options
    • -0
    • +7
    ./common/impl/LegacyConverter.cls.xml
  2. … 5 more files in changeset.
Tuesday 07 Aug 2018
3:45 pm

Kurt Callebaut

Friday 03 Aug 2018
4:07 pm

Wim Vermeulen