Commit graph

655 commits

Author SHA1 Message Date
Garry Collum
3a058fcc9d Bug 1607: Fixes the delete checkboxes in purchase suggestions.
If a user views suggestions of another user, syspref (OPACViewOthersSuggestions), a selection check box for deletion appeared next to all of the suggested titles whether the logged-in user owned them or not. - fixed

Also fixes the "Clear all" and "Select all" links which appeared for anonymouse users, if AnonSuggestions is turned on.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-29 22:34:26 -04:00
Joe Atzberger
b36269ea85 Bug 3056 - replace span hacking for terms in C4::Search
Reimplemented using jquery, added OSS plugin to both interfaces.  This is
another case where having a /common directory used by both OPAC and STAFF
would increase runtime efficiency and cut development time.

Removed unused variables.

I changed the OPAC .term color to match the staff interface, rather than being a
second shade of blue.  The highlight/unhighlight link is currently a bit bolder
than its neighbors, but my styling choices are provisional.  I expect the final
tweaking of CSS to come from another more capable designer, like Owen.

Note this patch may rely on previously submitted "Search.pm minor cleanup" patch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-29 22:06:55 -04:00
Kyle M Hall
38b356da8b Bugfix 2339 - Place hold button display with AllowOnShelfHolds OFF
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-29 21:15:13 -04:00
Nicole Engard
04db609d9e BUG 1255 Barred users will not see renew or renew all anymore
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-27 21:11:31 -04:00
Garry Collum
8582033e7c Bug 3408: Tag cloud says login when logged in - fix.
Changed the logic in the html form to have the login prompt only appear if a user is not logged in.  Separated the hidemylist functionality from the "show tags from other users" option.  Created a link for "hide my tags" or "show my tags" depending on context.

The user can now choose how many tags of other users to show, and can also choose whether to show their own tags or not.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-20 20:56:55 -04:00
Joe Atzberger
df7ef101e1 Remove bogus "selected" logic from OPAC adv. search.
The only reason the TMPL_VAR works where it does is because it is part
of the TMPL_LOOP.  It doesn't mean anything outside the loops, since
*everything* would be selected with the same variable, indiscriminantly.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-20 20:17:49 -04:00
Nicole Engard
9915d66589 Fix BUG #3406 - move tag cloud above 'my tags' for ease of reading
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 23:00:29 -04:00
Garry Collum
a1f827a611 Bug 3317: Adds author and added-author to email sent from basket (Cart).
Adds the author and added author fields to the email that is sent from the shopping cart.  This makes the data sent via email consistent with the data that is viewed in the basket.

