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
[ICT-2692][rvPVR] eCon TBX voor BIDI : PlaatMateriaal doorgeven van eCon/Legacy naar WSOrderLijn

- EDI LadeTBX aangepast : property PlaatMateriaal toegevoegd

[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

- Property ItemMinTolerantieOpIB toegevoegd (EDI LadeTBX)

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
[UST4130] VH4: TBX conversie geven problemen bij Mengvorm

- Nieuwe property op Edi-TBX (DesignCode)

- basisaanpassingen om conversietests al niet direct te laten falen

  1. … 1 more file in changeset.
[UST2415] Vh4: migratie productiewijze (tbx, tax en lbx)

- TBX legacy voorzien van productiewijze/onderdelenspeciaal

- Conversies aangepast van Econ tot Legacy

  1. … 7 more files in changeset.
  • More
  • CR-45
  • finished reviewing
In principe zouden er geen andere waarden in Product.FrontOndersteuningAantal mogen zitten dan "", "1", "2" of "3" ... en in ItemFS enkel 1,2,3,HS of A. Normaagezien zou er geen exceptie geraised m...

In principe zouden er geen andere waarden in Product.FrontOndersteuningAantal mogen zitten dan "", "1", "2" of "3" ... en in ItemFS enkel 1,2,3,HS of A. Normaagezien zou er geen exceptie geraised mogen worden... De andere "convert*" methods raisen ook geen excepties.
Qua strutctuur: op deze manier is het voor de geïnteresseerde lezer onmiddellijk duidelijk dat bij aantal="" er naar het type gekeken gaat worden. Bon, 't is nen uitleg gelijk een ander.

Caché is toch snel genoeg? (soit, ben al content dat "PeuterLadeCodeOpen" het gehaald heeft http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

Caché is toch snel genoeg?
(soit, ben al content dat "PeuterLadeCodeOpen" het gehaald heeft )

Beter een if elseif structuur zodoende niet alle if's moeten gevalideerd worden.

Beter een if elseif structuur zodoende niet alle if's moeten gevalideerd worden.

Geen drietaps if elseif else nodig met in den else een exceptie ??

Geen drietaps if elseif else nodig met in den else een exceptie ??

  • More
  • CR-45
  • started review
[UST2157] ECON-EDI Conversie
[UST2157] ECON-EDI Conversie
[UST2157] ECON-EDI Conversie

- Nieuwe logica voor frontondersteuning (type/aantal)

- Het frontondersteuningstype wordt toegevoegd op EDIlaag (property, enu)

  1. … 3 more files in changeset.
[UST1745] Labonorm: TBX DTO + VO interface programmatie

- Heeftdwarsverdeling als %boolean ipv string

[UST1794] Labonorm TBX: WS.TBX.Order opbouwen - Niet meeleveren

- HeeftGreepEnMeenemer -> GreepEnMeenemer rename

  1. … 5 more files in changeset.
[UST1746] Labonorm TBX: WS.TBX.Order opbouwen

- ZijkantHoogteValidator

  1. … 5 more files in changeset.
[UST1746] Labonorm TBX: WS.TBX.Order opbouwen

- Spoelbak en sifon: RugwandMateriaal en TussenrugMateriaal

[UST1746] Labonorm TBX: WS.TBX.Order opbouwen

- Spoelbak en sifon: eerste basistestjes, de rest volgt later

  1. … 4 more files in changeset.