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

BasisElement

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 20 Jul 20
Commits this week: 0
Total Lines of Code (LoC): 0
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:52 am

Sam Van Hoey

8:51 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
  1. ./BasisElement
    • -151
    • +0
    ./CrossFactory/Test.cls.xml
  3. ./CrossFactory
  4. … 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
    • -15
    • +15
    ./CrossFactory/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
    • -47
    • +53
    ./CrossFactory/Test.cls.xml
  2. … 4 more files in changeset.
Tuesday 07 Apr 2020
1:34 pm

SamV

[ICT-1374] SVG-031 Aanmaken CrossFactory

-aanpassing door review kleur anders dan default ingevuld + meer defaultwaarden voor dummyobj

Options
    • -10
    • +19
    ./CrossFactory/Test.cls.xml
Monday 06 Apr 2020
10:42 am

SamV

[ICT-1374] SVG-031 Aanmaken CrossFactory

-aanpassing door review andere invulwaarde en dummyObjecten

Options
    • -18
    • +21
    ./CrossFactory/Test.cls.xml
Thursday 02 Apr 2020
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)

Monday 23 Mar 2020
8:34 am

Tom Vermeulen

8:32 am

Tom Vermeulen

AssertClassName bestaat ook

AssertClassName bestaat ook

8:28 am

Tom Vermeulen

nu die constructor leeg is, kan je je de vraag stellen of hij hier überhaupt nog moet blijven staan. Een constructie als deze kan een potentieel risico inhouden: aangezien je enkel een Quit $$$OK ...

nu die constructor leeg is, kan je je de vraag stellen of hij hier überhaupt nog moet blijven staan.

Een constructie als deze kan een potentieel risico inhouden: aangezien je enkel een Quit $$$OK doet, maskeer je logica die in de constructor van de Superklasses zit. In dit geval is dat niet echt relevant, omdat het toch klasses zijn zonder enige inhoud (bijna). Wou het gewoon even melden.

