Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2946] [rvWV] Lodderkeukens W7 aanpassen SWS mail subject:

- De subject van de mail aan SWS wordt nu verrijkt met toeleveringsnummer en klantgegevens (klantnummer en -naam)

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt http://sub...

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt )
De impl in deze afgeleide klasse is wel correct.

De implementatie voor "Lengte" hieronder zou ik wel in een private method steken, analoog aan GeefFrontPlaatHoogte()
Zo blijft deze method simpel leesbaar en overzichtelijk. En dan is de scope ook duidelijk, nl. dat Kleur W7 enkel impact heeft op de kenmerk "Lengte, en niet op de "Breedte" van het Front.

copy-paste foutje : "... kan niet gemaild worden." (ook al is de conditie altijd false :-P )

copy-paste foutje :
"... kan niet gemaild worden."
(ook al is de conditie altijd false :-P )

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- ProductieDetailblad => nu ook gemaild naar Special Workshop indien IsProductMetBewerkingInSpecialWorkshop op ProductTypeApi = true

[VRB488] HXPlanning: Werkposten aansturen: TAX printen op andere A4-printer (ook VRB36):

- Na review => mini-refactortjes

- ProjectSettings IsActiefAnderePrinterVoorTaxBatchesBijHalux verwijderd

  1. … 10 more files in changeset.
ProjectSettings eruit gehaald

ProjectSettings eruit gehaald

Done!

Done!

$$$True ipv 1

$$$True ipv 1

geen .%New() in de methods. Afzonderen naar de constructor (zelfs als ge niet injecteert)

geen .%New() in de methods. Afzonderen naar de constructor (zelfs als ge niet injecteert)

Het logogedeelte is niet relevant in deze test. Kunt dat beter wegsmijten om op die manier ietske rapper aan de lezer duidelijk te maken wat de essentie is van deze test. (code wordt maar 1x geschr...

Het logogedeelte is niet relevant in deze test. Kunt dat beter wegsmijten om op die manier ietske rapper aan de lezer duidelijk te maken wat de essentie is van deze test. (code wordt maar 1x geschreven, maar veel keer gelezen)

[VRB488] HXPlanning: Werkposten aansturen: TAX printen op andere A4-printer (ook VRB36):
[VRB488] HXPlanning: Werkposten aansturen: TAX printen op andere A4-printer (ook VRB36):
[VRB488] HXPlanning: Werkposten aansturen: TAX printen op andere A4-printer (ook VRB36):

- PrinterID volgens config APPS.Halux.PPS.AansturingService.Printer.TAX indien TAX en ProjectSettings IsActiefAnderePrinterVoorTaxBatchesBijHalux