CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3866] GeoDynamics: trager werken
[UST3866] GeoDynamics: trager werken
[VRB530] Vh4: EconConfiguratieSleutel doorgeven
[VRB530] Vh4: EconConfiguratieSleutel doorgeven
Do ##super() vergeten !!!

Do ##super() vergeten !!!

Zelfde reactie als 'ginder' ;-P

Zelfde reactie als 'ginder' ;-P

3 --> ..#DataStartRij

3 --> ..#DataStartRij

Ik ben het volledig eens met Martijn : Zolang er geen test (of andere nood aan Injection) is, zet je gewoon : Set ..Console = ##class(TECH.Console.impl.TerminalConsole).%New() Wanneer later iemand ...

Ik ben het volledig eens met Martijn :
Zolang er geen test (of andere nood aan Injection) is, zet je gewoon :
Set ..Console = ##class(TECH.Console.impl.TerminalConsole).%New()
Wanneer later iemand toch iets moet injecten, dan zal hij zelf (alleen het gewenste object) aanpassen naar $$$Inject()
De meeste onder ons hebben de gewoonte om op deze laatste manier te werken. Eerder dan alles meteen inject-ready te maken
Deze code terugdraaien hoeft niet voor mij ... op voorwaarde dat je in de toekomst onze manier overneemt :-D

[VRB524] Vh4: BestelWinkelkar: emailadres gebruiker ontvangen van ITR en opslaan:
[VRB524] Vh4: BestelWinkelkar: emailadres gebruiker ontvangen van ITR en opslaan:
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: