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

-kleur en coordinaten aangepast

  1. … 2 more files in changeset.
[ICT-1610] SVG-034 Package SvgTools verplaatsen naar TECH

-Compile en toevoegen van CrossFactoryTest

    • -0
    • +151
    ./Tooling/CrossFactory/Test.cls.xml
[ICT-1125] Kaderdeurbeelden omzetten van VML naar SVG

-Code Cleanup

Indentatie (extreem rechts ? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif )

Indentatie (extreem rechts ? )

Deze lijn kan onder de if-structuur, moet niet telkens herhaald worden binnen de if.

Deze lijn kan onder de if-structuur, moet niet telkens herhaald worden binnen de if.

Ook doen of is het dan niet leesbaar genoeg?

Ook doen of is het dan niet leesbaar genoeg?

normaalgezien zou je heel deze if-elseif-else kunnen vervangen door Quit (Totaal && ##class(TECH.Math).IsEven(Totaal))

normaalgezien zou je heel deze if-elseif-else kunnen vervangen door Quit (Totaal && ##class(TECH.Math).IsEven(Totaal))

private

private

aaa, the good old wlip

aaa, the good old wlip

[ICT-1340] SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-toevoegen exceptie voor verkeerde lijsten

na opmerking van wim: testen toevoegen voor extra gedrag. Wat als er 0,1,2, veel coordinaten worden meegegeven?

na opmerking van wim: testen toevoegen voor extra gedrag.
Wat als er 0,1,2, veel coordinaten worden meegegeven?

Deze method is een leuke oefening (vanuit educatief standpunt), maar in de praktijk vind ik dit een nogal tricky interface. Als we deze willen blijven gebruiken/ondersteunen, dan is heb ik een rev...

Deze method is een leuke oefening (vanuit educatief standpunt), maar in de praktijk vind ik dit een nogal tricky interface.

Als we deze willen blijven gebruiken/ondersteunen, dan is heb ik een review-opmerking, namelijk : deze moet een InvalidInput-exception smijten indien het aantal "Punten" oneven is.
Let op : het tellen van Punten... is geen one-liner !

Ofwel overwegen om aan te passen naar $LB(X,Y) parameters, i.e.

MaakCoordinatenLijstVanXYPunten(lbXYPunten... As %List) 
[ICT-1350] SVG-027 Groene lijntjes laten verdwijnen in het project

-aanpassen variabelen

coordinatesservice en factory injecteren (en gebruiken in testen)

coordinatesservice en factory injecteren (en gebruiken in testen)

[ICT-1340]

SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-Testen van aangemaakte methodes

Ook al is performantie hier momenteel geen echt issue, toch zou ik opteren voor if - else if - else if - ...

Ook al is performantie hier momenteel geen echt issue, toch zou ik opteren voor if - else if - else if - ...

Later te bekijken : of we een andere manier, meer generieke methode, van "svg-objecten toevoegen" mogelijk is.

Later te bekijken : of we een andere manier, meer generieke methode, van "svg-objecten toevoegen" mogelijk is.

Accolades en indentation : graag volgens de VH-conventies aub.

Accolades en indentation : graag volgens de VH-conventies aub.

FYI : %Decimal is perfect ! %Numeric is ook aanvaardbaar (is een meer algemeen datatype, denk. Maar t maakt hoe dan ook ni echt uit aangezien Caché niet strong-typed is)

FYI :
%Decimal is perfect !
%Numeric is ook aanvaardbaar (is een meer algemeen datatype, denk. Maar t maakt hoe dan ook ni echt uit aangezien Caché niet strong-typed is)