Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
Ik stel voor dat we de private method MaakProductVanNietTbxKenmerken() hernoemen naar iets wat duidt op ons maarwerkframework, of alleszins dat ook voor TbxKenmerken werkt. Feel free om een naam te...

Ik stel voor dat we de private method MaakProductVanNietTbxKenmerken() hernoemen naar iets wat duidt op ons maarwerkframework, of alleszins dat ook voor TbxKenmerken werkt.
Feel free om een naam te bedenken, alleen of samen met mij

[ICT-2694][rvPVR] TBX MaakProductVanKenmerken : KenmerkDataDefinities toevoegen aan nieuwe calculatie...
[ICT-2694][rvPVR] TBX MaakProductVanKenmerken : KenmerkDataDefinities toevoegen aan nieuwe calculatie...
[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen
[ICT-2375][rvPVR] eCon TBX voor BIDI : MTIB als checkbox toevoegen
Volgens mijn beredenering zou er geen wijziging zijn voor reeds aanwezige TBXkenmerken. Dus enkel voor nieuwe bewaarde data. Ma toch een beetje "crossed fingers" :-P

Volgens mijn beredenering zou er geen wijziging zijn voor reeds aanwezige TBXkenmerken. Dus enkel voor nieuwe bewaarde data.
Ma toch een beetje "crossed fingers" :-P

Ik weet niet meer of we het hadden besproken maar, was "TBXKenmerken.RugwandMateriaal" voorheen al opgevuld? Zo niet zal het nu wel opgevuld worden en is er een herbereken van de canonische waarde ...

Ik weet niet meer of we het hadden besproken maar, was "TBXKenmerken.RugwandMateriaal" voorheen al opgevuld? Zo niet zal het nu wel opgevuld worden en is er een herbereken van de canonische waarde nodig.

[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-2373][rvPVR] eCon TBX voor BIDI : Rugwandmateriaal hout/staal beschikbaar
[ICT-2373][rvPVR] eCon TBX voor BIDI : Rugwandmateriaal hout/staal beschikbaar
[ICT-967] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TBX product-updaten - Implementatie:
[ICT-967] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TBX product-updaten - Implementatie:
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.