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

Hoekverbinder.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

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

Sam Van Hoey

8:56 am

Sam Van Hoey

8:55 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:51 am

Sam Van Hoey

Thursday 02 Apr 2020
6:21 pm

Wim Vermeulen

5:56 pm

Wim Vermeulen

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

12:07 pm

Tom Vermeulen

klasse newen in constructor

klasse newen in constructor

12:05 pm

Tom Vermeulen

indents ongedaan maken

indents ongedaan maken

12:03 pm

Tom Vermeulen

12:03 pm

Tom Vermeulen

Dit is opeetcode. Als er een exception geraised wordt, gaat niemand het weten. Is dit het gewenste gedrag? Indien niet: er iets mee doen (en testje(s) voor schrijven)

Dit is opeetcode. Als er een exception geraised wordt, gaat niemand het weten. Is dit het gewenste gedrag? Indien niet: er iets mee doen (en testje(s) voor schrijven)

12:01 pm

Tom Vermeulen

indenten tussen de accolades

indenten tussen de accolades

12:00 pm

Tom Vermeulen

11:41 am

Tom Vermeulen

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif.

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen .

11:39 am

Tom Vermeulen

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er i...

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er in deze klasse gebruikt worden

11:38 am

Tom Vermeulen

ge kunt het niet laten hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (hieronder nog een paar)

ge kunt het niet laten hé (hieronder nog een paar)

Monday 23 Mar 2020
4:35 pm

Sam Van Hoey

[ICT-1348] SVG-025 ' aanpassen naar $$$NOT() + naamgeving van variabelen uitbreiden
[ICT-1348] SVG-025 ' aanpassen naar $$$NOT() + naamgeving van variabelen uitbreiden
4:32 pm

Sam Van Hoey

[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
4:31 pm

Sam Van Hoey

[ICT-1365] SVG-029 Mock Testen uitbreiden
[ICT-1365] SVG-029 Mock Testen uitbreiden
Thursday 19 Mar 2020
2:44 pm

SamV

[ICT-1365] SVG-029 Mock Testen uitbreiden

-Hoekverbinder opgesplitst en mock testen toegepast

Options
    • -21
    • +7
    ./Hoekverbinder.cls.xml
  2. … 10 more files in changeset.
Monday 16 Mar 2020
4:16 pm

Wim Vermeulen

3:58 pm

Wim Vermeulen

Correctie : in dit uitzonderlijke geval lukt het niet om af te leiden van TECH.RegisteredObject --> in totaal zullen er 3 klassen zijn die dus afleiden van (%RegisteredObject,%XML.Adaptor)

Correctie : in dit uitzonderlijke geval lukt het niet om af te leiden van TECH.RegisteredObject
--> in totaal zullen er 3 klassen zijn die dus afleiden van (%RegisteredObject,%XML.Adaptor)

Thursday 12 Mar 2020
3:42 pm

SamV

[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger

-Toevoegen van ElementToevoeger en bewerken van het toekennen van variabelen

Options
    • -3
    • +4
    ./Hoekverbinder.cls.xml
  2. … 17 more files in changeset.