Bring 1.6.1-2.0 upgrade script in line with reality

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18463 dcc99617-32d9-48b4-a31d-7c20da2025e4

In 2.0+ we require PG 8.4+ which means we have WITH RECURSIVE support, so use that instead of tablefunc connect_by.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18462 dcc99617-32d9-48b4-a31d-7c20da2025e4

Dump some debugging output from clark; may later make this optional, but we usually need it when looking for the cause of problems.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18457 dcc99617-32d9-48b4-a31d-7c20da2025e4

Improved subject formatting on summary page, and better title bookmarking

Apply a modified version of Ben Ostrowsky's patch from
https://bugs.launchpad.net/evergreen/+bug/627133 - the modifications include
using BT.textContent() to retrieve the text of the subfields, and
getAttribute() instead of dojo.attr() for IE support.

From the description of 627133:

Currently the BibTemplate code in rdetail/rdetail_summary.xml for display
of subject headings only includes the 650. It should be expanded to:

  * include all standard subject added entry fields (or more broadly, all 6XX)
  * exclude control subfields (those whose subfield label is a digit) from display
  * only using the double-dash separator before subfields $x, $y, $z, and $v.
    The display text of other subfields should be separated from one another only
    by a space.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18456 dcc99617-32d9-48b4-a31d-7c20da2025e4

further protection against dreaded opac bank page; wrap settimeout redetail bibtemplate loading in an onload to ensure it runs after dojo onload events

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18454 dcc99617-32d9-48b4-a31d-7c20da2025e4

Use standard JS syntax; var [...] seems to be a Spidermonkey extension

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18452 dcc99617-32d9-48b4-a31d-7c20da2025e4

typo in setTimeout spotted by dbs

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18450 dcc99617-32d9-48b4-a31d-7c20da2025e4

Acq: fix minor bug, sometimes view/place orders interface failed if you clicked
"Add to Selection List" for a new order

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18448 dcc99617-32d9-48b4-a31d-7c20da2025e4

Move dojo loading around to avoid the "white screen of death" in Firefox and xulrunner (See: o.dojotoolkit.org/forum/dojo-foundation/general-discussion/firefox-3-fires-domcontentloaded-event-prematurely )

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18446 dcc99617-32d9-48b4-a31d-7c20da2025e4

Serials: in alt serial controls, subscription details, avoid cut-off autogrid

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18442 dcc99617-32d9-48b4-a31d-7c20da2025e4

ilsevent is no longer a number, but a string.  I knew that, but missed these tests

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18440 dcc99617-32d9-48b4-a31d-7c20da2025e4

adding database update missed in previous commit

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18435 dcc99617-32d9-48b4-a31d-7c20da2025e4

improve call number sorting

oils_text_as_bytea now does uppercasing in addition
to converting strings to bytea, working around
the limitation that json_query can't stack transforms

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18434 dcc99617-32d9-48b4-a31d-7c20da2025e4

use a transaction when searching for the nearest hold request

Solves the following problem in a setup that uses Slony - if
multiple items that can fulfill the same hold are checked
in one after another, occassionally two items will end up
being put in transit for the same hold request.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18432 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from tsbere for new tab versus current tab behavior with the button bar:
---
The attached patch to trunk adds the ability to set an open-ils.toolbar.defaultnewtab preference to cause the toolbar buttons to open in a new tab instead of the current tab. Holding the "accel" key (default of ctrl on windows/linux and command on mac) will do the opposite.

I only applied this to the five commands currently in use on the toolbar for now. Any other commands added to the toolbar will need to have their command handlers in menu.js updated.

Thomas Berezansky
Merrimack Valley Library Consortium

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18431 dcc99617-32d9-48b4-a31d-7c20da2025e4

added some time diagnostics to checkin process;  more org unit caching; do less user fleshing on hold user during checkin (only card and name is needed)

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18428 dcc99617-32d9-48b4-a31d-7c20da2025e4

These fields exist ... tell the IDL about them

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18427 dcc99617-32d9-48b4-a31d-7c20da2025e4

