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

Galen Charlton <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #12 from Galen Charlton <[email protected]> ---
(In reply to Jonathan Druart from comment #11)
> (In reply to Kyle M Hall from comment #10)
> > Excellent idea! I wonder if there is any way to add a check for out of order
> > sysprefs in koha-qa.pl, after this patch has been pushed.
> 
> Yep, it should certainly be possible to do something.

I like the idea of this patch, and will push it shortly unless I find something
wrong, but I'm opposed  to putting a sorting requirement into koha-qa.pl. 
Sorting the script provides a small convenience by reducing the chance of merge
conflicts, but doesn't per se provide a functional benefit.  I'd really hate to
see a patch fail QA on account of a new syspref getting inserted in the "wrong"
place.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to