Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Een vraag bij deze deploy-klasse: Zou dit ook werken met gewoon "Do ##class(APPS.Halux.PPS.DocumentDefinitie.GekoppeldeIK.MattenLijst).%New().MaakDocument()"? Dit roept SaveDocument op, wat volgens...

Een vraag bij deze deploy-klasse:
Zou dit ook werken met gewoon "Do ##class(APPS.Halux.PPS.DocumentDefinitie.GekoppeldeIK.MattenLijst).%New().MaakDocument()"?
Dit roept SaveDocument op, wat volgens mij hetzelfde doet. Maar misschien mis ik hier iets waardoor het manueel gedaan moet worden.

Anyway, het is zeker niet fout

[ICT-3622] Mattenlijst - Kolom 'Tot#' als eerste kolom zetten
[ICT-3622] Mattenlijst - Kolom 'Tot#' als eerste kolom zetten
[ICT-3622] Mattenlijst - Kolom 'Tot#' als eerste kolom zetten
  1. … 4 more files in changeset.
[ICT-3594] Mattenlijst - totalen weergeven
[ICT-3594] Mattenlijst - totalen weergeven
[ICT-3594] Mattenlijst - totalen weergeven

- Kolom Tot# toevoegen aan mattenlijst en breedte van enkele andere kolommen beetje verkleinen

- mattenuitvoeringinfo dto uitbreiden met nieuw kenmerk zodat deze kan worden weergegeven op het document

- Logica toevoegen die de berekening van het totaal aantal matten uitvoert + deze logica in 2 klassen toevoegen omdat TBX apart zit

  1. … 19 more files in changeset.
[ICT-2908] [rvTVE] Uitzoeken waarom LBX naar 5 karakters gegaan is voor zijn dossiercode

- Ook bij de mattenlijst moet nog een kolom breder want daar staan 4 karakters van de toelevering en in alle andere documenten maar 3

  1. … 8 more files in changeset.
Misschien kunnen deze filters ook in een meer algemene klasse in een (bestaande) common package?

Misschien kunnen deze filters ook in een meer algemene klasse in een (bestaande) common package?

[ICT-2426] Nieuwe aansturingsactie MVX: Mattenlijsten
[ICT-2426] Nieuwe aansturingsactie MVX: Mattenlijsten
[ICT-2426] Nieuwe aansturingsactie MVX: Mattenlijsten

- Door het aanmaken van de mattenlijsten voor MVX werd de code getriggerd die de kolomdefinitie update. De 2 klasses die in sync moesten blijven stonden niet exact in sync. Daardoor is er een wijziging in de outputfiles voor de mattenlijsten, deze commit lost het probleempje op en op deze manier staat de code weer in sync en correct.

  1. … 1 more file in changeset.
lange statements proberen vermijden.je kan vergelijkingen vervangen door methods. ..IsProductiegroepKast() bv

lange statements proberen vermijden.je kan vergelijkingen vervangen door methods. ..IsProductiegroepKast() bv

or gebruiken. binnen de "if" is het zelfde code.

or gebruiken. binnen de "if" is het zelfde code.

indent gelijk houden. we formatteren meestal else op dezelfde lijn van de accolades. } else {

indent gelijk houden.

we formatteren meestal else op dezelfde lijn van de accolades.
} else {

..BatchService bestaat al, geen nieuwe instantie nodig.

..BatchService bestaat al, geen nieuwe instantie nodig.

ook hier methods maken van vergelijken, zo wordt de lijn korter en leesbaarder. (boolean steeds "Is" -> bv ..IsProductieGroepKast() etc

ook hier methods maken van vergelijken, zo wordt de lijn korter en leesbaarder. (boolean steeds "Is" -> bv ..IsProductieGroepKast() etc

zelfde "insert" code. ik zou geen elseif gebruiken, maar OR statement.

zelfde "insert" code. ik zou geen elseif gebruiken, maar OR statement.

kleintje : indent aanhouden.

kleintje : indent aanhouden.

Proberen onnodige wijzigingen te committen.

Proberen onnodige wijzigingen te committen.

ik zou hier if $$$Not(..IsStuk(Batch) && $$$Not(..IsFlatpack(Batch)) zetten. maw vergelijkingen naar een nieuwe method brengen waarvan de naam duidelijk aangeeft wat hij doet. Dan is de if statemen...

ik zou hier
if $$$Not(..IsStuk(Batch) && $$$Not(..IsFlatpack(Batch)) zetten.
maw vergelijkingen naar een nieuwe method brengen waarvan de naam duidelijk aangeeft wat hij doet. Dan is de if statement makkelijk leesbaar.

Quit enkel op eind van method gebruiken. Opvangen door een variabele in het begin van de method. en variabele te zetten in de "if" en mee te geven bij de quit.

Quit enkel op eind van method gebruiken. Opvangen door een variabele in het begin van de method. en variabele te zetten in de "if" en mee te geven bij de quit.

BatchService zou ik als property maken en toevoegen aan New van deze klasse, dan kan je hier enkel ..BatchService.GeefBatch() als code zetten.

BatchService zou ik als property maken en toevoegen aan New van deze klasse,
dan kan je hier enkel ..BatchService.GeefBatch() als code zetten.

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