Currently, when displaying a list of ILL requests, or viewing a single request, the requester's userid is not displayed. This should be visible.
Created attachment 75817 [details] [review] Bug 20883: Add patron userid to display This patch adds the display of patron userid to both the "illlist" and "illview" pages. Test plan (see bug 7317 for ILL deployment details): 1) Ensure you have at least one ILL request 2) Go to the main "List requests" page by clicking "ILL requests" from the main staff interface TEST => Observe that you see a "Patron username" column the the table, containing the patron's correct userid 3) Click "Manage request" on a request TEST => Observe that, in the "Patron" row, you see the patron's correct userid in brackets (along with their name and barcode)
Created attachment 75821 [details] [review] Bug 20883: (follow-up) Fix column indexes
Created attachment 80609 [details] [review] Bug 20883: Add patron userid to display This patch adds the display of patron userid to both the "illlist" and "illview" pages. Test plan (see bug 7317 for ILL deployment details): 1) Ensure you have at least one ILL request 2) Go to the main "List requests" page by clicking "ILL requests" from the main staff interface TEST => Observe that you see a "Patron username" column the the table, containing the patron's correct userid 3) Click "Manage request" on a request TEST => Observe that, in the "Patron" row, you see the patron's correct userid in brackets (along with their name and barcode)
Created attachment 80610 [details] [review] Bug 20883: Add patron userid to display This patch adds the display of patron userid to both the "illlist" and "illview" pages. Test plan (see bug 7317 for ILL deployment details): 1) Ensure you have at least one ILL request 2) Go to the main "List requests" page by clicking "ILL requests" from the main staff interface TEST => Observe that you see a "Patron username" column the the table, containing the patron's correct userid 3) Click "Manage request" on a request TEST => Observe that, in the "Patron" row, you see the patron's correct userid in brackets (along with their name and barcode)
Created attachment 80611 [details] [review] Bug 20883: Add patron userid to display This patch adds the display of patron userid to both the "illlist" and "illview" pages. Test plan (see bug 7317 for ILL deployment details): 1) Ensure you have at least one ILL request 2) Go to the main "List requests" page by clicking "ILL requests" from the main staff interface TEST => Observe that you see a "Patron username" column the the table, containing the patron's correct userid 3) Click "Manage request" on a request TEST => Observe that, in the "Patron" row, you see the patron's correct userid in brackets (along with their name and barcode)
Hopefully this is recovered from a non-existent "Patch Does Not Apply" state, because I didn't want to leave it there. :)
Created attachment 80614 [details] [review] Bug 20883: Add patron userid to display This patch adds the display of patron userid to both the "illlist" and "illview" pages. Test plan (see bug 7317 for ILL deployment details): 1) Ensure you have at least one ILL request 2) Go to the main "List requests" page by clicking "ILL requests" from the main staff interface TEST => Observe that you see a "Patron username" column the the table, containing the patron's correct userid 3) Click "Manage request" on a request TEST => Observe that, in the "Patron" row, you see the patron's correct userid in brackets (along with their name and barcode) Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
(In reply to Andrew Isherwood from comment #0) > Currently, when displaying a list of ILL requests, or viewing a single > request, the requester's userid is not displayed. This should be visible. Why? :) We usually do not display the username anywhere AFAIK. Here you are adding a new column, maybe it could fit into the same column as the firstname and surname? On the detail view we have: Patron: firstname surname [42] [username] Not sure the [] are good for visibility and tell what we are displaying. I am just asking :)
It's a good question and I'm not 100% sure of the answer. We definitely had a request from a customer at some point to add the username as it would enable them then to filter by it. We do have another bug 21063 which enables the user to select the columns that are displayed, though username is not currently one of the columns that can be selected. So, maybe, rather than this bug which hardcodes this into the UI, we need an another enhancement bug, which is dependent on 20163, which adds username into the list of columns available for selection.
That gets a +1 from me.. perhaps mark this one as a duplicate and add the configurable column to that other bug :)
I was not aware of bug 21063. This should definitely be built on top of it. Re-assigning to you Andrew as more work is needed.
*** This bug has been marked as a duplicate of bug 21063 ***