Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-118
  • finished reviewing
?? Bedoeling van dit veld ? Is een herhaling van de KlantNaam niet ok ???

?? Bedoeling van dit veld ? Is een herhaling van de KlantNaam niet ok ???

Kunnen we het nummer ook niet halen vanuit die andere Utils ???

Kunnen we het nummer ook niet halen vanuit die andere Utils ???

Goed idee om die data vanuit een andere Utils te halen

Goed idee om die data vanuit een andere Utils te halen

Misschien moeten we het iets explicieter maken dat het gaat over "EenBelg" of misschien nog beter "DenBelg" --> natuurlijk zal dat wel het gevolg hebben dat we "DenBelg" als begrip overal moeten ge...

Misschien moeten we het iets explicieter maken dat het gaat over "EenBelg" of misschien nog beter "DenBelg" --> natuurlijk zal dat wel het gevolg hebben dat we "DenBelg" als begrip overal moeten gebruiken.
Dan lijkt het mij nog beter om dat "DeBelgischeOnderneming" te nemen

!!!!!!

!!!!!!

[UST2215R] Labonorm EDI: leverdatum invullen

- Classmethod voor aanvrager

    • -26
    • +24
    ./PlaatsBestelling/Aanvraag.cls.xml
  1. … 1 more file in changeset.
?????

?????

awel, dat ik het eerlijk gezegd zelf totaal niet zou weten. Ik had eerst wat extra wijzigingen in deze klasse (niets van doen met deze lijn, maar soit), maar die zijn allemaal ongedaan gemaakt (beh...

awel, dat ik het eerlijk gezegd zelf totaal niet zou weten. Ik had eerst wat extra wijzigingen in deze klasse (niets van doen met deze lijn, maar soit), maar die zijn allemaal ongedaan gemaakt (behalve die on-the-flight natuurlijk). Zonder die typfout had ik trouwens gewoon een revert gedaan en was er van bovenstaande niets meer te zien. Het was een mysterie, het is een mysterie en zal ten eeuwige dagen een mysterie blijven denk ik.

Kwestie van geen $$$ElkeWaarde in mijn VerwachtMethodCall te moeten steken

Kwestie van geen $$$ElkeWaarde in mijn VerwachtMethodCall te moeten steken

[UST2215R] Labonorm EDI: leverdatum invullen

- Aanvraag in Test\Utils\...

    • -0
    • +42
    ./PlaatsBestelling/Aanvraag.cls.xml
  1. … 2 more files in changeset.
Lijkt mij iets om de vhTest.Utils achter te laten en dat een beetje te algemenen

Lijkt mij iets om de vhTest.Utils achter te laten en dat een beetje te algemenen

Waarom wordt hier een DummyDto in de constructor meegegeven ?? en wat met de rest van de parameters ?

Waarom wordt hier een DummyDto in de constructor meegegeven ?? en wat met de rest van de parameters ?

Niet nodig om iedere keer hetzelfde opnieuw te testen..( DRY )

Niet nodig om iedere keer hetzelfde opnieuw te testen..( DRY )

Mock en Indien ??

Mock en Indien ??

Verwacht met Elke waarde ?

Verwacht met Elke waarde ?

Ja

Ja

mag het formaat zomaar gewijzigd worden ??

mag het formaat zomaar gewijzigd worden ??

waarom deze wijziging ??

waarom deze wijziging ??

lap daar gaat onze piloot-klant

lap daar gaat onze piloot-klant

Waar is de bewaar naar toe ???? ( ofwel moet ik er ergens naast kijken )

Waar is de bewaar naar toe ???? ( ofwel moet ik er ergens naast kijken )

Hiervoor zou ik toch de FormatterAPI gebruiken kwestie dat het toch wel allemaal iets duidelijker leest

Hiervoor zou ik toch de FormatterAPI gebruiken kwestie dat het toch wel allemaal iets duidelijker leest

[UST2215] Labonorm EDI: leverdatum invullen
[UST2215] Labonorm EDI: leverdatum invullen