Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Deze method op APPS.Halux.AAP.DnaCodeService zetten en van daaruit de repo oproepen. En eigenlijk ook testjes voorzien.

Deze method op APPS.Halux.AAP.DnaCodeService zetten en van daaruit de repo oproepen.

En eigenlijk ook testjes voorzien.

[ICT-5581] Veel orders blijven vastzitten in buffer
[ICT-5581] Veel orders blijven vastzitten in buffer
De lijst van de klantnummers is nu duplicate code. Kunnen we afspreken dat bij een volgende uitbreiding, dit zal ge-refactored worden. Dan krijg je gratis een method met een leesbare naam, wat de b...

De lijst van de klantnummers is nu duplicate code.
Kunnen we afspreken dat bij een volgende uitbreiding, dit zal ge-refactored worden. Dan krijg je gratis een method met een leesbare naam, wat de begrijpbaarheid van de code verhoogd.
Bijkomende optionele uitbreiding is om dit in Data te steken, dan is het tevens gedeeld met cache05, wat dan weer enkele aanpassingen uitspaart (misschien :-P )

Op dit moment is de code-change aanvaard.

Lijst van klantnummers : zie opmerking hieronder

Lijst van klantnummers : zie opmerking hieronder

[ICT-2483]Kvik-workaround voor andere klanten
[ICT-2483]Kvik-workaround voor andere klanten
Geen idee wat er gebeurd is, maar nu ineens staat bij mijn lokale lijst te committen bestanden deze laatste wijziging. Ik denk dat de vorige versie niet echt resultaat opgeleverd heeft http://subve...

Geen idee wat er gebeurd is, maar nu ineens staat bij mijn lokale lijst te committen bestanden deze laatste wijziging. Ik denk dat de vorige versie niet echt resultaat opgeleverd heeft

[ICT-2351] Aanpassing voorwaarden VRD
[ICT-2351] Aanpassing voorwaarden VRD
Ik weet het, het komt van ergens anders, maar toch http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Deze test gaat niet veel doen: ofwel renamen en zien ...

Ik weet het, het komt van ergens anders, maar toch Deze test gaat niet veel doen: ofwel renamen en zien dat ie werkt, ofwel (als hij niet relevant is) verwijderen

Ik weet dat deze code niet meer voor cache05 is, maar vroeger probeerden we overeenkomstige classes in trunk/deploy gelijklopend met trunk2010/deploy2010 te houden. Is dat principe nu totaal niet m...

Ik weet dat deze code niet meer voor cache05 is, maar vroeger probeerden we overeenkomstige classes in trunk/deploy gelijklopend met trunk2010/deploy2010 te houden.
Is dat principe nu totaal niet meer geldend?
En dit dan vermelden op weekplanning?

[ICT-2170] pa-ub-220: toelevering mee invullen bij maken van buffer
[ICT-2170] pa-ub-220: toelevering mee invullen bij maken van buffer
Goed opgemerkt, Jelle. Tom zal wel meteen zien dat de .ProductNummer binnen de Tech.Equals() moet staan. @ iedereen (vooral de jonkies) : Bovenal is de vraag : is er geen UT die dit aan het licht ...

Goed opgemerkt, Jelle.
Tom zal wel meteen zien dat de .ProductNummer binnen de Tech.Equals() moet staan.

@ iedereen (vooral de jonkies) :
Bovenal is de vraag : is er geen UT die dit aan het licht brengt?
Zal zelf al deels antwoorden :
Deze klasse BL.MB.UP.Receptie.cls bevat stokoude implementatie, en er is nauwelijks sprake van "scheiding van dependencies".
Dit laatste maakt het dan ook zeer moeilijk om UT hiervoor te schrijven.
@Tom : misschien toch een systeemke bedenken om de Dna-complexiteit af te zonderen in een helper-klasse, zodat die alsnog kan ge-UT-ed worden :
bvb. DnaCodeSoortBepaler .IsVoorKad() .IsVoorVhoss(), .IsVoorSpaceStep()
( *) woordkeuze "...Soort..." kan vast beter!

Geen Asserts na een "VerwachteExceptie". Deze worden toch niet uitgevoerd, kunnen daarom zelfs foute waarden tonen en niet falen. Dus sowieso de asserts hier weghalen. @iedereen : deze opmerking a...

Geen Asserts na een "VerwachteExceptie".
Deze worden toch niet uitgevoerd, kunnen daarom zelfs foute waarden tonen en niet falen. Dus sowieso de asserts hier weghalen.

@iedereen : deze opmerking altijd maken als reviewer, wanneer iemand hiertegen zondigt. En als developer hier best zelf proberen op te letten :-P