Cleanup code formatting for readability. No functional change.

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

opac-search - CORRECTION: remove diagnostic Dumper lines

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

stats.screen.pl - cleanup, conditionalize warns w/ Debug

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

smart-rules.pl - tighten regexp, Debug for warn statements, readable layout.

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

smart-rules.tmpl - cleanup display, remove units for zero/empty values

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

DB and syspref update for Tags and BakerTaylor

Feel free to apply this before the other related Tags/B&T patches.

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

Tags - module, script and template support for user tagging in OPAC.

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

Baker and Taylor OPAC integration, plus some logic for Tags

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

C4::Search - just cleanup, 1 FIXME added

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

C4/External/BakerTaylor.pm - Back end for B&T content.

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

opac.css - add styling for tag classes

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

opac-tags_subject.pl - touch up: this script is does not require Exporter.

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

BakerTaylor.pm - Initial implementation of B+T jacket cover and bookstore links.

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

Tagging - add basic template slot in search for tags and script responsiveness to tagging sysprefs

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

kohabug 1776 - try to locate Zebra during install

If running Zebra, try to locate zebrasrv and zebraidx
so that koha-zebra-ctl.sh can point to a functioning
zebrasrv.

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

added RUN_DATABASE_TESTS to install log

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

fixing version numbering from 71 to 72

Add dropbox mode to return.pl. Small API change to allow setting

returndate other than now().  Overdues will be handled in a forthcoming commit.

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

Small template cleanup: display library name and cat description instead of codes

in circ-menu include.  Also, change 'Default' on main login page to 'My Library'.

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

DB BUMP 71 : add formatstring to labels_conf for specifying labels content

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

Labels update- print any marc field and add csv output.

Make any MARC data available to labels module;
Also add a csv output option, allowing export of relevant
biblio data do a 3rd party label layout application.
Note: This patch REQUIRES a forthcoming updatedatabase patch !

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

Adding breadcrumbs (fixes Bug 2031, "No way back home from about page")

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

More cleanup and corrections for circulation reports.

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

Fix for Bug 2018 ("Incorrect focus")

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

Refining some circ reports interfaces, adding table sorting.

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

Fix for Bug 2016, navigation bar in moredetail.pl not the same as in detail.pl

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

Language corrections (partially fixes Bug 2027)

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

Language corrections (mostly fixes Bug 2025)

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

Language corrections (fixes Bug 2024)

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

Language corrections (fixes Bug 2023)

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

Fix for Bug 2015 ("subscription-renew button is disabled"), other markup corrections; Removing unused popup css declaration from doc-head-close; removing unused popup include file.

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

Minor change to fix button appearance.

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

Showing "OPAC (lang)" to further clarify location information. Adding calendar button to conform with other date entry form fields.

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

UNIMARC specific rebuild items correctly

note 995 for items is hardcoded, so it's really for UNIMARC only. The script exit if you're not UNIMARCflavour

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

removing <h1> for ISBD view

Also updating the breadcrumb (the title is not available, displaying the number)

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

look change for 2 radio buttons

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

reindenting + (very) small change in design

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

removing the wrong itemcallnumber link

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

IMPROVEMENT : serialsadditems at subscription level (bump 071)

The patch update the DB by adding a row in the subscription table (serialsadditems), that is filled with the value of the syspref.
then, the syspref is deleted.

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

Updating french installation

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

testing framework: adding more warnings about possible data loss

I'm adding some more warnings in Makefile.PL to indicate that you WILL lose the data
in your testing database.

I've also relaxed the requirement that the teting database have "test" in its name.

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

testing framework: replacing PERL5LIB in order to more conidently find libraries

This patch makes the test suite work even when $PERL5LIB isn't set to the install directory.
It replaces $PERL5LIB with '..' in order to help ensure that we're loading the C4 libraries
out of the installation directory.

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

Fix obvious error in 500.pl script. Please push up immediately.

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

kohabug 1776 - disable mergeserver if not installing PazPar2

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

kohabug 1776 - enable daemon controls scripts for 'dev' mode

The daemon control scripts (koha-zebra-ctl.sh, koha-zebraqueue-ctl.sh,
and koha-pazpar2-ctl.sh) are now copied and installed in a
runnable fashion for a 'dev'-mode install.  By default
they are installed in the bin subdirectory of the runtime
directory.

