July 2016 - Koha-bugs - lists.koha-community.org

[Bug 10148] 007/006 not filling in with existing values
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10148 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtompset(a)hotmail.com --- Comment #3 from M. Tompsett <mtompset(a)hotmail.com> --- The issue is in the template file. Anything that is 9595 lines long has got to have something wrong with it. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9259] batch delete marc batches
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9259 --- Comment #10 from Nick Clemens <nick(a)bywatersolutions.com> --- This is great! I think missing form what was requested is the ability to do this to multiple batches at once (batches of import batches) I can open a new bug and retitle this one if that makes the most sense, unless you want to build that functionality in too? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9259] batch delete marc batches
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9259 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #53007|0 |1 is obsolete| | --- Comment #9 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 53116 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53116&action=ed… Bug 9259: Ability to delete a staged file once it has been cleaned To test: 1) Go to Tools -> Staged MARC Management and clean a file. If you have no files to clean, go to 'Stage MARC for import' and upload one to clean following the necessary steps. 2) Confirm that once the file has been cleaned, the Action column now shows a Delete button. Confirm this button only shows for cleaned files. 3) Click the Delete button. 4) Confirm that clicking Cancel exits the pop-up message and does not delete the file. 5) Confirm that clicking OK refreshes the list of staged records and the one you just deleted is no longer on it (has been deleted). You can confirm this by checking for the file in mysql (SELECT * FROM import_batches WHERE import_batch_id = X;) 6) Run prove -v t/db_dependent/ImportBatch.t (have written unit tests for CleanBatch and DeleteBatch) Sponsored-by: Catalyst IT Signed-off-by: Liz Rea <liz(a)catalyst.net.nz> Catalyst sign off, so needs another one but YAY this is great. Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9259] batch delete marc batches
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9259 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14576] Allow arbitrary automatic update of location on checkin
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14576 --- Comment #26 from Nick Clemens <nick(a)bywatersolutions.com> --- (In reply to Owen Leonard from comment #7) > I think that should be "Item shelving location" (since "location" could be > confused with "library"). Changed > I think the text "Not an authorized value" is not helpful to the user and is > potentially scary. Think "UNAUTHOIRZED VALUE!!" Changed to: 'No description available' - I am happy to reword but I do think it is nice to indicate to the user that it isn't a defined authorized value somehow > - Text describing explicitly the correct format. > - Text explaining that an "_ALL_" rule overrides all others Added to syspref description in editor > - Some way to warn the user if their entry is incorrect. > - Some way to warn the user if they have set an "_ALL_" rule along with > other rules Added js check with warnings for basic syntax I opened a second bug for addition of full GUI - this patchset follows existing format in the system but I do think a full editor could be added as per bug 16698 once this feature is pushed -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14576] Allow arbitrary automatic update of location on checkin
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14576 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52365|0 |1 is obsolete| | --- Comment #25 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 53115 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53115&action=ed… Bug 14576 (QA Folllowup) Address issues use mock for tests use a separate variable for yaml fix indentation -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14576] Allow arbitrary automatic update of location on checkin
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14576 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52224|0 |1 is obsolete| | --- Comment #24 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 53114 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53114&action=ed… Bug 14576 - (QA followup) Fix message for use of value not in authorized values Signed-off-by: Cathi Wiggins <cwiggins(a)ArcadiaCA.gov> Signed-off-by: Megan Wianecki <mwianecki(a)mtpl.org> Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14576] Allow arbitrary automatic update of location on checkin
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14576 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52223|0 |1 is obsolete| | --- Comment #23 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 53113 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53113&action=ed… Bug 14576 - Unit tests Signed-off-by: Cathi Wiggins <cwiggins(a)ArcadiaCA.gov> Signed-off-by: Megan Wianecki <mwianecki(a)mtpl.org> Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14576] Allow arbitrary automatic update of location on checkin
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14576 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52222|0 |1 is obsolete| | --- Comment #22 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 53112 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53112&action=ed… Bug 14576 - Allow arbitrary automatic update of location on checkin This patch adds a new syspref "UpdateItemLocationOnCheckin" which accepts pairs of shelving locations. On check-in the items location is compared ot the location on the left and, if it matches, is updated to the location on the left. This preference replaces ReturnToShelvingCart and InProcessingToShelvingCart preferences. The update statement should insert values that replciate these functions. Note existing functionality of all items in PROC location being returned to permanent_location is preserved by default. Also, any items issued from CART location will be returned to their permanent location on issue (if it differs) Special values for this pref are: _ALL_ - used on left side only to affect all items _BLANK_ - used on either side to match on/set to blank (actual blanks will work, but this is an easier to read option) _PERM_ - used on right side only to return items to permanent location Test Plan: 1) Apply this patch 2) Run updatedatabase.pl 3) Set the new system preference UpdateitemLocationOnCheckin to the following (assuming sample data): NEW: FIC FIC: GEN 4) Create an item, set its location to NEW 5) Check in the item, note its location is now FIC 6) Check in the item again, note its location is now GEN 7) Check in the item again, note its location remains GEN 8) Test using _ALL_, _BLANK_ and _PERM_ for updates 9) Try entering various incorrect syntax in the pref and note you are warned Sponsored by: Arcadia Public Library (http://library.ci.arcadia.ca.us/) Middletown Township Public Library (http://www.mtpl.org/) Signed-off-by: Cathi Wiggins <cwiggins(a)ArcadiaCA.gov> Signed-off-by: Megan Wianecki <mwianecki(a)mtpl.org> Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14576] Allow arbitrary automatic update of location on checkin
by bugzilla-daemon＠bugs.koha-community.org 05 Jul '16

05 Jul '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14576 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.

1 0