Bug 1877 - Link to patron record from manage suggestions
Summary: Link to patron record from manage suggestions
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: Macintosh All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Owen Leonard
QA Contact:
URL: koha/suggestion/acceptorreject.pl?op=...
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-25 03:00 UTC by Nicole C. Engard
Modified: 2012-10-25 22:37 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
sample screen (27.60 KB, image/png)
2008-02-25 15:00 UTC, Chris Cormack
Details
Proposed fix (2.53 KB, patch)
2009-04-21 20:21 UTC, Chris Cormack
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Cormack 2010-05-21 00:35:39 UTC


---- Reported by nengard@gmail.com 2008-02-25 15:00:10 ----

It would be nice if on the list of purchase suggestions the patron's name was linked to his/her record.



---- Additional Comments From nengard@gmail.com 2008-02-25 15:00:38 ----

Created an attachment
sample screen





---- Additional Comments From nengard@gmail.com 2008-02-25 15:31:08 ----

I didn't think about the fact that some acq librarians may not have rights to see patron data - so adding a link may not be the best idea.  Think about it and maybe ask the community what's best.



---- Additional Comments From nengard@gmail.com 2009-04-19 14:53:45 ----

I have looked at the template for this.  I just need confirmation that I'm understanding this right.  Are we not pulling the patron information?  I'm not sure how to get the borrower number to put in the link.



---- Additional Comments From oleonard@myacpl.org 2009-04-21 20:21:52 ----

Created an attachment
Proposed fix

Borrowernumber was getting pulled by Suggestions.pm as "borrnumsuggestedby," which I added to the template. Suggestions.pm appeared to be pulling branchcode as well, but it wasn't coming through to the template. I thought it might be a variable name problem, so I renamed it "branchcodesuggestedby" and that worked.



---- Additional Comments From gmcharlt@gmail.com 2009-04-21 23:50:16 ----

Patch pushed -- please test and close.



---- Additional Comments From nengard@gmail.com 2009-04-22 10:56:32 ----

Awesome!!



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 00:35 UTC  ---

This bug was previously known as _bug_ 1877 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=1877
Imported an attachment (id=392)
Imported an attachment (id=393)

Actual time not defined. Setting to 0.0
The original submitter of attachment 392 [details] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
The original submitter of attachment 393 [details] [review] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.