This is a testing relic that nobody wants.

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

deleting items now implemented. All four of the CRUD family are done!

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

Fix problem identified by Bill: checkboxes and line numbers had gone away

Now they're back

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

quieter debug console logs and a little cleanup ahead of next task

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

Merge branch 'flattener-grid-sorting' of git://git.evergreen-ils.org/evergreen/equinox into flattener-grid

Merge branch 'flattener-grid-sorting' into flattener-grid

improvements to flattenerfilterdialog: use autowidget better

For explanation, see my last commit in this branch, plus this
[edited quickly] conversation:

08:44 < eeevil> senator: re "the solution", I think popping the field
off the path if path.length > 1 is universally right -- if we
can teach autowidget how to give us a downstream value (by passing
it shortname) instead of, say, an aou id automagically
08:45 < eeevil> senator: which I think is what you were planning ... I
just can't think of a reason to make the stop depth
configurable
10:57 < senator> eeevil: example of when i think it's not right:
10:58 < senator> core class acp, path call_number.record.tcn_value
10:58 < senator> there may be more... likely... examples, but sometimes
you'll actually want to deal with the end field
10:59 < senator> no?
11:00 < senator> one other idea i had, instead of making the stop depth
configurable, was to pop the end off iff the last field is the selector for
its class
11:01 < senator> then it seems fairly certain that an autowidget on the
preceding class would be the right thing. if you agree that's
the above is sane, i think i could live with that instead of
configurable stop depth
11:11 < eeevil> senator: call_number.record.tcn_value is a great example
selector came to mind for me too, I like that
11:12 < senator> very good, will make that so
11:12 < eeevil> senator: however ... selector would have to match the
leaf field, right?
11:14 < eeevil> senator: maybe that's still the answer ... a way to say,
on the <th>, that the named field is the selector for the class
in this instance, and if that's not there then look for an
IDL selector, and if neither of those is true, just show the
leaf in a simple widget
11:19 < senator> eeevil: i can't say i'm following you this time re
selector and leaf field
11:19 < eeevil> senator: for instance, shortname is the selector for
aou.id, but that -^ would let you have <th field='name' selector='true'/>
(maybe) and get a dropdown of names, instead of shortnames
11:20 < senator> oh, that is taking things a step further than i had in
mind.  you can tell autofieldwidget acp.circ_lib and get a
dropdown of shortnames, or you can tell it aou.shortname (or
aou.name) and get a text widget
11:21 < senator> getting a dropdown of, say, long names would seem to
mean teaching autofieldwidget a new trick, or developing
something new, which might be nice in the long run, but maybe
more new, which might be nice in the long run, but maybe
more effort than we need right now? unless i'm missing
something
11:21 < eeevil> senator: ahh... ok, so ... with a path of
acp.circ_lib.name, you get a text field, and with acp.circ_lib.shortname
you get a dropdown of shortnames?
11:22 < senator> right. not perfect, but faster to make reality, and
maybe a user's acp.circ_lib.name column shouldn't be a
filter:true column anyway
11:22 < eeevil> +1
11:22 < eeevil> senator++
11:23 < eeevil> since we're shoving the pkey in behind the scenes, would
that end up being the filtered column?
11:24 < eeevil> or are we just telling autofieldwidget that we want the
                shortname instead of the id as the value
11:25 < senator> when building the filter from the autowidget's
selection, we'll take the display value (or whatever that
attribute is called) instead of the item identifier like we would
for pcrudfilterdialog,
11:25 < senator> so our filter result is {"circ_lib_short_name": "BR1"}
11:25 < senator> which is what flattener wants. sound good?
11:32 < eeevil> senator: sounds like it'll work perfectly

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

column picker onSortChange support added to flattener grid

Signed-off-by: Bill Erickson <berick@esilibrary.com>

GridColumnPicker sort repairs

Ensure that the order of sort fields is correct when persisting them.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

_finishStartup works; todo: add the sorting

Signed-off-by: Bill Erickson <berick@esilibrary.com>

this works? sorta? maybe?

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

experiment 1

Signed-off-by: Bill Erickson <berick@esilibrary.com>

This is what victory looks like!  FlattenerFilterDialog

Heh, ok, it's not total victory.

FlattenerFilterDialog works (it's a small subclass of
PCrudFilterDialog), and it  cooperates with FlattenerGrid.  That's the
good news.  But the way it works is sometimes less than ideal due to
design under-think.

