[Bug 40699] New: Preferred name not displayed for instructors in course reserves
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Bug ID: 40699 Summary: Preferred name not displayed for instructors in course reserves Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Course reserves Assignee: koha-bugs@lists.koha-community.org Reporter: michael.adamyk@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org In the course reserves module, if a patron account used for an instructor on a course has a preferred name different from the patron first name, the course will still display firstname instead of the preferred name. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Michael Adamyk <michael.adamyk@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |michael.adamyk@bywatersolut | |ions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com --- Comment #1 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- https://github.com/Koha-Community/Koha/blob/5832521086b06c2221474791278ebd9b... We should use patron-title.inc here instead of grabbing firstname and surname directly. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 --- Comment #2 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 185745 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185745&action=edit Bug 40699: Use patron-title for course reserve instructor To test: 1 - have or make a course reserves course with an instructor 2 - give your instructor a preferred name, confirm the course shows their firstname instead 3 - apply patch, reload page 4 - confirm course now shows their preferred_name instead 5 - edit the patron to remove the preferred name, confirm course shows first name again -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |andrew@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Raina <raina@flo.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |raina@flo.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Noah <noah.tremblay@inlibro.com> 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=40699 Noah <noah.tremblay@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185745|0 |1 is obsolete| | --- Comment #3 from Noah <noah.tremblay@inlibro.com> --- Created attachment 186233 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186233&action=edit Bug 40699: Use patron-title for course reserve instructor To test: 1 - have or make a course reserves course with an instructor 2 - give your instructor a preferred name, confirm the course shows their firstname instead 3 - apply patch, reload page 4 - confirm course now shows their preferred_name instead 5 - edit the patron to remove the preferred name, confirm course shows first name again Signed-off-by: noah <noah@inlibro.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 --- Comment #4 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 187472 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=187472&action=edit Bug 40699: Correctly format patron everywhere in course reserves -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 --- Comment #5 from Jonathan Druart <jonathan.druart@gmail.com> --- Andrew, I found other places: * Create/edit a course * Course list Can you test and confirm it is needed as well please? :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff CC| |jonathan.druart@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 --- Comment #6 from Jonathan Druart <jonathan.druart@gmail.com> --- Note that I found several other places where they are not displayed correctly. I have fixed them on bug 32176. Please have a look :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |32176 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32176 [Bug 32176] Correctly display patrons when selected after autocomplete (was js/patron-autocomplete.js need another option) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #7 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #5)
Andrew, I found other places: * Create/edit a course * Course list
Can you test and confirm it is needed as well please? :)
Those places both look good! They reminded me to go look at the OPAC, where we show firstname rather then preferred_name in both the course list and the course details. I'll aim to get back there and add those, but if you beat me to it and assign the bug to yourself, I'll be happy to sign off on it. Thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 --- Comment #8 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Andrew Fuerste-Henry from comment #7)
(In reply to Jonathan Druart from comment #5)
Andrew, I found other places: * Create/edit a course * Course list
Can you test and confirm it is needed as well please? :)
Those places both look good! They reminded me to go look at the OPAC, where we show firstname rather then preferred_name in both the course list and the course details. I'll aim to get back there and add those, but if you beat me to it and assign the bug to yourself, I'll be happy to sign off on it. Thanks!
I would suggest to do that on a separate bug report, because of commit 4dba088549d1add43cfd575ebdd3d21689c4a700 Bug 15699 - Course reserves instructors should be in form "Surname, Firstname" for sorting purposes and this code in ./opac-course-reserves.tt 63 ><span class="inst_surname">[% i.surname | html %]</span>[% IF i.firstname %]<span class="instr_separator">, </span><span class="instr_firstname">[% i.firstname | html %]</span>[% END %]</div -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #9 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #8)
I would suggest to do that on a separate bug report
Fair enough! setting this back to NSO -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Preferred name not |Preferred name not |displayed for instructors |displayed for instructors |in course reserves |in course reserves in staff | |interface -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|32176 |40958 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32176 [Bug 32176] Correctly display patrons when selected after autocomplete (was js/patron-autocomplete.js need another option) https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40958 [Bug 40958] Move patron_to_html (from js-patron-format.inc) to a standalone JS file -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #10 from Jonathan Druart <jonathan.druart@gmail.com> --- Considering this one signed off. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Nick Clemens (kidclamp) <nick@bywatersolutions.com> 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=40699 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #186233|0 |1 is obsolete| | Attachment #187472|0 |1 is obsolete| | --- Comment #11 from Nick Clemens (kidclamp) <nick@bywatersolutions.com> --- Created attachment 188798 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188798&action=edit Bug 40699: Use patron-title for course reserve instructor To test: 1 - have or make a course reserves course with an instructor 2 - give your instructor a preferred name, confirm the course shows their firstname instead 3 - apply patch, reload page 4 - confirm course now shows their preferred_name instead 5 - edit the patron to remove the preferred name, confirm course shows first name again Signed-off-by: noah <noah@inlibro.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 --- Comment #12 from Nick Clemens (kidclamp) <nick@bywatersolutions.com> --- Created attachment 188799 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188799&action=edit Bug 40699: Correctly format patron everywhere in course reserves Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=41155 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |25.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 --- Comment #13 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Nice work everyone! Pushed to main for 25.11 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40699 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to main |Needs documenting CC| |Laura.escamilla@bywatersolu | |tions.com --- Comment #14 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- This is an enhancement and will not be backported to the 25.05.x branch. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org