Also:

* the control scripts now work if the EUID is other
  than root (as would be expected for a 'dev' or 'single'
  install).
* Split the SCRIPT_DIR installation target into
  SCRIPT_DIR (scripts to copy regardless of install mode)
  and SCRIPT_NONDEV_DIR (scripts to copy to SCRIPT_DIR
  unless the install mode is 'dev').

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

added labels to field selection page (koha2marclinks)

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

zebra conf: add reference to password file

Note: currently only zebraqueue_daemon.pl is known
to use the extended services that require the
Zebra r/w password.

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

kohabug 2019 - display entire subfield label

Don't truncate subfield label at 22 characters; instead
display whole label and wrap

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

test suite - various changes

* All configuration and run-time files created during
  a test suite run now live under t/run
* made sure scripts copied to t/run/bin are executable
* added t/run and t/test-config.txt to .gitignore

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

test suite: removed dep on 'acm' user

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

adding database-dependent test libs.

These test libs should have been included in the previous patch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

improvements to automated tests, including a testing database

I'm adding a bunch of additions to the test suite. There is now a test
database that is created, and a test instance of the zebra server and
daemon that are run before the tests in t/database-dependent.pl are
run. This means that you can test things that insert (or destroy)
things in the database.

To use these tests, after you 'make' your koha installation, you can
change to the 't' directory and 'make test' there.

There is now an additional question asked during installation. It asks
whether you would like to run the data-dependent tests. If so, It asks
you for some login information to a test database. I recommend that
you do not not use your production database here. It will delete all
of the data in it.

Things that need improvement:
* I suspect that there are large parts that are not very platform
  independent, so they will need to be improved.
* There are some parts of the installer that will probably be
  refactored to let this work a little bit better, including being run
  directly from a main-level 'make' target at some point.
