Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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)
[UST2991] VHIP184: Sync Prod (eenvoudige velden)

- refactoring om LangtekstBewaarder uitmockbaar te maken.

  1. … 5 more files in changeset.
  • 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.

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

- reviewopmerkingen verwerkt

    • -10
    • +2
    ./Acties/HerrekenProductViaIK/Test.cls.xml
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

- aparte commit :) om het onderscheid te testen tussen "datum van product" en "vandaag"

    • -7
    • +68
    ./Acties/HerrekenProductViaIK/Test.cls.xml
  1. … 1 more file in changeset.
[DEF603] PM: Matjes: Herbereken lukt voor vele matjes niet

- code in commentaar verwijderd

- ProductTypeApi geinjecteerd

- ProductTypeApi controleert type product

    • -13
    • +11
    ./Acties/HerrekenProductViaIK/Test.cls.xml
  1. … 1 more file in changeset.
[DEF603] PM: Matjes: Herbereken lukt voor vele matjes niet
[DEF603] PM: Matjes: Herbereken lukt voor vele matjes niet
[DEF603] PM: Matjes: Herbereken lukt voor vele matjes niet
    • -0
    • +65
    ./Acties/HerrekenProductViaIK/Test.cls.xml
  1. … 15 more files in changeset.
[ICT235] - OnBefores..OnAfters #super fixen
    • -16
    • +14
    ./TestWijzigActiefStatus.cls.xml
[ICT235] - OnBefores..OnAfters #super fixen
    • -16
    • +14
    ./TestWijzigKSProductStatus.cls.xml
[ICT235] - OnBefores..OnAfters #super fixen
[UST1740] TAX FPV1 VPK: GeefData

- specifieke mock verwijderd

- test aangepast: algemene mock wordt opgeroepen

    • -0
    • +0
    ./WijzigKorttekst/KorttekstUtilsMock.cls.xml
    • -28
    • +20
    ./WijzigKorttekst/TestVoerUit.cls.xml
  1. … 1 more file in changeset.
[DEF507] Slechte karakters preventiever voorkomen/wegwerken - BulkProductActie WijzigLangtekst ondersteunt nu ook $C(13,10) en $C(10) i.p.v. "~" als NewLine-separator
  1. … 1 more file in changeset.