Commit Graph

12718 Commits

Author SHA1 Message Date
Chris Nighswonger aeba77bca6 Beginning of Release Notes for 3.02.01.000 2010-11-17 17:09:59 -05:00
Marcel de Rooy e1523e21d5 Bug 5066 (Log timestamp consistently in _session_log)
Small fix with thanks to Galen.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 1bd67e28e5)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 17:01:07 -05:00
Owen Leonard f4edd2a949 Bug 2122 follow-up for authorities editor
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit c55f8ffca7)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 08:05:24 -05:00
Owen Leonard 4a34c5289a Fix for Bug 2122, Grayed out Fields not always visible
- Adding "readonly" class to readonly fields to guarantee
  that readonly state is indicated visually
- Adding "locked" icon to the background of readonly
  fields for additional visual feedback

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 240ce5a31c)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 08:05:24 -05:00
Owen Leonard b14213f703 Fix for Bug 5152 - confirm buttons different styles on lists
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 61c7ba4104)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 08:05:24 -05:00
Owen Leonard 8d5394a214 Fix for Bug 5037 - If patron category is empty it shouldn't show
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit bbb0819f0f)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 08:05:24 -05:00
Owen Leonard 6aee235e3c Fix for Bug 3013, Value builder for 006 and 008 need choices for all format types
This partial fix adds a complete list of values for 006/00, "Form of material,"
following the list found here: http://www.loc.gov/marc/bibliographic/bd006.html

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit b050029c65)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 08:05:23 -05:00
Colin Campbell f50c566b88 Bug 4074 Fix fail to parse simple qualified search
Searches such as au,phr and kw,wrdl were passing through the regexes
that should replace colon with equals
add wrdl and phr
use trn so not just rtrn is spotted
Where we were checking for multiple spaces specify that in the
regex not just two spaces

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit b97c4dba7a)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 01:01:21 -05:00
Chris Cormack 250ecc9bc8 Brooke caught my april fools trick, its not 20004 yet!
Ok it wasn't an april fools trick, but just a typo, april fools trick
sounds better though right?

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 0fa9ade9ff)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-17 01:01:20 -05:00
Galen Charlton 0de5e43fcd bug 4866 follow-up - minor code formatting cleanup
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 20be558069)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-15 08:27:41 -05:00
Marcel de Rooy fef4e30e26 Bug 4866 for master (Enable Change event for item plugins)
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit baee3e5c9b)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-15 08:27:40 -05:00
Owen Leonard bb3813956e Fix for Bug 4913 - Budget pages should show currency name instead of symbol
This patch adds a new include file to handle the display of the active
currency name and a warning if no currency is active.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 4b69538f9b)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-15 08:27:40 -05:00
Owen Leonard 5a862001b5 Fix for Bug 4912 - After editing private list redirect to private lists
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 38360b766c)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-15 08:27:40 -05:00
Owen Leonard 50c71c4801 Fix for Bug 5175, XSL view doesn't link publisher name to search
Fixes the doubled up display of the publisher.

This patch fixes the bug in both the OPAC and staff client

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit c0272a6b66)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-15 08:27:40 -05:00
Owen Leonard 7428c3f833 Fix for Bug 5210, back button on batch modification result page useless
I can't see any good use for a back button here, so I'm taking it out.

An improvement might be to build in a method to resubmit the same batch
of barcodes/item numbers again so that you could make additional
changes to the same set.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit fa43573b58)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-15 08:27:39 -05:00
Chris Nighswonger da2ea4a4d9 Bug 5381 - Fines in notices prints always 0.00
This was due to passing off the biblionumber to GetFines rather than the itemnumber.

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit c1463e7255)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-14 08:16:36 -05:00
Owen Leonard 26c368fc8f Additional fix for Bug 4423 - Staff Client XSLT is just a copy of the OPAC one
- Adding "Lists that include this title" links
 - Also correcting minor XHTML error in the same section of opac-detail.tmpl

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 0a26982378)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-14 08:16:18 -05:00
Galen Charlton 8dc027db12 bug 5400: new test case to find files with merge conflict markers
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 4cef5ec961)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 18:10:51 -05:00
Owen Leonard 4fed97397a Fix for Bug 4498 - Acq always shows '1 suggestions waiting'
- Removing 'suggestions waiting' link from left-hand menu. A plain
  'suggestions' link is still shown if suggestions are enabled.
