CacheAdminA_trunk2010/APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur

KaderToevoeger.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 09 Dec 20
Commits this week: 0
Total Lines of Code (LoC): 93
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
Wednesday 09 Dec 2020
10:33 am

SamV

[ICT-2059] SVG-040 Aanpassingen voor finale html

-ID toegevoegd aan tags + aanpassingen aan unitTests

Options
    • -5
    • +5
    ./KaderToevoeger.cls.xml
  2. … 9 more files in changeset.
Monday 23 Nov 2020
8:56 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:55 am

Sam Van Hoey

Monday 10 Aug 2020
12:08 pm

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-If om te kijken naar het type vulling toegevoegd zo dat als het "T" is deze niet groen kleurt maar ook lichtgrijs, werd in de oude code gedaan met opacity maar in svg verdwijnt dan ook de rand dus voeg nu de kleur grijs toe.

Options
    • -1
    • +4
    ./KaderToevoeger.cls.xml
10:59 am

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Formattering aangepast

Options
    • -10
    • +10
    ./KaderToevoeger.cls.xml
  2. … 7 more files in changeset.
Monday 20 Jul 2020
1:04 pm

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Aanpassing kleur veranderen van kader

Options
    • -1
    • +1
    ./KaderToevoeger.cls.xml
Tuesday 14 Jul 2020
3:14 pm

SamV

[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Kleuren aangepast + posities van verschillende rotaties

Options
    • -2
    • +2
    ./KaderToevoeger.cls.xml
  2. … 2 more files in changeset.
Tuesday 23 Jun 2020
12:06 pm

SamV

[ICT-1125] Kaderdeurbeelden omzetten van VML naar SVG

-Code Cleanup

Options
    • -0
    • +5
    ./KaderToevoeger.cls.xml
  2. … 8 more files in changeset.
Tuesday 07 Apr 2020
8:40 am

Tom Vermeulen

Thursday 02 Apr 2020
6:21 pm

Wim Vermeulen

6:10 pm

Wim Vermeulen

Ik ben nogal verward hoe deze test is opgebouwd. Mogelijk is dit veroorzaakt door misleidende naamgevingen. Ook conceptueel lijkt er iets niet te kloppen. Laten we zeggen dat de test aan het licht ...

Ik ben nogal verward hoe deze test is opgebouwd. Mogelijk is dit veroorzaakt door misleidende naamgevingen.
Ook conceptueel lijkt er iets niet te kloppen. Laten we zeggen dat de test aan het licht brengt dat de interfaces niet helemaal goed zitten.
Aan deze UT's (en code) gaan we best wat knutselen. Contact me

5:56 pm

Wim Vermeulen

4:54 pm

Wim Vermeulen

4:43 pm

Wim Vermeulen

Deze method gaan we eens samen bekijken: *afsplitsen in meerdere private methods : bevordert de leesbaarheid, maar bovenal vermijdt dependencies tussen stukken code die niks met mekaar te maken h...

Deze method gaan we eens samen bekijken:

  • afsplitsen in meerdere private methods : bevordert de leesbaarheid, maar bovenal vermijdt dependencies tussen stukken code die niks met mekaar te maken hebben.
  • oproep/interface geschikt maken om vanuit Prod.GADef.Kaderdeur op te roepen.
  • UT's rond GenereerSvg() zou ik een andere aanpak voorstellen. Contact me
4:20 pm

Wim Vermeulen

4:19 pm

Wim Vermeulen

Na een Catch() moet er altijd een "reporting" of een nieuwe throw gedaan worden. In dit geval zegt mijn gevoel dat hier geen try-catch moet staan, want het is aan de gebruiker/oproepende code om ze...

Na een Catch() moet er altijd een "reporting" of een nieuwe throw gedaan worden.
In dit geval zegt mijn gevoel dat hier geen try-catch moet staan, want het is aan de gebruiker/oproepende code om zelf te bepalen welke error-handling er nodig is.

Anderszijds lijkt dit meer op een stukje test-code, dan kan je dat best ook zo aangeven (in methodnaam of ev in de comment) of moet je deze testcode verplaatsen naar een zTryout-klasse.
Zo niet, dan krijg je de reviewer(s) op uw dak :-D

Thursday 26 Mar 2020
3:03 pm

SamV

[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger

-.%New() in constructor plaatsten indents aangepast

Options
    • -14
    • +12
    ./KaderToevoeger.cls.xml
  2. … 2 more files in changeset.
Wednesday 25 Mar 2020
12:21 pm

Tom Vermeulen

12:17 pm

Tom Vermeulen

Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets

Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets

12:13 pm

Tom Vermeulen

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

12:12 pm

Tom Vermeulen

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is (kleine kaartjes, rap committen, enz enz http://subversion02.vanhoecke.b...

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is

(kleine kaartjes, rap committen, enz enz )

12:10 pm

Tom Vermeulen

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen. Idem bij constructors andere boringen

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen.

Idem bij constructors andere boringen

12:07 pm

Tom Vermeulen

klasse newen in constructor

klasse newen in constructor

12:05 pm

Tom Vermeulen

indents ongedaan maken

indents ongedaan maken

12:03 pm

Tom Vermeulen

12:03 pm

Tom Vermeulen

Dit is opeetcode. Als er een exception geraised wordt, gaat niemand het weten. Is dit het gewenste gedrag? Indien niet: er iets mee doen (en testje(s) voor schrijven)

Dit is opeetcode. Als er een exception geraised wordt, gaat niemand het weten. Is dit het gewenste gedrag? Indien niet: er iets mee doen (en testje(s) voor schrijven)

12:01 pm

Tom Vermeulen

indenten tussen de accolades

indenten tussen de accolades

12:00 pm

Tom Vermeulen