BatchJobPakbonMagGeboektWorden.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

- refactoring code wat effecienter maken.

    • -17
    • +30
    ./BatchJobPakbonMagGeboektWorden.cls.xml
  1. … 1 more file in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

BatchJobPakbonMagGeboektWorden wlips verwijderen

    • -13
    • +4
    ./BatchJobPakbonMagGeboektWorden.cls.xml
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

BatchJobPakbonMagGeboektWorden methode VerwerkBon terug private maken

    • -0
    • +1
    ./BatchJobPakbonMagGeboektWorden.cls.xml
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

BatchJobPakbonMagGeboektWorden rekening houden met de dueoutime van het order, extra conditie: enkel het PakbonMagGeboektWorden bericht sturen als dit op vandaag staat.

Indien een order reeds vroeger gepicked wordt mag de bon nog niet verzonden worden op die avond als de dueouttime op overmorgen staat

    • -5
    • +35
    ./BatchJobPakbonMagGeboektWorden.cls.xml
  1. … 4 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Logging toevoegen als aanduiding start en einde BatchJobPakbonMagGeboektWorden

    • -0
    • +4
    ./BatchJobPakbonMagGeboektWorden.cls.xml
[ICT-1078] REVERT Fase 2 sturen bericht dat pakbon mag geboekt worden: newline weg
    • -1
    • +1
    ./BatchJobPakbonMagGeboektWorden.cls.xml
  1. … 7 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden: newline weg
    • -1
    • +1
    ./BatchJobPakbonMagGeboektWorden.cls.xml
  1. … 7 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Bugfix: batchjob enkel uitlevering met FakSoort = F verwerken

    • -0
    • +2
    ./BatchJobPakbonMagGeboektWorden.cls.xml
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Bugfix: if .hasnext vervangen door while .hasnext

    • -1
    • +1
    ./BatchJobPakbonMagGeboektWorden.cls.xml
De = $$$ True is in principe niet nodig (en wordt meestal niet gezet), maar daarover kan lang gediscussieerd worden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticon...

De = $$$ True is in principe niet nodig (en wordt meestal niet gezet), maar daarover kan lang gediscussieerd worden Zie de lijn erboven

Nitpicking, maar als het 123 is, is het niet random. Ofwel is het DummyWinkelkarId = $$$Integer als het echt niet belangrijk is, ofwel is het random en gebruik je de randomizer, ofwel moet het echt...

Nitpicking, maar als het 123 is, is het niet random. Ofwel is het DummyWinkelkarId = $$$Integer als het echt niet belangrijk is, ofwel is het random en gebruik je de randomizer, ofwel moet het echt 123 zijn

Kan je hier ##class(vhTest.Utils.APPS.VKP.Offerte).StandaardOfferte() of één van zijn vriendjes gebruiken?

Kan je hier ##class(vhTest.Utils.APPS.VKP.Offerte).StandaardOfferte() of één van zijn vriendjes gebruiken?

Ik vroeg me af of het interessanter is om de test in twee te splitsen zodat ze elk duiudelijk hun situatie testen, waardoor het beter opvalt wat er aan de hand is (door testomschrijving, en door va...

Ik vroeg me af of het interessanter is om de test in twee te splitsen zodat ze elk duiudelijk hun situatie testen, waardoor het beter opvalt wat er aan de hand is (door testomschrijving, en door variatiepunt in test. Een hele hoop van de asserts en opzet zijn dan gemeenschappelijk en kunnen in wat private methods gemikt worden.

Soit, 't is tijd om te stoppen, zal morgen verderdoen

fyi: in plaats van een hele lijst properties te asserten, zou je ook een AssertObject ofzo kunnen gebruiken.

fyi: in plaats van een hele lijst properties te asserten, zou je ook een AssertObject ofzo kunnen gebruiken.

ook hier werken met de fake datetimeapi

ook hier werken met de fake datetimeapi

Als de variabele ECHT geen bal uitmaakt, vervang hem dan door $$$DummyString (of $$$DummyInteger, ...). Dat maakt functioneel geen bal uit, maar maakt het voor de lezer duidelijk dat het echt niets...

Als de variabele ECHT geen bal uitmaakt, vervang hem dan door $$$DummyString (of $$$DummyInteger, ...).
Dat maakt functioneel geen bal uit, maar maakt het voor de lezer duidelijk dat het echt niets uitmaakt

Gebruik voor een vaste datum de DateTimeApi (bv Do ##class(vhTest.Fake.TECH.DateTime.DateTimeAPI).Activeer()) (voorbeelden in andere testen)

Gebruik voor een vaste datum de DateTimeApi (bv Do ##class(vhTest.Fake.TECH.DateTime.DateTimeAPI).Activeer()) (voorbeelden in andere testen)

Beter om de const te vervangen door een personality-enu

Beter om de const te vervangen door een personality-enu

ICT-1527: [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)
ICT-1527: [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor: gebruik maken van Flow/Uitlevering/HoofdingRepository ipv meerdere loops + unit testen aanpassen

    • -27
    • +21
    ./BatchJobPakbonMagGeboektWorden.cls.xml
  1. … 2 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor: opkuis code

    • -2
    • +0
    ./BatchJobPakbonMagGeboektWorden.cls.xml
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor: private maken van methode

    • -0
    • +1
    ./BatchJobPakbonMagGeboektWorden.cls.xml
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor: opsplitsen methode call

    • -2
    • +2
    ./BatchJobPakbonMagGeboektWorden.cls.xml
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor: rename methodes ZetVlagHoofding en GeefVlagHoofding naar specifiekere methodes met duidelijke naamgeving

    • -1
    • +1
    ./BatchJobPakbonMagGeboektWorden.cls.xml
  1. … 7 more files in changeset.
test of meerdere uitleveringen goed afgehandeld worden, cfr opm repository over flow.uitlevering.hoofding

test of meerdere uitleveringen goed afgehandeld worden, cfr opm repository over flow.uitlevering.hoofding

Repository maken over Flow.Uitlevering.Hoofding ? encapsulaties gebruiken waar het kan? overweeg met PRA om te zien of het de investering waard is. Indien niet gebruik maken van encapsulatie: nu wo...

Repository maken over Flow.Uitlevering.Hoofding ? encapsulaties gebruiken waar het kan? overweeg met PRA om te zien of het de investering waard is.
Indien niet gebruik maken van encapsulatie:
nu wordt 2 x $Order uitgevoerd voor één uitlevering. zo ook voor de volgende whiles
Dat kan beter
Na de rest van de code te bekijken : Repository maken (resulteert in één iterator en testbaarheid of meer dan één uitlevering correct afgehandeld worden.

Dit gaat alleen voor de eerste bon werken niet meer voor de 2de.

Dit gaat alleen voor de eerste bon werken niet meer voor de 2de.