There is still a problem with author authority records other than 100 tags appearing correctly in the shopping cart.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 22:54:04 -04:00
68b34874b0 Changes to lists to improve behavior for users with no javascript
This patch moves the creation of some js-based function links to the script so that non-functioning links will not display when javascript is turned off. Other links which require TMPL processing are hidden with CSS and shown with JS.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 22:25:01 -04:00
Nahuel ANGELINETTI
79fc6d81f7 (bug #3267) show items.uri in opac and intranet
This add a column in normal view, with a link to items.uri URL if it exists.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 08:26:33 -04:00
Joe Atzberger
b8a0cd9802 Search breadcrumb spacing
Also integrated both halves of the "searchdesc" conditional so they use the
same "No Result found!" message and <p> tags.
[LL 570]

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 08:14:07 -04:00
5a93f0b806 More changes to implement jQuery upgrade.
- Upgrading jquery checkboxes plugin
- Correcting references to checkboxes plugin
- Upgrading jquery hotkeys plugin
- Upgrading syntax for calls to hotkeys plugin
- Removing jquery dimensions plugin (functionality rolled into jQuery)

Signed-off-by: John Beppu <john.beppu@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 07:08:18 -04:00
John Beppu
dbbe53ac63 Upgraded from jQuery 1.2.3 to 1.3.2
http://docs.jquery.com/Release:jQuery_1.3
http://www.learningjquery.com/2009/03/3-quick-steps-for-a-painless-upgrade-to-jquery-13

Signed-off-by: John Beppu <john.beppu@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 07:08:06 -04:00
51aae63998 Fix for Bug 2713: Opac detail content overlaps when right column extends too far
Resubmitted. My proposed fix eliminates the OpacNav column on this page. It puts the "search for this title in..." links into a drop-down menu, and it moves the "similar items" display into a tab along with Holdings, Reviews, etc.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-14 06:58:56 -04:00
04d2b672f6 Moving the text of an error message from the script to the template where it can be translated. Partial fix for bug 1153.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-07-12 23:52:11 -04:00
Garry Collum
45069c1ca4 Bug 3229: Fixes sort-by feature for jquery tablesorter in lists.
This patch adds some variables for the sortfield that are passed through the URL so that the sort-by feature works when the jquery tablesorter is used for a list of titles in opac-shelves.tmpl.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-29 14:09:28 -05:00
Garry Collum
5ca73ecbcc Bug 3347: Follow-up patch. Added logic for non-loggedin user.
Adds a login message, instead of a "New List" link, to the public list table, if a non-logged in user views the table.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-29 14:02:37 -05:00
Garry Collum
1a8aac38b6 Bug 3347: Inconsistencies with public and private list tables in opac-shelves.tmpl.
This patch fixes inconsistencies with the public and private list tables in opac-shelves.tmpl.  It adds a "new list" link to the public list view, and it adds a "sort by" column.

This patch also removes the toggle variables from the tables in opac-shelves.tmpl and Page.pm, and uses the template __odd__ variable for table highlighting if javascript is turned off.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-29 14:02:28 -05:00
Garry Collum
b28fad3308 Bug 3346: Fixes invalid XHTML in opac-shelves.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-29 13:58:13 -05:00
471fe3927a Removing hard-coded $ symbols from the template, addressing Bug 2547
Pending a system for displaying the correct currency symbol based on the system's default currency all dollar-specific symbols should be removed in favor of no symbol.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-29 13:47:25 -05:00
adde6f914d Correcting validation errors.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-29 09:34:46 -05:00
Garry Collum
9c984ccade Bug 2889: Cleaned up toggle variable in opac-readingrecord.tmpl. Disregard previous patch.
Cleaned up toggle variable in opac-readingrecord.pl and tmpl.  Used __odd__ variable instead.

Missed </TMPL_UNLESS> in previous patch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-29 08:06:56 -05:00
Jane Wagner
437188f444 Revision to my earlier patch for displaying the user name on checked-out items
My earlier patch (using the new syspref OPACShowCheckoutName mistakenly
displayed the Koha borrower number instead of the patron barcode number
in the OPAC display; this revises the script/template to use barcode.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-17 11:48:04 -05:00
Garry Collum
ee167345d7 Bug 3317: Author displaying inconsistently or not at all. Revision to previous patch.
Revision to previous patch for bug 3317.  Removes the EXPR command from opac-basket.tmpl, which was polluting the apache logs.

NTS: Don't use EXPR.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-17 11:42:43 -05:00
Garry Collum
6f7d01718d Bug 3317: Author displaying inconsistently or not at all in Cart.
This patch fixes the author and added author fields for the 'More Details' view of opac-basket.tmpl.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-15 12:44:05 -05:00
Garry Collum
2e742f27b7 Bug 3325: Fixes XHTML errors in opac-basket.tmpl.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-15 12:43:53 -05:00
Garry Collum
c147650b4a Bug 3228: Fix for Type: label in XSLT displays.
Empty nodes must be generated in the XML for MARC21slim2OPACResults - typeOf008 and MARC21slim2OPACResults - materialTypeCode when the fields in the Marc record do not exist.  This patch fixes the "Type: " label.  More testing needs to be done to see if any other labels appear in the Opac if the data is not defined in the MARC record.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-15 12:43:49 -05:00
f306efb2c0 Add +(" ")+ to make few strings translatable (with Paul & Galen fix)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-11 06:10:55 -05:00
Galen Charlton
527898f6f3 fix XHTML error
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-08 11:07:37 -05:00
Matthias Meusburger
0a2a13ca13 bug 3292: in opac branchcode should be replaced with name
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-08 11:03:46 -05:00
Garry Collum
165f06c68b Bug 3081: Add description for category code.
Adds the category code to the description in "my personal details" in opac.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-07 13:07:41 -05:00
Galen Charlton
a8bfa5ffc0 retweak font-size of OPAC search box
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-05 18:26:10 -05:00
savitra.sirohi
6ae7b60962 Use Google Indic Transliteration API to allow users to transliterate english typed words into Indic languages.
This is available only on the masthead search box in the OPAC.

User has to enter a word in english and press space or return to transliterate the word. The destination language can be selected from a drop down list. User can temporarily turn on/off the feature by using "ctrl +g".

A system preference - GoogleIndicTransliteration can be used to turn on this feature.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-05 18:05:47 -05:00
Galen Charlton
b10cd79ba3 additional SCO template fixes
* removed sco.js, which is no longer used
* fixed XHTML warnings in sco-main.tmpl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-03 14:37:34 -05:00
Joe Atzberger
ece3a31fca Cleanup SCO - OPAC self checkout
~ vestigial broken scripts and templates removed
~ meaningless dependencies removed
~ Focus handling issues resolved for cross-browsers
~ Timeout only invoked for non-first screen.  This keeps the refresh from
flooding the logs continuously for no purpose.
~ two halves of "validuser" conditional linked in TMPL
~ elsif's used for $op conditionals

The focus should now appear on the "Return to Account Summary" button during errors.
The user can scan anything (w/ carriage return) and get back to the first screen.
Also, special functionality is added for the magic barcode "__KOHA_NEW_CIRC__".
This effectively ends the patron session and logs them out so the next patron can begin.
The purpose is for patrons to avoid having to use a keyboard at all, if libraries
print and have this special barcode available for patrons at the SCO station.

Enhancement was requested by Plano Independent School District.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-03 14:01:02 -05:00
Garry Collum
9f5869e8eb Bug 3275: Fixes typo (maxlentgh) in opac-suggestions.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-03 13:26:04 -05:00
Garry Collum
d24624865a Bug 3276: Minor XHTML error in opac-passwd.tmpl
Fixes a reference to a non-existent id.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-03 13:26:01 -05:00
Garry Collum
535046b5f3 Bug 3283: Fix for a minor XHTML error in opac-tags_subject.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-06-03 13:25:46 -05:00
Matthias Meusburger
f2dfe5b46e bug 3271 : Added tooltip when adding to a list with no item selected (same as for cart)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-29 08:47:17 -05:00
Galen Charlton
36c0d0922d fix minor translation bugs in two OPAC templates
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-28 20:46:01 -05:00
Michael Hafen
53922726fc bugfix: unbalanced TMPL_IF in opac-tmpl/.../masthead.inc
There used to be two TMPL_IF's here, now there's just one.  Missed removing
the second /TMPL_IF though.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-28 14:04:52 -05:00
Ryan Higgins
ca5e412c1b bug 3244: Support for multiple PAC interfaces.
Adds three env vars allowing url-specific search limits, and defining
how the limit interacts with 'SearchMyLibraryFirst'.
See http://wiki.koha.org/doku.php?id=en:development:rfcs3.2:rfc32_multiple_opac

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-27 10:52:23 -05:00
d9c63270d0 Quick fix to catch some instances of XSS vulnerability, there will be more probably, thanks for the heads up Mark
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-27 08:04:18 -05:00
400f2bad9e Bug 3169 - Google jacket display, remove preview image on result page
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-27 07:06:29 -05:00
Garry Collum
7457bc6d10 Bug 3261: Fix for invalid XHTML on Personal Details page.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-27 07:02:36 -05:00
Garry Collum
6d75e726d5 Bug 3241: Place hold button appears on results page if opacuserlogin is off
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-26 10:20:12 -05:00
dc7472aef9 Fix for Bug 3233, tabs on detail page change when clicking on MARC view
This change adds a check for the XSLTDetailsDisplay preference since different MARC view tabs display on the detail page if that pref is on or off.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-22 13:55:48 -05:00
Galen Charlton
391a6d3261 bug 2001, 2432: improve unAPI implementation
Improves Koha's unAPI support as follows:

* The implementation no longer depends on there being
  an accessible Zebra or SRW server.  Instead, responses
  are created by running the bib MARCXML through local
  stylesheets.
* unAPI now works for all bibs, not just ones that have
  an ISBN.  The ID format is changed from koha:isbn:ISBN
  to koha:biblionumber:BIBNUM.
* unAPI now correctly advertises the formats it supports.
* This implementation now passes validation testing at
  http://validator.unapi.info/ .
* MODS3 support now works correctly.  As a consequence, this
  patch fixes bug 2432 (Zotero support when using the XSLT
  OPAC stylesheets).

TODO: as additonal XSLT stylesheets are created to convert
UNIMARC bibs to additional formats, the stylesheet map
in opac/unapi should be updated.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-22 07:31:43 -05:00
Nahuel ANGELINETTI
02d23609b1 Show "no title", when a notice doesn't have a title
This patch just check if there is a title, else show "No Title".

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-21 11:13:05 -05:00
Galen Charlton
3f49ad62c8 bug 3236: remove disused scripts and a function
Removed opac/opac-dictionary.pl and catalogue/dictionary.pl,
which were not in use and not linked to from any active
template files.  According to Henri, the functionality that
these scripts implemented hasn't been working since 2.2.

Also removed C4::Search::findseealso(), which was used
only by the two scripts.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
2009-05-20 08:37:58 -05:00