[Koha-bugs] [Bug 7408] Expire holds that have been waiting too long

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Feb 24 20:17:57 CET 2012


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7408

--- Comment #21 from Chris Cormack <chris at bigballofwax.co.nz> 2012-02-24 19:17:57 UTC ---
(In reply to comment #20)
> This is a little discouraging to see some of the comments in this thread.  
> 
> This doesn't make much sense to me "trend of reimplementing
> people's work without prior discussion has got to stop"  If someone sees "in
> their opinion"  a different way to work out code for a particular BUG - then
> why discourage them from working on it and submitting the code?
> 
> Since there is competing code - why not just look at the two options and test
> them both?  In this situation I do agree - less chance of cronjobs stepping on
> each others toes - is a win.  Also for Koha - I don't think this is what one
> library wants but there are a lot of other libraries that want this sort of
> functionality to be included (I'm sure that I can find you plenty of libraries
> that are interested in the same functionality).  Holds Rewrite has been going
> on for years - maybe it would be best to have code that is ready now - be
> implemented.
> 
Sure, by all means,

But at least talk to the persons who patch you are marking obsolete with your
one before then. This is not the first bug this has happened on. 
Competing patches are fine, and now as Ian has unobsoleted his one, this is
what we have. Before we had someone replacing another persons patch with their
own.

The trend is not just this bug, its other ones too. Discuss first, talk to the
person who spent hours writing the patch, don't just mark his/hers obsolete and
replace with your own.

Chris

-- 
Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.


More information about the Koha-bugs mailing list