CacheAdminA_trunk2010/DOM/PM/Maatwerk/Calc/HF/impl

HFItemBerekenException.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 31 May 18
Commits this week: 0
Total Lines of Code (LoC): 28
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
Friday 01 Jun 2018
2:47 pm

Wim Vermeulen

2:24 pm

Martijn Vierstraete

12:08 pm

Wim Vermeulen

idd. ik heb daar ook aan gedacht. Maar blijkbaar wordt de "explicietgekozen" niet doorgegeven tot op de gekoppelde kenmerken. Geen idee of dit zo bedoeld is, of dat da eigenlijk ontbrekende functio...

idd.
ik heb daar ook aan gedacht.
Maar blijkbaar wordt de "explicietgekozen" niet doorgegeven tot op de gekoppelde kenmerken.
Geen idee of dit zo bedoeld is, of dat da eigenlijk ontbrekende functionaliteit is.

anyway, ik heb ervan geprofiteerd dat t nu zo is en heb het mij ook gerealiseerd dat het een beetje "kort door de bocht" is, maar als t correcter zou willen, dan moet er op één of andere manier een property bijkomen. Deze uitbreiding zou heel veel moeite kosten voor wat dit waard is. Vandaar mijn keuze om het zo te laten.

Ik hoop dat je het hiermee eens bent

11:40 am

Martijn Vierstraete

Waarom de controle of er expliciet een matmateriaal gekozen is? Zowel de standalone als gekoppelde matten moeten toch verplicht een matmateriaal bevatten? Bijgevolg kun je toch niet met zekerheid b...

Waarom de controle of er expliciet een matmateriaal gekozen is? Zowel de standalone als gekoppelde matten moeten toch verplicht een matmateriaal bevatten? Bijgevolg kun je toch niet met zekerheid bepalen of dit (apart) via de configurator ingegeven is?

Thursday 31 May 2018
12:08 pm

Wim Vermeulen

[ICT394] MatKenmerken melding aanpassen indien berekenFout
[ICT394] MatKenmerken melding aanpassen indien berekenFout
12:07 pm

Wim Vermeulen

[ICT394] MatKenmerken melding aanpassen indien berekenFout

- indien kenmerken ingegeven via MatConfigurator, dan de tekst "via MatConfigurator" toevoegen aan de foutmelding.

Options
    • -1
    • +18
    ./HFItemBerekenException.cls.xml
Wednesday 07 Jun 2017
11:14 am

Jo Claes

Tuesday 30 May 2017
3:24 pm

Kurt Callebaut

  • More
  • CR-235
  • finished reviewing
3:24 pm

Kurt Callebaut

De meegegeven parameter / data wordt ook in deze class meegegeven in de constructor en is dus al gekend en blijft ongewijzigd. Dumper kan dus zonder problemen naar constructor. Voglens mij kan de v...

De meegegeven parameter / data wordt ook in deze class meegegeven in de constructor en is dus al gekend en blijft ongewijzigd. Dumper kan dus zonder problemen naar constructor. Voglens mij kan de volledige Dumper...ToStream zelfs in een property gezet worden in de constructor van de klasse. Of kan dit opgelost worden via lazyload ?

3:15 pm

Kurt Callebaut

We kunnen best niet veronderstellen dat er een crash is. Deze test bevestigd niet dat het gegarandeerd zal crashen... bv als er een verkeerde builder wordt gebruikt en er geen crash optreedt omdat ...

We kunnen best niet veronderstellen dat er een crash is. Deze test bevestigd niet dat het gegarandeerd zal crashen... bv als er een verkeerde builder wordt gebruikt en er geen crash optreedt omdat de code niet doet wat we verwachten ?

3:10 pm

Kurt Callebaut

Volgens ik de code van de logger zie, kan je zonder problemen ##class(vhLib.Logger).%New() hergebruiken en 2x een Error method oproepen zonder problemen. Binnen de klasse is het dezelfde applicatie...

Volgens ik de code van de logger zie, kan je zonder problemen ##class(vhLib.Logger).%New() hergebruiken en 2x een Error method oproepen zonder problemen. Binnen de klasse is het dezelfde applicatie / groep.

11:24 am

JoC

[ICT251] - HalffabContext Foutmelding is voortaan een stream

- Verwerken review opmerkingen

Options
    • -6
    • +6
    ./HFItemBerekenException.cls.xml
  2. … 2 more files in changeset.
Monday 29 May 2017
3:53 pm

Jo Claes

Op gelijk welke context zal er een FoutieveBuilder worden toegevoegd dus het zal altijd crashen.

