Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[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

  1. … 1 more file in changeset.
[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Foutje dat SvgBox niet werd leeggemaakt dus meerdere keren volledig werd weergeven in de XML

  1. … 1 more file in changeset.
[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Extra specialeBoringen toegevoegd na testen van kaderdeuren en daarvoor aanpassingen voor bestaande testen

  1. … 4 more files in changeset.
[ICT-1663] SVG-038 Kleine aanpassingen tijdens het testen van Kaderdeuren

-Kleur aangepast van Tcross + coordinates + puntboring ipv rugboring

  1. … 7 more files in changeset.
[ICT-1656] SVG-036 HoekverbinderToevoeger + maker refactoren

-Aanpassingen aan Mock + Fakes

  1. … 2 more files in changeset.
[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

    • -4
    • +4
    ./Svg/BasisElement/CrossFactory.cls.xml
  1. … 12 more files in changeset.
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.

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é.

Deze klasse mag idd weg.

Deze klasse mag idd weg.

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.

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

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)

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)

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif.

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen .

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er i...

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er in deze klasse gebruikt worden

ge kunt het niet laten hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (hieronder nog een paar)

ge kunt het niet laten hé (hieronder nog een paar)

[ICT-1378] SVG-032 Toevoegers Mockable maken

-SchuineHoekenToevoeger.cls.xml mockable gemaakt en return types toegevoegd

    • -0
    • +27
    ./Svg/Kaderdeur/SchuineHoekenToevoeger.cls.xml
  1. … 2 more files in changeset.
[ICT-1378] SVG-032 Toevoegers Mockable maken

-LosProfielToevoeger.cls.xml mockable gemaakt en return types toegevoegd

    • -0
    • +27
    ./Svg/Kaderdeur/LosProfielToevoeger.cls.xml
  1. … 2 more files in changeset.
[ICT-1378] SVG-032 Toevoegers Mockable maken

-HoekverbinderToevoeger.cls.xml mockable gemaakt en return types toegevoegd

    • -0
    • +34
    ./Svg/Kaderdeur/HoekverbinderToevoeger.cls.xml
  1. … 2 more files in changeset.
[ICT-1378] SVG-032 Toevoegers Mockable maken

-KaderToevoeger.cls.xml mockable gemaakt en return types toegevoegd

    • -0
    • +34
    ./Svg/Kaderdeur/KaderToevoeger.cls.xml
  1. … 3 more files in changeset.