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-4457] [rvTVE] Aanpassing importfiles Ardis voor klant Snaidero (Häefele IT):

- In DataMToelevering blijkt de property EindPersonalityBepaler transient te zijn => dus OpenId i.p.v. new wegens gepersisteerd => dus de EindPersonalityBepaler op juist plek en in juiste geval alsnog newen

- In de global staan er dubbele quotes in de oorsprongketting => die weghalen bij het ophalen van de EindKlantId

  1. … 2 more files in changeset.
[ICT-4457] [rvTVE] Aanpassing importfiles Ardis voor klant Snaidero (Häefele IT):

- LegacyPartijAPI uitgebreid met GeefEindklantIdVolgensOorsprongKetting

- UnitTestje voor bovenstaande method

- Voor IsEindklantSnaideroOnderPersonalityHaefeleItalie wordt voor het Front de Rol op "Monster" gezet => Opdat de afdeelzaag gebruikt kan worden i.p.v. de afkortzaag.

  1. … 4 more files in changeset.
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.
[UST4131]EDI: MP: SFS: Alpnach nummer wijzigingen.

- Extra methods om handelprefixen toe te voegen of te verwijderen van handelklantids

  1. … 1 more file in changeset.
Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv ...

Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv op repository) Indien er enkel geraiset wordt vanuit verkoopservice, dan heeft de event waarschijnlijk een andere bedoeling/betekenis.