[Bug 17312] New: Typo in members-toolbar.inc / moremember-brief.tt / moremember.tt
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Bug ID: 17312 Summary: Typo in members-toolbar.inc / moremember-brief.tt / moremember.tt Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: arouss1980@gmail.com QA Contact: testopia@bugs.koha-community.org The following three templates are using [% guarantorborrowernumber %] while they should be using [% guarantor.borrowernumber %]: members/members-toolbar.inc members/moremember-brief.tt members/moremember.tt This doesn't result in any breakage; just a couple of 'Edit' links that do not pass the guarantorid in the URL, and one case where guarantor information is not shown in the staff client. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Andreas Roussos <arouss1980@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|oleonard@myacpl.org |arouss1980@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 --- Comment #1 from Andreas Roussos <arouss1980@gmail.com> --- Created attachment 55631 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55631&action=edit Bug 17312 - Typo in members-toolbar.inc / moremember-brief.tt / moremember.tt The following three templates are using [% guarantorborrowernumber %] while they should be using [% guarantor.borrowernumber %]: members/members-toolbar.inc members/moremember-brief.tt members/moremember.tt This doesn't result in any breakage; just a couple of 'Edit' links that do not pass the guarantorid in the URL, and one case where guarantor information is not shown in the staff client. This patch fixes that. Test plan: 0) [PREREQUISITE] Create a patron with a guarantor if you don't have one. 1) Go to Home > Patrons and search for a patron that has a guarantor. In the Details page for that patron, the 'Edit' link in the toolbar does not pass the guarantor's id in the URL (...&guarantorid=&...). 2) In the same page, the 'Edit' link under the patrons name (immediately under 'Guarantor') again does not include the guarantor id in the URL. 3) Go to Home > Patrons and click on 'New patron'. Pick any category from the drop down menu. Enter the Surname, First name, and Date of birth of the patron you used in step 1). This triggers the 'Duplicate patron record?' warning -- click on 'View existing record' and notice how the guarantor information is missing. 4) Apply the patch. 5) Repeat steps 1), 2), and 3) above. The URLs are fixed and patron info is showing. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Andreas Roussos <arouss1980@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #55631|0 |1 is obsolete| | --- Comment #2 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 55789 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55789&action=edit Bug 17312 - Typo in members-toolbar.inc / moremember-brief.tt / moremember.tt The following three templates are using [% guarantorborrowernumber %] while they should be using [% guarantor.borrowernumber %]: members/members-toolbar.inc members/moremember-brief.tt members/moremember.tt This doesn't result in any breakage; just a couple of 'Edit' links that do not pass the guarantorid in the URL, and one case where guarantor information is not shown in the staff client. This patch fixes that. Test plan: 0) [PREREQUISITE] Create a patron with a guarantor if you don't have one. 1) Go to Home > Patrons and search for a patron that has a guarantor. In the Details page for that patron, the 'Edit' link in the toolbar does not pass the guarantor's id in the URL (...&guarantorid=&...). 2) In the same page, the 'Edit' link under the patrons name (immediately under 'Guarantor') again does not include the guarantor id in the URL. 3) Go to Home > Patrons and click on 'New patron'. Pick any category from the drop down menu. Enter the Surname, First name, and Date of birth of the patron you used in step 1). This triggers the 'Duplicate patron record?' warning -- click on 'View existing record' and notice how the guarantor information is missing. 4) Apply the patch. 5) Repeat steps 1), 2), and 3) above. The URLs are fixed and patron info is showing. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #55789|0 |1 is obsolete| | --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 55792 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55792&action=edit [PASSED QA] Bug 17312 - Typo in members-toolbar.inc / moremember-brief.tt / moremember.tt The following three templates are using [% guarantorborrowernumber %] while they should be using [% guarantor.borrowernumber %]: members/members-toolbar.inc members/moremember-brief.tt members/moremember.tt This doesn't result in any breakage; just a couple of 'Edit' links that do not pass the guarantorid in the URL, and one case where guarantor information is not shown in the staff client. This patch fixes that. Test plan: 0) [PREREQUISITE] Create a patron with a guarantor if you don't have one. 1) Go to Home > Patrons and search for a patron that has a guarantor. In the Details page for that patron, the 'Edit' link in the toolbar does not pass the guarantor's id in the URL (...&guarantorid=&...). 2) In the same page, the 'Edit' link under the patrons name (immediately under 'Guarantor') again does not include the guarantor id in the URL. 3) Go to Home > Patrons and click on 'New patron'. Pick any category from the drop down menu. Enter the Surname, First name, and Date of birth of the patron you used in step 1). This triggers the 'Duplicate patron record?' warning -- click on 'View existing record' and notice how the guarantor information is missing. 4) Apply the patch. 5) Repeat steps 1), 2), and 3) above. The URLs are fixed and patron info is showing. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |kyle@bywatersolutions.com --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 16.11, thanks Andreas! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17312 Frédéric Demians <frederic@tamil.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |frederic@tamil.fr --- Comment #5 from Frédéric Demians <frederic@tamil.fr> --- Pushed in 16.05. Will be in 16.05.05. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org