* Lots more tests to add. Be bold! (see the t/lib directory)
* other tests in t/*.t can possibly benefit from being included in here.

There are two required perl modules for this part of the test suite. I
use:
'Test::Class' => 0.028,
'Test::Class::Load' => 0.02,
They are not listed as requirements in the top-level Makefile.PL
because they're not actually required to install or use Koha, but if
you want to run the test suite, you can install them from CPAN.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

refactor web installer code to created C4::Installer

Created a new module, C4::Installer, which embodies
most of the code from installer/install.pl that handles
identifying and loading SQL scripts and setting the
Version, marcflavour, FrameworksLoad, and NoZebra
system preferences.

With C4::Installer, it is now easier to automate
initializing and populating a Koha database.  This will
help allow the test suite to set up a test database
in order to run DB-dependent tests, as well as write
a command-line installer which may be of use to hosts
that need to support multiple Koha databases.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

added another check to MARC framework test

Check verifies that all subfields for each tag
are assigned to the same editor tab (or are ignored).

This situation can lead to data corruption, because
if an incoming MARC record has a field whose subfields
are assigned to different tabs, the first time that
record is saved in the bib editor, the field
will be split into multiple versions, each containing
the subfields assigned to a given tab.

Errors identified by this test known
in the MARC21 frameworks - I'm working on a patch to
correct those.

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

dep change: Data::Dumper 2.121_08 => 2.121

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

fix crash when saving MARC21 bib with NoZebra and BiblioAddsAuthority ON

Needed to identify correct subfield for retrieving
the authority type.

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

kohabug 2022 - fixed fine and issuing rules editors

Because of the way that the older fine and issuing
rule editors generate the HTML form, if a branch code,
patron category code, or item type code happened to have a
'-' or '.', the HTML form would not be parsed properly, thus
adding an implicit (rather than explicit) limit on the
characters allowed in one of those codes.

This fix removes this limitation by Base64-encoding the codes
when constructing the names for the <input> elements.

Two functions are added to C4::Koha:

  str_to_base64() - UTF-8 string to Base64
  base64_to_str() - reverse

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

Update yuipath syspref

If syspref points to online 2.3.1 YUI version, edit it to 2.5.1.

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

XHTML correction - readonly attribute can contain only "readonly"

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

rebuild_zebra.pl - add -y option

rebuild_zebra.pl will now mark all zebraqueue entries
of the affected record type(s) done when run in
normal mode to index all records (as opposed to running
it with -z to just process the zebraqueue).  This prevents
any running zebraqueue_daemon processes from attempting
to reindex the same records, redundantly.

The new -y swtich overrides this new behavior; in other words, if
running rebuild_zebra.pl without -z, you can specify
-y to *not* mark zebraqueue done.

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

Completeing port of changes in DrawSpineText API

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

Adding back text wrap funcionality for lines exceeding $text_wrap_cols; also adding use of C4::Debug

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

Adding trinaries to handle unpopulated itemcallnumber and/or barcode fields

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

Update 'yuipath' syspref to accomodate YUI 2.5.1 use rather than 2.3.1

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

Adding missing <!-- /TMPL_IF -->

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

Bug Fix : Blocker 2017

disabled values are not passed to CGI.
So that if you disable entry, values and subfield codes are mixed up.
I changed disabled=tru to readonly and it fixes the bug.

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

Tweak bullmarcimport.pl

* Add a new parameter -o to begin importing input file after skiping
  n records.
* Enclose input file reading in an eval directive to avoid abording
  import if few records are corrupted: they are now skipped.
* Help formating.

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

Updates to go with upgrade to YUI 2.5.1

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

Upgrading YUI library to 2.5.1

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

Correction to cat-search.inc to avoid focus-stealing problem; Change to detail.tmpl to play nice with moredetail.tmpl; Minor changes to search pages to allow greater flexibility with background customizations.

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

Accept or reject : Bug Fix and improvements.

Improving workflow : A suggestion can be edited and status can be changed if mistaken as soon as it is not deleted or acquired.
Adding dates display.
Adding deletion of suggestions.

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

Two fixes to speed up (a little bit) installation process

Add a explanation on DBD::mysql installation without test suite.
Add /misc/translator/install-code.pl script that creates templates
for specified language codes.

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

Porting SimpleSearch return changes to all code calling it.

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

More work on SimpleSearch

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

Enabling SimpleSearch to return the total number of hits to caller

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

added preview MARC links to manage staged import

In the list of bibs for an import batch, the sequence
number is now a link to a preview of the original
MARC record - uses the same JavaScript widget as the
preview for Z39.50 search results.

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

/TMPL_ELSE is invalid, should be TMPL_ELSE

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

BUGFIX (blocking) : AddSubscription was bugguy

making the creation of a subscription silently fail, but modif is impossible.

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

BUGFIX : highlighting term fixes

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

Correcting code to pass total number of hits returned to the template.

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

removing some spaces in record.abs, that make the file invalid for Zebra

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

my .gitignore, to avoid having files suggested to be commited.

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

BUGFIX : don't displayt "Place Hold" when RequestOnOpac is OFF

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

BUGFIX : encoding problem & security problem

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

Correct bad encoding of fr-FR data loaded by web installer

Some informations imported into MySQL DB by web installer
were wrongly encoded. It was the case for example with
opac_news table.

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

adding license tab to about page

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

Displays Languages selector on opac-main.pl page

On bottom of each OPAC pages, a languages selector is displayed
(depending of syspref). But this selector wasn't available
on OPAC home page.

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

Total overhaul of bor_issues_top report, removal from CGI::scolling_list.

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

Add authorised values descriptions to selector for "location".

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

bug 1953: removing possible SQL injections from C4::Acquisition::GetHistory

I think this is the last sub in this module that needs to be changed.
there should be no functionality or no documentation changes with this patch.
http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=1953

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

Remove redundant name from breadrumbs, make casing consistent in patron edit

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

show zipcode with address in patron search results list

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

Bug #1855

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

Correcting class.labels_conf to be classification.labels_conf

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

Bug Fixing : Forcing USMARC format for xml

using as_xml uses default MARC::File::XML format which is UNIMARC for marcflavour UNIMARC
And thus, causing a problem when encoding simple items marc records.

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

Item-level_itypes fix, addition of ccode vector and filter.

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

bug 1953: removing possible SQL injections from C4::Acquisition::GetLateOrders

I decided to not make chagnes the the query that's executed on databases other than MySQL as I have no good way to test that.
This change provides no functinality change and requires no documentation change.

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

BUGFIX : encoding problem & security problem

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

Bugfix: Handling cases where the image is the correct pixel dimensions

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

avoid crashes if record in result set is corrupted

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