CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Done!

Done!

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Die worden allebei in de caller van deze method al uitgesloten

Die worden allebei in de caller van deze method al uitgesloten

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:

Hier is geen fake voor nodig, dit kan evengoed met de echte implementatieklasse.

Hier is geen fake voor nodig, dit kan evengoed met de echte implementatieklasse.

[ICT482] EDI - niet meer over Vhintra voor LBX/TAX/beslag
[ICT482] EDI - niet meer over Vhintra voor LBX/TAX/beslag
[UST4167] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)
[UST4167] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)
[UW727] DoorHandelMeeleverenBepalers voor Pavanello, en Ferexpert + refactor
[UW727] DoorHandelMeeleverenBepalers voor Pavanello, en Ferexpert + refactor
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?
[UST4169] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Kostprijs:...
[UST4169] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Kostprijs:...
object LadeOnderdeel doorgeven, i.p.v. Rol As enu dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

object LadeOnderdeel doorgeven, i.p.v. Rol As enu
dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux". Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de kla...

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux".
Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de klant meegeeft.

Laat voorlopig maar staan, maar hou er wel rekening mee, dat dit soort defaultings "gedurfd" is

De bijhorende UT geeft aan dat *ID = "" geeft "" *ID = 0 geeft "" Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten? Want voor deze waarden is het eigenlijk zinloos ...

De bijhorende UT geeft aan dat

  • ID = "" geeft ""
  • ID = 0 geeft ""
    Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten?
    Want voor deze waarden is het eigenlijk zinloos om de bepaler op te roepen.
Deze lijn zou ik binnen de Else zetten. Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

Deze lijn zou ik binnen de Else zetten.
Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
[BUS689] TAX: LL: ander kostitem-bedrag voor laser logo/kostenplaats toevoegen.
[BUS689] TAX: LL: ander kostitem-bedrag voor laser logo/kostenplaats toevoegen.
[DEF740] VHIP901: ORDER2ADMIN SFS bestelopmerking toegevoegd bij elke update uit AX
[DEF740] VHIP901: ORDER2ADMIN SFS bestelopmerking toegevoegd bij elke update uit AX