CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
beetje aangepast

beetje aangepast

1. aangepast 2. aangezien dat zo uit de pre-refactor-code komt, ga ik dat in eerste instantie niet aanpassen, want dan maskeer ik mischien gedrag dat ik nu niet ken. (en ondertussen hebben we hier ...

1. aangepast
2. aangezien dat zo uit de pre-refactor-code komt, ga ik dat in eerste instantie niet aanpassen, want dan maskeer ik mischien gedrag dat ik nu niet ken. (en ondertussen hebben we hier op den bureau er al een hele boom over opgezet )

Test met lege personality toevoegen? Zit nu eigenlijk wel vervat in de test "met crappy data", maar deze zie ik graag toch als aparte testmethod. (in de veronderstelling dat er een exception wordt ...

Test met lege personality toevoegen?
Zit nu eigenlijk wel vervat in de test "met crappy data", maar deze zie ik graag toch als aparte testmethod.
(in de veronderstelling dat er een exception wordt gesmeten, i.p.v. fallback op de VH-personality)

LET OP: hier kan je beter wel de exceptie smijten, en de oproeper laten beslissen om al dan niet terug te vallen op VH-personality.

LET OP: hier kan je beter wel de exceptie smijten, en de oproeper laten beslissen om al dan niet terug te vallen op VH-personality.

Ik verkies om hier geen one-liner meer te gebruiken om volgende reden : Voorheen was de property straight-forward gevuld, dus met z'n échte waarde : ..LEVERINGDATUM = piece 2 van ^KUL Nu wordt het ...

Ik verkies om hier geen one-liner meer te gebruiken om volgende reden :
Voorheen was de property straight-forward gevuld, dus met z'n échte waarde : ..LEVERINGDATUM = piece 2 van ^KUL
Nu wordt het veld misbruikt om een andere waarde te tonen.
Ik ben van mening dat het duidelijker is om dit in 2 lijnen code te zeten :
var VerzendDatum = ..GeefDatumOpBon() --> deze lijn doet écht wat ze zegt
set ..LEVERINGDATUM = VerzendDaum --> deze lijn geeft aan dat het veldje "misbruikt" wordt.

Als je dit in een one-liner zet, dan verberg je zelfs dat er gefoefeld wordt, of zal iemand anders zich meer vragen stellen of dit wel juist is.

oude code is nu weg.

oude code is nu weg.

zelfde als hierboven: oude code mag weg

zelfde als hierboven: oude code mag weg

oude code mag weg hoor, zie svn waarom geen one-liner?

oude code mag weg hoor, zie svn
waarom geen one-liner?

[VRB484] HX: TAX ProductieGroepBepaler met Opstartfase
[VRB484] HX: TAX ProductieGroepBepaler met Opstartfase
[VRB484][VRB485] HX: TAX ProductieGroepBepaler met Opstartfase
[VRB484][VRB485] HX: TAX ProductieGroepBepaler met Opstartfase
[UST3795] Docbase leverbon - verzenddatum ipv leverdatum vermelden
[UST3795] Docbase leverbon - verzenddatum ipv leverdatum vermelden
[UST3767] LBXSY3: PPS-kenmerk voor kleur van sifonkap
[UST3767] LBXSY3: PPS-kenmerk voor kleur van sifonkap
Volgens de code komen de nietmeeleverens binnne als list en worden deze geconverteerd naar deze properties, maar aangezien deze intern nerges meer gebruikt zal worden, lijkt me dat ok te zijn. We g...

Volgens de code komen de nietmeeleverens binnne als list en worden deze geconverteerd naar deze properties, maar aangezien deze intern nerges meer gebruikt zal worden, lijkt me dat ok te zijn. We gaan dit in ieder geval op test valideren.

Volgens de code komen de nietmeeleverens binnne als list en worden deze geconverteerd naar deze properties, maar aangezien deze intern nerges meer gebruikt zal worden, lijkt me dat ok te zijn. We g...

Volgens de code komen de nietmeeleverens binnne als list en worden deze geconverteerd naar deze properties, maar aangezien deze intern nerges meer gebruikt zal worden, lijkt me dat ok te zijn. We gaan dit in ieder geval op test valideren.

[UST3802] InventoryInConverter igv HerverpakteSopr: Aantal:
[UST3802] InventoryInConverter igv HerverpakteSopr: Aantal:
voor de reformat http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/thumbs_down.gifvoor early quits

voor de reformat
voor early quits

Niets fout, maar http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif 1. Constructor injects als optionele parameters 2. Api's newen ipv ophalen van context,...

Niets fout, maar
1. Constructor injects als optionele parameters
2. Api's newen ipv ophalen van context, indien mogelijk

Idem opmerking LBX

Idem opmerking LBX

Kan het kwaad dat de storage index wijzigt?

Kan het kwaad dat de storage index wijzigt?

Zelfde opmerking als bij LBX

Zelfde opmerking als bij LBX

nieuwe wijziging daarnet on-the-fly door WV gereviewed

nieuwe wijziging daarnet on-the-fly door WV gereviewed

Voor alle zekerheid de 'storage-name' behouden, je weet nl. niet of dto's als property mee bewaard worden

Voor alle zekerheid de 'storage-name' behouden, je weet nl. niet of dto's als property mee bewaard worden

[UST3798] HNLT: Mogelijke leverdatums aanpassen
[UST3798] HNLT: Mogelijke leverdatums aanpassen