Imagine core class acp, and you have a map for the flattener with these
columns, because these are the things you want to see in the grid:

{
    "barcode": "barcode",
    "shelving_loc": "location.name",
    "shelving_loc_owning_lib": "location.owning_lib.shortname"
}

The FlattenerFilterDialog figures out the terminal fields for each of
those paths, which are acp.barcode, acpl.name, and aou.shortname.  It
naturally gives you the appropriate AutoWidget for each of those things.

Which is a TextBox. In every case.

Because all of those are text field.

See the problem?

They do /work/, but as a user you probably want to get an org unit
dropdown instead of being expected to type "BR1" or "ARL-ATH" or
whatever.

So I guess the way to make this better is to give an option in the
FlattenerGrid per-column (which will pass this option onto
FlattenerFilterDialog) that tells us how many steps /from/ the end to
stop when we're looking for a field to inform our choice of AutoWidget?
So instead of [acp.]location.owning_lib.shortname = aou.shortname, if we
stop at one step from the end we get acpl.owning_lib instead, which
would give us a much more appropriate autowidget.

yeah.

that'll be the solution

Also, as a bonus, you can now leave out the header name between <th> and
</th>, and it will be filled in from the IDL.  This is surprisingly
often not what you really want, but it's a better reasonable default
than the raw field name.

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

put name= back into Content-Type for IE8

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

Merge branch 'flattener-grid' of git://git.evergreen-ils.org/evergreen/equinox into flattener-grid

these things:

    1) actually use map key, refetching if it has expired
    2) make csv output actually work
    3) make htmlish output actually work, mostly, but still has
        unwanted '0' after the end of the html output?

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

AutoGrid reloads structure after columnHelpers loads

Otherwise, the newly added columns are not recognized by the grid
(unless you happen to have the column picker enabled).

Signed-off-by: Bill Erickson <berick@esilibrary.com>

GridColumnPicker avoid sorting when no settings have persisted

Avoid sorting cells whose _sort value is null/undefined, which can
happen when no column settings have been persisted for the grid.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

GridColumPicker hides sort selection for non-sortable fields

Query the grid to determine when a column is sortable.  When not, simply
hide the sort priority selector.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Add CSV output (Content-Disposition: attachment) to FlatFielder mod_perl

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

create and edit working. not quite delete yet. also,

broke out line numbers and selectors into a separate dojo module for
flattener grid and autogrid to share. don't quite have it right though,
as flattenergrid works but for autogrid you can no longer see the lineno
and selector columns (although they are present in
grid.structure[0].cells[0])

hrm

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

have editing working. at. last. create should be next.

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

Grid column picker sort direction

Added support for specifying a descending sort direction.  Selecting a
positive sort priority results in an ascending sort.  A negative sort
priority resutls in a descending sort.

Sort order is still determined by the (absolute) value of the sort
priority.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Add an HTML-output stylesheet and teach FlatFielder to use it

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

AutoGrid column picker sort handler

Adds a default sort handler for column picker onSortChange events.  The
handler simply reloads the grid with the new columns and assumes all
columns are fields on the base class.

Also supports a user custom onSortChange handler for more complicated
situations.

Initial grid load now happens after the column picker loads when the
column picker is enabled to sure the grid only loads once on the initial
load.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

GridColumnPicker mult-column support selectors

Adds a number spinner to each sortable column in the grid column picker.
A Sort Priority of '0' means no sorting is applied, a priority of 1 is
highest priority, 2 is second, etc.

Adds support for an onSortchange handler, which is passed the list of
fields to sort in order.

Adds support for an onLoad handler, which is called after the initial
retrieval of the user persist settings.  This is important for the
initial grid load, which may need to wait until after the sort columns
are known (to avoid re-fetching on the initial load).

Signed-off-by: Bill Erickson <berick@esilibrary.com>

GridColumnPicker terminology repairs

Change the configuration attribute columnPickerPrefix to
columnPersistKey.  Most notably, it's not a prefix (it's a suffix), nor
is its pre/suffixiness relevant to the user.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Fix a bug in magic scroll; start hooking up Edit{Pane,Dialog}

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

Set the columnPickerPrefix on flattener_test

To persist settings, you'll need to do this:

INSERT INTO config.usr_setting_type
    (name, label, datatype)
