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

Test.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 05 Mar 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:51 am

Sam Van Hoey

Thursday 19 Mar 2020
2:34 pm

Tom Vermeulen

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
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:17 am

SamV

[ICT-1210]

SVG-015 Boringen toevoegen aan de kaderdeur

-toevoeging specialevormbeslagboring

Options
    • -18
    • +0
    ./Test.cls.xml
  2. … 2 more files in changeset.
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(....)

8:41 am

Tom Vermeulen

normaalgezien geen commentlijnen laten staan. In dit geval wil je de inject blijkbaar niet meer gebruiken. Op zich kan het niet echt kwaad om hem te laten staan, omdat BasisElementFactory toch gede...

normaalgezien geen commentlijnen laten staan.
In dit geval wil je de inject blijkbaar niet meer gebruiken. Op zich kan het niet echt kwaad om hem te laten staan, omdat BasisElementFactory toch gedefault binnenkomt in de constructor
Belangrijker: waarom niet meer injecten? Geen testen die daar gebruik van moeten maken? <evil grin>

8:39 am

Tom Vermeulen

Do ipv DO

Do ipv DO

8:39 am

Tom Vermeulen

#dim ipv #Dim

#dim ipv #Dim

8:38 am

Tom Vermeulen

Hier (en in andere XMLExport methods) moogt ge gerust een oneliner van maken: Quit ..SvgBox.XMLExport(jadajadajada)

Hier (en in andere XMLExport methods) moogt ge gerust een oneliner van maken: Quit ..SvgBox.XMLExport(jadajadajada)

8:37 am

Tom Vermeulen

Vraagje voor WV: is het aan te raden om in het kader van 'nieuwe schrijfstijl' hier een BoringenIterator (als OneDimIterator ofzo) op te zetten ipv met de Array te werken, of is dat zeuren over det...

Vraagje voor WV: is het aan te raden om in het kader van 'nieuwe schrijfstijl' hier een BoringenIterator (als OneDimIterator ofzo) op te zetten ipv met de Array te werken, of is dat zeuren over details?

8:32 am

Tom Vermeulen

over het opsplitsen van deze lange method hebben we het gisteren ook gehad, dus daar ga ik nu geen epistel over schrijven (maar 't is maar voor het geval dat WV het zou lezen)

over het opsplitsen van deze lange method hebben we het gisteren ook gehad, dus daar ga ik nu geen epistel over schrijven (maar 't is maar voor het geval dat WV het zou lezen)

8:31 am

Tom Vermeulen

Voor alle testen hieronder: beter te testen op objectniveau ipv svg-strings: maw: een cross heeft twee horizontal lines (desnoods ook de properties van die dingen checken), maar hier hebben we het ...

Voor alle testen hieronder: beter te testen op objectniveau ipv svg-strings: maw: een cross heeft twee horizontal lines (desnoods ook de properties van die dingen checken), maar hier hebben we het gisteren al over gehad

8:27 am

Tom Vermeulen

deze test doet niet zo veel

deze test doet niet zo veel

Tuesday 03 Mar 2020
4:36 pm

Sam Van Hoey

[ICT-1210]
[ICT-1210]
Tuesday 18 Feb 2020
8:34 am

SamV

[ICT-1210]

SVG-015 Boringen toevoegen aan de kaderdeur

-Veranderen van tests

Options
    • -12
    • +0
    ./Test.cls.xml
  2. … 2 more files in changeset.
Thursday 06 Feb 2020
11:27 am

Sam Van Hoey

11:26 am

Sam Van Hoey

[ICT-1162]
[ICT-1162]
Tuesday 28 Jan 2020
2:48 pm

SamV

[ICT-1162]

SVG-009 aanmaken boringen

-wijziging naam methode

Options
    • -4
    • +4
    ./Test.cls.xml
  2. … 1 more file in changeset.
2:43 pm

SamV

[ICT-1162]

SVG-009 aanmaken boringen

-aanmaken boringen en test

Options
    • -0
    • +30
    ./Test.cls.xml
  2. … 4 more files in changeset.