lp:evergreen

Branch merges

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

20668. By Galen Charlton on 2016-06-02

LP#1373578: fix translatability of hold placement errors

This patch fixes a circumstance where ->maketext() could be
passed an empty string, causing it to emit the PO file header text
rather than a desired string.

To test
-------
[1] Configure Evergreen to use a TPAC localization (e.g.,
    the Czech translation).
[2] Enable the "Has Local Copy Alert" library setting.
[3] In the OPAC, change locale and attempt to place a
    hold that would trigger the alert saying that there's
    already a local copy available.
[4] Note that the alert reads as "Project-Id-Version: evergreen
    Report-Msgid-Bugs-To: ..."
[5] Apply the patch.
[6] Try step 3; this time the alert will be translated correctly,
    e.g., "Ve vaší domovské knihovně je již dostupný exemplář."

Signed-off-by: Galen Charlton <email address hidden>
Signed-off-by: Ben Shum <email address hidden>

20667. By Jason Stephenson on 2016-06-02

LP 1548993: Remember the Show More/Fewer Details button selection.

The state of the Show More/Fewer Details button selection was
previously not remembered across searches or across changes to
advanced search. This commit attempts to remedy that.

Signed-off-by: Jason Stephenson <email address hidden>
Signed-off-by: Jeanette Lundgren <email address hidden>
Signed-off-by: Kathy Lussier <email address hidden>

20666. By Jason Stephenson on 2016-06-02

LP 1548993: Restore Show More/Show Fewer Details button functionality.

When using a show_more_details.default setting of true in config.tt2,
the Show More/Show Fewer Details button functionality is broken. The
detail record view is always shown, regardless of the button's state.
This patch attempts to repair the behavior so that the user can toggle
the detail record view even though the default is set to true.

Signed-off-by: Jason Stephenson <email address hidden>
Signed-off-by: Jeanette Lundgren <email address hidden>
Signed-off-by: Kathy Lussier <email address hidden>

20665. By Galen Charlton on 2016-05-25

2.10.3-2.10.4 schema upgrade script

Signed-off-by: Galen Charlton <email address hidden>

20664. By Galen Charlton on 2016-05-25

fix typos in 2.10.4 release notes

Signed-off-by: Galen Charlton <email address hidden>

20663. By Kathy Lussier on 2016-05-25

Docs: Adding 2.10.4 Point Release Notes

Signed-off-by: Kathy Lussier <email address hidden>

20662. By Galen Charlton on 2016-05-24

LP#1584801: fix tagging of previous circs in metarecord search results

This patch fixes a bug where previously-checked-out items were not
indicated as such in search results when doing a metarecord search.

To test:

[1] Ensure that the opac.search.tag_circulated_items library setting
    is enabled and that you are testing with a user that has opted-in
    to retaining circ history.
[2] Make a couple loans to that user.
[3] Log into the public catalog using that user.
[4] Perform a catalog search, choosing search terms such that
    the previously-loaned items appear in results.
[5] Verify that "I have checked this item out before" is displayed
    on the relevant hits.
[6] Click the "Group Formats and Editions" checkbox; note that
    after the page refreshes, previously-borrowed hits are /not/
    tagged.
[7] Apply the patch and restart the open-ils.search service.
[8] Repeat step 6; verify that this time previously-borrowed hits
    are tagged.

Signed-off-by: Galen Charlton <email address hidden>
Signed-off-by: Kathy Lussier <email address hidden>

20661. By Jason Boyer on 2016-05-24

LP1567514 - Don't Output Null Byte for Spool Files

If a spool file is defined on the marc_stream_importer.pl
command line, no null byte is output after the record(s)
are processed.

To test:

[1] Use marc_stream_importer.pl to import a record from
    a file; verify that the output messages do not include
    a null character.

Signed-off-by: Jason Boyer <email address hidden>
Signed-off-by: Galen Charlton <email address hidden>

20660. By Josh Stompro <email address hidden> on 2016-05-24

LP#1517556 - Exclude inactive event defs from find_event_def_by_hook.

Currently when find_event_def_by_hook is called to find an appropriate
event_def based on the hook, it doesn't exclude inactive event defs. So
strange behavior can happen if inactive event defs are present.

I ran into this issue when I was trying out different versions of the self
check receipt event def. I had one ative and one inactive while I was
troubleshooting, and when the system would grab the inactive event def by
chance, the self check UI would freeze. I think it is a resonable expectation
that an event def marked inactive doesn't get used by the system.

To test this out, clone one of the self check event defs, such as
"Self-Checkout Items Out Receipt" and mark the copy as inactive. Now
go and trigger this event in the self check interface by attempting to
print from the Items Out screen. It should now fail 50% of the time, since
both the active and inactive defs are returned.

After the fix the inactive event def will always be excluded, so the action
should no longer fail.

Signed-off-by: Josh Stompro <email address hidden>
Signed-off-by: Galen Charlton <email address hidden>

20659. By Galen Charlton on 2016-05-24

LP#1580676: fix error message

This patch fixes a typo in the error message presented
if the queued record could not be cleaned up for some
reason; it also reports the correct type of the record.

Signed-off-by: Galen Charlton <email address hidden>