fixed spelling error, removed extra debug logging

Fixed ClassCastException crash on search with no results.

removed files that were renamed

App is signing in OK now.  Reimplemented LoadingTask in terms of a raw Thread because it was hanging.  Starting with HONEYCOMB, tasks are executed on a single thread, and so LoadingTask->Authenticator->AsyncTask would hang.

forgot this file

Checkpoint.  Not working but getting less cluttered with thread/session/exception handling.

Factored out authtoken retrieval to LoginController.  Still forwarding to JunkActivity but seems to be working OK.

checkpoint: starting app on new activity and getting an app token and forwarding to JunkActivity, just until the kinks are worked out

Removed AndroidAuthenticator tree, preferring to have that code in-app

Tweaking resources.

Added working EG Authenticator to the build, but haven't tied it to the app yet.  Start of library-specific app customization via "ou_" (organizational unit) resources, e.g. ou_app_name and ou_gateway_url.

removed confusing commented out code

checkpoint: moving auth package inside Evergreen app tree.  Dealing with external Android library projects in eclipse has been a giant hassle and I don't really need it.

checkpoint.  builds but authenticator dies with NoClassDefFoundError: org.opensrf.net.http.HttpConnection

updated android support library to rev 19

Added an android authenticator based on Apache 2.0 licensed code from https://github.com/Udinic/AccountAuthenticator

The main problem in the EG app in the android-minor-cleanup branch is that when the auth token expires or you rotate the device at the wrong time you get repeated crashes.  My hypothesis is that using native android.accounts facilities for getting access to auth tokens will clean up the auth handling and also make authentication accessible to Services.

ignore some android cruft

Eliminated warnings by using textSize 16sp not 16dp.  Allow dashboard_account view to go landscape.

Improvements surrounding organizations.

Make initialization faster by (1) parsing OrgTree.js organization array as
JSON.

Do not include in the library list organizational units that are marked as
not visible in OPAC.  This reduces the list from 338 items to 181 items.
Improved readability of the list by removing space-padding that is done
for the web drop-down list, and putting the spinner in dialog mode.

Sort organizations by name, with (1) home library selected, and (2) root
organization first.  This narrows the results of default searches.

Added more missing classes: cbreb, cbrebn, cbrebi, cbrebin

removed unused class

spelling fix

More status message cleanup.

Rephrase awkward message

Removed unused import and simple-xml-2.6.4.jar

Bah, added a couple more missing classes to fm_IDL.xml.  In developer/debug mode, failure to deserialize an OSRF response will throw an NPE; in production it will return null.

Minor tweaks of progress messages (that really should be in resources).

Do not throw an empty ServerErrorMessage exception if the renewal succeeds.

Step 2 in IDL speedup: Use assets copy of /reports/fm_IDL.xml?class=a&class=b.  This is a much smaller xml file.  Only danger is we run the risk of a NPE if we get a response of a class we haven't parsed.  That should be handled in OSRF but for now we optimize for speed.

Slightly better tracing of OSRF methods, and a better comment.

moved

annotating a couple sources of NPEs and fixed one.  Login handling needs to be rewritten.  I'm almost there

Step 1 in IDL speedup: Use the fm_IDL.xml from assets instead of
downloading; shrink fm_IDL.xml by removing unused elements.  Under the
emulator parsing the IDL is ridiculously slow (35s).  Still loads of work
to do; it's only slightly faster on the emulator (30s) and still
noticeably slow on hardware (~5s).

Removed openils_idl.jar and instead put the modified sources under
Android/src/java/org/open_ils/.  Nat at least it's obvious that it's
modified version of the code in Open-ILS/src/java.  It's modified because
the javax.xml.stream parser is not available on Android and was replaced
with an xmlpull parser.

Whoa there!  It's not fair to take Apache licensed source code from
https://code.google.com/p/android-imagedownloader/ (blog at
http://android-developers.blogspot.com/2010/07/multithreading-for-performance.html)
and claim to have authored it.  I changed the Copyright back to its
original before I do anything else.

Adding release notes for tracking improvements.

Work around an issue with the request handler crashed server side and client got an empty payload / null converted response.

Improving holds.
* Hold status was all mixed up; it now properly says "Waiting for copy"
  instead of "Transit".  Hold status now indicates both #holds #copies and
  position in queue (user requested).
* Removed email checkbox and phone number checkbox from hold updates.  At
  least at my library web site you can't update phone/email preferences
  by editing holds.  hold_request.phone_notify is a text not boolean.

Need RECEIVE_BOOT_COMPLETED permission to register a receiver; this used
to say permission "REBOOT" which was wrong and disallowed in sdk 18.  Not
sure any notifications work anyway since app currently crashes when
attempting to edit notifications.

Enable moving the app to the SD card.  Requires minSdkVersion="8" (Android 2.2)
but that reaches 95% of the installed devices.

more titles in Title Case

Updated targetSdkVersion to 18. minSdkVersion is still 3.

Updated targetSdkVersion to 18. minSdkVersion is still 3.