Op gelijk welke context zal er een FoutieveBuilder worden toegevoegd dus het zal altijd crashen.

3:51 pm

Jo Claes

Huidige werking van de logging is zo dat er eigenlijk per te loggen item een nieuwe instantie moet opgevraagd worden.

Huidige werking van de logging is zo dat er eigenlijk per te loggen item een nieuwe instantie moet opgevraagd worden.

3:47 pm

Jo Claes

Dumper werkt zo dat hij via constructor zijn data moet binnen krijgen. dus verplaatsen naar constructor is niet aan de orde

Dumper werkt zo dat hij via constructor zijn data moet binnen krijgen. dus verplaatsen naar constructor is niet aan de orde

3:44 pm

Jo Claes

DRY Exceptie staat er al in dus het zou eerder GeefDetail moeten zijn , om toch een duidelijk verschil te hebben tussen de string en stream implementatie is er voor GeefDetailStream gekozen

DRY Exceptie staat er al in dus het zou eerder GeefDetail moeten zijn , om toch een duidelijk verschil te hebben tussen de string en stream implementatie is er voor GeefDetailStream gekozen

3:41 pm

Jo Claes

Er bestaat een test voor dus zou niet weten wat je hier wil zeggen

Er bestaat een test voor dus zou niet weten wat je hier wil zeggen

3:39 pm

Kurt Callebaut

Tab gebruiken en accolades verduidelijkt het code blok.

Tab gebruiken en accolades verduidelijkt het code blok.

2:44 pm

Kurt Callebaut

Een voorstel om linefeeds gebruiken zodat de samenhorende blokken code duidelijker worden of zelfs methods zoals AssertSelectieKenmerk.

Een voorstel om linefeeds gebruiken zodat de samenhorende blokken code duidelijker worden of zelfs methods zoals AssertSelectieKenmerk.

2:42 pm

Kurt Callebaut

Hier wordt verondersteld dat het steeds een nieuwe context is. Als dezelfde context gebruikt wordt dan hebben we hier het risico dat er geen exceptie optreed en dat er nog gebruik gemaakt wordt van...

Hier wordt verondersteld dat het steeds een nieuwe context is. Als dezelfde context gebruikt wordt dan hebben we hier het risico dat er geen exceptie optreed en dat er nog gebruik gemaakt wordt van Berekeningsproblemen van een vorige melding waardoor de test onterecht kan slagen. We hebben hier dus een controle nodig dat de exceptie wel degelijk gegooid is.

2:40 pm

Kurt Callebaut

idem testnaam

idem testnaam

2:39 pm

Kurt Callebaut

Als er nog wat tijd over is, testnaam verduidelijken.

Als er nog wat tijd over is, testnaam verduidelijken.

2:26 pm

Kurt Callebaut

Iterator naam verduidelijken, vb door BerekeningProblemenIterator.

Iterator naam verduidelijken, vb door BerekeningProblemenIterator.

2:25 pm

Kurt Callebaut

.%New() naar constructor en private property van maken zodat het duidelijk is dat Dumper gebruikt wordt in deze klasse. Ingegevenkenmerken worden ook in constructor meegegeven en worden dus niet me...

.%New() naar constructor en private property van maken zodat het duidelijk is dat Dumper gebruikt wordt in deze klasse. Ingegevenkenmerken worden ook in constructor meegegeven en worden dus niet meer gewijzigd.

2:24 pm

Kurt Callebaut

vhLib.Logger).%New naar constructor brengen en private property van maken..

vhLib.Logger).%New naar constructor brengen en private property van maken..

2:16 pm

Kurt Callebaut

Eventueel deze method opsplitsen in VoegToeIngegevenKenmerken, VoegToeBerekenDatum, VoegToeHalfFab,... kleine methods met duidelijke benamingen.

Eventueel deze method opsplitsen in VoegToeIngegevenKenmerken, VoegToeBerekenDatum, VoegToeHalfFab,... kleine methods met duidelijke benamingen.

2:14 pm

Kurt Callebaut

Meer duidelijk benamingen gebruiken zoals ExceptieDetail. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Meer duidelijk benamingen gebruiken zoals ExceptieDetail.

2:14 pm

Kurt Callebaut

HalfFabIterator als naam gebruiken

HalfFabIterator als naam gebruiken

2:09 pm

Kurt Callebaut

Geen unittest voor deze implementatie.

Geen unittest voor deze implementatie.

2:05 pm

Kurt Callebaut

Algemeen casing.

Algemeen casing.