Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1432] HR: Brandlijst: oude code opkuisen DOM.HR.*

- Queries om ProtimeNr van Medewerker of Medewerker van ProtimeNr op te halen verwijderd

[UW441] Leveringsbon naar eerste persoon met gewenstedocs: correcte spelling "LeveringBon"
Whaaaw!!! dit is niet zomaar een klein addertje onder het gras :-/ #anaconda ;-P .

Whaaaw!!!

dit is niet zomaar een klein addertje onder het gras :-/
#anaconda ;-P
.

Aandacht voor de correcte schrijfwijze! geen tussen "-s" (of alternatief "bon" met kleine letter, zodat "leveringsbon" één kernwoord is) Voor mij persoonlijk optie 1 : geen tussen "-s" LET OP : (r...

Aandacht voor de correcte schrijfwijze!
geen tussen "-s" (of alternatief "bon" met kleine letter, zodat "leveringsbon" één kernwoord is)
Voor mij persoonlijk optie 1 : geen tussen "-s"

LET OP : (reminder) indien je de property wijzigt, dan niet vergeten om ook aan te passen in de storage
.

[UW441] Leveringsbon naar eerste persoon met gewenstedocs [ leveringsbon, ipv AK2 (voor DKG). Extra...
[UW441] Leveringsbon naar eerste persoon met gewenstedocs [ leveringsbon, ipv AK2 (voor DKG). Extra...
Merged revision(s) 49940 from trunk2010:

[UW441] Leveringsbon naar eerste persoon met gewenstedocs [ leveringsbon, ipv AK2 (voor DKG).

........

[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- UniekID op Derde.Klant

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
[UST3333] VHIP504, VHIP901, VH4_SVC: AxLeveradresId gebruiken

- Index weer weg, staat daar niet juist

[UST3333] VHIP504, VHIP901, VH4_SVC: AxLeveradresId gebruiken

- Derde.Klant.LevAdres uitgebreid met AxLeveradresId

Commentaar.
  • More
  • CR-197
  • finished reviewing
Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen. Geen testen ...

Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen.
Geen testen ==> needs resolution -> De testen zitten dus in de svn-repo

  • More
  • CR-188
  • finished reviewing