CacheAdminA_trunk2010/vhUnitTest/AXimpl/Dynamics/SalesOrderIn/Converter/SalesOrderConverter

LeverAdresMaker

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 22 Dec 17
Commits this week: 0
Total Lines of Code (LoC): 138
Net change in LoC this week: 0

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

11:05 am

Kurt Callebaut

[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?
[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?
Wednesday 21 Feb 2018
1:46 pm

Tom Vermeulen

1:17 pm

Tom Vermeulen

Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/...

Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn )

Zal het er wel bijsteken voor de volledigheid

11:19 am

Tommy Hebb

11:10 am

Tommy Hebb

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

11:04 am

Tommy Hebb

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

11:04 am

Tommy Hebb

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

11:01 am

Tommy Hebb

KlantId of KlantNummer?

KlantId of KlantNummer?

11:01 am

Tommy Hebb

KlantId of KlantNummer?

KlantId of KlantNummer?

10:54 am

Tommy Hebb

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

8:47 am

Tom Vermeulen

[UST3670] LeverAdresRepository aanpassen
[UST3670] LeverAdresRepository aanpassen
Monday 22 Jan 2018
2:31 pm

Tom Vermeulen

2:31 pm

Tom Vermeulen

Dit is ondertussen al door anderen aangepast (zie boven: file outdated)

Dit is ondertussen al door anderen aangepast (zie boven: file outdated)

2:29 pm

Tom Vermeulen

Er komt geen foutmelding, zie testjes

Er komt geen foutmelding, zie testjes

2:27 pm

Tom Vermeulen

file outdated, niet meer relevant

file outdated, niet meer relevant

Friday 19 Jan 2018
3:31 pm

Tom Vermeulen

Absoluut, die van VH zelfs. Aangezien het niet leeg mag gelaten worden hebben mensen die nog slimmer zijn dan wij beslist om daar het VH nummer in te zetten voor het geval een chauffeur ergens vast...

Absoluut, die van VH zelfs. Aangezien het niet leeg mag gelaten worden hebben mensen die nog slimmer zijn dan wij beslist om daar het VH nummer in te zetten voor het geval een chauffeur ergens vaststaat en geen opmerkingen op zijn blad ziet. Kan ie in extremis naar hier bellen.

Monday 15 Jan 2018
2:21 pm

Wim Vermeulen

  • More
  • CR-870
  • finished reviewing
2:21 pm

Wim Vermeulen

Het (input) AxId moet in de foutmelding vermeld worden. dit is gekoppeld aan mn opmerking in de impl-klasse.

Het (input) AxId moet in de foutmelding vermeld worden.
dit is gekoppeld aan mn opmerking in de impl-klasse.

12:50 pm

Wim Vermeulen

Exact dezelfde asserts als de method hierboven. Is dit toeval of niet? --> refactor?

Exact dezelfde asserts als de method hierboven.
Is dit toeval of niet? --> refactor?

12:32 pm

Wim Vermeulen

Indien ResultSet.Next() = 0 ga je nu een foutmelding krijgen à la <SUBSCRIPT> of "lege waarde is ongeldig" of "ongeldige KlantID "K||" En dan word je meteen op verkeerde spoor gezet, want je denkt ...

Indien ResultSet.Next() = 0 ga je nu een foutmelding krijgen à la <SUBSCRIPT> of "lege waarde is ongeldig" of "ongeldige KlantID "K||"
En dan word je meteen op verkeerde spoor gezet, want je denkt dat er een veld met lege waarden bestaat.

Dus beter: deze twijfel meteen weg te nemen door if .Next() = 0 then Throw Exception("Geen leveradres gevonden voor "_AXLeverAdresId)

11:53 am

Wim Vermeulen

beter: nen échte Boolean van maken. bvb ... >0 of met een '' (not(not( ... ))

beter: nen échte Boolean van maken.
bvb ... >0 of met een '' (not(not( ... ))

11:28 am

Wim Vermeulen

Wat is dan de conclusie? *is er dan totaal geen order gemaakt? *of wel een order, maar geen leveradres gekoppeld? *Moet men iets in AX ondernemen? Of aanpassing te doen in Admin? Indien mogelij...

Wat is dan de conclusie?

  • is er dan totaal geen order gemaakt?
  • of wel een order, maar geen leveradres gekoppeld?
  • Moet men iets in AX ondernemen? Of aanpassing te doen in Admin?


Indien mogelijk, graag verduidelijken wat de verwachte handelingen zijn.
Al staat er in de message iets als "zie wiki voor mogelijke oplossingen ... " of whatever.

11:21 am

Wim Vermeulen

hard-coded telefoonnummer ???

hard-coded telefoonnummer ???

Friday 29 Dec 2017
1:49 pm

Tom Vermeulen

[UST3333] VHIP504, VHIP901, VH4_SVC: AxLeveradresId gebruiken
[UST3333] VHIP504, VHIP901, VH4_SVC: AxLeveradresId gebruiken