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

Boringen.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 30 Oct 20
Commits this week: 0
Total Lines of Code (LoC): 209
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:56 am

Sam Van Hoey

8:55 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:51 am

Sam Van Hoey

8:51 am

Sam Van Hoey

Friday 30 Oct 2020
1:32 pm

Wim Vermeulen

REDO van commit 61921 (opgesplitst in 3 delen) :

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

-Wlips verwijderd + formattering

........

Options
    • -11
    • +5
    ./Boringen.cls.xml
  2. … 3 more files in changeset.
1:28 pm

Wim Vermeulen

UNDO van commit 61921 :

Merged revision(s) 61921 from trunk2010:

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

-Wlips verwijderd + formattering

........

Options
    • -5
    • +11
    ./Boringen.cls.xml
  2. … 5 more files in changeset.
Monday 10 Aug 2020
4:12 pm

SamV

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

-Wlips verwijderd + formattering

Options
    • -11
    • +5
    ./Boringen.cls.xml
  2. … 5 more files in changeset.
3:19 pm

SamV

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

-Aanpassingen van rotation 270 en 90 voor kadFreesRect heeft XPos nodig, dus toegevoegd aan methode

Options
    • -1
    • +2
    ./Boringen.cls.xml
10:59 am

SamV

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

-Formattering aangepast

Options
    • -28
    • +28
    ./Boringen.cls.xml
  2. … 7 more files in changeset.
10:26 am

SamV

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

-write weggehaald en YPos meegegeven aan BoringSpecialeVorm

Options
    • -2
    • +1
    ./Boringen.cls.xml
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
    • -2
    • +2
    ./Boringen.cls.xml
  2. … 7 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
    • -1
    • +3
    ./Boringen.cls.xml
  2. … 14 more files in changeset.
Tuesday 23 Jun 2020
12:06 pm

SamV

[ICT-1125] Kaderdeurbeelden omzetten van VML naar SVG

-Code Cleanup

Options
    • -3
    • +4
    ./Boringen.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: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: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

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