Deploy20161020xUpdatePasDocMattenlijst.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
els if van maken controle op count() is gemeenschappelijk en kan in 1 instructie. if (count()=0) ( insert mat) else if (overeenkosmtigemat gevonden) (insert mat)

els if van maken controle op count() is gemeenschappelijk en kan in 1 instructie.

if (count()=0) ( insert mat)
else if (overeenkosmtigemat gevonden) (insert mat)

Algemene richtlijn voor ons, niet blokkerend, maar naar leesbaarheid spatie gebruik bij = en + tekens.

Algemene richtlijn voor ons, niet blokkerend, maar naar leesbaarheid spatie gebruik bij = en + tekens.

idem

idem

ik zou Set Mat gebruiken of bij dim een type meegeven.

ik zou Set Mat gebruiken of bij dim een type meegeven.

[ICT-1115] HLX: mattenlijst voor TBX & LBX (toeleveringsnummer)
[ICT-1115] HLX: mattenlijst voor TBX & LBX (toeleveringsnummer)
[ICT-1115] HLX: mattenlijst voor TBX & LBX (toeleveringsnummer)

- Kolomheader aangepast van "Toe:Dos" naar "Toelev" voor alle mattenlijsten

- Opbouw van de mattenlijsten voor TBX werden aangepast zodat matten met dezelfde afmetingen en toelevering gegroepeerd werden naar 1 regel maar dan met een hoger aantal te fabriceren matten

    • -2
    • +2
    ./Deploy20161020xUpdatePasDocMattenlijst.cls.xml
  1. … 2 more files in changeset.
LET OP: ALTIJD wanneer je storage-data aanpast rechtstreeks in de globals, dan nakijken of deze klasse indexen bevat. In dat geval kan je best de ...%BuilldIndices(,1) oproepen, zelfs wanneer dat g...

LET OP: ALTIJD wanneer je storage-data aanpast rechtstreeks in de globals, dan nakijken of deze klasse indexen bevat.
In dat geval kan je best de ...%BuilldIndices(,1) oproepen, zelfs wanneer dat geen verschil zou geven. Play safe

Ook al is het maar een deploy-klasse, toch zou ik duidelijk maken dat ID 35 vast is. Bijvoorbeeld door als variabele te definieren : #dim ActivieitGroepId As %Integer = 35 ; Hard-coded, want vast...

Ook al is het maar een deploy-klasse, toch zou ik duidelijk maken dat ID 35 vast is.
Bijvoorbeeld door als variabele te definieren :

#dim ActivieitGroepId As %Integer = 35   ; Hard-coded, want vaste ID
Set ^APPS.HaluxA368.ActiviteitGDA7FD(ActivieitGroepId)=$lb("~APPS.Halu... 
Voor mij persoonlijk is dit één stapje te ver gerefactored. De method ZetProductGegevens() zou ik niet gebruiken en MaatwerkProduct niet als property maar gewoon als variabele (wordt slechts 1x geb...

Voor mij persoonlijk is dit één stapje te ver gerefactored.
De method ZetProductGegevens() zou ik niet gebruiken en MaatwerkProduct niet als property maar gewoon als variabele (wordt slechts 1x gebruikt). Voor ingegevenKenmerken wel O.K.

Dan zou de code er zo uit zien :

#dim MaatwerkProduct As DOM.PM.MaatwerkProduct = ..ProductTypeApi.GeefMaatwerkProduct(ProductId)
Set ..IngegevenKenmerken = MaatwerkProduct.GeefIngegevenKenmerken()
ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
[ICT-930] Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)

- fix matten definitie van de kolommen.

Do ##class(vhDeploy.Deploy20161020xUpdatePasDocMattenlijst).ExecCode() overschrijft data die gegenereerd wordt door maakdocument bij wijzigen van activiteiten via Do ##class(vhDeploy.Deploy20161020xUpdatePasDocMattenlijst).ExecCode().

Door gebruik te maken van deze laatste klasse was er een oude definite terug gezet.

maw (nog te bekijken/refactoren) zodat enkel via jusite weg de global aangepast wordt.)

    • -0
    • +2
    ./Deploy20161020xUpdatePasDocMattenlijst.cls.xml
  1. … 2 more files in changeset.
[UST1925] - MAT-OP-PPS-DOCS : Type en kleur op mattenlijst wordt één veld

- Legende ook verwijderd - was afgesproken met APO dit om papier te besparen.

- Voorlopig is het nog niet de correct mat beschrijving -> volgt later

    • -0
    • +35
    ./Deploy20161020xUpdatePasDocMattenlijst.cls.xml
  1. … 19 more files in changeset.