CacheAdminA_trunk2010/APPS/Halux/PPS/ALUFrees/Svg

Kaderdeur.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 16 Aug 23
Commits this week: 0
Total Lines of Code (LoC): 209
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
Thursday 17 Aug 2023
10:53 am

Wim Vermeulen

10:29 am

Jelle Baele

Wednesday 16 Aug 2023
4:01 pm

Wim Vermeulen

ICT-3829: [rvJBA] KAD rendering probleem (CEFSharp) indien de SVG een "é" bevat
ICT-3829: [rvJBA] KAD rendering probleem (CEFSharp) indien de SVG een "é" bevat
Monday 23 Nov 2020
8:57 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:56 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:51 am

Sam Van Hoey

8:51 am

Sam Van Hoey

8:51 am

Sam Van Hoey

Friday 15 May 2020
12:24 pm

Wim Vermeulen

11:55 am

Wim Vermeulen

Standaard regel voor onze UT's : naamgeving = "vhUnitTest." + naam van de klasse " + "." + naam van de methode "." + "Test" Dus in dit geval was het bijna juist http://subversion02.vanhoecke.be/sta...

Standaard regel voor onze UT's : naamgeving = "vhUnitTest." + naam van de klasse " + "." + naam van de methode "." + "Test"
Dus in dit geval was het bijna juist
--> vhUnitTest.APPS.Halux.PPS.ALUFrees.Svg.Kaderdeur.GenereerSvg.Test

Er zijn uitzonderingen, maar hier is standaard regel zeker oké.
Graag ook je andere UT's eens nakijken. Aanpassen kan je best via : 

Do ##class(Tools.CacheSourceUtils).MoveClass("","")
Friday 08 May 2020
10:53 am

Tom Vermeulen

10:22 am

Tom Vermeulen

kommaneukerij, maar ik zou "uitvoeren van methodes" niet in de testnaam zetten, anders kan hij misschien te lang worden en 't is nogal triviaal

kommaneukerij, maar ik zou "uitvoeren van methodes" niet in de testnaam zetten, anders kan hij misschien te lang worden en 't is nogal triviaal

10:20 am

Tom Vermeulen

toch eens naar de rest gekeken en vraag me af wat de zin is van die write (buiten tijdens development) - > weg die handel http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/em...

toch eens naar de rest gekeken en vraag me af wat de zin is van die write (buiten tijdens development) - > weg die handel

10:19 am

Tom Vermeulen

niet gekeken wat hierboven nog gebeurt, maar je kan best w !,"standaard .... gebruiken. Dat ! geeft een CRLF

niet gekeken wat hierboven nog gebeurt, maar je kan best
w !,"standaard ....
gebruiken. Dat ! geeft een CRLF

Wednesday 06 May 2020
1:13 pm

Sam Van Hoey

[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods
[ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods
Tuesday 07 Apr 2020
8:40 am

Tom Vermeulen

Thursday 02 Apr 2020
6:21 pm

Wim Vermeulen

6:10 pm

Wim Vermeulen

Ik ben nogal verward hoe deze test is opgebouwd. Mogelijk is dit veroorzaakt door misleidende naamgevingen. Ook conceptueel lijkt er iets niet te kloppen. Laten we zeggen dat de test aan het licht ...

Ik ben nogal verward hoe deze test is opgebouwd. Mogelijk is dit veroorzaakt door misleidende naamgevingen.
Ook conceptueel lijkt er iets niet te kloppen. Laten we zeggen dat de test aan het licht brengt dat de interfaces niet helemaal goed zitten.
Aan deze UT's (en code) gaan we best wat knutselen. Contact me

4:54 pm

Wim Vermeulen

4:43 pm

Wim Vermeulen

Deze method gaan we eens samen bekijken: *afsplitsen in meerdere private methods : bevordert de leesbaarheid, maar bovenal vermijdt dependencies tussen stukken code die niks met mekaar te maken h...

Deze method gaan we eens samen bekijken:

  • afsplitsen in meerdere private methods : bevordert de leesbaarheid, maar bovenal vermijdt dependencies tussen stukken code die niks met mekaar te maken hebben.
  • oproep/interface geschikt maken om vanuit Prod.GADef.Kaderdeur op te roepen.
  • UT's rond GenereerSvg() zou ik een andere aanpak voorstellen. Contact me
4:20 pm

Wim Vermeulen

4:19 pm

Wim Vermeulen

Na een Catch() moet er altijd een "reporting" of een nieuwe throw gedaan worden. In dit geval zegt mijn gevoel dat hier geen try-catch moet staan, want het is aan de gebruiker/oproepende code om ze...

Na een Catch() moet er altijd een "reporting" of een nieuwe throw gedaan worden.
In dit geval zegt mijn gevoel dat hier geen try-catch moet staan, want het is aan de gebruiker/oproepende code om zelf te bepalen welke error-handling er nodig is.

Anderszijds lijkt dit meer op een stukje test-code, dan kan je dat best ook zo aangeven (in methodnaam of ev in de comment) of moet je deze testcode verplaatsen naar een zTryout-klasse.
Zo niet, dan krijg je de reviewer(s) op uw dak :-D

Wednesday 25 Mar 2020
12:21 pm

Tom Vermeulen

12:17 pm

Tom Vermeulen

Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets

Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets

12:13 pm

Tom Vermeulen

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

12:12 pm

Tom Vermeulen

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is (kleine kaartjes, rap committen, enz enz http://subversion02.vanhoecke.b...

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is

(kleine kaartjes, rap committen, enz enz )

12:10 pm

Tom Vermeulen

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen. Idem bij constructors andere boringen

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen.

Idem bij constructors andere boringen