(logica van de superklasse oproepen: do ##super() )

Thursday 19 Mar 2020
2:34 pm

Tom Vermeulen

11:42 am

Sam Van Hoey

[ICT-1374] SVG-031 Aanmaken CrossFactory
[ICT-1374] SVG-031 Aanmaken CrossFactory
11:22 am

SamV

[ICT-1374] SVG-031 Aanmaken CrossFactory

-Toevoegen van CrossFactory en test + verwijderen test cross objecten + aanpassen cross objecten + Aanpassing BoringELementFactory/Test.cls.xml

Options
  1. ./TCross
    • -0
    • +133
    ./CrossFactory/Test.cls.xml
  3. ./CrossFactory
  4. ./CrossVH
  5. ./CrossDiag
    • -36
    • +0
    ./TCross/Test.cls.xml
    • -36
    • +0
    ./CrossVH/Test.cls.xml
    • -34
    • +0
    ./CrossDiag/Test.cls.xml
  9. … 12 more files in changeset.
Thursday 12 Mar 2020
6:26 pm

Wim Vermeulen

6:21 pm

Wim Vermeulen

Sam, deze implementatie gaan we in "pair reviewen" (wss rechtstreeks vanop de code). We kunnen hier immers vele kanten mee uit http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ico...

Sam, deze implementatie gaan we in "pair reviewen" (wss rechtstreeks vanop de code).
We kunnen hier immers vele kanten mee uit

5:42 pm

Wim Vermeulen

Al opgelost? Indien niet, dan mag je een "UnimplementedException" throwen (voorbeeldjes via ctrl-shift-find) Als je echt wil om deze te implementeren, dan kan deze boring best voorgesteld worden do...

Al opgelost?
Indien niet, dan mag je een "UnimplementedException" throwen (voorbeeldjes via ctrl-shift-find)
Als je echt wil om deze te implementeren, dan kan deze boring best voorgesteld worden door de combinatie van een "interne zijboring" + een "externe zijboring".

5:26 pm

Wim Vermeulen

Ja, ik geef ook de voorkeur aan nieuwe conventies, namelijk : While (Iterator.HasNext() ) i.p.v. While ($IsObject(Boring)) Merk op : testen met $$$HasLength() gebruiken we niet (meer) bij objecten...

Ja, ik geef ook de voorkeur aan nieuwe conventies,
namelijk : While (Iterator.HasNext() ) i.p.v. While ($IsObject(Boring))

Merk op : testen met $$$HasLength() gebruiken we niet (meer) bij objecten, mag alleen nog bij DataTypes, als %String. Voor objecten is het correcter om te testen met $IsObject()

Opm. 2: (naming convention) niet BoringenIterator maar BoringIterator (enkelvoud)

5:06 pm

Wim Vermeulen

Beter :   #dim sc As %Status Set sc = ..TopLine.XMLExport(...) Do ##class(TECH.Exceptions.StatusException).ThrowIndienNietOK(sc,"Error bij XmlExport van TopLine bij object "_$this) Set sc = ....

Beter : 

 
	#dim sc As %Status

	Set sc = ..TopLine.XMLExport(...)
	Do ##class(TECH.Exceptions.StatusException).ThrowIndienNietOK(sc,"Error bij XmlExport van TopLine bij object "_$this)
	Set sc = ..BottomLine.XMLExport(...)
	Do ##class(TECH.Exceptions.StatusException).ThrowIndienNietOK(sc,"Error bij XmlExport van BottomLine bij object "_$this)

	Quit $$$OK
Tuesday 10 Mar 2020
10:29 am

SamV

[ICT-1326]

SVG-023 Uitbreiden testen svg-project

-Uitbreiden van testen svg objecten en svg tooling

Options
    • -0
    • +12
    ./CrossDiag/Test.cls.xml
    • -1
    • +15
    ./TCross/Test.cls.xml
    • -0
    • +14
    ./CrossVH/Test.cls.xml
  4. … 6 more files in changeset.
Thursday 05 Mar 2020
3:54 pm

Tom Vermeulen

zie opmerking bij Doorboring: dit kan gerust een oneliner worden. (idem bij andere BasisElementen)

zie opmerking bij Doorboring: dit kan gerust een oneliner worden. (idem bij andere BasisElementen)

3:53 pm

Tom Vermeulen

fancy, kende deze syntax eigenlijk niet. ik zou een gewone && gebruikt hebben Dit wil wel zeggen dat je bij een DoorBoring ook altijd een Zijboring hebt. Klopt dat? (zie BoorType="Z" hierboven)

fancy, kende deze syntax eigenlijk niet. ik zou een gewone && gebruikt hebben

Dit wil wel zeggen dat je bij een DoorBoring ook altijd een Zijboring hebt. Klopt dat? (zie BoorType="Z" hierboven)

3:48 pm

Tom Vermeulen

heb het hieronder al een paar gezegd $$$Not ipv '

heb het hieronder al een paar gezegd $$$Not ipv '

3:47 pm

Tom Vermeulen

$$$Not ipv ' (en pas en passant de casing en spacing efkes aan (if hier en else hieronder)

$$$Not ipv ' (en pas en passant de casing en spacing efkes aan (if hier en else hieronder)

3:46 pm

Tom Vermeulen

$$$Not ipv '

$$$Not ipv '

Wednesday 04 Mar 2020
8:56 am

Tom Vermeulen

deze sc gaat overschreven worden door de volgende (2 lijnen lager). Dus: ofwel is ie echt zinloos en mag het weg, ofwel moet hieronder iets staan in de zin van "if sc heeft een of andere toestand, ...

deze sc gaat overschreven worden door de volgende (2 lijnen lager). Dus: ofwel is ie echt zinloos en mag het weg, ofwel moet hieronder iets staan in de zin van "if sc heeft een of andere toestand, dan doe de VerticalLine-stuff"

Uwen TCross-test had dat kunnen uitwijzen, als ge daar gaat asserten dat er twee lijnen aanwezig zijn

Deze opmerking geldt voor de andere crossen ook

8:42 am

Tom Vermeulen

en ge moogt dit ineens in een Set steken Set CrossDiag = ##class(dinges).%New(....)

en ge moogt dit ineens in een Set steken
Set CrossDiag = ##class(dinges).%New(....)