fix to have the branch search on OPAC being really a branch search, not kw

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

display itemnote on check-out (added, was not here) & check-in (red-ed, was already here)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add MARC preview to cataloguing search, and add a link to add holdings. Once the edititems permission patch is accepted, this page should be updated to require edititems instead of editcatalogue.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

add single-transaction flag to mysqldump in example backup script.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Restrict OPAC tags display to approved=>1.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Correction to: Comment out confirmation popups (on successful events).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Comment out confirmation popups (on successful events).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - settle on application/xml Content-type for feeds

Required by IE6 to at least get a feed to display as XML
in the browser instead of being downloaded.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - send correct Content-type for search feeds

OPAC search RSS and ATOM feeds now have the correct
Content-type sent - "application/rss+xml" and "application/atom+xml",
respectively.

As part of this patch, added an optional fourth parameter
to C4::Output::output_html_with_http_headers to specify
the content type.  If that parameter is now supplied, or if
the value of the parameter does not contain at least a "/",
the default type of "text/html" is returned.

No documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - removed ersatz ESCAPE="html" added in prior patch

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - fixed channel links in Atom and RSS

query_cgi already includes the "q=" bit.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

OPACBaseURL - do not tack on :443 if using SSL

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - fix feed validation errors

Fix several validation errors in RSS and Atom feeds
generated from the OPAC:

- add missing guid to RSS elements
- add missing feed ID and element ID to Atom elements
- reflect OPACBaseURL changes
- fix atom:link self links
- add HTML escaping to fields comming from bib record
- set default timestamp for Atom updated elements

Issues identified but not solved in this patch:

- setting Atom updated element for each bib (presumably
  from biblio.timestamp)
- possible problems performing paging of Atom feed

Based on successful validation of feeds by feedvalidatory.org,
it is expected that feeds should now work in IE7 and IE6.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

calculate OPACBaseURL instead of using syspref

Calculate OPACBaseURL from the
CGI environment variables instead of using the
system preference.  As a result of this patch,
it should be possible to change the hostname,
protocol, or port of the OPAC without having to
reset a system preference.

Also added a FIXME to opac/unapi - the URL of
the SRU/W server has no necessary relationship
to the URL of the OPAC.

Once this patch is confirmed, the syspref can
be removed.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2180 - improve display when fines block checkouts

If a patron's fine balance is over the limit set by the
noissuecharges syspref, checkouts are blocked.  However,
this was not made very clear in the interface.

Improved the display by doing the following:

[1] If the fine balance is over the noissuecharges limit, use the "blocker"
styling (i.e., make all of the text red).
[2] Add wording to explicitly signal whether or not the fine balance is
blocking charges.

Documentation changes: new screenshots of blocked checkouts.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Apply item-level_itypes syspref to MARC export.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Independent-branchify MARC export. Add option to remove all holdings except your own, and force that behavior for independent branches. Also add accession date limit to export.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Sort dropdowns for libraries by name instead of branchcode in several staff-side interfaces.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Remove commented deprecated code and a useless query on items.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Remove a couple of warns.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Items from Bug 2167 ("URL handling in staff client inconsistent"): Removing display of URL from moredetail.pl. URLs are now displayed on detail.pl just as they are in the OPAC. Also removing display of biblio-level call number. Item level call number is displayed already.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix for bug 2183, "Account tab needs spacing between description and stadard text" Please note that this is a band-aid for a larger issue, described in Bug 2188

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix 1979 - display hold numbers. Note that hold priorities are still buggy.

But that is a separate bug from whether they display or not.
Also added basic error handling if biblionumber is not received, or
if biblionumber is bad, since that otherwise crashes the page.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Addresses Bug 2073, clarifying OpacCloud's limitations.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Printers code paring, related to bug 2045. This does not resolve the bug,

but it converts the template to using loop_context_var, and removes unused
variables from the script.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix typo in field name, apply branchcode selector logic from fines-ll.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

FIXME added to note discrepancy.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Wrap up Tags integration - add sysprefs for new installs.

Note that when the user is not logged in, the Tags input box/button will not display.
The user will see a message telling them to log in to add tags.  However, in order to
keep the results display clean, the message will display only once, utilizing the loop
context variable "__first__".  Useful stuff!

