Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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)

Verwarrend commentaar (vooral lijn 62 & 64)

Verwarrend commentaar (vooral lijn 62 & 64)

Test: Test:

Test: Test:

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.

[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?
[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?
Dit is ondertussen al door anderen aangepast (zie boven: file outdated)

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

Er komt geen foutmelding, zie testjes

Er komt geen foutmelding, zie testjes

file outdated, niet meer relevant

file outdated, niet meer relevant

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.

  • More
  • CR-870
  • finished reviewing
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.

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?

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)

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

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.

hard-coded telefoonnummer ???

hard-coded telefoonnummer ???

[UST3333] VHIP504, VHIP901, VH4_SVC: AxLeveradresId gebruiken
[UST3333] VHIP504, VHIP901, VH4_SVC: AxLeveradresId gebruiken
  • More
  • CR-804
  • finished reviewing
[UST3161] [rvTVE] VHIP Migratie 0103KlantenAdres:
[UST3161] [rvTVE] VHIP Migratie 0103KlantenAdres:
  • More
  • CR-749
  • finished reviewing
Gebruik volgende keer best DyyyymmddxBlaBlaBla, met die x ertussen

Gebruik volgende keer best DyyyymmddxBlaBlaBla, met die x ertussen

[UST2894]VHIP900: EventHandling Caché to CRM/AX bij live afzetten
[UST2894]VHIP900: EventHandling Caché to CRM/AX bij live afzetten
  • More
  • CR-315
  • finished reviewing
[UST2581] : AX: MessageId op juiste plaats in interfaces
[UST2581] : AX: MessageId op juiste plaats in interfaces