provided ability to append the table rows from one edit pane to an existing table.  this is useful for inserting edit panes for different objects into a single, consistently formatted table

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

repaired some mis-named fields; added an ADMIN_* version of invoice method pcrud perms

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

initial invoice sub-module

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

added ability to flesh PO and PL in lineitem retrieve

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

be smarter about holdings fleshing, particularly where only URIs are requested

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

stumbled upon an incorrect sequence

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

better regexp with new paging functionallity

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

removed call to removed function

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

in related viewer mode, go ahead and show the li attrs in the detail page since we're using the bib-level attrs for the summary display

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

add embedded holdings paging support to biblio-record_entry type unAPI tag URIs -- also works for holdings_xml format directly -- and teach the local call number BibTemplate opac slot how to use this (show first 10 local call numbers, instead of all of them)

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

add metarecord support to BibTemplate

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

related items now returns LI's that may be attached to a selection lists only (no po)

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

de-dupe li ids in picklist li fetcher.  if an LI has multiple attrs for the sort field, the join collects 1 row per attr, resulting in duplicate attrs

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

Added a "View / Place Orders" option to the exta actions drop-down for a
bib record in the OPAC.  Lineitem related LI's page can now take a
bib ID as the param (instead of lineitem).  For consistency, show
title, author, etc. from the catalog record (via BibTemplate) instead
of the selected lineitem.

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

patch from Ben Ostrowski to address a i18n-affecting typo in stock group creation

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

Create a separate function for adding datatype information
from the database to the IDL information.  That way we can
call it in other contexts, e.g. the forthcoming
query set server.

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

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

Replace almost all remaining reference to the MODULENAME macro
with references to a corresponding variable name.

This change is part of a project to split up the oils_cstore module
into an interface module (consisting mostly of osrfAppChildInit(),
osrfAppInitialize(), and osrfAppChildExit()) and an implementation
module (consisting of pretty much everything else).  The implementation
module will then be available for reuse in other contexts.

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

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

flesh the copy location and set a tooltip for its flags

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

localized bool values for display

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

display the shortnames for orgs in the Item Details interface (and set the full names as tooltips)

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

typo

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

localized date/time values for Item Details interface

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

Reduce dispatchCRUDMethod() to little more than a case structure,
which delegates all work to other functions (instead of doing some
of the work itself).  Besides making the code arguably tidier, this
change will make it easier to isolate much of the logic to a separate
module.

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

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

add format selector for e-books

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

copy-n-paste-o on variable name

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

when auth session times out with embedded catalog, launch the xul login dialog (instead of flopping back to the home page)

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

Flesh and show cancel_reason in the user request rejected template.  I'm just correcting the existing upgrade script and seed data rather than roll a new one.

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

augment the xulG that the OPAC wrapper gives its content

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

Or unit setting acq.holds.allow_holds_from_purchase_request: Allows patrons to create automatic holds from purchase requests

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

added wrapper to new XUL get_new_session call.  when embedded ui's lose their session, launch the staff client's login dialog instead of the local web-only dialog

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

respond_complete before processing a/t events to the caller does not have to wait

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

Adds a xulG.get_new_session() convenience function that can be called by embedded dojo interfaces, etc.

Side effect: Sets a session cookie and updates the global OpenILS.data store with the newly acquired authtoken.

Return Value: True if a new authtoken was acquired, otherwise False.

Optional Param: Object with optional keys 'callback' and 'operator_change'.

