CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
[BUS662] HFL NL en BE: forceren dat OV haalbare leverdatum kiest bij sluiten order vanop multipersonality...
[BUS662] HFL NL en BE: forceren dat OV haalbare leverdatum kiest bij sluiten order vanop multipersonality...
[DEF624] Maken van toeleveringen: mag niet voor bouwstenen °INK of °EXP
[DEF624] Maken van toeleveringen: mag niet voor bouwstenen °INK of °EXP
Testen voor requestconverter samengevoegd in UST3184

Testen voor requestconverter samengevoegd in UST3184

Voor de aangevraagde change werd er in eerste instantie van uitgegaan dat de filter in de iterator nodig was. Toen die opgezet was en het resultaat werd afgetoetst bleek dat de change op een andere...

Voor de aangevraagde change werd er in eerste instantie van uitgegaan dat de filter in de iterator nodig was. Toen die opgezet was en het resultaat werd afgetoetst bleek dat de change op een andere manier moest aangepakt worden. I.p.v. de filter terug te verwijderen heb ik deze oplossing gekozen. Het was bovendien de laatste migratie die nog geen dynamische iterator met filter gebruikte, hetzij dan dat deze voorlopig altijd true teruggeeft. Het is meer dan waarschijnlijk dat er alsnog filtering bij het itereren moet worden doorgevoerd. Het staat daarvoor klaar nu.