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

PuntBoring

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

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

8:47 am

Sam Van Hoey

Thursday 02 Apr 2020
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

Monday 23 Mar 2020
10:43 am

Tom Vermeulen

10:43 am

Tom Vermeulen

deze test gaat niet echt uitgevoerd worden. Ofwel wel iets (nuttigs) laten doen, ofwel wegsmijten (idem voor andere boringtesten)

deze test gaat niet echt uitgevoerd worden. Ofwel wel iets (nuttigs) laten doen, ofwel wegsmijten (idem voor andere boringtesten)

10:40 am

Tom Vermeulen

zie opmerking in kaartje van CrossFactory

zie opmerking in kaartje van CrossFactory

10:38 am

Tom Vermeulen

well well well http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (en er staan er nog een paar)

well well well (en er staan er nog een paar)

Thursday 19 Mar 2020
2:45 pm

Sam Van Hoey

[ICT-1367] SVG-030 Aanmaken BoringElementFactory
[ICT-1367] SVG-030 Aanmaken BoringElementFactory
2:34 pm

Tom Vermeulen

Tuesday 17 Mar 2020
3:59 pm

SamV

[ICT-1367] SVG-030 Aanmaken BoringElementFactory

-Voorlopig uitschakelen van testen v/ boringen

Options
    • -2
    • +2
    ./Test.cls.xml
  2. … 3 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
    • -1
    • +8
    ./Test.cls.xml
  2. … 8 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 '

11:11 am

Wim Vermeulen

11:11 am

Wim Vermeulen

Width en Height : zullen we later nog eens verder in detail bekijken. bvb: bij puntboring moet de "gebruiker" eigenlijk GEEN afmetingen doorgeven, bij doorboring of rugboring zou de gebruiker eerde...

Width en Height : zullen we later nog eens verder in detail bekijken.
bvb: bij puntboring moet de "gebruiker" eigenlijk GEEN afmetingen doorgeven, bij doorboring of rugboring zou de gebruiker eerder de straal/diameter van de boring doorgeven.
Laat voorlopig effe staan zoals het is. Komen hier later nog op terug

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