Titles should be in Title Case, and obvious.

Construct BufferedInputStream with explicit 8k buffer to avoid android logcat warning.

Fixed bug 16: app leaks password to System.out (visible in logcat)

Removed backup file

Wordsmithing:
    (title) checkout items -> Checked Out
    (button) Check out items -> Items Checked Out
    EG server: -> Library Catalog URL:
    Load more data -> Fetching more results...

organized imports

Fetching a URL to see if the network is available is not a good plan.
For now, return true if the url is non-empty.  Also cleaned up some
java warnings.

Changes made by eclipse

Add missing space to record header

Also, while we are at it, make it part of strings.xml.

Signed-off-by: Dan Wells <dbw2@calvin.edu>

complete fix store server url

Fixed shared prefs login and MyAccount buttons

Merge branch 'master' of git://git.evergreen-ils.org/Evergreen into android

LP#867691: Authority match sets are not working

Authority match sets are not supported so hide the option for now.
It's confusing having it there.

[with minor conflict resolution]

Signed-off-by: Mark Cooper <markchristophercooper@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Remove deprecated, old, unused GridColumnPicker

Spotted by Joseph Lewis in LP #804462.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

.gitignore additions for (new) generated files

Add these to Evergreen .gitignore:

Open-ILS/examples/apache/eg.conf
Open-ILS/examples/apache/eg_startup
Open-ILS/examples/apache/eg_vhost.conf
Open-ILS/src/support-scripts/eg_db_config

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP1147286 - Allow translating more of the staff client portal page.

Add translatable strings for image alt-tags and the Search button on
the staff client portal page.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Ben Shum <bshum@biblio.org>

Stamping upgrade for LOC z39.50 target seed data

Signed-off-by: Ben Shum <bshum@biblio.org>

Upgrade script for LP#1105225: LOC z39.50 seed data

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1105225: update LOC z39.50 target seed data

This patch updates the seed data from the target list at
http://www.loc.gov/z3950/lcserver.html#addr.  It also changes the
structure attribute for the 'lccn' search attribute to wordlist,
which means that LCCNs can be searched without having to enter
the leading spaces.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1022582: MARC Exporter includes deleted copies in holdings export

Added check to filter out callnumbers/copies if deleted.

Signed-off-by: Mark Cooper <markchristophercooper@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Stamping upgrade for truncate-to-max-fine

Signed-off-by: Mike Rylander <mrylander@gmail.com>

