CacheAdminA_trunk2010/APPS/EC/impl/ConfiguratorService

KenmerkMogelijkhedenPostProcessor.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 9
Last Commit: 03 Mar 23
Commits this week: 0
Total Lines of Code (LoC): 4.672
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 16 Mar 2023
8:52 am

Peter Vercoutter

8:35 am

Peter Vercoutter

[ICT-3470] Opkuisen code LBX LadeKleur PS-M en BodemKleur LG
[ICT-3470] Opkuisen code LBX LadeKleur PS-M en BodemKleur LG
Monday 08 Aug 2022
1:08 pm

Peter Vercoutter

9:31 am

Jelle Baele

9:11 am

Jelle Baele

Hier staat nog een regel commentaar dat waarschijnlijk weg mag

Hier staat nog een regel commentaar dat waarschijnlijk weg mag

9:09 am

Jelle Baele

Geen idee of dit van u is, maar hier staat een schrijffout http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Geen idee of dit van u is, maar hier staat een schrijffout

Friday 05 Aug 2022
12:47 pm

Peter Vercoutter

[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
Friday 24 Jun 2022
1:46 pm

Jelle Baele

11:05 am

Peter Vercoutter

10:59 am

Peter Vercoutter

wlipke vergeten weg te doen

wlipke vergeten weg te doen

10:59 am

Peter Vercoutter

Moet bodemkleur hier ook tussen staan of niet?

Moet bodemkleur hier ook tussen staan of niet?

10:28 am

Jelle Baele

10:27 am

Jelle Baele

[ICT-2894] [rvPVR] LBX - Introductie nieuw kleur ladezijkant CS-M
[ICT-2894] [rvPVR] LBX - Introductie nieuw kleur ladezijkant CS-M
Thursday 11 Feb 2021
1:01 pm

Peter Vercoutter

11:30 am

Tom Vermeulen

Tuesday 09 Feb 2021
11:15 am

Peter Vercoutter

[ICT-1482] TA’ORBOX front op een LEGRABOX lade
[ICT-1482] TA’ORBOX front op een LEGRABOX lade
Thursday 28 May 2020
1:19 pm

Peter Vercoutter

Tuesday 26 May 2020
4:03 pm

Tommy Hebb

4:03 pm

Tommy Hebb

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

Tommy Hebb

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

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

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.

Friday 22 May 2020
1:38 pm

Peter Vercoutter

[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte
[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte
Thursday 10 Oct 2019
9:58 am

Kurt Callebaut

Thursday 26 Sep 2019
10:35 am

Wim Vermeulen

10:35 am

Wim Vermeulen

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

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

Wednesday 25 Sep 2019
10:56 am

Martijn Vierstraete

10:56 am

Martijn Vierstraete

Wordt de code ivm wolvilt mettertijd weggewerkt?

Wordt de code ivm wolvilt mettertijd weggewerkt?

10:51 am

Martijn Vierstraete

[ICT-684] Uitfaseren wolvilt matjes
[ICT-684] Uitfaseren wolvilt matjes
Thursday 12 Sep 2019
3:52 pm

Tommy Hebb