Activity CacheAdminA_trunk/UI/MB/UGLYPicking/dlgOrderSel.cls.xml

11:36 am

Tom Vermeulen

More
Re: CR-2821
commented on review

Hoe dan ook meer en kleinere commits. *klasse toegevoegd *filters aangepast *nieuwe klasse aangesproken *... Kan alleen maar helpen voor als ge iets om zeep helpt tijdens ontwikkeling om te kun...

Hoe dan ook meer en kleinere commits.

klasse toegevoegd
filters aangepast
nieuwe klasse aangesproken
...

Kan alleen maar helpen voor als ge iets om zeep helpt tijdens ontwikkeling om te kunnen reverten zonder heel de nest te moeten terugdraaien. Helpt anderen ook om in de commitlog te zien wat er allemaal gebeurd is.Kan ook helpen om een taak die halverwege gestopt is (door andere prio's ofzo) eenvoudig door iemand anders te laten verderzetten.

Soit, iets met gewoontes kweken enzo

11:03 am

Tom Vermeulen

More
Re: CR-2821
commented on review

Is het niet beter om hier een andere Locknaam te gebruiken? Ik heb niet teveel de code allemaal nagelezen, dus mogelijks zorgt het niet voor een probleem (of moet het expres dezelfde zijn). Hoe dan...

Is het niet beter om hier een andere Locknaam te gebruiken? Ik heb niet teveel de code allemaal nagelezen, dus mogelijks zorgt het niet voor een probleem (of moet het expres dezelfde zijn). Hoe dan ook moet je de 'scope' van een lock zo beperkt mogelijk houden (maar het gedrag mag ook niet verkeerd kunnen gaan natuurlijk)

10:40 am