add start_date and end_date to scap serialization

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18425 dcc99617-32d9-48b4-a31d-7c20da2025e4

don't allow inactive cards with web-selfcheck

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18423 dcc99617-32d9-48b4-a31d-7c20da2025e4

Remove accidental duplicated line

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18421 dcc99617-32d9-48b4-a31d-7c20da2025e4

Fix a set of supercat bugs in handling records with subscriptions attached. The
most noticeable manifestation was that you wouldn't see local call numbers in
the opac for any such record.

Credit to miker for help with the bug squashing and filling in missing pieces.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18419 dcc99617-32d9-48b4-a31d-7c20da2025e4

Action/trigger: restore the environment-flesher's ability to try reporter-store
if cstore is not among the controllers on a needed class

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18416 dcc99617-32d9-48b4-a31d-7c20da2025e4

reporter.materialized_simple_record isn't really /just/ for reporter, as it's
linked to from bre

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18414 dcc99617-32d9-48b4-a31d-7c20da2025e4

1.6.1-2.0 upgrade script needs to fall in line with reality, re 0442

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18411 dcc99617-32d9-48b4-a31d-7c20da2025e4

Prevent "Validate" from matching against deleted authority records

2.0 introduces the ability to delete authority records. The routines in
OpenILS:Application:Storage:Publisher:authority check for matching values
of authority.full_rec without concern about whether the corresponding
entry in authority.record_entry has been deleted or not.

This change removes entries from authority.full_rec when an
authority.record_entry row is deleted, with the (possibly incorrect)
assumption that there won't be many cases where users will want to
search for deleted authority records and undelete them. If that
assumption turns out to be incorrect, then further changes can follow;
this change at least makes the current behaviour for user-visible
actions work as expected.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18409 dcc99617-32d9-48b4-a31d-7c20da2025e4

Alt serials interface tweak

Add a shortcut for the user to jump back from the streams interface to the
distributions tab of the subscription details interface

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18407 dcc99617-32d9-48b4-a31d-7c20da2025e4

during checkin, an item's hold may have a fleshed pickup_lib.  added code to detect and protect against treating a ref as an ID.  minor cleanup

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18402 dcc99617-32d9-48b4-a31d-7c20da2025e4

Summary: Patch from Thomas Berezansky providing an alternate implementation of Hard Due Dates.

Overview:
 * Moves hard due date configuration from the duration rule to the circ matrix
 * Implements backward-compatible API for circ scripts
 * Provides a stored procedure which updates hard due date values on demand

Further extension by Mike Rylander:
 * Provide a srfsh script and crontab.example entry to fire the stored procedure nightly
 * Expose stored procedure and add to the upgrade script

TODO: Create UIs for configuring hard due dates; integrate into version upgrade scripts

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18396 dcc99617-32d9-48b4-a31d-7c20da2025e4

Use a transaction when clearing the expired circ history

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18395 dcc99617-32d9-48b4-a31d-7c20da2025e4

expand the scope of BT on the result list page, allowing simpler slots

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18392 dcc99617-32d9-48b4-a31d-7c20da2025e4

update 1.6.1 to 2.0 upgrade script for 0441

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18390 dcc99617-32d9-48b4-a31d-7c20da2025e4

fix mapping of the crossed D character in naco_normalize

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18388 dcc99617-32d9-48b4-a31d-7c20da2025e4

