Activity CacheAdminA_trunk/UI/PR/GA/TandemBox.cls.xml

3:02 pm

Wim Vermeulen

More
62809
changed 2 files
root:

[ICT] TandemBox ingave + dlgOpties form :

- extra logging toegevoegd omdat de dlgOpties-form niet zichtbaar was bij KAE.

- nu met ClientIP.

- -0
- +5
./TandemBox.cls.xml
- History
- Source
- Diff
./TandemBox.cls.xml
… 1 more file in changeset.

1:19 pm

Admin

More
61025
changed 1 file
root:

[ICT-1438] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte (commit By PeterV - PVR)

- Verwerken review

- -13
- +13
./TandemBox.cls.xml
- History
- Source
- Diff
./TandemBox.cls.xml

4:03 pm

Tommy Hebb

More
Re: CR-2073
commented on review

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...

4:00 pm

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.

3:55 pm

Tommy Hebb

More
Re: CR-2073
commented on review

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.

3:44 pm

Tommy Hebb

More
Re: CR-2073
commented on review

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.

1:38 pm

Peter Vercoutter

More
CR-2073
started review

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

8:02 am

Admin

More
60997
changed 2 files
root: CR-2073

[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