Galen Charlton [Thu, 8 Oct 2015 21:28:53 +0000 (21:28 +0000)]
webstaff: improve MARC record deletion
The MARC editor now asks the user to confirm
whether to delete the record, and in the case of
deleting bibs, now uses open-ils.cat.biblio.record_entry.delete
so as to catch things like volumes still attached
to the bib and cancelling holds in the bib record.
Galen Charlton [Thu, 8 Oct 2015 16:43:42 +0000 (16:43 +0000)]
webstaff: tweak layout of the merge interface
The lead record is now displayed using the same sort
of tabset as the subordinate records. While, of course,
there can be only one lead record at a time, doing it
this way gives us two things:
[1] Display of the lead bib's ID
[2] Better vertical alignment of the MARC records
Galen Charlton [Thu, 8 Oct 2015 16:16:01 +0000 (16:16 +0000)]
webstaff: automatically advance focus in MARC editor
Upon completing a tag, indicator, or subfield code
input, focus will now automatically advance to the next
input, allowing users to enter records with less need
to use the tab button.
Adds a new scrollable-menu CSS class. It's (provisionally?) limited to
larger screens, w/ the assumption that smaller screens handle scrolling
for you.
New class is applied to org unit selects only so far.
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Conflicts:
Open-ILS/src/templates/staff/css/style.css.tt2
Galen Charlton [Thu, 8 Oct 2015 01:05:48 +0000 (01:05 +0000)]
webstaff: start teaching egHatch about sessionStorage
This patch adds getSessionItem(), setSessionItem(), and
removeSessionItem(), which are all wrappers around
$window.sessionStorage. This is done to support settings
whose values are sticky for the duration of a session,
which at present is the lifetime of the browser tab.
An alternative would be using session cookies; some user
testing is needed to see which lifetime makes the most
sense.
Galen Charlton [Wed, 7 Oct 2015 17:49:19 +0000 (17:49 +0000)]
LP#1503782: fix horizontal scrollbar in MARC import queue inspector
In the XUL client (and webstaff), attempting to view the contents
of a MARC record import queue displays a table that, due to
its width, will almost always have a horizontal scrollbar. However,
attempting to actually use the scrollbar often causes the
focus to jump to the top of the table instead. This patch
fixes that.
To test:
[1] Create a bib queue that has at least a dozen records or so.
[2] Inspect the contents of the queue, scroll to the bottom of
the table, and interact with the horizontal scrollbar.
[3] Observe that doing so typically results in jumping up
to the top of the table.
[4] Apply the patch, clear cache and refresh, and try step #2
again.
[5] This time, the horizontal scrollbar should act normally.
Galen Charlton [Wed, 7 Oct 2015 15:22:34 +0000 (15:22 +0000)]
webstaff: fetch MVR to help build record summary
Although this adds another OpenSRF request, using the
MVR rather than fleshing simple_record() gives us
two benefits:
* getting access to the edition
* fields like title and author are now displayed in
their proper case, rather than the normalized lowercase
supplied by simple_record().
Galen Charlton [Tue, 6 Oct 2015 18:24:16 +0000 (18:24 +0000)]
webstaff: teach MARC editor some more tricks
[1] Now accepts a flat-only attribute specifying that
the editor should be restricted to flat mode.
[2] Now accepts an embedded attribute specifying that
various stuff that isn't useful when the editor
is embedded in another control should be ignored.
[3] The width of the flat editor's textarea is now
bounded by its container.
Galen Charlton [Fri, 2 Oct 2015 20:50:03 +0000 (20:50 +0000)]
webstaff: more changes to holdingsSvc
- convert holdingsSvc from a singleton to a provider
of holdings service instance objects
- adjust the catalog app accordingly
- add a new egVolumeList directive that displays
a list of volumes attached to a bib record, including
owning library, call number, and copies
Galen Charlton [Fri, 2 Oct 2015 14:19:52 +0000 (14:19 +0000)]
webstaff: make overlay record modal wider
This patch defines a CSS class, eg-wide-modal, that can be
set as the windowClass of 'lg' Boostrap modals and increase
their width from a fixed width to 95% of the window.
Galen Charlton [Thu, 1 Oct 2015 20:31:08 +0000 (20:31 +0000)]
webstaff: move some "mark for" buttons to record summary
This patch moves the "Mark for..." overlay, volume transfer,
and conjoined items buttons from the MARC editor to
the record summary (next to the "Add To Bucket" button).
Galen Charlton [Thu, 1 Oct 2015 19:24:57 +0000 (19:24 +0000)]
webstaff: add drop-down for item type Z39.50 search field
It should be noted that this works *only* for OCLC, as
not all targets support use attribute 1001, and the list
of values for that use attribute are target-defined.
A more general solution might be creating a new
config.z3950_attr_value_list table.
webstaff: adjust URL when navigating catalog search results
Navigating from record to record after performing a catalog
search now sets the URL in the browser to /eg/staff/cat/catalog/record/{record_id}.
Clicking on the "Back To Results" button will set the URL
back to /eg/staff/cat/catalog/index.
This allows staff to copy and paste record URIs more conveniently.
webstaff: give more feedback for Z39.50 edit-then-import
The "edit then import" modal now varies the label of the
MARC editor's save button based on whether one has yet
to import the current record or if one is just making
changes to it after having imported it. The dialog now
also has a button for going to the newly-imported record
in catalog view.
When editing a record that has not yet been saved
to Evergreen (either because it is new or because it
is a record coming from a Z39.50 target), ensure that
the context menu 'delete field' operation works.
Jason Stephenson [Thu, 17 Sep 2015 19:12:40 +0000 (15:12 -0400)]
LP 1496977: Duplicate Code in OpenILS::Application::Cat::AssetCommon
It looks like two lines of code got duplicated in commit e2f6cca0 in
AssetCommon. This duplication can lead to data loss for the function
update_fleshed_copies. It also lead to the following warning when
you use OpenILS::Application::Cat::AssetCommon in a script:
"my" variable $sc_entries masks earlier declaration in same scope at
/usr/local/share/perl/5.18.2/OpenILS/Application/Cat/AssetCommon.pm
line 371.
After this patch, the warning disappears.
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Bill Erickson [Thu, 10 Sep 2015 00:52:14 +0000 (20:52 -0400)]
webstaff: Phys Char Wiz various additions and repairs
* Show <unset> as selector value when no value is selected
* Properly handle gaps (undefined slots) in 007 field value
* Return to origin value when dialog Cancel is chosen
Signed-off-by: Bill Erickson <berickxx@gmail.com> Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>