Make editing of asset.copy_template a little more friendly (AutoGrid w/ pcrud
can't deal with editor/creator fields for you).

Incidentally add createPaneOnSubmit and editPaneOnSubmit attributes to
AutoGrid that take a function name as their values, specifying an alternate
callback function to use instead of pcrud.create/update

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18386 dcc99617-32d9-48b4-a31d-7c20da2025e4

Serials: add functionality meant for "Previous Issuance Copy Location" setting

Simply put, if you're receiving an item a branch for which the org-unit
setting "serial.prev_issuance_copy_location" has a value, the previously
received copy matching the same distribution and stream as the item you're
receiving now will be moved to that copy location specified in the org-unit
setting.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18384 dcc99617-32d9-48b4-a31d-7c20da2025e4

Protect biblio.extract_located_uris() from a NULL uri_use value by using the href as a fallback

If an 856 field contained a subfield "9" but did not contain a subfield
"z", "2", or "n", then ingest of the MARC record would fail as it attempted
to retrieve an asset.uri row where uri_use = NULL - which was never true.

This change adds the subfield "u" to fall back to for the uri_use variable,
just like we do for the uri_label variable, and prevents ugly ingest errors
from getting thrown back at the client.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18382 dcc99617-32d9-48b4-a31d-7c20da2025e4

stop sign page was incorrectly coming up for all penalties

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18380 dcc99617-32d9-48b4-a31d-7c20da2025e4

make Circulation -> Enter Offline Interface open a new window instead of a tab, like the button from the login window.  This works around a permission issue with loading the offline receipt templates

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18377 dcc99617-32d9-48b4-a31d-7c20da2025e4

fix billing interface so that %title% and %barcode% work in bill_payment template

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18375 dcc99617-32d9-48b4-a31d-7c20da2025e4

correct default bills_historical template to use %mbts_id% instead of %id%.  Also add in %title%

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18374 dcc99617-32d9-48b4-a31d-7c20da2025e4

go ahead and use the new index if we need to

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18366 dcc99617-32d9-48b4-a31d-7c20da2025e4

need to update this index as well

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18365 dcc99617-32d9-48b4-a31d-7c20da2025e4

use a function to wrap up escaping of solidus and casting to bytea, propogate to indexing and where/order_by

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18364 dcc99617-32d9-48b4-a31d-7c20da2025e4

tweak expanding search field aliases

Avoids a glitch that can occur if a search field alias
exists that happens to have the same name as a search field.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18353 dcc99617-32d9-48b4-a31d-7c20da2025e4

If an item is captured for a hold, but not in transit (i.e. on holds shelf), set the destination_location equal to the pickup library (i.e where it's supposedly on the shelf).  This is useful for autmated sorting so that the item will return to the branch whose shelf where it belongs.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18351 dcc99617-32d9-48b4-a31d-7c20da2025e4

force granularity-only when any granularity is specified

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18349 dcc99617-32d9-48b4-a31d-7c20da2025e4

"as" not "to" ... you pointed that out the first time, miker, what is your deal?

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18343 dcc99617-32d9-48b4-a31d-7c20da2025e4

repaired upgrade version

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18342 dcc99617-32d9-48b4-a31d-7c20da2025e4

backward compat indexing for label instead of label_sortkey

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18336 dcc99617-32d9-48b4-a31d-7c20da2025e4

cast label_sortkey to bytea in order to get ascii-betical sorting in any locale, even C. stupid glibc ...

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18333 dcc99617-32d9-48b4-a31d-7c20da2025e4

patch from Steve Callendar to avoid resetting the passwd every time a phone number changes

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18331 dcc99617-32d9-48b4-a31d-7c20da2025e4

correct top-half ordering by label_sortkey

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18329 dcc99617-32d9-48b4-a31d-7c20da2025e4

change the Swap Editor button to a persisted checkbox

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18326 dcc99617-32d9-48b4-a31d-7c20da2025e4

default to standard cstore instead of rstore for env building editor

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18323 dcc99617-32d9-48b4-a31d-7c20da2025e4

clean up duplicate system-controlled penalties during penalty calculation; perform penalty trigger event creation after the standalone commit occurs.  todo, handle non-standalone post-commit penalty trigger event creation

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18322 dcc99617-32d9-48b4-a31d-7c20da2025e4

Changes to the treatment of ORDER BY:

1. For json_query: when ORDER BY is expressed as an object keyed on class
(instead of an array of field specifications), and the class is not in
scope, error out instead of silently ignoring the class.

The other changes affect only methods other than json_query:

2. When the ORDER BY list is provided as a raw text string: block any
string containing a semicolon, in order to block simple SQL injections.
For now we make no exceptions for quoted semicolons, which are not
likely ever to appear an an ORDER BY clause.

3. Keep virtual fields out of the ORDER BY clause.  For now we silently
ignore them, as we ignore non-existent fields.  In both cases we should
perhaps error out.

4. Don't require that a class referenced in the ORDER BY clause also be
referenced in the SELECT clause.  Just make sure it's in scope.

M    Open-ILS/src/c-apps/oils_sql.c

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18321 dcc99617-32d9-48b4-a31d-7c20da2025e4

overzealous trimming of cat.properties

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18313 dcc99617-32d9-48b4-a31d-7c20da2025e4

Use just one transaction, and inside an rstore editor no less, for fleshing env paths

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18312 dcc99617-32d9-48b4-a31d-7c20da2025e4

when the target for an event is no longer around, immediately invalidate the event and prevent the event from bubbling up for further processing

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18311 dcc99617-32d9-48b4-a31d-7c20da2025e4

missing semicolon

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18310 dcc99617-32d9-48b4-a31d-7c20da2025e4

thinko supporting multiple rules of the same type in in-line merge rulesets

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18303 dcc99617-32d9-48b4-a31d-7c20da2025e4

double-arg, backwards logic

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18300 dcc99617-32d9-48b4-a31d-7c20da2025e4

arg, missing semicolon

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18299 dcc99617-32d9-48b4-a31d-7c20da2025e4

deduplicate bibs going into the merge queue

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18298 dcc99617-32d9-48b4-a31d-7c20da2025e4

entry points for miker_'s batch marc editor

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18294 dcc99617-32d9-48b4-a31d-7c20da2025e4

when we have no target field, add the entire source field, even when we have a subfield designation

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18293 dcc99617-32d9-48b4-a31d-7c20da2025e4

Get rid of commas, apparently confusing; make action button label more generic / less "developery"

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18291 dcc99617-32d9-48b4-a31d-7c20da2025e4

more terminology cleanup and inline documentation

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18290 dcc99617-32d9-48b4-a31d-7c20da2025e4

new record merge UI.  needs r18287, r18288.  I'll backport them all together

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18289 dcc99617-32d9-48b4-a31d-7c20da2025e4

vertical bib summary (mainly for use in the Merge Record interface).  I tried using just bib_brief.xul with document.loadOverlay to choose between bib_brief_overlay_vertical.xul and bib_brief_overlay.xul based on a param, but was running into too much pain going that route.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18288 dcc99617-32d9-48b4-a31d-7c20da2025e4

allow util.deck to be instantiated with either a xul deck object or a xul deck id

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18287 dcc99617-32d9-48b4-a31d-7c20da2025e4

add option for hiding entire embedded browser toolbar

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18286 dcc99617-32d9-48b4-a31d-7c20da2025e4

improve labeling and documentation

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18285 dcc99617-32d9-48b4-a31d-7c20da2025e4

Be sure to copy new JS files from OpenSRF on a FULL update/install

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18282 dcc99617-32d9-48b4-a31d-7c20da2025e4

now that we are working again, hush the warnings (osrfLogInternal to the rescue)

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18281 dcc99617-32d9-48b4-a31d-7c20da2025e4

and ... use a function name that exists

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18280 dcc99617-32d9-48b4-a31d-7c20da2025e4

rearrangement for variable existance

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18279 dcc99617-32d9-48b4-a31d-7c20da2025e4

skip unknown order_by entries instead of erroring

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18278 dcc99617-32d9-48b4-a31d-7c20da2025e4

Incorporate upgrade # 0433 into the consolidated upgrade script.

M    Open-ILS/src/sql/Pg/1.6.1-2.0-upgrade-db.sql

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18277 dcc99617-32d9-48b4-a31d-7c20da2025e4

avoid race condition with post-save patron editor refresh and replicated databases

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18274 dcc99617-32d9-48b4-a31d-7c20da2025e4

make the source selection less confusing, and improve some wording in the template section (more to come)

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18272 dcc99617-32d9-48b4-a31d-7c20da2025e4

dogfooding cleanup

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18271 dcc99617-32d9-48b4-a31d-7c20da2025e4

initialize these lists just once to prevent display glitch

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18269 dcc99617-32d9-48b4-a31d-7c20da2025e4

Working template-based batch bib updater!

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18264 dcc99617-32d9-48b4-a31d-7c20da2025e4

Tighter verbose format for skipped POs

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18262 dcc99617-32d9-48b4-a31d-7c20da2025e4

supply a dummy leader, as required by the Perl MARC modules

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18261 dcc99617-32d9-48b4-a31d-7c20da2025e4

Method registration cleanup

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18260 dcc99617-32d9-48b4-a31d-7c20da2025e4

Clean up method registration sigs/descs

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18259 dcc99617-32d9-48b4-a31d-7c20da2025e4

Pull out into a separate function: the code in SELECT() that builds a
comma-separated list of ORDER BY expressions from a JSON_ARRAY.

Invoke that function, not only from SELECT(), but also from the
buildSELECT() function.

As a result, the select methods will be able to use the same array
syntax as json_query for ORDER BY clauses, as an alternative to the
existing hash syntax.

M    Open-ILS/src/c-apps/oils_sql.c

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18258 dcc99617-32d9-48b4-a31d-7c20da2025e4

wait to run-pending if a specific granularity used

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18256 dcc99617-32d9-48b4-a31d-7c20da2025e4

mod_perl handler to allow batch update bib records from an ephemeral template

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18255 dcc99617-32d9-48b4-a31d-7c20da2025e4

mild fixes for a/t interface admin interface.  sort by def name instead of hook, which probably makes more sense to a human.  hide the opt-in and max-delay columns to free up some badly needed horizontal space.  use percentage width for name column, which acts a lot like 'auto', but allows the user to manually resize

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18253 dcc99617-32d9-48b4-a31d-7c20da2025e4

typo

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18251 dcc99617-32d9-48b4-a31d-7c20da2025e4

Be a little more tolerant of aberrant data when clearing the holds shelf

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18249 dcc99617-32d9-48b4-a31d-7c20da2025e4

fix logic error where we were just testing for the presence of a user setting instead of its value

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18247 dcc99617-32d9-48b4-a31d-7c20da2025e4

moving users in and out of groups produced dialogs that expected vertical patron summaries.  this fixes that

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18245 dcc99617-32d9-48b4-a31d-7c20da2025e4

add a method to overlay a special bib container full of records (or bucket + template)

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18244 dcc99617-32d9-48b4-a31d-7c20da2025e4

Tidied up buildSELECT() a bit:

1. Sprinkled the const qualifier here and there.

2. Moved some variable declarations to get them closer to the point of
first use, and to limit their scope.

3. Renamed some variables to better reflect their meaning.

4. Split a couple of variables into multiple variables, instead of using
them for multiple unrelated purposes.

5. Plugged a memory leak in the case of an error return.

6. Added comments, including a Doxygen-style comment at the top of the
function.

M    Open-ILS/src/c-apps/oils_sql.c

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18240 dcc99617-32d9-48b4-a31d-7c20da2025e4

Avoid scary SSL / HTTPS errors in Apache configuration

When port 443 is the last listener port, Apache generates lots
of "unknown protocol speaking not SSL to HTTPS port!?" errors in
the logs - which are scary, but harmless. Putting port 80 last
avoids those errors entirely, per http://wiki.apache.org/httpd/InternalDummyConnection

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18239 dcc99617-32d9-48b4-a31d-7c20da2025e4

do not apply superpage limit inside bib search joins

Oddly, empirical evidence suggests that this might actually
be faster, and if this pans out, search results will definitely
be more complete.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@18233 dcc99617-32d9-48b4-a31d-7c20da2025e4

progress to ProgressDialog means total, not incremental

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18231 dcc99617-32d9-48b4-a31d-7c20da2025e4