[Koha-bugs] [Bug 5888] opac-detail subject/author links improvements

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #9 from Jared Camins-Esakov <jcamins at 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.