I also threw OPACShelfBrowser sysprefs.sql, but it still needs added to updatedatabase.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2186 - improve display of empty import batches

[1] Display the "No records have been staged" message only
    if no import batches exist.
[2] When displaying a batch that has no bibs in it, display
    a "There are no records in this batch to import".  Also,
    do not display the "import record into catalogue" button
    for an empty batch.
[3] Add CSS class "problem" to both messages.

Documentation changes: minor; only required if screenshots
exist of the manage staged MARC records page that has no
import batches on it.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIX issuingrules : total for all itemtype was not properly calculated

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding orderby parameter : Fixes changing page in auth_finder.pl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

From HDL:

I agree that opac-suggestions.tmpl modification is Not required any longer.
But still, the modification in opac-suggestions.pl is really necessary.
If you donot do that, ppl are presented with ALL suggestions and not
their suggestions by default, which is not what My Suggestions suggests.

Adding search on ISSN for z3950_search.pl

seems that search on ISSN was not taken into account adding @or @attr 1=8

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

removing autotruncate for authors.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fixing typo in DB update

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

add borrower note to check-in

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

enhance the patron note on check-out (was not enough visible, now is red)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

If the catalogue contains a value that is not in authorised values and it should, display the code

very usefull for us (UNIMARC) to find languages that are in the database but not in the auth values.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Displaying branchcode and notes fields in full serials issue table.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Respect BiblioDefaultView when search returns only 1 result & jump directly to the detail

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

small (UNIMARC) fix, error in regexp writing

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

change in button description to explain that we will import into catalogue

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

add script to install koha on a fresh debian.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

nomenclature cleanup: s/Branch/Library/ s/Document Type/Item Type/

Adding branchcode to opac-full-serial-issues

Suggestion : We may add some jquery ordering on those tables
Can we owen ?

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

DB Revision 087

adding default 'account details' notice.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug-2149, var fix - changing borr.cardnumber to borr.userid.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug-2149, added 'AutoEmailOpacUser' and 'AutoEmailPrimaryAddress' sysprefs

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug-2149, added new block to C4::Letters::SendAlerts() to email 'account creation' notice, and new block in memberentry.pl to call SendAlerts().

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Repair Labels code to accomodate patroncards. Purged EXPR.

Major FIXME's still remain, like the use of GET instead of POST.
The code is also a bit too INCLUDE-happy to net good performance.
The entire mechanism of adding to a batch should probably be proper
AJAX instead of the GET-centric opener.location approach.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fixing some typos, including, I believe, Bug 2174 ("spelling error in web installer")

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

minor cleanup, remove $sth->finish's. No documentation impact.

For details: see http://search.cpan.org/~timb/DBI-1.14/DBI.pm
"There's no need to call finish if you're about to destroy or re-execute the statement handle."
In all these cases the sth is falling out of scope, about to be destroyed.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

is_ajax function added. pagination_bar now intelligent: it has

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Removing dbh line altogether as discussed on koha-patches

Bug Fix : Creating a new authority type would fail on creating subfield_structure

removing "link" field which has been removed from table auth_subfield_structure
Removing a $dbh->disconnect

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2164 - display of guarantor relationship

When editing a child or professional patron, display
the drop-down of relationship types only if at least
one relationship type is defined by the borrowerRelationship
syspref.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Tags AJAX integration on OPAC details page, so it behaves like results.

Also, prevent duplicate tags (same user, biblio and term).  Strip
leading/trailing whitespace from term.  Block whitespace terms.
The CSS for details could be enhanced to emphasize tagstatus more.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Make printslip and printpage buttons use popup.

This bug was reported by existing users.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Correct typo "printslilp" to "printslip"

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fines repair. Make fines2.pl work, give feedback, improve comments and perldoc.

Remove $dbh->disconnect statements as counterproductive.
Prevent description field from begining with whitespace.
Added robust debug elements.  Test script behavior with:
    perl misc/cronjobs/fines2.pl -v
and:
mysql> select * from accountlines;

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding basic get_filters to Tags, centralizing "counts" code for Terms Summary.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Remove string processing (redundant to new pagination_bar)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Single FIXME comment inserted.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Commented out fixaccounts (not used anywhere). Also improved getnextacctno.

