CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-684
  • finished reviewing
  • More
  • CR-679
  • finished reviewing
'k niet direct vinden vanwaar deze fout komt maar ik zou toch opteren voor '' in de tekst. Product met id 'OnbestaandProduct' bestaat niet.

'k niet direct vinden vanwaar deze fout komt maar ik zou toch opteren voor '' in de tekst.

Product met id 'OnbestaandProduct' bestaat niet.

  • More
  • CR-743
  • finished reviewing
Heb je rekenining gehouden met ProcedureBlock 0/1 ? indien ja, geen verdere opmerkingen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Heb je rekenining gehouden met ProcedureBlock 0/1 ?
indien ja, geen verdere opmerkingen

  • More
  • CR-742
  • finished reviewing
Dit ruikt naar feature envy

Dit ruikt naar feature envy

op zich is het geen probleem maar ik zou toch opteren voor quit:$$$Not(TranslationIterator.HasNext()) ; zodoende het duidelijker is dat dit een guard clause is en je dan geen indentatie lager is

op zich is het geen probleem maar ik zou toch opteren voor quit:$$$Not(TranslationIterator.HasNext()) ; zodoende het duidelijker is dat dit een guard clause is en je dan geen indentatie lager is

  • More
  • CR-666
  • finished reviewing
Als je dan toch de constructor hebt aangepast, gelieve de Langtekstbewaarder dan te instantieren in de %OnNew

Als je dan toch de constructor hebt aangepast, gelieve de Langtekstbewaarder dan te instantieren in de %OnNew

  • More
  • CR-654
  • finished reviewing
!!!!!!!! niet vergeten den productie url ooit in te vullen op productie !!!!!!

!!!!!!!! niet vergeten den productie url ooit in te vullen op productie !!!!!!

  • More
  • CR-653
  • finished reviewing
  • More
  • CR-739
  • finished reviewing
Code duplicatie, zit al in producttypeapi, maar werkt daar rechtstreeks op KPR, dus alleen voor bewaarde producten, en zit daar dus niet op zijn plaats volgens mij.

Code duplicatie, zit al in producttypeapi, maar werkt daar rechtstreeks op KPR, dus alleen voor bewaarde producten, en zit daar dus niet op zijn plaats volgens mij.

Code duplicatie, zit al in producttypeapi, maar werkt daar rechtstreeks op KPR, dus alleen voor bewaarde producten, en zit daar dus niet op zijn plaats volgens mij.

Code duplicatie, zit al in producttypeapi, maar werkt daar rechtstreeks op KPR, dus alleen voor bewaarde producten, en zit daar dus niet op zijn plaats volgens mij.

productID ipv 'productnummer met productID' http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

productID ipv 'productnummer met productID'

omschrijving exception juist? Kan toch ook nog ander type product zijn?

omschrijving exception juist? Kan toch ook nog ander type product zijn?

  • More
  • CR-733
  • finished reviewing
  • More
  • CR-730
  • finished reviewing
  • More
  • CR-629
  • finished reviewing
Een beetje te kort door de bocht wijziging -> laattijdig reviewen is wel handig -> bewijs wordt in de loop van geleverd http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emot...

Een beetje te kort door de bocht wijziging -> laattijdig reviewen is wel handig -> bewijs wordt in de loop van geleverd

  • More
  • CR-614
  • finished reviewing
  • More
  • CR-604
  • finished reviewing
Het feit dat er één aanpassing is en dat er 5 testen moeten aangepast worden dit moet toch één of andere bellentje doen rinkelen dat de structuur toch niet zo 'bien' is -> niet voor nu op te lossen...

Het feit dat er één aanpassing is en dat er 5 testen moeten aangepast worden dit moet toch één of andere bellentje doen rinkelen dat de structuur toch niet zo 'bien' is -> niet voor nu op te lossen , maar te aanschouwen als een verbeterpuntje

Geen goed idee om dit iedere keer te herhalen -> beter één maal in basis test , die het standaard gedrag test , maar voor de rest een stub gebruiken , met eventueel een dummy product repo -> aandac...

Geen goed idee om dit iedere keer te herhalen -> beter één maal in basis test , die het standaard gedrag test , maar voor de rest een stub gebruiken , met eventueel een dummy product repo -> aandacht gaat teveel naar zaken die er niet toe doen in de testen.

!!! Testen moeten duidelijk zijn -> werken is maar een bijzaak , indien ze niet werken moeten ze makkelijk aan te passen zijn -> dus hoe duidelijker hoe beter;