http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #9 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- (In reply to comment #8)
(In reply to comment #7)
I noticed a couple of problems with this patch: * The magnifying glass to link to authority records produces an invalid link for authority-controlled headings. I'm not sure to understand. Could you provide such a link as an example?
It doesn't include the authid at all. The URL just ends authid= (or something like that... I can't check right at the moment)
* When you click on a subdivision that appears in more than one subject, the dropdown has that subdivision checked in every single subject in the list, which is confusing. * If you choose all the subdivisions of a particular subject (and only those subdivisions), a user would expect it to use the authid for the search if UseAuthoritiesForTracings is turned on, but it does not, which is confusing and not what I'd expect. Ok I will fix those things.
Thanks.
* Although comment 6 says "Fixed display when xslt is enabled," it seems that the feature simply doesn't show up at all when XSLT is enabled, which is fine, but should be indicated somewhere, at least in the commit message. Even better would be if the feature could be made to work with XSLT as well, of course. An indication would make it clear to a tester whether the fact that it doesn't work with XSLT means that the patch failed QA, or just means that it works as intended. To be clear, this feature is not intended to work when using XSLT for opac-detail.pl. But with XSLT enabled subjects and authors must be listed only once (the problem was subjects were listed twice).
Okay. A note "affects 'regular mode' only" would clarify greatly. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.