Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4627] [rvPVR] VHLOG MES M1 - TBX afbeeldingcode blijft leeg in de orderoptie:
[ICT-4627] [rvPVR] VHLOG MES M1 - TBX afbeeldingcode blijft leeg in de orderoptie:
[ICT-4627] [rvPVR] VHLOG MES M1 - TBX afbeeldingcode blijft leeg in de orderoptie:

- Afbeeldingservice uitbreiden voor TBX

  1. … 5 more files in changeset.
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-2692][rvPVR] eCon TBX voor BIDI : PlaatMateriaal doorgeven van eCon/Legacy naar WSOrderLijn

- method LegacyNaarWebOrderConverter() : ItemMinTolerantieOpIB wordt overgenomen (van APPS Legacy naar WebOrderLijn)

[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen
[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen
[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen

- Bugfix in LegacyNaarWebOrderConverter : fout gemaakt bij refactoring, nl. aantal parameters bij method VulInWebOrderLijn() gewijzigd.

[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen

- LegacyNaarWebOrderConverter : ItemMinTolerantieOpIB wordt overgenomen (van APPS Legacy naar WebOrderLijn)

- Refactoring om public method ConverteerLegacyNaarWebOrderLijn() toe te voegen

    • -5
    • +22
    ./LegacyNaarWebOrderConverter.cls.xml
[ICT-2455][rvPVR] eCon TBX voor BIDI : ProductieWijze doorgeven van Legacy naar WSOrderLijn
[ICT-2455][rvPVR] eCon TBX voor BIDI : ProductieWijze doorgeven van Legacy naar WSOrderLijn
[ICT-2455][rvPVR] eCon TBX voor BIDI : ProductieWijze doorgeven van Legacy naar WSOrderLijn (LegacyNaarWebOrderConverter)

- method VulInWebOrderLijn() aangepast : Property ItemOnderdelenSpeciaal overzetten van DOM TBXKenmerkenLegacy naar WsOrderLijn

- TODO : refactor de andere methods VulInWebOrderLijn() zijn kopieën van deze. Best allemaal samen laten komen.

zal wel geweest zijn omdat de method op GADataProduct "LangTekst" heet

zal wel geweest zijn omdat de method op GADataProduct "LangTekst" heet

Had er over gekeken. Ik heb inderdaad de gewoonte om standaardgedrag bovenaan te zetten, waarschijnlijk daarom niet goed opgelet

Had er over gekeken. Ik heb inderdaad de gewoonte om standaardgedrag bovenaan te zetten, waarschijnlijk daarom niet goed opgelet

Tja, is een beetje misleidend hé. Deze is bedoeld om nu reeds op te roepen, en bij het eerste concrete geval eventueel al (deels) te implementeren. Want als t em hiere crasht, dan is deze debugInfo...

Tja, is een beetje misleidend hé.
Deze is bedoeld om nu reeds op te roepen, en bij het eerste concrete geval eventueel al (deels) te implementeren.
Want als t em hiere crasht, dan is deze debugInfo van levensnoodzakelijk belang

vhUnitTest.APPS.PM.Maatwerk.impl.TBX.WebOrderNaarGADataConverter.Convert.Test.cls(Test WebOrder geeft GADataProduct): Method Test WebOrder geeft GADataProduct Deze geeft een GADataProduct terug me...

vhUnitTest.APPS.PM.Maatwerk.impl.TBX.WebOrderNaarGADataConverter.Convert.Test.cls(Test WebOrder geeft GADataProduct): Method Test WebOrder geeft GADataProduct

Deze geeft een GADataProduct terug met een verwachte prijs van 31.3281 EUR.
Ik kan hier nog 100 andere properties/DataItems controleren, maar mijn conclusie is dat, als de prijs correct is, dan is het berekende product ook correct.
Dit is voor mij (voorlopig) voldoende controle wat deze converter moet doen.

De method staat wel onderaan (lees: halverwege). Zou het beter zijn als ik die bovenaan, als eerste zet?

Is opgelost. Zie mail met screenshot.

Is opgelost.
Zie mail met screenshot.

Method niet meer nodig

Method niet meer nodig

cosmetische opmerking: de oude lijnen staan proper uitgelijnd, de nieuwe niet...

cosmetische opmerking: de oude lijnen staan proper uitgelijnd, de nieuwe niet...