CacheAdminA_trunk2010/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: 28 Jan 20
Commits this week: 0
Total Lines of Code (LoC): 250
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:55 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:51 am

Sam Van Hoey

8:47 am

Sam Van Hoey

Monday 20 Jul 2020
12:14 pm

SamV

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

-Coordinates aangepast van zijboring voor wanneer deze x Coordinaat 0 is

Options
    • -0
    • +3
    ./BoringElementFactory.cls.xml
11:57 am

SamV

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

-Kleur aangepast van Tcross + coordinates + puntboring ipv rugboring

Options
    • -1
    • +1
    ./BoringElementFactory.cls.xml
  2. … 7 more files in changeset.
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
    • -1
    • +1
    ./BoringElementFactory.cls.xml
  2. … 2 more files in changeset.
Monday 29 Jun 2020
9:08 am

SamV

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

-Wlipjes weg

Options
    • -2
    • +0
    ./BoringElementFactory.cls.xml
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
    • +9
    ./BoringElementFactory.cls.xml
    • -1
    • +1
    ./PuntBoring.cls.xml
    • -1
    • +1
    ./DoorBoring.cls.xml
    • -1
    • +1
    ./ZijBoring.cls.xml
    • -1
    • +1
    ./RugBoring.cls.xml
  6. … 10 more files in changeset.
Monday 06 Apr 2020
11:04 am

SamV

[ICT-1367] SVG-030 Aanmaken BoringElementFactory

-specifiekere getallen bij aanroepen methodes + dummyObjecten + groeperen

Options
    • -11
    • +16
    ./BoringElementFactory.cls.xml
  2. … 1 more file in changeset.
Thursday 02 Apr 2020
6:21 pm

Wim Vermeulen

5:58 pm

Wim Vermeulen

5:58 pm

Wim Vermeulen

Deze method hoort niet thuis in deze klasse, want heeft niets met BeslagBoringen te maken. We zullen samen bekijken hoe we dit best kunnen afsplitsen.

Deze method hoort niet thuis in deze klasse, want heeft niets met BeslagBoringen te maken.
We zullen samen bekijken hoe we dit best kunnen afsplitsen.

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

10:53 am

Wim Vermeulen

10:51 am

Wim Vermeulen

Bij iedere UT moet je denken "Kan deze setup fouten verbergen?" of anders gezegd "Kan ik een programmeerfout aan het licht brengen, puur door andere input te kiezen?" Dus de (30, 30) zou je beter ...

Bij iedere UT moet je denken "Kan deze setup fouten verbergen?" of anders gezegd
"Kan ik een programmeerfout aan het licht brengen, puur door andere input te kiezen?"

Dus de (30, 30) zou je beter aanpassen naar verschillende waarden (zoals bvb. erboven 25,30)

10:49 am

Wim Vermeulen

Zoals het er nu staat, kan je enigszins verwarring scheppen tussen de echte MaakLijn() en de VerwachtMethodCall, aangezien deze (eerste) dezelfde parameters hebben. Bij voorkeur 2 verschillende (du...

Zoals het er nu staat, kan je enigszins verwarring scheppen tussen de echte MaakLijn() en de VerwachtMethodCall, aangezien deze (eerste) dezelfde parameters hebben.
Bij voorkeur 2 verschillende (dummy) Lijn-objecten gebruiken bij de DanReturn :

DummyLine1 = MaakLijn(0,0,0,0)
DummyLine2 = MaakLijn(0,0,0,0)

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