Welcome aboard the Koha devs. I'm afraid we don't accept github pull requests at this time. It's come up a few times, but to keep our workflows clear for the largest group of people we've stuck to Bugzilla and mailbox patches so far (They're easy to apply to all our other infrastructure for testing etc).
Onto the patch itself.
I've come across this exact case a month or so now.. in fact, I have a vague recollection of pointing it out to someone on IRC recently.. perhaps that was you.
Your code looks mostly reasonable to me, but I'm also not close enough to how the linking all fits together to know what side effect it might have.. :(.
If you're happy to give git-bz a try as Nicolas suggested, that would give us a head start in terms of onboarding fellow testers and devs.. if not let me know and I can extract your patch from github for you and submit it to BZ.
Martin Renvoize, MPhys (Hons) |
|
Head of Development and Community Engagement | |
| |
Phone: +44 (0) 1483 378728 | Mobile: +44 (0) 7725 985 636 |
| |
| Sign up for our newsletters here or by scanning the QR code | |
|
Registered in the United Kingdom No. 06416372 VAT Reg No. 925 7211 30 The information contained in this email message may be privileged, confidential and protected from disclosure. If you are not the intended recipient, any dissemination, distribution or copying is strictly prohibited. If you think that you have received this email message in error, please email the sender at info@ptfs-europe.com |