Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronde...

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronder verifiëren?

overbodige comment?

overbodige comment?

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten
[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten
Dit ruikt naar feature envy

Dit ruikt naar feature envy

op zich is het geen probleem maar ik zou toch opteren voor quit:$$$Not(TranslationIterator.HasNext()) ; zodoende het duidelijker is dat dit een guard clause is en je dan geen indentatie lager is

op zich is het geen probleem maar ik zou toch opteren voor quit:$$$Not(TranslationIterator.HasNext()) ; zodoende het duidelijker is dat dit een guard clause is en je dan geen indentatie lager is

  • More
  • CR-666
  • finished reviewing
Als je dan toch de constructor hebt aangepast, gelieve de Langtekstbewaarder dan te instantieren in de %OnNew

Als je dan toch de constructor hebt aangepast, gelieve de Langtekstbewaarder dan te instantieren in de %OnNew

[UST2991] VHIP184: Sync Prod (eenvoudige velden)
[UST2991] VHIP184: Sync Prod (eenvoudige velden)
  • More
  • CR-577
  • finished reviewing
Verder zou ik ook nog een test toevoegen wanneer er een typo in vandaag staat dat het ook een andere datum is.

Verder zou ik ook nog een test toevoegen wanneer er een typo in vandaag staat dat het ook een andere datum is.

Omschrijving van test kan duidelijker zodat er meer gefocused wordt op de eigenlijke functionaliteit : De berekendatum wordt bepaald adhv een stringwaarde.

Omschrijving van test kan duidelijker zodat er meer gefocused wordt op de eigenlijke functionaliteit :

De berekendatum wordt bepaald adhv een stringwaarde.

Beter $$EXTDATE^..... gebruiken daaraan kan je het formaat meegeven en dat in het ook iets duidelijker om te lezen, trouwens er bestaat een $$$AssertDate of $$$AssertEqualsDate die er voor zorgt da...

Beter $$EXTDATE^..... gebruiken daaraan kan je het formaat meegeven en dat in het ook iets duidelijker om te lezen, trouwens er bestaat een $$$AssertDate of $$$AssertEqualsDate die er voor zorgt dat je al die formatteren zelf moet doen

+$H is niet altijd vandaag als er gebruik gemaakt wordt van de API

+$H is niet altijd vandaag als er gebruik gemaakt wordt van de API

[DEF603] PM: Matjes: Herbereken lukt voor vele matjes niet
[DEF603] PM: Matjes: Herbereken lukt voor vele matjes niet