Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Gebruik mss ##class(DOM.common.enu.Personality).HaefeleIT()

Gebruik mss ##class(DOM.common.enu.Personality).HaefeleIT()

[ICT-4457] [rvTVE] Aanpassing importfiles Ardis voor klant Snaidero (Häefele IT):
[ICT-4457] [rvTVE] Aanpassing importfiles Ardis voor klant Snaidero (Häefele IT):
[ICT-1116] [rvTVE] VHIP900: SyncKlant AxToAdmin: Verwijdering telefoon e.d. breekt klant in Cache:
[ICT-1116] [rvTVE] VHIP900: SyncKlant AxToAdmin: Verwijdering telefoon e.d. breekt klant in Cache:
ja, maar code vliegt er over twee weken toch weer uit

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

nope, code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

Eén instantie maken van legacypartij lijkt me voldoende.

Eén instantie maken van legacypartij lijkt me voldoende.

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.

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.

"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.

Duplicate testcode op klantid na ?

Duplicate testcode op klantid na ?

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.)

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.)

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().

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 ?

equals() ?

equals() ?

stringutils.equals ?

stringutils.equals ?

2 ifs samenvoegen om duplicate code te vermijden.

2 ifs samenvoegen om duplicate code te vermijden.

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()

injecteren zodat er kan uitgemocked worden ?

injecteren zodat er kan uitgemocked worden ?

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

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

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() ?

[UST4131]EDI: MP: SFS: Alpnach nummer wijzigingen.
[UST4131]EDI: MP: SFS: Alpnach nummer wijzigingen.
Lol... deuh http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif Aangepast!

Lol... deuh Aangepast!