VALUES (
    'ui.grid_columns.conify.flattener_test',
    'Flattener Test Columns',
    'object'
);

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Added GridColumnPicker support to FlattenerGrid

The columnpicker is assumed to be enabled.  Even when the user setting
required to persist the settings does not exist, the columpicker can be
used on a per-page basis.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

GridColumnPicker support column ordering, cleanup

This adds support for detecting and saving state information for
re-ordered grid columns.  Also, with each instantiation of the column
picker dialog (via control-click), the dialog is re-rendered to match
the order of columns in the grid in case they have moved.

ColumnPicker now produces a useful console message when the user setting
required for persistance does not exist.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

sorting bug fixed, FlattenerGrid's API stabilized

when I say its API is stablized I mean how you build the map, using the
<th> elements and the mapExtras attribute on the <table> element.

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

paging kinda works, but sorting is not right. not sure what to blame yet

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

progress, still doesn't page past first set of results yet

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

it LIIIIIIVVVVVVVVVVVESSSSSSSSSS!

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

:wq;lsdkfjpasdhflajhf

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

wth
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

early tests
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Amend design doc with an example that you can actually test without...

... adding new pcrud controllers to your IDL.

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

Handle all ways of getting arrays at the leaves

We can get arrays of values be either:
 - ending on a $field within a has_many reltype
 - passing through a path that is a has_many reltype

This detects either and array-ifies the value on the flat row.

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

Check auth on flattener search calls

Otherwise, the auth will fail at a lower level (pcrud) and result in
sending a less-than-useful error message to the caller.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Just a little syntax cleanup and the like to make things run as intended

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

Tell apache about the flattener

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

mod_perl service providing JSON and XML flattened, fleshed pcrud search results

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

Flattener stores might_have links as arrays

E.g. "aou", {"addrs" : "addresses.id"}

{ "addrs" : [ 1,2,3 ] }

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Flattened searching: generalized data retrieval via public service

For a better overview of what this feature is about than what I could
write here, see docs/TechRef/Flattener/design.txt in this commit.

This is the first new feature (as far as I know) to take advantage of
PCRUD fleshing. Very briefly, imagine issuing a query to PCRUD with lots
of arbitrarily deep fleshing, and getting back a set of flat rows with
the fields you need for display/editing/whatever all neatly picked out
as if ready to be displayed in a table or grid-based UI.

A Dojo grid that knows how to use this can potentially replace and avoid
lots of relatively complex (AutoGrid + custom middle layer
methods)-powered interfaces.  AutoGrid interfaces that just work with
one fieldmapper class at a time, more or less, can just keep doing what
they're doing. Little or no advantage to switcihing to flattened data
in that case.

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

.gitignore update - ignore more built files

Signed-off-by: Dan Scott <dan@coffeecode.net>

TPAC: Tiny fix for + instead of _ string cat operator

Quiet down, Apache logs.

Signed-off-by: Dan Scott <dan@coffeecode.net>

TPAC: suppress warning about splice() when the array in question is too short

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Shooting 2 obvious thinkos d-e-d dead. Pardon the rogueness.

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

Fix Auditor Boost upgrade script

CREATE FUNCTION needs an OR REPLACE included.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

TPac; credit card assumes all xacts if none selected

For consistency with the initial payment form and to preserve backwards
compatibility, if no transactions are selected, assume the user wants to
pay for all transactions.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

TPac; credit card payment "Processing..." page

To prevent impatient patrons from re-submitting credit card payment
request forms, insert a Processing... page between the initial form
submission and the actual payment processing.

Steps 3-5 are new or modified.

1. User selects transactions to pay
2. User fills in payment form and Submits
3. Payment form data is cached in memcache with a temporary token
4. Processing page is displayed with message about how processing the
   payment can take time, do not refresh, etc.
5. Page is refreshed via http meta refresh to submit the form using
   the temp token.  The page is not visibly changed, though, since it
   takes time for the form submission to begin returning data.
6. User is directed to the payment results page.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

TPac; support context-genereted refresh content

If the tpac context object contains a "refresh" attribute, it will be
used as the content attribute in a <meta http-equiv="refresh" ...>
element in the page head.  If present, it will override the default
refresh content for the authentication timeout.

While we're in there, let's use the more correct form of the content
attribute:  content="x; url=foo"

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Add the session to Circ set_audit_info calls

