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

[Bug 20825] New: The method is not covered by tests!
by bugzilla-daemon＠bugs.koha-community.org 06 Jun '18

06 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20825 Bug ID: 20825 Summary: The method is not covered by tests! Change sponsored?: --- Product: Koha Version: master Hardware: HP OS: Linux Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: sergeinik(a)tut.by QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Created attachment 75575 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75575&action=ed… Message in browser When I try to issue an instance and enter a barcode, the program displays the following message: Software error: The method is not covered by tests! -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 37

[Bug 12159] Duplicate borrower_add_additional_fields function
by bugzilla-daemon＠bugs.koha-community.org 06 Jun '18

06 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12159 --- Comment #23 from Josef Moravec <josef.moravec(a)gmail.com> --- (In reply to Jonathan Druart from comment #20) > IMO we should push this one along with bug 20443 (which does not have > patches yet). Otherwise we are going to have 2 different ways to fetch > patron's attributes. > Before we only had GetBorrowerAttributes. Does make sense. I'll try to look at it. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12159] Duplicate borrower_add_additional_fields function
by bugzilla-daemon＠bugs.koha-community.org 06 Jun '18

06 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12159 --- Comment #22 from Josef Moravec <josef.moravec(a)gmail.com> --- (In reply to Marcel de Rooy from comment #18) > FAIL Koha/Patron/Attribute.pm > forbidden pattern: Warning: The 'sub type' may be wrong is declared in a > Koha::* package (see bug 15446) (line 127) > > Josef: > This is a more nasty one. > Could you consider renaming this method? > The Koha objects also use _type and it might be confusing. > What do you think? We could rename it of course, but isn't that the reason why the type in Koha::Object was renamed to _type? To avoid conflicts with type accessor if needed? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12159] Duplicate borrower_add_additional_fields function
by bugzilla-daemon＠bugs.koha-community.org 06 Jun '18

06 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12159 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com --- Comment #21 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- I agree with Jonathan here -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7651] Add separate permission for managing currencies and exchange rates
by bugzilla-daemon＠bugs.koha-community.org 05 Jun '18

05 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7651 --- Comment #19 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 75828 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75828&action=ed… Bug 7651: put TT directive on a single line to help translation Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7651] Add separate permission for managing currencies and exchange rates
by bugzilla-daemon＠bugs.koha-community.org 05 Jun '18

05 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7651 --- Comment #18 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 75827 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75827&action=ed… Bug 7651: (QA follow-up) Fix warning about undefined active currency If no active currency is defined we show a warning. This fixes the link shown below to check the correct permission. Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7651] Add separate permission for managing currencies and exchange rates
by bugzilla-daemon＠bugs.koha-community.org 05 Jun '18

05 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7651 --- Comment #17 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 75826 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75826&action=ed… Bug 7651: (follow-up) Correct visibility on admin sidebar This patch corrects visibility of links to the different acquisition pages from the admin sidebar. It also makes a correction to the links on the acq start page: - Budgets = period_manage - Funds = budget_manage To test: - Follow the test plan of the first patch Signed-off-by: Charles Farmer <charles.farmer(a)inLibro.com> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7651] Add separate permission for managing currencies and exchange rates
by bugzilla-daemon＠bugs.koha-community.org 05 Jun '18

05 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7651 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #75573|0 |1 is obsolete| | Attachment #75574|0 |1 is obsolete| | Attachment #75788|0 |1 is obsolete| | --- Comment #16 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 75825 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75825&action=ed… Bug 7651: Add a new permission for managing currencies and exchange rates At the moment staff users need parameters or parameters_remaining_permissions in order to be able to change exchange rates for acquisition orders. This patch adds a new separate permission currencies_manage and updates staff users currently having those permissions to get the new permission as well. To test: - Create some staff users with different permission sets 1) superlibrarian 2) parameters 3) parameters_remaining_permissions 4) manage_circ_rules, but not parmeters_remaining_permissions 5) all acquisition permissions - Apply patch and run database update - Verify new permission has been added and staff users updated 1) remains the same 2) + 3) will have currencies_manage 4) remains unchanged, doesn't have new permission 5) remains the same, will have access now because of having the top level acquisition permission - Verify the changed pages work correctly: - navigation on admin home page NOTE: the acquisition parameters section will now honor all different related permissions (edi_manage, budget_manage,...) - navigation on acquisition home page - try to access currencies page directly Signed-off-by: Charles Farmer <charles.farmer(a)inLibro.com> Signed-off-by: Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7651] Add separate permission for managing currencies and exchange rates
by bugzilla-daemon＠bugs.koha-community.org 05 Jun '18

05 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7651 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 20880] New: Upgraded 17.11 to 18.05: Malformed Bootstrap Theme Filenames
by bugzilla-daemon＠bugs.koha-community.org 05 Jun '18

05 Jun '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20880 Bug ID: 20880 Summary: Upgraded 17.11 to 18.05: Malformed Bootstrap Theme Filenames Change sponsored?: --- Product: Koha Version: 18.05 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: rcmcdonald91(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org My 18.05 (from 17.11) OPAC is trying to download files like this: 'https://library.mydomain.com/opac-tmpl/bootstrap/lib/modernizr.min_18.05000…' These files do not exist. it should be: 'https://library.graa.com/opac-tmpl/bootstrap/lib/modernizr.min.js' -- You are receiving this mail because: You are watching all bug changes.

1 6