Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2694][rvPVR] TBX MaakProductVanKenmerken : KenmerkDataDefinities toevoegen aan nieuwe calculatie

- routine tmpDev2 aangepast : GenereerDataDefinitie() oproepen indien "DDEFS=1", zelfs wanneer de oude BomBol-berekening niet volledig gelukt is, i.e. Prod.ValidatePR() = false.

- In feite wordt de code nu wel echt DIRTY, en dus bij een volgende aanpassing/uitbreiding toch echt overwegen om de code terug cleaner te maken.

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

- BTItemsFromWSLijn : ItemPlaatMateriaal toegevoegd aan BTValues; indien leeg, dan valt de calculatie sowieso terug op de default "HS" (via MetaItem); Dat was tot nu toe ALTIJD het geval voor deze routine.

[ICT-2694][rvPVR] TBX MaakProductVanKenmerken : KenmerkDataDefinities toevoegen aan nieuwe calculatie

- routine tmpDev2 aangepast : (bugfix) variabele GemaaktProductId ingevuld.

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-2694][rvPVR] TBX MaakProductVanKenmerken : KenmerkDataDefinities toevoegen aan nieuwe calculatie

- routine tmpDev2 aangepast : i.p.v. KPRCreate() wordt nu GenereerDataDefinitie() opgeroepen, indien Opties de flag "DDEFS=1" bevat; dan worden ^KPR en ^PRBS hier niet meer ingevuld.

[ICT] bugfix bij Berekenen van TBX-product

- BerekeningServiceProduct.TbxHelper : propere foutmelding i.p.v. een runtime-exception

- variabele "GemaaktProductId" geeft geen <Undefined> meer in geval van Fout bij berekenen TBX

  1. … 1 more file in changeset.
[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

- routine bfwBTItemsFromWSLijn() : MinTolerantieOpIB toegevoegd aan de BTValues

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-2359] TBX BOMBOL builders porten naar .NET

- Ter voorbereiding van het porten heb ik wat testen aangemaakt. Deze wijzigingen zijn nodig omdat de testen anders een error gooien: "Variabelen probleem: Aantal voor test is minder dan na de test"

  1. … 1 more file in changeset.
[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-2373][rvPVR] eCon TBX voor BIDI : Rugwandmateriaal hout/staal beschikbaar (BerekeningServiceProduct TbxHelper)

- BTItemsFromWSLijn : ItemRugwandMateriaal toegevoegd, mapping naar "TBStalenRug" via lokale variabele CodeStalenRug

[ICT-2455][rvPVR] eCon TBX voor BIDI : ProductieWijze doorgeven van Legacy naar WSOrderLijn

- BTItemsFromWSLijn : ItemOnderdelenSpeciaal toegevoegd, mapping naar "TBOndSpecial"