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

[Bug 14610] Add ability to place article requests in Koha
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

1 0

[Bug 14793] New cataloguing plugin unimarc_field_225a_bis
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14793 Zeno Tajoli <z.tajoli(a)cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |z.tajoli(a)cineca.it Text to go in the| |This plugin is used to fill release notes| |225$a with a value already | |existing in table | |biblioitems.collectiontitle -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15280] Switch default CHECKOUT notice to Template Toolkit
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15280 --- Comment #10 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Kyle M Hall from comment #9) > (In reply to Jonathan Druart from comment #8) > > Ok so, beginning of a release cycle, let's take decisions! > > > > What is the plan of attack for these notices? > > I think we need to know if 1. everybody agrees with the TT syntax for > > notices and 2. we have dedicated time in a release cycle to move all of the > > existing notices to TT syntax. > > I don't think it's a good idea not to move all of the notices at the same > > time (ie. during the same release cycle). > > I totally agree with that. I think first we need to start a mailing list > topic for this which should end with a simple vote of whether to continue to > use the existing syntax by default or to switch to TT by default and > eventually deprecate the existing syntax. What do you think? We already had one :) http://lists.koha-community.org/pipermail/koha-devel/2016-February/042316.h… -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15280] Switch default CHECKOUT notice to Template Toolkit
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15280 --- Comment #9 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Jonathan Druart from comment #8) > Ok so, beginning of a release cycle, let's take decisions! > > What is the plan of attack for these notices? > I think we need to know if 1. everybody agrees with the TT syntax for > notices and 2. we have dedicated time in a release cycle to move all of the > existing notices to TT syntax. > I don't think it's a good idea not to move all of the notices at the same > time (ie. during the same release cycle). I totally agree with that. I think first we need to start a mailing list topic for this which should end with a simple vote of whether to continue to use the existing syntax by default or to switch to TT by default and eventually deprecate the existing syntax. What do you think? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15280] Switch default CHECKOUT notice to Template Toolkit
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15280 --- Comment #8 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Ok so, beginning of a release cycle, let's take decisions! What is the plan of attack for these notices? I think we need to know if 1. everybody agrees with the TT syntax for notices and 2. we have dedicated time in a release cycle to move all of the existing notices to TT syntax. I don't think it's a good idea not to move all of the notices at the same time (ie. during the same release cycle). -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14567] Browse interface for OPAC
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14567 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #32 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Scalar value @_[0] better written as $_[0] at /home/vagrant/kohaclone/Koha/SearchEngine/Elasticsearch/Search.pm line 126. Scalar value @_[0] better written as $_[0] at /home/vagrant/kohaclone/Koha/SearchEngine/Elasticsearch/Search.pm line 128. Can't use string ("(d)") as an ARRAY ref while "strict refs" in use at /home/vagrant/kohaclone/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm line 108. Please do a *complete* pass on this patchset before sending it back to the QA queue: - Remove debug statements - Add a test plan: - Provide full test coverage - The opac-browse template looks wrong (does not follow the usual pattern, is it expected?) - And finally, test -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8361] Issuing rule if no rule is defined
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8361 --- Comment #7 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to M. Tompsett from comment #6) > Perhaps add an about.tt/about.pl check for issuing rules being defined? I'd even suggest to bump up a warning on the main page. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8361] Issuing rule if no rule is defined
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8361 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtompset(a)hotmail.com --- Comment #6 from M. Tompsett <mtompset(a)hotmail.com> --- Perhaps add an about.tt/about.pl check for issuing rules being defined? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8361] Issuing rule if no rule is defined
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8361 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57704|0 |1 is obsolete| | --- Comment #5 from M. Tompsett <mtompset(a)hotmail.com> --- Created attachment 57717 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57717&action=ed… [SIGNED-OFF] Bug 8361: Do not allow checkouts if no rules are defined We should require a circulation rule to allow checkouts and reject them if no rules are defined. Test plan: - Delete all issuing rules - Check an item out => Without this patch the checkout is allowed => With this patch applied it is rejected NOTE: Nicely fails when test is unchanged too. Signed-off-by: Mark Tompsett <mtompset(a)hotmail.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8361] Issuing rule if no rule is defined
by bugzilla-daemon＠bugs.koha-community.org 23 Nov '16

23 Nov '16

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

1 0