Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5643] CPConfig TBX ondersteunen in cache
  1. … 8 more files in changeset.
[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren

- Reviewopmerking

- Productiewijze niet hardcoderen op volledig want er zijn natuurlijk ook nog andere productiewijzen

  1. … 4 more files in changeset.
Ik denk het wel, het zit namelijk wat raar in elkaar en ik denk dat het enkel op deze manier werkt als ik het me goed herinner

Ik denk het wel, het zit namelijk wat raar in elkaar en ik denk dat het enkel op deze manier werkt als ik het me goed herinner

De constructie in deze klasse is volgens mij een beetje scheefgetrokken : Bovenstaande lijnen horen thuis in private method VulAanMatKenmerken() Het is niet aangewezen vanuit een private method opn...

De constructie in deze klasse is volgens mij een beetje scheefgetrokken :
Bovenstaande lijnen horen thuis in private method VulAanMatKenmerken()
Het is niet aangewezen vanuit een private method opnieuw de Accept($this) op te roepen.
En de "Set ..IngegevenKenmerken = MatKenmerken" maakt deze visitor helemaal tricky.
Best effe samen overleggen, om te verbeteren.

N.B.: De logica lijkt momenteel wel te werken, dus t is geen showstopper, mag gedeployed worden. Maar niet forever laten staan.

"RH" is RugMetHouders --> zowel in de enum ECON en DOM. Dus lijn 169 is identiek aan lijn 165. Voor mij kan deze laatste daarom weg (denk ik) Bij BodemEnRug zou ik lijn 167 aanpassen naar :       ...

"RH" is RugMetHouders --> zowel in de enum ECON en DOM.
Dus lijn 169 is identiek aan lijn 165. Voor mij kan deze laatste daarom weg (denk ik)

Bij BodemEnRug zou ik lijn 167 aanpassen naar :

        If (Kenmerken.ProductieWijze = "RB") Set ProductieWijze = ..Enum.ProductieWijze.BodemEnRug 

idem RFB ( ? )

Commentaar mag weg, of desnoods aan te passen naar : Mengvorm (design) valt ook onder Variant "TBX_Antaro", want het zit volledig in hetzelfde model. Voor het onderscheid met mengvorm wordt enkel L...

Commentaar mag weg, of desnoods aan te passen naar :
Mengvorm (design) valt ook onder Variant "TBX_Antaro", want het zit volledig in hetzelfde model. Voor het onderscheid met mengvorm wordt enkel LadeDesign en DesignCode gebruikt.

FYI :
In eCon staat DATA.Kenmerken.Variant = "TBX", dit is eigenlijk de definitief juiste versie!
In Caché wordt "TBX" vertaald/geconverteerd naar "TBX_Antaro" omdat we nog steeds de oude waarde/benaming volgen. Dat zorgt dus helaas voor verwarring :-P

Deze speciale constructie begrijp ik eigenlijk helemaal niet. If "Enkele Uitsparing" dan is RugX (breedte) ook gewoon gelijk aan TbxKenmerken.UitsparingPositie1 M.a.w. voor mij is RugX altijd gelij...

Deze speciale constructie begrijp ik eigenlijk helemaal niet.
If "Enkele Uitsparing" dan is RugX (breedte) ook gewoon gelijk aan TbxKenmerken.UitsparingPositie1
M.a.w. voor mij is RugX altijd gelijk aan UitsparingPositie1

Zie ik iets over het hoofd?

De oude Rolnamen blijven bestaan, en de nieuwe worden erbij gevoegd. Zou je een Replace() overwegen? Dat voelt bij mij alvast iets "cleaner" aan. Zo ja, geef ik als tip : maak gebruik van deze Util...

De oude Rolnamen blijven bestaan, en de nieuwe worden erbij gevoegd. Zou je een Replace() overwegen? Dat voelt bij mij alvast iets "cleaner" aan.
Zo ja, geef ik als tip : maak gebruik van deze Util :

    Do ##class(TECH.ListUtils).ReplaceValuesInListOfDataTypes(NietMeeleveren,VervangArray)    

Dan zeker nakijken in .NET dat je toch nergens gebruik maakt van de oude benaming hé.

Er is een mismatch bij de MinTolerantie : *in de lijn hierboven wordt ze opgevraagd via de LegacyKenmerken *in de private methods hieronder gebruiken we de GUIKenmerken --> ..GeefInterneBreedte()...

Er is een mismatch bij de MinTolerantie :

  • in de lijn hierboven wordt ze opgevraagd via de LegacyKenmerken
  • in de private methods hieronder gebruiken we de GUIKenmerken --> ..GeefInterneBreedte() en ..GeefMinTolerantieTBX()


Ik denk dat we dit best effe samen moeten bekijken.

iets voorzichtiger maar vooral iets duidelijker (je enige doel is om aan te passen wanneer de derde letter een "S" is, right?) :         $$$If($E(TbxKenmerken.BodemKleur,3)="S", $Extract(TbxKenmer...

iets voorzichtiger maar vooral iets duidelijker (je enige doel is om aan te passen wanneer de derde letter een "S" is, right?) :

        $$$If($E(TbxKenmerken.BodemKleur,3)="S", $Extract(TbxKenmerken.BodemKleur,0,2), TbxKenmerken.BodemKleur)
[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren
[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren
[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren
[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren

- Sepciaal ladekleur vanuit econ opvangen in convertor

  1. … 1 more file in changeset.
[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren

- Er ontbrak een {}

[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren

- MinTolerantieOpIB toevoegen aan convertor

[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

- Conversieketting : EconNaarApps : bij afwijkende MTIB de InterneBreedte rechtstreeks doorgeven (van Econ naar Apps, zonder de tussenstap EdiNaarLegacy) om een andere canonische Waarde te bekomen bij het bewaren (DOM-laag)

[ICT-2359] TBX BOMBOL builders porten naar .NET

- Configsetting is ook geldig om extra conversie te doen

[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren

- Verder uitwerken EconNaarApps convertor voor TBX

[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren

- Properties in alfabetische volgorde zetten van tbx dotnetkenmerken

- Dom naar dotnetconvertor verder uitwerken

- VoorwandVerhogingstype aanvullen in econ naar appsconvertor

  1. … 2 more files in changeset.
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Zijkanthoogte wordt goed opgevuld door ConverteerGegenereerdeProperties -> niet overschrijven

[ICT-2359] TBX BOMBOL builders porten naar .NET

- Nieuwe extra code onder projectsetting zetten in EconNaarApps, anders zouden we wel es voor verrassingen kunnen komen te staan in de canonische waarde :|

[ICT-2365] Opstellen .NET TBX dto en uitbreiden bestaande TBX convertoren

- EconNaarApps convertor uitbreiden wanneer projectsetting actief is

Kaartje aangemaakt: VRB614 http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Kaartje aangemaakt: VRB614