CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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;

Van een MaakReceptieVoorToelevering zou ik eerder een toelevering object / toeleveringid verwachten en geen differse parameters -> misschien is MaakReceptie voldoende

Van een MaakReceptieVoorToelevering zou ik eerder een toelevering object / toeleveringid verwachten en geen differse parameters -> misschien is MaakReceptie voldoende

GeefProductIdVoorLeverancierId lijkt mij toch wel een rare naamgeving ; misschien is een GeefProductId al voldoende

GeefProductIdVoorLeverancierId lijkt mij toch wel een rare naamgeving ; misschien is een GeefProductId al voldoende

Lijkt mij beter om de productlijn in een variabele te steken en deze hier te gebruiken ipv ProductLijnen.GetAt(1) -> verhoogt de leesbaar -> begrijpbaarheid

Lijkt mij beter om de productlijn in een variabele te steken en deze hier te gebruiken ipv ProductLijnen.GetAt(1) -> verhoogt de leesbaar -> begrijpbaarheid

StandaardToelevering hoofding met flexibele in te stellen data lijkt mij niet echt standaard te zijn -> lijkt mij eerder iets in om in .Fake. te zetten

StandaardToelevering hoofding met flexibele in te stellen data lijkt mij niet echt standaard te zijn -> lijkt mij eerder iets in om in .Fake. te zetten

Opletten met negatieve resultaten in functies ; beter IsReceptieNodig -> dubbele ontkenningen etc..

Opletten met negatieve resultaten in functies ; beter IsReceptieNodig -> dubbele ontkenningen etc..

  • More
  • CR-576
  • finished reviewing
[ICT323] - enu..Verwerk.cls -> enu..Verwerker.cls
[ICT323] - enu..Verwerk.cls -> enu..Verwerker.cls
[BUS661] -TA'OR : Baliko concept -> andere afmetingen
[BUS661] -TA'OR : Baliko concept -> andere afmetingen
  • More
  • CR-652
  • finished reviewing
[UST3136] VHIP Migratie: Vrijgegeven Producten: GrootteOrdeVerkoop:
[UST3136] VHIP Migratie: Vrijgegeven Producten: GrootteOrdeVerkoop:
[UST3184] VHIP481: BOMBOL: Unieke ConfigVariantId
[UST3184] VHIP481: BOMBOL: Unieke ConfigVariantId