Because I apparently wasn't thinking about how I wrote the apputils piece
when I added them.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

LP914821 template selection should be sticky

in Unified Volume/Copy Editor.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

AutoSuggest: Have the server cache for us if we're not using cookies

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

AutoSuggest: Don't try to store cmc and cmf caches in a cookie. Too big.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Fix a couple of templates with multiple class attributes in tags.

There should only be one class attribute on a HTML tag. However,
multiple classes can be combined in a single attribute argument.

See: http://www.w3schools.com/tags/att_standard_class.asp

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Backdate into grace period - include last second

The last second of the grace period is ignored on backdated checkins, but
should still count as "in" the grace period.

This fixes that.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

AutoSuggest: don't force-scroll browser; that's irritating

By default, Dijits and things that inherit from Dijits (at least in
Dojo 1.3.3 that we're still using), have a true value for an attribute
called scrollOnFocus.  This makes your browser jump to ensure that the
widget that just received focus is in view.

You don't always want this though.  Try expanding the extras on the
record detail page of the TPAC without this commit, but with AutoSuggest
enabled, for example.

This commit sets the scrollOnFocus attribute to false for the
AutoSuggest widget.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Propagate search location on "Another Search" link

In commit 27b2037d72b I had intentionally prevented "Another Search" on
the results page from propagating the current search location, in the
belief that sites would want to use that as a way of starting a brand
new search session; after all, both the "Advanced search" and "Search
toolbar" offer propagation of the current parameters.

However, a number of people including Kathy Lussier and Ben Shum
indicated that they expected "Another Search" to maintain scope as well,
so we will propagate state here.

Sites may want to customize this link or the top-left logo link if they
want to provide a "Start a new search session" option.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Prevent AutoGrid death using a certain helper with null OU fields

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Stamping upgrade script for reingest betterment

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

New metabib.reingest_metabib_field_entries func

In addition to acting like the old one by default, support new params:

skip_browse - Skip autosuggest (re)population
skip_facet - Skip facet (re)population
skip_search - Skip search (re)population

Can be specified by name with PostgreSQL named param features:

metabib.reingest_metabib_field_entries(bib_id, skip_search := true);

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Stamping upgrade script (and updating 2.1-2.2 version upgrade script)

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

Call set_audit_info and clear_audit_info DB funcs

Set whenever we can (including automatically via pcrud)
Clear whenever we disconnect (hopefully) from a location we set from.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Auditor Function Overhaul

Alter create_auditor_history to include two new columns:
    audit_user - To refer to the user who made this change
    audit_ws - To refer to the workstation they were logged into

Alter create_auditor_func to include the new columns, as well as make it
stop assuming that the auditor table and the table it is based on use the
same column ordering. In additon, make it use OR REPLACE so it can be run
again later.

Alter create_auditor_lifecycle similarly to create_auditor_func, sans the
OR REPLACE functionality.

Add five new functions:

set_audit_info - To set the user and workstation IDs
get_audit_info - Used by the auditor functions to retrieve said IDs
clear_audit_info - To remove the stored IDs for whatever reason
fix_columns - To add/remove/update columns that have changed since any
    auditors were initially created
update_auditors - To run a full pass of dropping all lifecycle views,
    update for all column changes, re-create the auditor functions, and
    re-create the auditor lifecycle views.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Unwrapped auditor upgrade script

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

TTPAC material type icon compression

When cleaning up the images directory for TTPAC, I discovered
that the mattype icons had been saved with a poor compression
scheme and where taking up nearly 3MB, which was just too
high for a small set of icons.

I ran them through Photoshop CS5 and saved us a whopping 2.85MB
of disk space used by the images.  The entire lot decreased from
3MB to just 152KB.  Average icon size is now just about 1KB.

This should speed up load times when these images are displayed.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Password Reset Submit buttons should look like other OPAC buttons

This branch stylizes the submit buttons on the Password Reset form
in TTPAC so they match the other types of OPAC buttons throughout
the catalog.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#707167 Make open-ils.actor.usrname.exists not ignore deleted users

Currently, open-ils.actor.usrname.exists filters out deleted users.  This
results in a user recieving indication as to why a user update
failed, when the reason is the username they're attempting to use
is in use, but is "deleted" in the database.

This branch makes open-ils.actor.usrname.exists look at deleted users
since they still prevent re-use of the delted user's username.

More information is in the original bug, https://bugs.launchpad.net/evergreen/+bug/707167.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

New features for telephony currently in use at KCLS

I must ask the community's forgiveness for sitting on this for so long.
This code was developed in fits over time, and it had to be disentangled
from some site-specific configuration and workarounds before it could
be usefully published.

Enhancements over what's already committed to master include:

- a feature that allows an Evergreen system to query the PBX (where
  PBX here refers to an Asterisk system plus the eg-pbx-* scripts from
  Evergreen - particularly eg-pbx-mediator.pl) for the A/T event IDs
  associated with failed notifcations, so that those A/T events can be
  "rolled over" into new ones with a different event definition, and
  thus a different notification method

- holiday awareness in eg-pbx-allocator (the piece run by cron whose
  main responsibility is dripping files from a staging directory into
  Asterisk's real spool for call files).  There's a script (set_pbx_holidays)
  that can be run by cron on an Evergreen system to periodically update the
  PBX's table of holidays based on a given org unit's closed_date ranges.

- smart retry.  In certain situations, if you put too many callfiles
  into Asterisk's spool at once, Asterisk will try to make too many
  calls at once, and all such calls just fail.  That's what the
  allocator is meant to prevent.  Smart retry is about moving calls that
  have been tried once, and will be retried again later due to resulting
  in a busy signal or other problem, out of the spool to make room for
  other calls that could be made in the mean time.

- miscellaneous helper scripts and tricks for spreading more
  reportable/greppable information among A/T output and cleaning up
  callfiles that result in no-op's for Asterisk. Lock file checking for
  the allocator. Etc.

I also have tried to throw in some useful documentation.  I admit things
look a bit jumbled, but I'll be as available as I can make myself to help
anyone who steps up to clarify documentation or improve code in order to
make general adoption of these features a little easier.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Use getItemType() instead of getForm() in searchBarInit().

This seems to actually fix the problem with the item type limiter
being lost when going from advanced search to the results page.

Looks like the wrong method was being used to set the form_selector
value in search_bar.js. It was being set to the web form itself and
not to the value of the it (item type) parameter.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

When sorting a search results set the isFilterSort variable is passed
to searchBarSubmit.  In side searchBarSumbit there is a call to the
opac_utils.js function clearSearchParams function.  This removes all
search paramaters when sorting a result set, which is not the desired
effect.  I have wraped clearSearchParams in a check to make sure
isFilterSort is not set before calling the clearSearchParams function,
and this check ensures that the current search's paramaters are passed
to sort operations.

Signed-off-by: Liam Whalen <lwhalen@evergreen-dev.catalogue.nrcan.gc.ca>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP918020: Fix regression in biblio.extract_located_uris

The 2.0-2.1 upgrade script contained an old version of
biblio.extract_located_uris that functioned poorly. Fix the 2.0-2.1
upgrade script and ensure that sites upgrading to 2.1.2 get the fix as
well.

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

lp928937, null date sorting

Changes util.date.db_date2Date so that nulls are not converted into the epoch.
This fixes date sorting in XUL-based lists in the staff client such that unset
dates are listed before actual dates.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Avoid JSPAC hold placement interface problems when no holdable copies

This addresses a bug brought to my attention by John Jones.

When you're trying to place certain holds (at least title level holds) on
records for which there are no holdable formats (no descriptors as returned by
open-ils.search.metabib.record_to_descriptors), then
holdArgs.recordDescriptors in holds.js can be an empty array.  There is
a line of code that assumes this will never happen.

The end user-visible result of the javascript failing here is an alert()
dialog about phone number format, but this is a red herring.  That only
happens because the problem code is in one big function that has too
many responsibilites.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

TPac: avoid showing replaced addresses

Replaced (negative-id) addresses should never be visible to the patron.
Remove them from the collected list of addrs.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Tpac (etc): improve payment history sorting/paging

Payments within a transaction will have the same timestamp.  With a
large enough set of such payments, sorting by timestamp alone when
retrieving payments is not enough to guarantee unique pages of payments.
This adds a secondary sort on payment ID as a tie-breaker.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Fix AccessControl plugin extraction

Apparently they ditched a containing folder in the zip...lets autodetect
that from now on.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

patron search result order sorting

The default patron search "order by" on the server side is last name, first
name, and actor.usr.id.

The staff client intended to request last name, first name, middle name, and
date of birth (when actor.usr.id to be tacked on in the middle layer), but a bug
prevented this.

This fixes that.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Paging repairs for Event definition admin UI

If there are multiple event definitions with the same name that lie along
the page boundaries, it's possible to page through event definitions
seeing some def's multiple times and others never.  This is a result of
sorting solely on event def. name when fetching event def's for display.
This patch adds secondary (and tertiary, and quaternary, and quinary)
sort columns as tie breakers to guarantee a unique set of defs across
pages.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Do not exclude "identifier" facets by default

No longer ignore "identifier" fields when collecting and caching
facet data.  This change causes all config.metabib_field's that are
marked as facet_fields to be taken into consideration.

Apart from marking a field as facet_field=false, it's also possible to
control which fields are visible within the jspac and tpac (pending LP
911908)

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Fix some small errors in Apache log for staff client splash page

Fixes the path to the background image in the splash page's search
box.

Also removes references to javascript files that don't seem to be
present.

Signed-off-by: Warren A. Layton <warren.layton@gmail.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

TPAC: Prevent OU-invisible copies from being displayed to public

Per LP# 959577, "We've found that copies from OU's with OPAC visible set
to false are displaying on the record details page of tpac."

This fixes that by including the pertinent clause in mk_copy_query()'s
version of what in-db unapi does. Some day we will unite under the same
logic and be stronger!

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Disable mod_deflate on XMLENT filtered files

Default configs for Ubuntu, at least, enable mod_deflate too broadly, and
XMLENT filtering needs to happen before mod_deflate kicks in.

Because we can't change the filter order, instead disable mod_deflate for
those files we have enabled XMLENT filtering, by use of the no-gzip ENV.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Stamping BluRay videorecording format upgrade

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

Teach Evergreen about Blu-Ray VR Format

Also, reingest blu-ray records in an update script to populate it.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Layout tweak to return-to-search-results staff button

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Staff client: add a back-to-search-results button in the chrome that ...

... wraps the OPAC (whether JSPAC or TPAC).

You don't really need this when in OPAC view, because the OPAC page
itself will have a link that does the same thing.  However, if you're
browsing through records in a search result set one at a time in some
other view, say the MARC view, then this button can come in handy.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Stamping upgrade script for ACQ/Vandelay integration branch

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

Vandelay-based record matching and import for Acquisitions

From the Lauchpad description:

In its current form, Acquisitions record importing is inflexible and
unforgiving. The user is given practically no control over how inbound
records match against existing records nor how to deal with record
collisions. This proposal attempts to resolve this problem by leveraging
Vandelay for matching, merging, overlaying, and importing acquisitions
records.

Suggested work flow:

1. Any time an ACQ record should be loaded into the catalog or simply
   linked to an existing catalog record, the user is presented with an
   additional set of Vandelay upload options to control the matching and
   import behavior. This would include the vendor file upload UI, PO
   activation, and PO "Load Bibs and Items". The Vandelay options would
   include the full set of Match Sets, Merge Profiles, import options, etc.
   the user normally has.

2. Once options are selected and, when applicable, the MARC file is uploaded,
   the acquisitions records are added to a new type of ACQ Vandelay queue and
   processed like any other Vandelay records.

3. Vandelay will be taught to recognize ACQ records and update the lineitems
   to reflect any imported/linked catalog records.

4. Any records that fail to import will be available for manual inspection
   within the Vandelay interface an can be re-imported from there.

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

AutoGrid line number CSS application cleanup

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

add Line# column to autogrid

hidden by default, but we need to make it show up in autogrid's column picker

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

make ordinal column work in Holdings Maintenance

brute force kludge

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

adds a Line# column to all xul lists

displays the ordinal position of the row

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

row_node vs my_node in util.list

Both of these should be holding the same xul treeitems, and I don't recall
wanting different names here, so I'm renaming both to treeitem_node.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

pcrud fleshing bug fixes

1. Avoid perm checks on org tree root lookup.
2. Test for child objects to flesh into has_a / might_fields differently
to avoid segfault when kids array is empty

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Fix a bug spotted by Jason Stephenson and Thomas Berezansky

I was leaving local variables called 'cache' high and dry in many
functions calling the initSessionCache() function that was introduced to
reduce code repetition.

This seems to fix that

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