Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1078] AT 1467: Pakbon pas aanmaken op moment van transport

Aanpassen unit testen volgens nieuwe code implementatie

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

Pakbon code aanpassen, gebruik maken van Bon.GeefVerzendwijzeID ipv Bon.GeefVerzendwijze

  1. … 2 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

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

    • -2
    • +2
    ./PakbonMagGeboektWordenHelper/Test.cls.xml
    • -3
    • +3
    ./PakbonMagGeboektWordenHelper/IsVerzendwijzeMetUitgesteldPakbonBoekenAxTest.cls.xml
  1. … 6 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.

ik zou dit toch over een paar lijnen spreiden, gaat wat leesbaarder worden dan.

ik zou dit toch over een paar lijnen spreiden, gaat wat leesbaarder worden dan.

Private?

Private?

Wat doen we met Data?

Wat doen we met Data?

grrrrr...

grrrrr...

grrrr....

grrrr....

Maak hier een specifieke method voor waarvan de naam betekenis heeft. 63 zegt niet veel. cfr andere opmerking op de impl hiervan.

Maak hier een specifieke method voor waarvan de naam betekenis heeft. 63 zegt niet veel. cfr andere opmerking op de impl hiervan.

cfr andere opmerking in implementatie

cfr andere opmerking in implementatie

Lijkt me 2 mooie methods om private te zetten en dan in specifiek genaamde methods voor speciefieke gegevens te gebruiken. Nu staat er ergens anders in de code een getal 63 te blinken....

Lijkt me 2 mooie methods om private te zetten en dan in specifiek genaamde methods voor speciefieke gegevens te gebruiken.
Nu staat er ergens anders in de code een getal 63 te blinken....

[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactoring volgens coding guidelines + wlip's verwijderen

    • -43
    • +33
    ./PakbonMagGeboektWordenHelper/Test.cls.xml
    • -8
    • +8
    ./PakbonMagGeboektWordenHelper/IsVerzendwijzeMetUitgesteldPakbonBoekenAxTest.cls.xml
  1. … 5 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Gebruik maken van de helper klasse zijn IsVerzendwijzeMetUitgesteldPakbonBoekenAx methode en de gelijkaardige methode uit de klasse AXimpl.Dynamics.InventoryOut.EventHandler.PickingCompleet.PakbonMagGeboektWorden verwijderen en helperklasse injecten + test verwijderen , worden in de helperklasse zijn testen opgevangen

  1. … 3 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Toevoegen testen methode IsVerzendwijzeMetUitgesteldPakbonBoekenAxTest uit helperklasse

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

Toevoegen testen methode BonMarkerenAlsVerzondenTest uit helperklasse

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

Hernoemen GeefVlag en zetvlag naar GeeftVlagHooofdign en ZetVlagHooding

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

Toevoegen AfhandelenBijWijzigenVerzendwijze aan helper klasse en bijhorende testen

Methode aanroepen vanuit de KFVZW zijn SET methode

    • -0
    • +106
    ./PakbonMagGeboektWordenHelper/Test.cls.xml
  1. … 2 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactoren code, toevoegen van een helper klasse

  1. … 3 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Hernoemen methode RaisePakbonMagGeboektWordenEvent naar AfhandelaarRaisePakbonMagGeboektWordenEvent om verwarring te voorkomen + referenties aanpassen

  1. … 4 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor code /PickingCompleet/PakbonMagGeboektWorden Methode RaisePakbonMagGeboektWordenEvent en VerzendwijzeUitgesteldPakbonBoekenAx + injecteren van OrderApi

  1. … 1 more file in changeset.
Geen engelstalige stukken in naamgeving Code wordt overbodig want GeefBon gebruiken en dan verder ook Bon.GeefVerzendwijze

Geen engelstalige stukken in naamgeving
Code wordt overbodig want GeefBon gebruiken en dan verder ook Bon.GeefVerzendwijze

// GeefVerzendwijze methode gebruiken uit apps.common.dataservice ipv zelf rechtstreeks %OpenId te doen

// GeefVerzendwijze methode gebruiken uit apps.common.dataservice ipv zelf rechtstreeks %OpenId te doen

//$Not van maken ipv '

//$Not van maken ipv '

// GeefBon --> Geef klant --> via objecten oproepen, niet rechtstreeks via de global

// GeefBon --> Geef klant --> via objecten oproepen, niet rechtstreeks via de global

//$Not van maken in plaats van ..VerzendwijzeUitgesteldPakbonBoekenAx(bon, klant) = $$$False

//$Not van maken in plaats van ..VerzendwijzeUitgesteldPakbonBoekenAx(bon, klant) = $$$False