If 'operator_change' is passed in with a value of true, then the dialog will resemble the Admin->Operator Change dialog, but it is not yet completely implemented (doesn't preserve the existing authtoken).  Otherwise, we get the normal Session Expired dialog.

Callback may be a function that resembles this:

    function( success, authtoken, authtime ) {
        return success; // whatever gets returned here will also be returned by xulG.get_new_session()
    }

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

Patch from Joe Atzberger: Last batch of method reg docs and perltidy.

Note spellcheck now returns [] immediately if no term is given.
No point initalizing and fetching stuff with no term.

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

adjust stored proc to use the appropriate STABLE or IMMUTABLE flag for speed

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

Use the AssetCommon version of update_fleshed_copies so we can stay within the same transaction.  However, no more magic hold retargeting.  The CRON job can take care of any holds that do not get cancelled

From my open-ils.acq.lineitem.cancel branch

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

Use utility method for firing events

From my open-ils.acq.lineitem.cancel branch

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

fields for supporting bib record ownership and sharing

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

Patch from Joe Atzberger:  Large overhaul of misleading and erroneous method reg notes.

Including one deprecated method and its current replacement.

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16093 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Joe Atzberger:  Cleanup subs in Actor

For the first, @xact was never used.
The 3 part conditional only had two different outcomes.
There is still a problem with user_transactions reproducing a lot
of the functionality of user_transaction_history, which it already
calls to do the real query.  But it doesn't reproduce all of it.
Recommend relying on user_transaction_history as much as possible,
but that kind of work is out of scope for this review.

For the other, remove unused var and unnecessary array.
You don't need to build an array when all you do at the end is count its elements.

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16092 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Joe Atzberger:  A great blast of perltidy

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16091 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Joe Atzberger:  More method reg docs in Biblio.pm and Actor

Signed-off-by: Joe Atzberger <atz@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16090 dcc99617-32d9-48b4-a31d-7c20da2025e4

added claim sub-module.  Added API call to retrieve claim-ready items.  Added API call to initiate a claim for a given lineitem_detail or add a claim event to an existing claim

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

some stubs for future facet UI support

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

Acq: ML logic to support the "not" operator in unified acq search queries

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

fixed function name in lineitem price setting code.  update local copy of lineitem after price changes to prevent clobbering of value

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

added a view which returns all lineitem details that are ready to be claimed.  returns po, lineitem, lineitem_detail, and claim policy action data

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

missed the environment when upgradeing the worksheet before.  added copy location to lineitem worksheet template

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

give the trigger autocreate wrapper a clearer name

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

fire hold_request.cancel.cancelled_order events for any cancelled holds after transaction commit

From my open-ils.acq.lineitem.cancel branch

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

Cancel holds targeting the bib associated with a lineitem if it has become deleted.  Note concerns about rolling back transactions and using open-ils.cat

From my open-ils.acq.lineitem.cancel branch

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

don't make the call to open-ils.cat if there are no real copies to delete.  I'm assuming we want to be able to call this method against lineitems that might not even be associated with a purchase order, much less have copies, etc.  Need to think about that permission check at the beginning which is using purchase order -> ordering agency as a context org

From my open-ils.acq.lineitem.cancel branch

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

param tweak to get copies to flesh on lineitem details, and yes, open-ils.cat will handle volume and bib deletion, as well as hold retargeting.  But we still need to cancel holds in the case where the bib gets deleted.  If the bib isn't deleted (because other copies were added outside of acq), holds that came from acq may or may not target those non-acq copies.  If they don't and any holds become unfulfillable (say due to hold permit policies), we'll just let them lay for now rather than try to figure that out.

From my open-ils.acq.lineitem.cancel branch

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

results are fancier in perl; adjust the result test from open-ils.cat

From my open-ils.acq.lineitem.cancel branch

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

Har, I looked in Lineitem.pm for an existing lineitem cancel method before I started, but not Order.pm.  Remove the one from Lineitem.pm and tweak the one in Order.pm.  Still need smoketest, etc.

From my open-ils.acq.lineitem.cancel branch

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

Rough cut of 'open-ils.acq.lineitem.cancel', which "Cancels a lineitem, any of its detail entries and corresponding copies and call numbers, and potentially related holds (if the bib becomes empty)."  Need to smoke test it, add hold deletion if the bib becomes empty (and add an org unit setting for optionally deleting the bib as well), and check to see if volume deletion is being handled automatically by open-ils.cat.asset.copy.fleshed.batch.update

From my open-ils.acq.lineitem.cancel branch

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

show progress dialog while waiting for the lineitem worksheet to load

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

added support for progress dialog messages in the markup

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

Seed data for org settings and override permissions for managing the
behavior when the systems finds a copy that could be used to fulfill a
hold at the requesting branch before the hold is placed.

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

fleshed out the docs for the open-ils.circ.hold.has_copy_at method

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

Acq: Indicate funds at stop/warning balance level with color and warnings

Specifically, in the lineitem detail (copy) interface,
where there are fund dropdowns.

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

added API docs for open-ils.actor.user.stage.create.  Added support for passing in staged statcats

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

added pcrud entries for claim_policy and claim_policy_action

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

Patch from Joe Atzberger to add additional code docs and more code cleanup

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

correct paging behavior with new staged search stored proc and results

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

syntax

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

syntax.  this time I broke the file :)

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

