LP#1441634: add release notes for removal of biblios.net Z39.50 target

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1441634 Remove biblios.net from Z39.50 sources

The biblios.net Z39.50 server has not been functional for years.
Remove it from the list of Z39.50 servers that we populate in a
default install of Evergreen.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Docs: Change all .PNG to .png

There were files with uppercase .PNG and referneces to .PNG in the docs.

Change all to lowercase .png for consistency.

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

Docs: Delete remnant .JPG files that had already been converted to .jpg

In a previous commit, these files were already re-added as lowercase .jpg,
so let's go ahead and remove the .JPG versions of those files.

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

Docs: Change all .JPG to .jpg

There were files with uppercase .JPG and references to .JPG in the docs.

Change all to lowercase .jpg for consistency.

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

Docs: Minor edits to 2.8 release notes

Fix spelling mistake for "Berkless" to "Berklee" in the acknowledgements.

Also, change from "2_8" to "2.8" for consistency in reading.

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

Release Notes acknowledgements addition

Adding KCLS to the acknowledgements.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Release Note Acknowledgements

Added acknowledgements to the Release Notes. Also added a missing feature and
did some minor cleanup.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>

2.8 release notes creation and cleanup

* Create RELEASE_NOTES_2_8.txt
* Clean up RELEASE_NOTE_NEXT
* Bump doc header level for security announcement

Signed-off-by: Bill Erickson <berickxx@gmail.com>

Forward-porting 2.7.4-2.8.0 DB upgrade

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP 1435966: Clark Kent should write lock file after it forks.

Rearrange the code in clark-kent.pl so that the lock/pid file is
opened and written to after the reporter daemonizes itself.  This
way, the pid in the file can actually be used to stop the running
reporter daemon.

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

LP#1234220 Stamping DB upgrade copy/ratio messages

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1234220 hold ratio renewal override perms

New permissions for overriding renewal failure events:

TOTAL_HOLD_COPY_RATIO_EXCEEDED.override
AVAIL_HOLD_COPY_RATIO_EXCEEDED.override

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1234220 Improve hold/copy ratio renewal messages

Display friendlier messages in the TPAC and staff client client when a
hold/copy ratio rule prevents a renewal.

This creates top-level ILS events for total/avail hold copy ratios,
since that's what best works with the staff client.  The TPAC displays a
simple "item is needed for a hold" message, whereas the staff client
displays the raw event description, which is slightly more descriptive.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Docs: Link optional docs from Apache config

This commit links to the new optional Apache config docs from the Apache
config step in the installation instructions.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1413624 Docs: Move Access Handler docs to official docs

This commit moves the new Access Handler doc file to the official docs
folder, adds the file reference to the root.txt, changes the main header
text for easier linking, adds the missing block delimiter lines around a
code block, and adds a blank line at the end of another doc file to
prevent the docs from running together.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1413621 Docs: Minor changes to Apache Rewrite Tricks

This commit moves this doc file from TechRef into the official docs
area, adds it to the root.txt file, and adds line breaks for easier
viewing and editing of the file.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1413621 Docs: Apache Rewrite Tricks

Because multiple sites have found this kind of thing useful in the past.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: Fix AsciiDoc processing errors

This commit fixes an AsciiDoc link syntax error, and renames section ids
to prevent duplicate names in the preview version of the Web Client
docs.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: Eliminate accidental invalid level 5 AsciiDoc headings

To properly integrate updated receipt template editor documentation
we origimally used invalid level 5 AsciiDoc headings. Asciidoc only
supports 4 levels of headings

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: Add authority section and Authority subject heading thesaurus

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: 2.8 feature correction

This commit corrects a mistake in the new feature documentation for
deleting copy locations.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: 2.8 feature "Deleted flag for copy locations"

This commit documents the new 2.8 feature, the ability to delete copy
locations. It also updates the screenshot, and adds lines to document
other common features of the interface.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1403967: show 'subject heading thesaurus' value release notes

The 'subject heading thesaurus' basically defines the issuing authority
that created the authority record. For example, a value of "a" means that
it came from LoC. A value of "n" means that it is a locally created
authority. Note that within the authority MARC editor the 'Subject
Heading Thesaurus' is stored/displayed in the 'Subj' fixed field.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Forward-port 2.7.3-2.7.4 upgrade script

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

LP#1424755: Add IMPORTANT SECURITY INFORMATION release note.

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

LP#1424755: Org Unit Setting view permissions can be bypassed

