CacheAdminA_trunk2010/vhUnitTest/APPS/VKP/impl

VerzendwijzeMock

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 1
Total Committers: 4
Last Commit: 19 Jun 15
Commits this week: 1
Total Lines of Code (LoC): 20
Net change in LoC this week: 4

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 24 Jun 2021
8:16 am

Jo Willems

Thursday 19 Mar 2020
11:12 am

Jo Willems

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

11:04 am

Jo Willems

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.

Wednesday 18 Mar 2020
6:24 pm

Jo Willems

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.

5:59 pm

Jo Willems

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.

5:56 pm

Jo Willems

Private?

Private?

5:54 pm

Jo Willems

Wat doen we met Data?

Wat doen we met Data?

4:53 pm

Jo Willems

grrrrr...

grrrrr...

4:52 pm

Jo Willems

grrrr....

grrrr....

10:57 am

Jo Willems

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.

10:42 am

Jo Willems

cfr andere opmerking in implementatie

cfr andere opmerking in implementatie

8:52 am

Jo Willems

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....

Thursday 16 Jan 2020
1:54 pm

Laura Landuyt

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

1:52 pm

Laura Landuyt

// 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

1:51 pm

Laura Landuyt

//$Not van maken ipv '

//$Not van maken ipv '

1:51 pm

Laura Landuyt

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

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

1:50 pm

Laura Landuyt

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

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

1:49 pm

Laura Landuyt

een else kant toevoegen met een log else{ // log toevoegen dat event uitgesteld zal worden verzonden }

een else kant toevoegen met een log

else

Unknown macro: { // log toevoegen dat event uitgesteld zal worden verzonden }
1:48 pm

Laura Landuyt

// var hernoemen naar GeneratePakbonMagGeboektWordenResult en ook het vereder gebruik , argument in methode aanpassen

// var hernoemen naar GeneratePakbonMagGeboektWordenResult en ook het vereder gebruik , argument in methode aanpassen

1:40 pm

Laura Landuyt

Niet gewoon gebruik maken van de globals maar de gemapte objecten gebruiken zoals Geefbon Daaraan een methode toevoegen als en Zetvlag(new) De zet vlag zal dan onderstaande set piece bevatten

Niet gewoon gebruik maken van de globals maar de gemapte objecten gebruiken zoals Geefbon
Daaraan een methode toevoegen als en Zetvlag(new)
De zet vlag zal dan onderstaande set piece bevatten

1:38 pm

Laura Landuyt

ctor toevoegen en injecteren van de geeforderapiservice niet via context

ctor toevoegen en injecteren van de geeforderapiservice niet via context

1:38 pm

Laura Landuyt

VerwerkRecord hernoemen naar VerwerkBon (dit is iets duidelijker)

VerwerkRecord hernoemen naar VerwerkBon (dit is iets duidelijker)

1:36 pm

Laura Landuyt

Naamgevingen verduidelijken *Geen engelse stukken vb VerzendwijzeLONG.. *{} plaatsen volgens cache standaard *Zie ook confluence notities voor dit ticket
  • Naamgevingen verduidelijken
  • Geen engelse stukken vb VerzendwijzeLONG..
  • {} plaatsen volgens cache standaard
  • Zie ook confluence notities voor dit ticket
Wednesday 15 Jan 2020
8:34 am

Laura Landuyt

[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden
Monday 25 Mar 2019
8:52 am

Kurt Callebaut

Thursday 14 Mar 2019
10:19 am

Martijn Vierstraete

10:18 am

Martijn Vierstraete

Je zou de LandRepository ook kunnen toevoegen aan AxToAdminUtils, en hier DlvAddrCountry ipv landnr meegeven (zoals ook Dlv omgezet wordt naar Verzendwijze in de utils klasse)

Je zou de LandRepository ook kunnen toevoegen aan AxToAdminUtils, en hier DlvAddrCountry ipv landnr meegeven (zoals ook Dlv omgezet wordt naar Verzendwijze in de utils klasse)

9:49 am

Martijn Vierstraete

Verwarrend commentaar (vooral lijn 62 & 64)

Verwarrend commentaar (vooral lijn 62 & 64)

9:42 am

Martijn Vierstraete

Test: Test:

Test: Test:

Wednesday 13 Mar 2019
11:06 am

Kurt Callebaut

blijkt dat AX voor salesorderin iso2 doorstuurt en dus niet landcode. (test aangepast.) Voor andere zoals adres stuurt AX dan weer iso3.

blijkt dat AX voor salesorderin iso2 doorstuurt en dus niet landcode. (test aangepast.)
Voor andere zoals adres stuurt AX dan weer iso3.