Link to TPAC library pages from copy table in results and KPAC for consistency

[Dan Scott]

* Evergreen master

Per Dan Wells' comment in #1261939 and mirroring #1201982 we should be consistent about where we're linking to on the search results copy table.

Thus, please merge the top commit in http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbs/tpac_library_results

[Dan Scott (denials)]

Removing pullrequest tag until I address the KPAC copy table as well. (3 copy tables, really?)

[Dan Scott (denials)]

I've pushed some updates to this branch. Going deeper down the KPAC rabbit hole, I refactored some of the library page bits so we could call them from both TPAC and KPAC contexts without too much code duplication. It's working now, although you lose the KPAC breadcrumb context when you jump to the library page (because we're not maintaining the GET params in the link to the library page). Maybe look at REFERER as a cleaner way of pulling the previous pages' params?

[Dan Scott (denials)]

I also added some reasonable/minimal CSS so the KPAC library info page didn't look ridiculous. At this point, I think it's worth putting the pullrequest tag back in effect. This branch improves significantly over what is currently in master.

[Ben Shum (bshum)]

The KPAC side looked very nice to me Dan. Thanks for your hard work on that piece of it.

Couple issues though for the regular TPAC -- the record and result (with more details) views were broken with apache Internal Server Errors. Looking at my logs I found this:

open-ils.cstore 2014-01-24 23:54:03 [ERR :3345:oils_sql.c:5585:13906237963958117] open-ils.cstore: Error with query [SELECT * FROM actor.org_unit_ancestor_setting( 'lib.info_url', '' ) AS "actor.org_unit_ancestor_setting" ;]: 0 ERROR: invalid input syntax for integer: ""
LINE 1: ... actor.org_unit_ancestor_setting( 'lib.info_url', '' ) AS "a...
                                                             ^

Digging deeper, I found that when we pulled out the code that makes the URLs into its separate file, there was a move to make "copy_info = copy;" but this was not present in the results/copy_table.tt2 file (which broke all more detailed results). Unfortunately, the record one was also broken, because it was backwards, and it really wanted "copy = copy_info;"

I've signed off on the commits in the branch, and added a new commit at the top to address the variables issue.

See branch: user/bshum/tpac_library_results

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/tpac_library_results

[Dan Scott (denials)]

Thank you Ben! Clearly my head got too focused on sorting out KPAC; I'm sure I tried manually testing TPAC after the refactoring, but might have just been getting cached results from hitting "reload" instead of a fresh search. No excuse, really... but thank you!

Pushed to master.