Summary: | Patron category dropdown empty in header search form on patron entry page | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Patrons | Assignee: | Owen Leonard <oleonard> |
Status: | RESOLVED DUPLICATE | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | normal | ||
Priority: | P5 - low | CC: | chris, gmcharlt, kyle.m.hall, m.de.rooy, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 12982 - Patron category dropdown empty in header search form on patron entry page
[SIGNED-OFF] Bug 12982 - Patron category dropdown empty in header search form on patron entry page |
Description
Owen Leonard
2014-09-23 18:53:34 UTC
Created attachment 31850 [details] [review] Bug 12982 - Patron category dropdown empty in header search form on patron entry page The include file which contains the patron search form assumes that the page in which it is included will have access to a set of patron categories passed from the script. This is often not the case. This patch adds a new template plugin for getting patron categories and adds usage of it to patron-search.inc. The follow templates include patron-search.inc: members\boraccount.tt members\deletemem.tt members\files.tt members\mancredit.tt members\maninvoice.tt members\member-flags.tt members\member-password.tt members\member.tt members\memberentrygen.tt members\members-update.tt members\moremember.tt members\notices.tt members\pay.tt members\paycollect.tt members\purchase-suggestions.tt members\readingrec.tt members\routing-lists.tt members\statistics.tt tools\import_borrowers.tt tools\picture-upload.tt To test, view those pages and confirm that the patron category dropdown is present in the patron header search at the top of the page (shown by clicking the [+] link). This is my first try at modifying a template plugin, so I hope I've done it correctly. Should the addition have a test? Created attachment 32006 [details] [review] [SIGNED-OFF] Bug 12982 - Patron category dropdown empty in header search form on patron entry page The include file which contains the patron search form assumes that the page in which it is included will have access to a set of patron categories passed from the script. This is often not the case. This patch adds a new template plugin for getting patron categories and adds usage of it to patron-search.inc. The follow templates include patron-search.inc: members\boraccount.tt members\deletemem.tt members\files.tt members\mancredit.tt members\maninvoice.tt members\member-flags.tt members\member-password.tt members\member.tt members\memberentrygen.tt members\members-update.tt members\moremember.tt members\notices.tt members\pay.tt members\paycollect.tt members\purchase-suggestions.tt members\readingrec.tt members\routing-lists.tt members\statistics.tt tools\import_borrowers.tt tools\picture-upload.tt To test, view those pages and confirm that the patron category dropdown is present in the patron header search at the top of the page (shown by clicking the [+] link). Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> A test would be great Owen. t/Koha_Template_Plugin_Cache.t is a good starting point QA Comment: Looks good. But I still have a question: Would this be the place to also provide the branches if they are not passed in? For instance, moremember now shows the Categories, but it still does not give me Libraries? I would not mind to pass QA on this patch, but if you want to include that now, please go ahead. Changing status to reflect need for clarification :) (In reply to M. de Rooy from comment #5) > QA Comment: > Looks good. But I still have a question: Would this be the place to also > provide the branches if they are not passed in? Of course that would be a good idea, but since this bug specifically references patron categories I think it's out of its scope. However, I'll leave this as Failed QA since it still lacks a test. This patch fixes an issue, but also implements a way to get the patron categories through TT plugins. I think it has already been addressed by bug 7380. Feel free to report a new bug if the mentioned bug is already present for you (I didn't manage to reproduce it, on top of master/7380). *** This bug has been marked as a duplicate of bug 7380 *** |