You can test getnextacctno like:
perl -e 'use C4::Accounts; print getnextacctno(33), "\n";'
where 33 is a borrowernumber out of the accountlines table.  Get that number like:
mysql> select borrowernumber,accountno from accountlines LIMIT 100;

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Correct number of args to getnextacctno.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Corrected wrong number of arguments to getnextacctno.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix 2026 - Comments handling overhauled. Scrubber and Error feedback added.

Note: we CANNOT rely on window.close in onSubmit or $().submit to close our popups.
On a relatively slow connection with a relatively large POST, commonly the close finishes
*before* the POST completes, as reported with our New Zealand clients.  Despite success in
trivial cases, this should be obvious, since the event is necessarily before the submission.
It also assumes success and prevents any kind of error feedback.  Other popups are likely
to exhibit this same defective behavior.

Some FIXME's outstanding: need to allow users to delete their own comments,
need to enforce and feedback on max comment length.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix 1978 - if invalid duedate specified, return error, do not issue.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fixed session cookie name in opac-logout.pl

Session Cookie name is CGISESSID in Auth.pm

Signed-off-by: Michael Hafen <mdhafen@washk12.org>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Unescape Comment, now that we trust Scrubber to block bad markup.

Obviously if we are going to allow good markup, we can't then escape it.
Documentation reference: allowed tags for comments are:
br b i em big small strong

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Tags AJAX and JSON for OPAC side.

Note, all Tags js will live under KOHA.Tags namespace.
See opac-tags.pl perldoc for AJAX/JSON examples.  The capability is already
there to be very web-servicey, even moreso than the current OPAC implementation
will utilize.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Single slash added in test text string.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add error feedback and Scrubber to OPAC tags page. Beginnings of ajax code.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Initial commit for Tags back-end moderation. Requires AJAX functions from Output.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

New subs for moderation, fixed bug to allow multiple sort fields, added POD.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add AJAX output sub, EXPORT_TAGS and enable loop_context_vars for HTML::Template::Pro.

The loop_context_vars will be very useful in display.  They are:
__first__, __last__, __inner__, __odd__, __counter__.
Note: apparently __even__ does not exist, so instead use something like:
<!-- TMPL_UNLESS NAME="__odd__" -->

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add cn_source to authorised values dropdown in marc structure editor.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

small update to french translation

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fixing the css property for tablesorter js

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIXING authority link in detail.pl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

small fix for UNIMARC 600$a that is also subject

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding Image::Magick version info

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Update Catalog Stats report: / Remove lccn/loc filter, which seems to conflate marc21 010 and 050. / Update labels to reflect 3.0 terminology.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fill statistics table properly for renewals; and issue->checkout, return->checkin

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add ANSCR to default cn_source list.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix pagination error on subfield structure editor

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

add URLLinkText syspref to opac detail, and add some extra handling for MARC 21 urls.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Show 856$3 (materials specified) by default in MARC21 .

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Update MARC21 simple frameworks with biblio.serial and items.enumchron

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Translatable facets in librarian interface

Pushes modified facets display from OPAC to librarian interface.
Corrects also OPAC version which were omitting Titles facet.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Facets (current) translatable via template

This patch allows to translate facets label with standart
templates / PO files. Facets are still hard coded in Koha.pm.
Template wait those hard coded facets: Topics, Places, Titles,
Autors and Libraries.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

removing warnings from C4::Members::GetMember

C4::Members::GetMember emitted a few unnecessary warnings when no $type was passed in. This patch prevents that.

No functional or documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bug 2047: some tests for C4::Biblio::get_biblio_authorised_values

I had these tests laying around for a while. I just forgot to commit them.

No functional or documentation changes needed.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding bookfund search to aqbudget.tmpl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding routing list link to table of information about each subscription (Bug 2059, "Link to Serial Collection routing list missing on menu")

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix for bug 2130, "Patron Image Missing on Permissions Screen"; Correction to allow proper tab highlighting when viewing patron log.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix for Bug 2120, "

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Authorities detail page was missing toolbar, breadcrumbs.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Markup error caused layout problems.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>