Fix private org. unit setting leakage by forcing the $auth argument
to true if not passed in when open-ils.actor.ou_setting.ancestor_default
or open-ils.actor.ou_setting.ancestor_default.batch are called.

Other than a change to the desc of the public methods to reflect that
they now check permissions if permissions are required, there are no
required api changes to the back or the front ends.

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

LP#1206589: Temporary Fix for Org. Unit Settings History Bug

This commit makes two changes to the coustl IDL entry.

1. It adds a retrieve permission of STAFF_LOGIN.  This at least
requires someone with staff permission to be able to view settings
history.

2. It fixed the primary key field from date_applied to id.  The
prior setting was just an outright error.

NOTE: This is intended as a short term fix for the Library Settings
Editor history feature.

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

lp1427331: Remove relator information from author links

Removes the display and linking of relator information from the author field
on the search results page. The display of subfield e was limiting the number
or results retrieved when clicking these links.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Revert LP#1205072

It was decided that these changes caused too much potential for
confusion, and would be better served by a simultaneous change to
making 'granularity' case insensitive.  Pulling back for now.

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

LP#1287370: minor textual cleanup

- remove trailing spaces
- remove debug logging

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Erica Rohlfs <erohlfs@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1287370: apply filter/offset persistence to the fund search page

This patch completes the AutoGrid work and applies the
filter & offset persistence to the fund search page.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Erica Rohlfs <erohlfs@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1287370: allow AutoGrid to persist filter state and page offset

By turning on a new AutoGrid option called urlNavigation,
the current filter and offset is stored in a URL parameter
called djgridops. This permits using the back button
to return to an AutoGrid page at the place in the search
results where the user left it.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Erica Rohlfs <erohlfs@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP #1406350 Mobile Device Navigation Issue Fix for Shelf Browser

Currently, when a browser window is narrowed too much, whether
in a mobile browser, or on a desktop or laptop machine, the
"Previous Page" and "Next Page" navigation links disappear. The
only solution that seems to solve this is by removing the
"THEAD" element from the shelf browser table.

This patch removes that tag, adds a new row to imitate a new
border as existed before, and modifies the CSS in .../opac/
css/style.css.tt2 to address that difference.

Signed-off-by: Adam Bowling <abowling@emeralddata.net>
Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1205072: Correct and clarify the upgrade notes

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

LP#1205072 - Assorted fixes for action trigger granularity settings

- Added "weekdays" option to match up with example crontab file
- Change granularity example back to lowercase
- Change granularity labels to lowercase in the docs

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1205072 A/T granularity upgrade notes

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1205072 A/T granularity UI sane default, honors case

Present A/T granularity options as untranslated, lower-case strings to
match the crontab examples.  Also, honor alternate case variations for
granularity values so that "Daily" and "daily" are both seprate, valid
options.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1287791: Restrict authority browse to controlled subfields

Back in bug 1175308 I added some logic that improves the behavior
of the MARC editor's right-click authority popup. Now we need to
go a little farther and adjust that logic to be more careful about
which subfields it uses to browse for authority records. Specifically,
it should only use controlled subfields.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1155313: fix mangling of schema upgrade script name

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1155313: recalculate normalized labels during upgrade

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1155313: pin upgrade script to 0914

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1155313: fix copy-and-paste-o in test case

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1155313: upgrade script and pgTAP tests for monograph_part label normalization fix

Signed-off-by: Daniel Pearl <dpearl@cwmars.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1155313: Repair generation of label_sortkey for monograph_part entries

The evergreen.lpad_number_substrings function attempts to codify numeric
fields within labels.  It does this by finding the strings, padding them
to a given size, and replacing them in the source string.  For instance:
       3 => 0000000003
    15.4 => 00000000150000000004

The algorithm was fooled by repeated characters, like in 15.1:
    15.1 => 00000000000000000150000000001  INCORRECT

This change will result in the correct value.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1395842: fix labels for A/T environment and parameter ID fields

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1426133: followup fix seed data sequence

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

LP#1425967: look for Business::CreditCard in settings-tester.pl

and not Business::CreditCard::Object

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

LP#1426133: Stamping upgrade script for set merge_profile_id_seq

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

LP#1426133: Set merge_profile_id_seq explicitly

Many database tables need to explicitly set the id sequence (often to
100) to provide room for stock data with static ids. This commit does so
for vandelay.merge_profile, and also updates the ids of any non-stock
profiles (both in the merge_profile table and in related settings).

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1425191 Restore summarization of serial units

