Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1438] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte (commit By PeterV - PVR)

- Uitfasering voor antislip: Matkleur Zwart en ladediepte 500,550,650 ook niet meer mogelijk

- Uitfasering voor antislip: Matkleur Antraciet en ladediepte 500,550,650 ook niet meer mogelijk

    • -7
    • +5
    ./impl/AntislipMattenUitfaseren.cls.xml
[ICT-1438] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte (commit By PeterV - PVR)

- Uitfasering voor antislip: Matkleur Zwart en ladediepte 550 ook niet meer mogelijk

    • -1
    • +1
    ./impl/AntislipMattenUitfaseren.cls.xml
[ICT-1438] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte (commit By PeterV - PVR)

- Eerste fase uitfasering voor antislip: Matkleur Zwart en ladediepte 450 niet meer mogelijk

    • -5
    • +3
    ./impl/AntislipMattenUitfaseren.cls.xml
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-1438] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte

- Momenteel staat de toegevoegd code nog in commentaar, op deze manier is het makkelijk om de code te actieveren onder de juiste condities

- Zowel een change in ladediepte als in matkleur zullen checken indien de combinatie mat/ladediepte wel nog geldig is en zal een melding weergeven indien dit niet zo is

    • -0
    • +30
    ./impl/AntislipMattenUitfaseren.cls.xml
  1. … 1 more file in changeset.
[BUS610] TBX mengvorm TOB : TECH.RegisteredObject

Merged revision(s) 44708 from trunk2010:

[BUS610] TBX mengvorm TOB : MengvormTDesign toegevoegd.

........

  1. … 1 more file in changeset.
Merged revision(s) 41307 from trunk2010:

[BUS610] TBX mengvorm TOB (enkel intern)

- de aanpassing heeft niks met intern/extern te maken. (alleen het feit dat mengvorm intern is :-P )

........

  • More
  • CR-356
  • finished reviewing
[UST2537] TBX: Sifon type 3 : BOM/BOL aanpassen in TBX MetaDesigner.xls
[UST2537] TBX: Sifon type 3 : BOM/BOL aanpassen in TBX MetaDesigner.xls
[UST2537] TBX: Sifon type 3 : BOM/BOL aanpassen in TBX MetaDesigner.xls

- Berekening van HoutschroevenVoorBodemRug (method opgeroepen vanuit de MetaDesigner)

    • -2
    • +2
    ./impl/AantalHoutschroevenVoorBodemRugBepaler.cls.xml
  1. … 1 more file in changeset.
[UST2537] TBX: Sifon type 3 : BOM/BOL aanpassen

- AantalHoutschroevenVoorBodemRug

    • -1
    • +4
    ./impl/AantalHoutschroevenVoorBodemRugBepaler.cls.xml
  • More
  • CR-213
  • finished reviewing
[BUS596][BUS614] TBX Tipon-geleiders uitfaseren (volledig) / TBX 30kg TOB toelaten voor...
[BUS596][BUS614] TBX Tipon-geleiders uitfaseren (volledig) / TBX 30kg TOB toelaten voor...
[BUS596] TBX Tipon-geleiders uitfaseren - Validatie (+melding) toegevoegd voor...
[BUS596] TBX Tipon-geleiders uitfaseren - Validatie (+melding) toegevoegd voor...
Zinloze commentaar opkuisen

Zinloze commentaar opkuisen

Kuist de kak eens op

Kuist de kak eens op

  • More
  • CR-208
  • finished reviewing
[BUS585] TBX 30kg TOB toelaten (voor intern)
[BUS585] TBX 30kg TOB toelaten (voor intern)
[BUS614][BUS596][BUS585] TBX Tipon-geleiders uitfaseren (volledig) / TBX 30kg TOB toelaten voor ladeDiepte 600mm

- Draagkracht 65kg via "automatisch aangepast" ipv. via "IsUitgefaseerd" te misbruiken :-P

- zie vorige commit (rev.nr 41396)

    • -3
    • +0
    ./impl/GeleiderBeschikbaarBepaler.cls.xml