CacheAdminA_trunk2010/vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement

Boring

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 23 Jan 20
Commits this week: 0
Total Lines of Code (LoC): 365
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
Monday 23 Nov 2020
8:56 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:51 am

Sam Van Hoey

8:47 am

Sam Van Hoey

8:47 am

Sam Van Hoey

Monday 20 Jul 2020
11:57 am

SamV

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

-Kleur aangepast van Tcross + coordinates + puntboring ipv rugboring

Options
    • -6
    • +6
    ./BoringELementFactory/Test.cls.xml
  2. … 7 more files in changeset.
Tuesday 14 Jul 2020
3:25 pm

SamV

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

-Testfiles aangepast door aanpassingen kleur + test aanpassingen door grotere cirkel

Options
    • -2
    • +2
    ./BoringELementFactory/Test.cls.xml
  2. … 5 more files in changeset.
Wednesday 24 Jun 2020
3:23 pm

SamV

[ICT-1610] SVG-034 Package SvgTools verplaatsen naar TECH

-De package svg tooling verplaatst naar TECH door move package zonder svnmove om fouten te voorkomen

Options
    • -8
    • +8
    ./BoringELementFactory/Test.cls.xml
  2. … 14 more files in changeset.
Tuesday 23 Jun 2020
12:15 pm

SamV

[ICT-1125] Kaderdeurbeelden omzetten van VML naar SVG

-Code Cleanup

Options
    • -0
    • +8
    ./BoringELementFactory/Test.cls.xml
  2. … 4 more files in changeset.
Tuesday 07 Apr 2020
1:50 pm

SamV

[ICT-1367] SVG-030 Aanmaken BoringElementFactory

-dummy objecten generieker maken + kleur en rotation apart van default invullen en controleren in svg resultaat

Options
    • -22
    • +22
    ./BoringELementFactory/Test.cls.xml
Monday 06 Apr 2020
11:04 am

SamV

[ICT-1367] SVG-030 Aanmaken BoringElementFactory

-specifiekere getallen bij aanroepen methodes + dummyObjecten + groeperen

Options
    • -42
    • +42
    ./BoringELementFactory/Test.cls.xml
  2. … 1 more file in changeset.
Thursday 02 Apr 2020
6:21 pm

Wim Vermeulen

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

12:12 pm

Wim Vermeulen

beetje muggeziften, maar maakt de method wel iets overzichtlijker : --> zet de 2 lijnen met Doorboring.SvgBox mee bovenaan, dan staan de (3) lijnen met Doorboring-object bij elkaar en de (3) lijnen...

beetje muggeziften, maar maakt de method wel iets overzichtlijker :
--> zet de 2 lijnen met Doorboring.SvgBox mee bovenaan, dan staan de (3) lijnen met Doorboring-object bij elkaar en de (3) lijnen met Circle-object staan dan ook bij elkaar.

Een andere volgorde, op basis van een andere logica, is wellicht ook oké.

12:04 pm

Wim Vermeulen

Deze klasse mag idd weg.

Deze klasse mag idd weg.

12:03 pm

Wim Vermeulen

Zelfde opmerking als bij ICT-1374 CrossFactory UT : Welgemikte getallen kiezen : bvb MaakBoring (60, 40, 5, Kleur.blauw) Een essentiele verantwoordelijkheid van de UT is om bij (x,y)-coordinaten e...

Zelfde opmerking als bij ICT-1374 CrossFactory UT :
Welgemikte getallen kiezen : bvb MaakBoring (60, 40, 5, Kleur.blauw)

Een essentiele verantwoordelijkheid van de UT is om bij (x,y)-coordinaten een foutieve (y,x)-switch te detecteren.

11:52 am

Wim Vermeulen

Zelfde opmerking als bij ICT-1374 CrossFactory UT : DummyCirkel en (misschien) DummySvgBox --> dit laatste is mogelijk een beetje tricky :-P

Zelfde opmerking als bij ICT-1374 CrossFactory UT :
DummyCirkel en (misschien) DummySvgBox --> dit laatste is mogelijk een beetje tricky :-P

11:47 am

Wim Vermeulen

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