http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8782
David Cook <dc...@prosentient.com.au> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |dc...@prosentient.com.au
--- Comment #4 from David Cook <dc...@prosentient.com.au> ---
I'm glad to see this enhancement! This issue came up with a client back when I
first started using Koha, so it's nice to see a fix for what was one of my
first issues!
The patch seems rather large though, so I'm not completely sure how to even
start testing to see if it breaks anything. Knowing Jonathan's work, I'm sure
that the patch does what it describes (yay), but what's the best way of testing
if it breaks something else? Unit tests? Are the following tests enough?
prove t/
prove t/db_dependent
prove xt/
prove xt/author
Thoughts?
I'd like to start testing and signing off on larger patches, but I want to make
sure that I'm doing a thorough enough job.
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/