truncate fines to max fine amount (LP#1145284)

When the max fine amount is not a multiple of the recurring fine amount,
fines will actually max out at an amount greater than the max fine
value.  This commit adds YAOUS and some simple functionality to
optionally truncate the final amount billed to the max fine amount.

For example, if max fine is $5.00 and recurring fine is $0.30, the 17th
billing will bring the total amount billed to $5.10 (17 x $0.30),
thereby exceeding the max fine amount.  With this commit, if
circ.fines.truncate_to_max_fine is true, the final billing amount will
be reduced and the total amount billed will be $5.00.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1136084: drop silent dependency on 'bsdmainutils' Debian/Ubuntu package

colrm is a perfectly good command, but isn't guaranteed to be
present on a minimal Ubuntu system.  This patch replaces ti with
with cut, which is more likely to be present (including on *BSD
servers).

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Remove dependency on libreadline5-dev from Debian and Ubuntu.

libreadline5-dev is old and superseded by libreadline-dev.

Also OpenSRF installs libreadline-dev, so we should not have to
install it from Evergreen's Makefile.install.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: move up event listeners in place_hold.js

The preceding two functions need the event listeners to be defined before
use. Move up to prevent errors.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1086458: fix typo

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: add destructor to caption binding

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: add destructor to messagecatalog

Based on patch by Jason Etheridge.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: clean up event listener functions

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

move EventListenerList out of global_util.js

So that we can more easily pull it into main.xul (which we also do here)

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: remove references to windows during cleanup

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: clear util.list lists and trees during cleanup

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: add missing cleanup call

Thanks to Jason Etheridge for the catch.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: clean up after event listeners in circ/patron interface

Upon window unload, now removes event listeners, both ones explicitly
created by the page JavsScript as well as ones created by
util.list, util.controller, and persist_helper().

The same approach of defining cleanup fnctions used by unload
events can be applied to the rest of the staff client, but
this patch focuses on circulation first.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: add more event listener tracking

util.list and util.controller now have cleanup()
methods that can be invoked to unregister the event
listeners they create.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: add way to clean up persist_helper event listeners

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1086458: add class to manage event listeners

EventListenerList allows one to maintain a list
of event listeners, then remove them all when it's
time to clean up a window.

Usage is:

var list = new EventListenerList();
// attach an event listener
list.add(node, 'command', function(ev) { alert('BOO!'); }, false);
...
// get rid of them
list.removeAll();

Based on an idea by Jason Etheridge.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Version upgrade script from 2.4-alpha1

Signed-off-by: Mike Rylander <mrylander@gmail.com>

Updated translations for 2.4-alpha1

Signed-off-by: Mike Rylander <mrylander@gmail.com>

Stamping upgrade scripts for ingest partitioning

Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1066326: protect against flags being entirely missing from config.internal_flag

Belt, meet suspenders.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1066326: use internal flags; sync with master

After discussion with Mike Rylander, moved the new flags
to internal flags, as exposing them to the staff client's
global flag admin page could lead to records getting incompletely
ingested without there being an easy way to rectify the problem
from the staff client.

Also, sync up with other recent work on ingest.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1066326: Allow component partitioning of ingest

The component parts of the indexing process are fairly flexible. In some
places they can be called with parameters that control the behavior, and
other places will respond to global and/or internal flags.  However, there
is some inconsistency to which parts will react to each mechanism.

Previous to this commit, there was no way to cause ingest to skip specific
parts of the indexing process (search / facet / browse, in particular)
even though the stored procedure responsible for this had the ability to
perform, and parameters for the control of, this behavior.  Now, when any
of the parameters are false (the default, saying "do not skip") it will
check the value of a related global flag and, if enabled, skip that part
of the ingest.

This will allow one to do perform massive, batch, parallel ingests of bib
records, something that was effectively blocked by the way browse ingest
is performed.  By disabling browse ingest during the main record loading,
and then performing only the browse ingest later, one can significantly
reduce the time required to load massive batches of records.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

TPAC: Don't display serial headers without content

Currently, the TPAC serial display code will print a section
header, then see if there is any content to show.  This causes
headers to show up when they are unnecessary or unwanted.

Now, we only show the header if we have content needing heading.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Provide some level of support for legacy install locations

If someone really wants to keep OpenSRF and Evergreen installed in
/openils, then enable the configure script to find the osrf_config
script and add the library location to the test to prevent it from
failing.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Clean-up unused Apache header/library config vars

apxs is used to build the Apache modules, so there is no need to
configure the Apache / APR header and library paths.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Reduce hard-coded configure defaults

We can and should be smarter about how we set configuration variables.
Rather than using hard-coded defaults for the OpenSRF header and library
locations, ask osrf_config what values to use.

Also be smarter about how we check for the OpenSRF library as a dependency.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

eg_config should be installed

We were not installing eg_config, which is a useful script for returning the
configured settings for an Evergreen installation.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Generate Apache config files for increased portability

Continue weaning ourselves off of hardcoded paths such as /openils in config
files. Also, install the Apache config example files into a docs directory
instead of just leaving them in the source tree. Similarly, remove some
hardcoded paths from eg_db_config.pl.

At the same time, drop the '.pl' suffix from the scripts that we're touching,
and make the Apache startup script a little less generic in name.

The change from "startup.pl" to "eg_startup" and from "eg_db_config.pl"
to "eg_db_config" also results in updates to installation and upgrading
documentation.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Remove broken error message

Checkout fills related hold causes an empty array sometimes, so don't throw
out errors for that.

Also, the original error would itself fail, throwing an error that would
then fail as well, likely due to a copy/paste issue.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Include barcode in pre-cat confirm dialog

This allows staff to see what was scanned into the field as part of
the dialog window prior to confirming creating a pre-cat for it.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Reporter: Sort contents of remote selector widgets

Simple sort based on the display value.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Stamping upgrade script for Bib Merge Ignore Deleted Call Numbers

Signed-off-by: Ben Shum <bshum@biblio.org>

Changed bib merge to ignore deleted call numbers.

The asset.merge_record_assets function was not checking for deleted call numbers
and merging items onto deleted call numbers that of course causes issues.

Signed-off-by: Steve Callender <stevecallender@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

lp1092644 refresh row with Item Status Edit Volume

Assuming the Trim List box is not selected, this simply refreshes selected rows
in the Item Status interface that were used to provide context for the Actions
for Catalogers -> Edit Volumes command. If Trim List is selected, then the items
are instead re-added to top of the list.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Repair clear holds shelf transaction timeout

Initiate our cstore transaction after making the initial, potentially
long-running call to collect the clear-shelf holds to process.
Otherwise, the cstore transaction may time out while waiting on the
holds retrieval to complete.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

set title during receipt/slip printing

This is to workaround a bug in CUPS where long window titles will
cause the print job to fail.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Speed up sorted-related-holds query

In several interfaces, we use a server side method which gathers statistics
about a hold: related holds, it's position in the (approximate) queue, the
estimated wait time, etc.  Within this method is a relatively complicated
json_query that returns the list of related, (FIFO-ish) sorted holds -- ones
that could be filled by a copy which could fill the hold in question.  This
commit restructures that query so as to make it faster when the list of
related holds is large, by removing duplicate (cartesian product, actually)
hold ids that were being fed into an INNER JOIN clause.

Testing shows a speed increase of 4x for related-hold queue of around 675
holds [~2s -> ~0.5s] on a relatively large Evergreen installation,
appropriately tuned.  The speed improvement gets larger with longer queues.
There is no observed decrease in speed for smaller queue sizes.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>