In a previous commit, code was added for force summaries to be limited
to a certain type (to prevent cross type contamination of summaries).
This had the negative side-effect of preventing summaries on units, as
the units were not requesting a 'type' for their summary.

While we could force the unit code to send a type, it seemed generally
more flexible to restore the removed support for returning all summary
information in the absense of a requested type.  This could lead to
cross-type summaries for units, but that is a larger issue altogether
(i.e. how we can properly support cross-type units at all).

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

lp1373203: Allow users to escape metarecord holds

Provides a means for users to escape a metarecord hold if they accidentally
clicked the Advanced Hold Options link. A Basic Hold Options link will be
provided to allow the user to return to a title hold. If the hold starts as a
metarecord hold, the basic hold options link won't display since we don't know
which bib the patron wants.

Also fixes an issue where the hold cancel button always returned the user to
the previous screen. We will instead utilize the existing hold_source_page
parameter to identify the return page.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1402797 Open item status in a new tab

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

LP#1402797 Allow (and use) a default cancel cause for holds by passing around a scalar instead of an object

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

LP#1402797 Do not allow workstations as org units that cannot have user

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

LP#1402797 Test value directly, and invert test for the a-disabled test

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

LP#1402797 Move aDisabled from patron/app to services/ui so it can be used everywhere

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

LP#1402797 Attempt to implement disabling tests for orgs in the magic dropdown

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

LP#1402797 clean up action rows and styling

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

LP#1402797 Add (stub) fast item add, and (working) bib source selection

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

LP#1402797 Mangle the 005 when saving, and support any record_entry type

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

LP#1402797 handle up/down arrow (navigate) and ctrl + up/down (duplicate field)

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

LP#1402797 canonicalize field.postion after inserting or removing fields

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

LP#1402797 insert 00[678], building 008 from record data

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

LP#1402797 Improve styling; marcrecord bug fixes; implement field/subfield insert and remove

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

LP#1402797 Start styling marc editor via css

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

LP#1402797 Code layout cleanup

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

LP#1402797 Should prompt when there is unsaved data, but does not.  More to do...

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

LP#1402797 Global undo/redo stack stack

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

LP#1402797 Make grid action context menu safe for multiple grids per page

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

LP#1402797 Allow forgive-fines on forced checkout

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

LP#1402797 If no row is currently selected on right-click, select the clicked row

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

LP#1402797 Make grid context menu safer and more state-aware

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

LP#1402797 Typo

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

LP#1402797 use ng-options so that the default option works

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

LP#1402797 Attach the context menu to the body for proper coordinate mapping

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

LP#1402797 Grid row context menu from Actions dropdown

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

LP#1402797 add a Retrieve All These Patrons button

in the Item Status -> Circ History List

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1402797 Add user-related fields to some hold interfaces

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

LP#1402797 Add missing columns to patron search result grid

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

LP#1402797 Remove Uncancel Hold action where not useful

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

LP#1402797 Hide cancel/uncancel as appropriate

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

LP#1402797 Teach the autogrid how to hide actions

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

LP#1402797 remove Group Actions from Group Member Details

place "Move Another Patron To This Group" under Actions

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1402797 change Retrieve Selected Patron

to Retrieve Selected Patrons for Other -> Group Member Details interface

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1402797 Context menu infrastructure

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

LP#1402797 Initial MARC editor -- load, edit content, save, see the Breaker version

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

LP#1402797 Use jquery xml parser, and appendChild()

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

LP#1402797 Initial commit and sourcing of the jquery-ified marcrecord.js module

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

LP#1402797 Remove DOB valid test for now; Change alert valid test (expressions must be simple)

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

LP#1402797 Add Item Status to the circ menu

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

LP#1402797 Make the save box smaller to help avoid overlapping the input form

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

LP#1402797 Add DOB and address validity styling

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

LP#1402797 Add DOB to patron summary

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

LP#1402797 Profile Group no longer causes second search to fail, but it does not yet propagate

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

LP#1402797 Add "forget" mode for grid limit; Use in hold clearing mode

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

LP#1402797 Pass the top-level class so that the proper tooltip can be generated

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

LP#1402797 Focus the Print button

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

LP#1402797 Make Strict Barcode checkbox sticky upon actual use

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

LP#1402797 Pre-force tab change.  There have been reports of the tab not changing on automatic redirect.

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