February 2018 - Koha-bugs - lists.koha-community.org

[Bug 20222] New: The bread crumb for cleanborrowers.pl does not match the link text in tools-home.pl
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20222 Bug ID: 20222 Summary: The bread crumb for cleanborrowers.pl does not match the link text in tools-home.pl Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Tools Assignee: koha-bugs(a)lists.koha-community.org Reporter: barton(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org In Home › Tools (tools/tools-home.pl), the link to the patron batch deletion tool is Batch patron deletion/anonymization When you click into the page, the breadcrumb says Home › Tools › Clean Patron Records The breadcrumb should match the link text. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 9

[Bug 11897] Stock Rotation for Koha
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11897 --- Comment #183 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Comment on attachment 70521 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70521 Bug 11897: Add Stock Rotation atomic update schema. Review of attachment 70521: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=118…) ----------------------------------------------------------------- ::: installer/data/mysql/atomicupdate/stockrot_tables.sql @@ +6,5 @@ > + description text NOT NULL default '', -- Description for this rota > + cyclical tinyint(1) NOT NULL default 0, -- Should items on this rota keep cycling? > + active tinyint(1) NOT NULL default 0, -- Is this rota currently active? > + PRIMARY KEY (`rota_id`) > +) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci; 2) Should now be: ... ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci; (see bug 18336) ::: installer/data/mysql/kohastructure.sql @@ +4156,5 @@ > + description text NOT NULL default '', -- Description for this rota > + cyclical tinyint(1) NOT NULL default 0, -- Should items on this rota keep cycling? > + active tinyint(1) NOT NULL default 0, -- Is this rota currently active? > + PRIMARY KEY (`rota_id`) > +) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci; 3) Same here as above: ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci; ::: installer/data/mysql/sysprefs.sql @@ +601,5 @@ > ('XSLTResultsDisplay','default','','Enable XSL stylesheet control over results page display on intranet','Free'), > ('z3950AuthorAuthFields','701,702,700',NULL,'Define the MARC biblio fields for Personal Name Authorities to fill biblio.author','free'), > +('z3950NormalizeAuthor','0','','If ON, Personal Name Authorities will replace authors in biblio.author','YesNo'), > +('StockRotation','0',NULL,'If ON, enables the stock rotation module','YesNo'), > +('RotationPreventTransfers','0',NULL,'If ON, prevent any transfers for items on stock rotation rotas, except for stock rotation transfers','YesNo') 4) Preferences in this file are sorted alphabetically, please move into the right spot. ::: koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc @@ +21,4 @@ > [%- CASE 'lists' -%]Lists > [%- CASE 'clubs' -%]Patron clubs > [%- CASE 'ill' -%]Create and modify Interlibrary loan requests > + [%- CASE 'stockrotation' -%]Manage stockrotation operations 5) Descriptions for the sub-permissions appear to be missing? ::: koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref @@ +879,5 @@ > + - pref: RotationPreventTransfers > + choices: > + yes: Enable > + no: Disable > + - "Prevent branchtransfers on items in stockrotation rotas" 6) These preferences don't form sentences and are not nicely formatted. Please avoid use of 'branch' in favor of 'library'. Also, should it be stock rotation or stockrotation? Both are used here. It should probably be something like: - yes: Enable - no: Disable - "library transfers on items in stockrotation rotas." -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11897] Stock Rotation for Koha
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11897 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #182 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- I've tried to get these patches to apply, but I'd like a proper rebase. 1) Database update doesn't work DEV atomic update: stockrot_tables.sql C4::Installer::load_sql returned the following errors while attempting to load /home/vagrant/kohaclone/installer/data/mysql/atomicupdate/stockrot_tables.sql: Will continue comments using splinter. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 20237] New: Don't allow adding same user multiple times to be notified on receive
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20237 Bug ID: 20237 Summary: Don't allow adding same user multiple times to be notified on receive Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Keywords: Academy Severity: trivial Priority: P5 - low Component: Acquisitions Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org To test: - create a basket - add an order line - try to add the same user to be notified on receive multiple times Probably same fix as for bug 20148 can be implemented here. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 2

[Bug 15336] Script for merging vendors
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15336 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15336] Script for merging vendors
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15336 --- Comment #36 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Comment on attachment 68676 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68676 Bug 15336 - New command-line script: merge_bookseller.pl Review of attachment 68676: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=153…) ----------------------------------------------------------------- Hi Alex, started QA on this, but found some issues, please fix: 1) QA tools fail: FAIL Koha/Acquisition/Bookseller.pm FAIL pod empty =head3 in file Koha/Acquisition/Bookseller.pm *** ERROR: FAIL pod coverage POD is missing for 'merge_with' 2) Please fix your commit message to include the correct script name! I think you also change subscriptions and items? Please document in your test plan. ::: Koha/Acquisition/Bookseller.pm @@ +77,5 @@ > + > + my $from_vendor = Koha::Acquisition::Booksellers->find( $mergefrom ); > + my $result = $from_vendor->merge_with($mergeto); > + > +merge source vendor (-f) into target vendor (-t) 3) ...merge with :) ::: misc/migration_tools/merge_vendors.pl @@ +102,5 @@ > + > +=item B<--confirm> > + > + --confirm Commit the changes in database. Running without this > + parameter is like testing. It is recommanded to run without 4) ... recommended @@ +107,5 @@ > + --confirm first to be sure of the changes. > + > +=item B<--from> > + > + --from=<vendorid> The source vendor identifier that will be merged in 5) Suggestion: The identifier of the source vendor that will be merged with the target vendor. @@ +112,5 @@ > + target vendor. > + > +=item B<--to> > + > + --to=<vendorid> The target vendor identifier in which merge 6) Suggestion: The identifier of the target vendor that the source vendor is merged with. ::: t/Bookseller.t @@ +1,3 @@ > #!/usr/bin/perl > # > +# This Koha test module is a stub! 7) Could possibly remove this now :) All tests pass for me. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 20242] New: items marked as checkout marked as check in automatically in koha
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20242 Bug ID: 20242 Summary: items marked as checkout marked as check in automatically in koha Change sponsored?: --- Product: Koha Version: 3.22 Hardware: PC OS: Windows Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: ahmad.najeh(a)majancollege.edu.om QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Hi All, items marked as checkout marked as check-in automatically in koha. This issue happened when I used inventory tool. Can you help? Please. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 3

[Bug 14749] Add API route to get top issues
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |katrin.fischer(a)bsz-bw.de, | |tomascohen(a)gmail.com --- Comment #10 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Hi Julian, could you add your proposal to https://wiki.koha-community.org/wiki/REST_api_RFCs please? I think we'd like to change a few things like branch -> library, biblionumber -> biblio_id. Also adding Tomas to stop me if I am mistaken :) -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 20110] New: Adding same user multiple times to same budget fund
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '18

18 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20110 Bug ID: 20110 Summary: Adding same user multiple times to same budget fund Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Acquisitions Assignee: koha-bugs(a)lists.koha-community.org Reporter: pasi.kallinen(a)joensuu.fi QA Contact: testopia(a)bugs.koha-community.org It is possible to add the same user multiple times into the same budget fund -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 8

[Bug 9573] Ability to download items lost report
by bugzilla-daemon＠bugs.koha-community.org 17 Feb '18

17 Feb '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9573 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #67 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Pushed to master for 18.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.

1 0