- Putting back the 'pending suggestions' box that used to appear
  on the Acquisitions home page. This will show, with a count
  of pending suggestions, only if suggestions are turned on and
  there are pending suggestions.

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit d9de6ebab3)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 18:10:36 -05:00
Chris Nighswonger 8f2e2f07ac Updating Version Number to 3.02.00.007
Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 13:17:18 -05:00
Owen Leonard 8e0b7a53d2 Bug 5396: New system preference UseTablesortForCirc
This system preference allows the librarian to toggle on or off whether the JQuery tablesort is used on the
circ/circulation.pl page.  For patrons with many checkouts, this sort may cause very slow page loading (and
therefore slow circulation transaction time), depending on the browser and local machine used.  Staff can
still see a sortable list of current checkouts on the members/moremember.pl page.

My revision prevents the template from including *any* tablesorter code if the
preference is feature is turned off: both the circulation list and
the holds list will have no jQuery tablesorter.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit d9db65a17e83f64c5858c6fd7ad8dc0d1e7660ab)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 12:51:11 -05:00
Galen Charlton 7e49b79b73 bug 4359: make other pages in the OPAC respect noItemTypeImages
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 7a945dbc28)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 08:58:48 -05:00
Galen Charlton fcaee0d1f4 bug 4359 follow-up: noItemTypeImages should just control image display
Tweaked so that the item type column is always displayed in the
checkout history in the OPAC; noItemTypeImages now just controls
whether the icon displays.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 3357d811ee)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 08:58:29 -05:00
Owen Leonard 41e99198db Fix for Bug 4359, noItemTypeImages not active in OPAC
- Auth.pm put the noItemTypeImages preference under "intranet only."
  Moved it to "OPAC or intranet."
- Added check for the preference on the OPAC advanced search page.
- Improved logic for displaying images on the reading history page.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit d4f02c469b)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 08:58:11 -05:00
Owen Leonard 8257c0eb63 Fix for Bug 3271, Missing message when adding to a list with no items selected
There is no visible Lists button to "attach" our warning to
as there is in the OPAC, so warning message is displayed via
a javascript alert.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit fcb3668512)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 08:56:36 -05:00
Owen Leonard 1fa6ac5479 Partial fix for Bug 4924, Public/Internal notes missing in staff normal view
This fix adds only public notes. A div is added as a hook for styling

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit be54744836)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 08:56:18 -05:00
Owen Leonard 956ed669b7 Fix for Bug 4933, Link to subfield's edit tab from subfield structure admin
- Also corrects framework display in the case of the default

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 1586306866)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-13 08:56:02 -05:00
Colin Campbell 4d3888ce2c Bug 5392 Initialize vars used in building url strings
If no failures are to be added to the to the strings for redirect
the corresponding variables are undefined causing warnings
to be logged. Initialize them so that the error log is not
being filled with trivia.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit bd93bfdbc4)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-12 21:19:42 -05:00
Katrin Fischer 300bd7baba BUG 5058: small layout correction of vendor form
Rebased to HEAD, should also apply to 3.2

Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit d39dd6d857)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-12 21:19:32 -05:00
Katrin Fischer c7fa47668a Bug 4979: Acq: input fields for new record are too short
Patch rebased to HEAD, should also apply to 3.2

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit ed0ccd9f57)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-12 21:19:23 -05:00
Galen Charlton af37b7d4e6 bug 5393 [SIGNED-OFF]: add test case to check well-formedness of XML/XSLT files
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 35d2008ef0)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-12 21:14:35 -05:00
Colin Campbell 8912a613a5 Bug 4305: Add test case for ISBN-13 to ISBN-10 conversion
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 51eb4fa12e)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-12 13:25:07 -05:00
Galen Charlton dab153a622 bug 5389: mark Business::ISBN as a required dependency
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 11d54188f8)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 20:46:08 -05:00
Owen Leonard 55cb2cf1c6 Fix for Bug 5075 - Terms not highlighted w/ xslt on
- XSLT didn't contain the class markup which acts as a hook
  for the JavaScript highlighting system.
