KenmerkMogelijkhedenPostProcessor

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte

- Enkele testen faalden zonder voor de hand liggende reden, deze wijziging laat de testen slagen en zo blijft de configurator in vhintra op dezelfde manier werken

[ICT1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte

- Volledig onbeschikbaar maken van Matmateriaal Antislip

    • -16
    • +20
    ./AntislipMattenUitfaseren.cls.xml
[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte (commit voor PeterV)

- Uitfaseren Antislip ZWART voor ladediepte 350

[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte (commit voor PeterV)

- Refactor AntislipMattenUitfaseren : Lijst van uitgefaseerde dieptes zit nu in 2 aparte methods (1 voor Zwart en 1 voor Antraciet)

- Bij melding van PM aan te passen : methods onderaan , alsook bij VerwerkLBXKenmerkenVoorAntislip() het tweede deel, i.e. "... en omgekeerd ... verwijderen bij ongeldige MatMaterialen)"

- Een eerste voorbeeldje in volgende commit

    • -24
    • +35
    ./AntislipMattenUitfaseren.cls.xml
[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte

- 2e en 3e fase uitfaseringen: Matmateriaal AS en Matkleur zwart zijn niet meer beschikbaar voor ladediepte 500,600,650

- Matmateriaal AS en Matkleur Antraciet zijn niet meer beschikbaar voor ladediepte 500,550,650

- Bij TAX alles van matmateriaal antislip uitfaseren in alle kleuren en materiaal zelf

    • -92
    • +34
    ./AntislipMattenUitfaseren.cls.xml
[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte

- 1e fase uitfaseringen: Matmateriaal AS en Matkleur zwart zijn niet meer beschikbaar voor ladediepte 450 en 550

    • -19
    • +24
    ./AntislipMattenUitfaseren.cls.xml
  1. … 1 more file in changeset.
[ICT-1529] Code ivm MatMateriaal Wolvilt opkuisen in Cache
    • -168
    • +0
    ./WolvitMattenUitfaseerder.cls.xml
  1. … 2 more files in changeset.
Properder: If ((MatKleur = "Z") & (LadeDiepte > 450)) Dat geeft at runtime exact hetzelfde resultaat, maar is 1 If ipv een geneste If, dus leesbaarder. Idem voor die hieronder...

Properder: If ((MatKleur = "Z") & (LadeDiepte > 450))
Dat geeft at runtime exact hetzelfde resultaat, maar is 1 If ipv een geneste If, dus leesbaarder.

Idem voor die hieronder...

Hier een beetje hetzelfde... nieuwe code, ook al is het gebaseerd op een copy/paste, graag refactoren volgens huidige conventies. Je hoeft het nu niet meer aan te passen. Onthoud het voor de volgen...

Hier een beetje hetzelfde... nieuwe code, ook al is het gebaseerd op een copy/paste, graag refactoren volgens huidige conventies. Je hoeft het nu niet meer aan te passen. Onthoud het voor de volgende keer.

"$lb" = $ListBuild
"$lg" = $ListGet
"n" = New
"s" = Set
"q" = Quit
Wat spaties hier en daar voor en na =-teken en na komma en zo.

Opletten voor casing graag http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Variabele-namen beginnen met hoofdletter (Msg, Bool). Misschien de variabele...

Opletten voor casing graag

Variabele-namen beginnen met hoofdletter (Msg, Bool).
Misschien de variabele Bool dan ineens hernoemen naar het meer beschrijvende: IsValid.
Set met hoofdletter.

P.s.: OMG wat een klasse!!! lol Je hoeft niet de hele klasse aan te passen in dit geval (veel te veel werk), maar als je code toevoegt, zet die dan wel ineens volgens onze huidige conventies.

Als alle lijnen in comment staan, misschien toch een extra comment-lijntje toevoegen met de reden waarom. Ik vermoed in afwachting van dat de code actief moet gezet worden? Maar de call zelf staat ...

Als alle lijnen in comment staan, misschien toch een extra comment-lijntje toevoegen met de reden waarom. Ik vermoed in afwachting van dat de code actief moet gezet worden? Maar de call zelf staat toch in comment in de klasse APPS.EC.impl.ConfiguratorService.KenmerkMogelijkhedenPostProcessor?

Anderzijds... moet je hier niet enkel diegene voorzien die in klasse APPS.EC.impl.ConfiguratorService.KenmerkMogelijkhedenPostProcessor nog niet voorzien waren?

Louter ter info: Ook nog even meegeven dat iedere enum, die dus per definitie af moet leiden van TECH.Enumeration, automatisch daarom ook beschikt over een ValueListIterator. Als je dus alle enumwaarden van een enum moet overlopen, dan is gebruik maken van die iterator wellicht overzichtelijker dan voor iedere waarde een codelijn te voorzien.

[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte
[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte
[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte

- Klaarzetten code voor uitfasering antislip voor LBX, TAX en MAT configurator

    • -0
    • +219
    ./AntislipMattenUitfaseren.cls.xml
  1. … 1 more file in changeset.
Ja, idd. Nadat de nieuwe matten (ago-Flock) volledig ingefaseerd zijn.

Ja, idd.
Nadat de nieuwe matten (ago-Flock) volledig ingefaseerd zijn.

Wordt de code ivm wolvilt mettertijd weggewerkt?

Wordt de code ivm wolvilt mettertijd weggewerkt?

[ICT-684] Uitfaseren wolvilt matjes
[ICT-684] Uitfaseren wolvilt matjes
[ICT-684] Uitfaseren wolvilt matjes

- WolviltUitfaseerder : laatste versie van (openstaande wijzigingen op) Cache01 gecommit.

    • -27
    • +23
    ./WolvitMattenUitfaseerder.cls.xml
[ICT-684] Uitfaseren wolvilt matjes

- WolvitMattenUitfaseerder niet abstract

[ICT-684] Uitfaseren wolvilt matjes

- KenmerkMogelijkheden : code voor Wolvilt afgezonderd in aparte klasse (specifiek voor de KenmerkMogelijkhedenPostProcessor

    • -0
    • +173
    ./WolvitMattenUitfaseerder.cls.xml