CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Zelfde reply als 'ginder'

Zelfde reply als 'ginder'

De JoC heeft mij da zo geleerd, als zijnde 'good coding convention', voor mocht er in de toekomst iemand eens een andere resource willen meegeven

De JoC heeft mij da zo geleerd, als zijnde 'good coding convention', voor mocht er in de toekomst iemand eens een andere resource willen meegeven

Niet echt http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Had die idd beter niet opgenomen in de review, sorry

Niet echt Had die idd beter niet opgenomen in de review, sorry

Is deze klasse nog relevant?

Is deze klasse nog relevant?

Zelfde opmerking als bij BestandUploader

Zelfde opmerking als bij BestandUploader

Er zijn geen tests op deze klasse, waarom constructor injections?

Er zijn geen tests op deze klasse, waarom constructor injections?

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:
[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:
Daar zijn "ze" van op de hoogte en dat zou zo de bedoeling zijn. Anyway... stuff voor ander kaartje, indien alsnog nodig.

Daar zijn "ze" van op de hoogte en dat zou zo de bedoeling zijn. Anyway... stuff voor ander kaartje, indien alsnog nodig.

Moeten er ook wijzigingen zijn in de AppsNaarDom? En bij uitbreiding: in hoeverre moet dit ook tot in Econ geraken? Dan moeten die convertoren ook nog aangepast worden.

Moeten er ook wijzigingen zijn in de AppsNaarDom?

En bij uitbreiding: in hoeverre moet dit ook tot in Econ geraken? Dan moeten die convertoren ook nog aangepast worden.

Er gaat geen probleem zijn indien het Leveradres geen landnummer heeft? Ik weet dat het bij ons in principe geen kwaad kan, maar als we in ax defaulten naar BE geeft dat misschien rariteiten? Soit,...

Er gaat geen probleem zijn indien het Leveradres geen landnummer heeft? Ik weet dat het bij ons in principe geen kwaad kan, maar als we in ax defaulten naar BE geeft dat misschien rariteiten? Soit, just wondering.

[UW521] Alpnach SFS: nieuwe korting 27.7%
[UW521] Alpnach SFS: nieuwe korting 27.7%
Beter dan het verwijderen van de test-method, is in dit geval : de test aanpassen en controleren op waarde "" (leeg), net zoals de wijziging in de code.

Beter dan het verwijderen van de test-method, is in dit geval : de test aanpassen en controleren op waarde "" (leeg), net zoals de wijziging in de code.

UT: in bestaande testclass : method toevoegen voor ZijkantLogoID = "UT_logo" 1) Ik zal eerlijk zeggen dat ik zelf eerst twijfelde om deze review-opmerking te maken, maar toen viel m'n oog op de "c...

UT: in bestaande testclass : method toevoegen voor ZijkantLogoID = "UT_logo"

1) Ik zal eerlijk zeggen dat ik zelf eerst twijfelde om deze review-opmerking te maken, maar toen viel m'n oog op de "complexiteit van de ()" "haakjes in caché", en dus sowieso beter om effe in een testje vast te leggen.
2) Of je de waarde "0" expliciet gaat testen, laat ik aan u over.
3) Leeg "" moet je niet testen, want zit impliciet in de andere tests.

[UST3856] EDI-verwerking DKG - land leveradres niet correct verwerkt:
[UST3856] EDI-verwerking DKG - land leveradres niet correct verwerkt:
[UST3863] TAX-LL-001 : opstartfase voorzien voor zijkantlogo
[UST3863] TAX-LL-001 : opstartfase voorzien voor zijkantlogo