[Bug 5888] New: opac-detail subject/author links improvements
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Bug #: 5888 Summary: opac-detail subject/author links improvements Classification: Unclassified Change sponsored?: --- Product: Koha Version: rel_3_4 Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P5 Component: OPAC AssignedTo: oleonard@myacpl.org ReportedBy: chris@bigballofwax.co.nz QAContact: koha-bugs@lists.koha-community.org Blocks: 5584 When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on this is very convenient when, for example, subjects are : France, history, kingdom, war to do a search on "France + war" easily -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|oleonard@myacpl.org |julian.maurice@biblibre.com --- Comment #1 from Julian Maurice <julian.maurice@biblibre.com> 2011-03-17 15:42:26 UTC --- Created attachment 3345 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=3345 Patch from BibLibre -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent Patch Status|--- |Needs Signoff -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |5871 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> 2011-03-17 17:08:27 UTC --- The patch for Bug 5871 contains the CSS changes required to make this bug work properly. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #3 from Owen Leonard <oleonard@myacpl.org> 2011-03-17 17:17:30 UTC --- Some comments, after testing: 1. This completely breaks subject searching for users with JavaScript disabled. It should gracefully degrade. 2. The behavior of these menus doesn't follow that of other menus in the OPAC like the Lists menu or the "More searches" menu. If we're going to add this now we really should have it use the YUI Menu library so the interactions are consistent. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #4 from Owen Leonard <oleonard@myacpl.org> 2011-03-17 17:23:19 UTC --- If OPACXSLTDetailsDisplay is turned on subjects are listed twice: once in the section output by the XSL and once with the new display. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|julian.maurice@biblibre.com |paul.poulain@biblibre.com -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Needs Signoff |Failed QA -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #5 from MJ Ray (software.coop) <mjr@ttllp.co.uk> 2011-04-09 09:25:18 UTC --- This bug is mentioned in: Bug 5888: opac-detail.pl: adding of search rebound on author and subject http://lists.koha-community.org/pipermail/koha-patches/2011-March/014116.htm... -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |5871 Depends on|5871 | -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|paul.poulain@biblibre.com |julian.maurice@biblibre.com -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #3345|0 |1 is obsolete| | --- Comment #6 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 9738 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9738&action=edit opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on -- Recreated patch against current master - Fixed display when xslt is enabled - Fixed searching when javascript is disabled -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #7 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- 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. * 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. * 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. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #8 from Julian Maurice <julian.maurice@biblibre.com> --- (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?
* 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.
* 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).
-- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
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.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #9738|0 |1 is obsolete| | --- Comment #10 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 9855 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9855&action=edit Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Also added a link to opac-authoritiesdetail.pl when possible. Only affects 'Normal view'. Does not affect XSLT display. --
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) Fixed.
* 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. Fixed.
* 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. Fixed but UseAuthoritiesForTracings is only used with XSLT, so I don't use it in this patch. If there is an authid and all subdivisions are checked, then the authid is used. Otherwise the strings are used.
-- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #9855|0 |1 is obsolete| | --- Comment #11 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Created attachment 9856 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9856&action=edit Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Also added a link to opac-authoritiesdetail.pl when possible. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |8210 -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |8212 -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #12 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- *** Bug 5871 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|PATCH-Sent (DO NOT USE) |P5 - low CC| |m.de.rooy@rijksmuseum.nl Version|rel_3_4 |master --- Comment #13 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Just a *dumb* question: If I click on a subject or author, wouldn't it be logical to start searching for it directly instead of presenting another menu where I have to click Search again? I understand that I now have the option to select multiple subjects, but if I wanted that, wouldn't I have typed them in the search box already? Also note that I would not prefer to let normal view and xslt view deviate in this regard. Confusing for users? -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #14 from Julian Maurice <julian.maurice@biblibre.com> --- What do you suggest? Let the link behave like currently in master and add a new link after subjects and authors for displaying this menu? For sure, you can type what you want in the search box, but I think it's easier to check boxes than to type complete names of authorities in a search box. I do not understand your comment about normal view and xslt view. Do you want to enable this feature for xslt view too? If someone uses his own xslt and want this feature, he has to modify his xslt by himself. Anyway, opac users do not use both normal and xslt views, right? How can this be confusing for them? -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #15 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to comment #14)
What do you suggest? Let the link behave like currently in master and add a new link after subjects and authors for displaying this menu? Well, that could be the best of both! If I click on a name, I directly get what I want. If I want to select multiple names/topics, I get your menu. I would be in favor of that; but I do not want to force a personal opinion. Your are welcome to raise this point on the mailing list (before just changing your code) ...
I do not understand your comment about normal view and xslt view. Do you want to enable this feature for xslt view too? If someone uses his own xslt and want this feature, he has to modify his xslt by himself. You are right about custom xslt, but if the default xslt could also support this feature somehow, that would be nice. (Note that this could be handled in a followup.)
Anyway, opac users do not use both normal and xslt views, right? How can this be confusing for them? Correct. Maybe it is more confusing for developers :-)
-- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|koha-bugs@lists.koha-commun |m.de.rooy@rijksmuseum.nl |ity.org | -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
(In reply to comment #14)
What do you suggest? Let the link behave like currently in master and add a new link after subjects and authors for displaying this menu? Well, that could be the best of both! If I click on a name, I directly get what I want. If I want to select multiple names/topics, I get your menu. I would be in favor of that; but I do not want to force a personal opinion. Your are welcome to raise this point on the mailing list (before just changing your code) ... I'm not a fan of this option. There is already a magnifying glass when your entry is an authority. Adding another image will result in less readability I
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |paul.poulain@biblibre.com --- Comment #16 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #15) think. I think that the option we choose with Nimes (that sponsored the feature) is the best compromise. Note that this enhancement is a tiny one, and if it does not pass QA as the feature is submitted, Nîmes & BibLibre have agreed to abandon it.
I do not understand your comment about normal view and xslt view. Do you want to enable this feature for xslt view too? If someone uses his own xslt and want this feature, he has to modify his xslt by himself. You are right about custom xslt, but if the default xslt could also support this feature somehow, that would be nice. (Note that this could be handled in a followup.) I agree this is independant from this patch = the XSLT could also support this feature without this patch.
QA comment: * why isn't the javascript in a separate file ? That would: - reduce the network load (js is cached) - ease the introduction of this feature at XSLT level -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #17 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Julian, With permission I would like to break this patch into two parts: one that links to the authority record, which I think is fairly uncontroversial, and one that adds the cool new subject search pop-up. Any objections? I would, of course, be sure to retain your authorship. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #18 from Julian Maurice <julian.maurice@biblibre.com> --- Jared, I have no objections at all. You can do what you want with these patches ;-) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #9856|0 |1 is obsolete| | --- Comment #19 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Created attachment 11586 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11586&action=edit Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Split out of the original patch for bug 5888. The links to the authority records are added by bug 8210. Additional signoff required since I split the patch up. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|8210, 8212 | Depends on| |8210 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #20 from Chris Cormack <chris@bigballofwax.co.nz> --- Woooo how cool is that! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #21 from Chris Cormack <chris@bigballofwax.co.nz> --- This works exactly as described, it pops up a list you can search on, and the search works. Their appears to be no noticable performance hit on the page load. Passes xt/tt_valid.t xt/author/valid_templates.t and xt/author/translatable-templates.t While we could put the js in its own file. That is a QA decision, I'm happy to do an initial signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11586|0 |1 is obsolete| | --- Comment #22 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 11592 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11592&action=edit Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Split out of the original patch for bug 5888. The links to the authority records are added by bug 8210. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #23 from Owen Leonard <oleonard@myacpl.org> --- I know this is already signed off, but when I went to take a look myself I couldn't get it to apply: Applying: Bug 5888: opac-detail subject/author links improvements fatal: sha1 information is lacking or useless (koha-tmpl/opac-tmpl/prog/en/css/opac.css). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 Bug 5888: opac-detail subject/author links improvements -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #24 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- (In reply to comment #23)
I know this is already signed off, but when I went to take a look myself I couldn't get it to apply:
Applying: Bug 5888: opac-detail subject/author links improvements fatal: sha1 information is lacking or useless (koha-tmpl/opac-tmpl/prog/en/css/opac.css). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 Bug 5888: opac-detail subject/author links improvements
Did you apply the patch for 8210 first? I broke them up into two. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11592|0 |1 is obsolete| | --- Comment #25 from Mason James <mtj@kohaaloha.com> --- Created attachment 12308 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12308&action=edit Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Split out of the original patch for bug 5888. The links to the authority records are added by bug 8210. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |mtj@kohaaloha.com --- Comment #26 from Mason James <mtj@kohaaloha.com> ---
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
patch looks good, passing QA… $ koha-qa.pl testing 1 commit(s) (applied to commit c74b72b) * e5b58d3 Bug 5888: opac-detail subject/author links improvements koha-tmpl/opac-tmpl/prog/en/css/opac.css koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt * koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt OK -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #27 from Paul Poulain <paul.poulain@biblibre.com> --- QA comment: There is a bug in Author links: if you have, say, 3 authors, and click "All", then the query built is on and an=NNNN and an=MMMM and an=PPPP The initial "on and" is wrong (and comes from the fact that the query is built using the 'All' checked -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12308|0 |1 is obsolete| | --- Comment #28 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 12343 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12343&action=edit Bug 5888: opac-detail subject/author links improvements Rebased on master. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #29 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 12344 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12344&action=edit Bug 5888: Fix building of query in authors links 'all' checkbox was taken into account, resulting in the string 'on and' added to beginning of query. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12343|Bug 5888: opac-detail |[SIGNED-OFF] Bug 5888: description|subject/author links |opac-detail subject/author |improvements |links improvements -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply --- Comment #30 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Sorry, this patch does not apply any more: error: patch failed: koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt:360 error: koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt: patch does not apply Did you hand edit your patch? It does not apply to blobs recorded in its index. Cannot fall back to three-way merge. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12343|0 |1 is obsolete| | --- Comment #31 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 13631 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13631&action=edit [SIGNED-OFF] Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Also added a link to opac-authoritiesdetail.pl when possible. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Split out of the original patch for bug 5888. The links to the authority records are added by bug 8210. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12344|0 |1 is obsolete| | --- Comment #32 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 13632 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13632&action=edit Bug 5888: Fix building of query in authors links 'all' checkbox was taken into account, resulting in the string 'on and' added to beginning of query. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff --- Comment #33 from Julian Maurice <julian.maurice@biblibre.com> --- Patches rebased on master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Melia Meggs <melia@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |melia@bywatersolutions.com Patch complexity|--- |Medium patch --- Comment #34 from Melia Meggs <melia@bywatersolutions.com> --- I can't get this to apply, sorry. I tried first with git bz apply 5888 Bug 5888 - opac-detail subject/author links improvements [SIGNED-OFF] Bug 5888: opac-detail subject/author links improvements Apply? [yn] y Applying: Bug 5888: opac-detail subject/author links improvements error: patch failed: koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt:293 error: koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt: patch does not apply Patch failed at 0001 Bug 5888: opac-detail subject/author links improvements I also tried with git am -iu3 Applying: Bug 5888: opac-detail subject/author links improvements Using index info to reconstruct a base tree... Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/opac-tmpl/prog/en/css/opac.css Auto-merging koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt CONFLICT (content): Merge conflict in koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt Failed to merge in the changes. Patch failed at 0001 Bug 5888: opac-detail subject/author links improvements -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply --- Comment #35 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #34)
I can't get this to apply, sorry. I tried first with git bz apply 5888 Auto-merging koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt CONFLICT (content): Merge conflict in koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt Failed to merge in the changes. Patch failed at 0001 Bug 5888: opac-detail subject/author links improvements
Melia, in this case, you can set the status to "Patch doesn't apply" -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #36 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 13973 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13973&action=edit [SIGNED-OFF] Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Also added a link to opac-authoritiesdetail.pl when possible. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Split out of the original patch for bug 5888. The links to the authority records are added by bug 8210. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13632|0 |1 is obsolete| | --- Comment #37 from Julian Maurice <julian.maurice@biblibre.com> --- Created attachment 13974 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13974&action=edit Bug 5888: Fix building of query in authors links 'all' checkbox was taken into account, resulting in the string 'on and' added to beginning of query. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13631|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff --- Comment #38 from Julian Maurice <julian.maurice@biblibre.com> --- Patches rebased on master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13973|0 |1 is obsolete| | --- Comment #39 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 14140 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14140&action=edit [SIGNED-OFF] Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Also added a link to opac-authoritiesdetail.pl when possible. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Split out of the original patch for bug 5888. The links to the authority records are added by bug 8210. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13974|0 |1 is obsolete| | --- Comment #40 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 14141 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14141&action=edit [SIGNED-OFF] Bug 5888: Fix building of query in authors links 'all' checkbox was taken into account, resulting in the string 'on and' added to beginning of query. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |kyle@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #41 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- QA: Looking at this one again.. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |In Discussion --- Comment #42 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- QA Comment on 5888: Still have my doubts on this feature. The usability is IMO the biggest issue. Please note my earlier comment 13. If I just want to search on the author I click on, I do not want to click again on Search. If I have some more authors and subject, the page even needs to be scrolled down before reaching the buttons. This would mean 1 click, 1 scroll and another click instead of the previous 1 click. If I want to search on two subjects, I will type them just as fast in the search box as selecting them in the menu? All together, I would prefer this option to be controlled by a syspref (default disabled), or offer the new menu in a different way. You could e.g. add a last option to the subject line like "Combined subject search" (only if #subjects>1) and present the menu then. Comment 16 suggests that this is less readable. But note that is only one extra link at the end. No large effect on readability, but usability is kept or even improved. Another minor point: If I want to remove the menu again, it should be possible without the need to scroll and click. In the case of more authors and subjects, the Cancel button cannot be reached rightaway. Could you add a Close or X button at the top to remove the menu? Another minor point: It is working on normal view only, but not on xslt. Since all comments are on a functional level, I will set this patch to In Discussion and first send a mail to the QA team. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #43 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I think a syspref would make sense to me - libraries are sensitive about such things, so giving them a choice is always good. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m --- Comment #44 from Jonathan Druart <jonathan.druart@biblibre.com> --- Since some people find this feature cool and others not, I follow Katrin : a syspref could be an easy way to gain consensus. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #45 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 15230 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15230&action=edit Bug 5888: Followup add a syspref (OPACPopupAuthorsSearch) New syspref OPACPopupAuthorsSearch. If it is disabled, the development has no effect. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #46 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 15231 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15231&action=edit Bug 5888: Add label for each entry in the authors list -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Needs Signoff --- Comment #47 from Jonathan Druart <jonathan.druart@biblibre.com> --- Feel free to rename the syspref name/description if needed. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #14140|0 |1 is obsolete| | Attachment #14141|0 |1 is obsolete| | Attachment #15230|0 |1 is obsolete| | Attachment #15231|0 |1 is obsolete| | --- Comment #48 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15341 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15341&action=edit Bug 5888: Followup for preference description Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Followup added for trivial string change in finalizing QA. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15341|0 |1 is obsolete| | --- Comment #49 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15342 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15342&action=edit Bug 5888: opac-detail subject/author links improvements When the user clicks on a subject or an author, the list of subjects (authors) is displayed and the user can choose more than 1 subject (author) to search on. Also added a link to opac-authoritiesdetail.pl when possible. Only affects 'Normal view'. Does not affect XSLT display. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> Split out of the original patch for bug 5888. The links to the authority records are added by bug 8210. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #50 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15343 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15343&action=edit Bug 5888: Fix building of query in authors links 'all' checkbox was taken into account, resulting in the string 'on and' added to beginning of query. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #51 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15344 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15344&action=edit Bug 5888: Followup add a syspref (OPACPopupAuthorsSearch) New syspref OPACPopupAuthorsSearch. If it is disabled, the development has no effect. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #52 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15345 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15345&action=edit Bug 5888: Add label for each entry in the authors list Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #53 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15346 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15346&action=edit Bug 5888: Followup for preference description Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Followup added for trivial string change in finalizing QA. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15345|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15346|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #54 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15347 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15347&action=edit Bug 5888: Add label for each entry in the authors list Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #55 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 15348 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15348&action=edit Bug 5888: Followup for preference description Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Added for trivial string change in finalizing QA. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #56 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- QA Comment: Little struggle with git bz.. With the pref, this looks good to me. Adjusted pref description by adding "subjects" and "opac detail". Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #57 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Additional thought: Could a followup "easily" add this to opac xslt too, or should we further adjust the pref description when only limited to no-xslt view? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #58 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |9711 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5888 --- Comment #59 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- *** Bug 5440 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org