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.
[ICT-1432] HR: Brandlijst: oude code opkuisen DOM.HR.*

- Method 'GeefPartijViaProtimeNr' verwijderd uit de PartijApi

  1. … 2 more files in changeset.
[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:
[rvTVE] VHIP900: SyncKlant AxToAdmin: Verwijdering telefoon e.d. breekt klant in Cache:

- KK1-data wordt nu bij wijziging van partij eerst opgehaald om later te vergelijken en te weten welke oude KKL moet gemerged worden met de eventueel nieuwe

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 ?