- Standard results page also contained a typo which
  prevented data other than title and author from
  being highlighted

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit b558c0db5a)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 20:01:34 -05:00
Owen Leonard 6080451a35 Fix for Bug 5254 - no need to scroll left to right on acq z search
- Adding inline preview/import menu to match cataloging z39.50 search
- Improving behavior of inline menu on both pages
- Adding white space to ISBN/ISSN output on both pages to allow
  line breaks, improving table width and horizontal scrolling.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit f987d35a22)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 20:00:55 -05:00
Nicole Engard 199e82d7d8 bug 5235 : circ receipt should include name
Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit ace0ce3f8d)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 20:00:08 -05:00
Colin Campbell 519b2c03ef Bug 5315: Remove references to obsolete variable nbstatements
Trying to see the wood from the trees

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit 9e5366734b)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:59:52 -05:00
Galen Charlton 79e0f57153 bug 5309: tweak so that background progress doesn't exit immediately
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit 48ab366b57)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:53:22 -05:00
Galen Charlton 14e76be09b fix use of outdated boilerplate
* New code *must* have an accurate copyright statement;
  it isn't 2000-2002 any more.
* Enable warnings in new code, don't do

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit 0187e38ad7)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:53:21 -05:00
Srdjan Jankovic 204ba1c125 bug 5309: Adjust progres bar
Also added some test facilities
wr73785

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit e2e1dfd86e)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:53:21 -05:00
Srdjan Jankovic ab774516fb bug5027: skip unticked checkboxes when submitting in the background.
Signed-off-by: Robin Sheat <robin@catalyst.net.nz>
Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit 3a4f36b002)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:53:21 -05:00
Chris Cormack fc78b73764 Bug 5385 - Fixing an error that crept in with the POD cleanup
(cherry picked from commit 9c6db56a3f)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:48:15 -05:00
Galen Charlton 57b3924b9a bug 4305: add a couple test cases for _isbn_cleanup
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit bcedff69bc)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:48:15 -05:00
Colin Campbell c82169bc32 Bug 4305 Improve code flow
Use the ISBN Object's method to format the string
rather than add extra code to do it
Clean up the code flow for clarity
Benchmarking shows its a bit faster too
Don't add to the number of Test::Perl::Critic warnings
 for no benefit

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit c95b5153e0)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:48:15 -05:00
Andrew Elwell 7b7ee3303d Bug 5385: POD Cleanups (part 2)
More podchecker cleanups to eliminate warnings / errors

Signed-off-by: Andrew Elwell <Andrew.Elwell@gmail.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit efa66f1f55)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:48:14 -05:00
Andrew Elwell 78a598f509 Bug 5385: POD Cleanups (part 1)
working through the master branch to eliminate all
podchecker warnings/errors

Actual improvement to the quality of the POD will
come later (hopefully with assistance of others)

Signed-off-by: Andrew Elwell <Andrew.Elwell@gmail.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 9fa574f609)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:48:14 -05:00
Katrin Fischer e9877ebb4c Bug 5112: Organisation does not show links to professionals
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 63fdd8768a)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 19:48:13 -05:00
Chris Cormack 77c39f6cfc Merge remote branch 'kc/new/bug_5386' into kcmaster
(cherry picked from commit af1d829031)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 12:52:41 -05:00
Chris Cormack 3b4cf45f9b Test modules compile
Script to test modules compile, when used with a pre-commit hook this
can test before a commit

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 7ecf2c2dc4)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 12:52:36 -05:00
Chris Cormack 0fffa19267 Merge remote branch 'kc/new/bug_5105' into kcmaster
(cherry picked from commit 235cf872a5)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-11 08:53:14 -05:00