return the whole blob for an undefined field key

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

index is already there, do not recreate it

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

Acq: PO interface now tells you if activating a PO would breach fund limits.

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

add makefile and autogen support for the facet definition bits; more minor bug fixing of facet definition JS generator

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

moved to auto-grid for view funding-source UI (for paging, etc.).  added pcrud entries for funding_source credits

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

arg, typos

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

add initial cut of slim facet structure autogen script

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

reinstated the links for funding_source code in the fund allocations grid

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

moved to auto-grid for allocations and debits for paging support, etc.  hiding percent-based allocation option for now since they are not supported by the back-end

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

added pcrud entries for fund allocations and fund debits

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

Patch from Lee Dickens and Galen Charlton to correct a thinko in the recent change to pg_loader -- ignore auto-pkey tables, as those have had their sequences adjusted, um, automatically

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

consolidated the bib and authority record import logic to get the new functionality into the authority import process.  1match is possible if all matches point to the same record, not just if there is only 1 match.

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

removed some unnecessary intermediate code that snuck into the commit

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

forgot to change my version placeholder

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

added support for using alternate context-specific spell-check dictionaries w/ sample config.  now de-duping suggestions that only differ by case.  added a new boolean value to the response object to indicate if the original word was found in the dictionary, but still return alternate suggestions

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

picking an upgrade version number

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

paste-o

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

fix paste-o for facet_field (do we need facet_xpath?) and add in missing reference to id column

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

reorder these so that they work

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

seed data

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

the reason I wanted to pass in an editor here, so I could flesh the users for their home_ou's needed for the event

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

We'll be renaming this before it goes into master

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

use editor

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

create acq.created and acq.rejected events

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

more user request related templates, and add all of them to the db seed data.  misc fixes to db seed data, and i18n for previous user request data

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

Acq: In fund transfer, add destination amount field when warranted

When warranted means when source and dest fund have differing currencies.

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

In query.stored query: make from_clause nullable, so that we can
represent UNION, INTERSECT, and EXCEPT queries.

M    Open-ILS/src/sql/Pg/002.schema.config.sql
M    Open-ILS/src/sql/Pg/008.schema.query.sql
A    Open-ILS/src/sql/Pg/upgrade/0220.schema.query.nullable-from.sql

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

removed unused / nonfunctional method stub

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

Acq: fund transfer UI

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

Acq: this line unneeded; TagManager doesn't go here.

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

plugged in paging for viewing fund propagation results.  added context org and org child selector to rollver actions.  moved some css to external acq css file

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

When propagating funds: propagate the balance_warning_percent
and balance_stop_percent columns.

M    Open-ILS/src/sql/Pg/200.schema.acq.sql
M    Open-ILS/src/sql/Pg/002.schema.config.sql
A    Open-ILS/src/sql/Pg/upgrade/0219.schema.acq.propagate-fund-bal-pct.sql

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

more patron update docs

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

More method reg docs and a minor touchup.

$err variable was not used.
trinary op makes clearer that logger only gets called once.

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