Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam v...

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam verschillend (cq. AantalDwarsverdelingen in APPS).

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logi...

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logica.
Maar ik geloof best, zonder het na te kijken, dat in deze klasse nog code staat die vertrekt van de ingegevenKenmerken i.p.v. halffabItem.

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties() hiervoor is geen "uitzondering nodig". Tijdens de uitbreidingsfase en review lijkt het me handig om te zien ...

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties()
hiervoor is geen "uitzondering nodig".
Tijdens de uitbreidingsfase en review lijkt het me handig om te zien dat beide properties zijn toegevoegd. Maar eenmaal in productie (en ondersteund door automatische testen) kan dit het standaard pad volgen.
Of zie ik iets over t hoofd?

Tekst correct? Zonder kar --> geen vakken, right?

Tekst correct?
Zonder kar --> geen vakken, right?

Staat al in productie en is voordien al uitgebreid end-to-end getest.

Staat al in productie en is voordien al uitgebreid end-to-end getest.

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
Hmm, terechte review opmerking. Maar volgens mij is het wel oké, want via EDI kan men geen keuze maken voor PlaatMateriaal, dus is het steeds "HS". Indien men later dit kenmerk toch specifiek voor ...

Hmm, terechte review opmerking.
Maar volgens mij is het wel oké, want via EDI kan men geen keuze maken voor PlaatMateriaal, dus is het steeds "HS".
Indien men later dit kenmerk toch specifiek voor EDI zou beschikbaar maken, dan zal een test meteen uitwijzen dat iets anders dan "HS" niet werkt.
FYI : Edi zal niet meer via caché binnenkomen, dus deze klasse is weldra Deprecated.

Een alternatief, properdere oplossing zou zijn om volgende toe te voegen aan de klasse EXT.Unishop.PurchaseOrder001.CreateOrder.LadeEigenschappenTBX :

Property PlaatMateriaal As %String(XMLPROJECTION = "NONE") [ InitialExpression = "HS" ];
Heb mijn twijfels bij deze hargecodeerde opvulling. Dit kenmerk is inderdaad niet terug te vinden bij de input van deze convertor maar is het safe om het zo achter te laten?

Heb mijn twijfels bij deze hargecodeerde opvulling. Dit kenmerk is inderdaad niet terug te vinden bij de input van deze convertor maar is het safe om het zo achter te laten?

[ICT-2692][rvPVR] eCon TBX voor BIDI : PlaatMateriaal doorgeven van eCon/Legacy naar WSOrderLijn
[ICT-2692][rvPVR] eCon TBX voor BIDI : PlaatMateriaal doorgeven van eCon/Legacy naar WSOrderLijn
[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen
[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen
[ICT465] EDI TAX tag EdgeBandColour voorzien
[ICT465] EDI TAX tag EdgeBandColour voorzien
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!

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.

[UST4130] VH4: TBX conversie geven problemen bij Mengvorm
[UST4130] VH4: TBX conversie geven problemen bij Mengvorm