September 2017 - Koha-bugs - lists.koha-community.org

[Bug 11580] returnBeforeExpiry and closed day
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580 Patrick Robitaille <patrick.robitaille(a)ccsr.qc.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |patrick.robitaille(a)ccsr.qc. | |ca -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 7960] choice to not show the text labels for item types
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7960 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle(a)bywatersolutions.com --- Comment #10 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Marcel de Rooy from comment #9) > (In reply to Aleisha Amohia from comment #8) > > (In reply to Marcel de Rooy from comment #7) > > > Have the impression that we should not hide labels with a syspref but > > > resolve this with custom css. Please convince me that I am wrong. > > > > The syspref just provides the functionality for the library to hide or show > > the text, without having to play around with css or ask us to do it for > > them. I don't think it's fair to assume that librarians know how to write > > css. > > We have to find a balance between adding a syspref for showing/hiding > elements and css customization. Yes, not each librarian writes css. I will > not block this patch and change the status to Signed off again. Another QA > opinion please. I must agree, my first thought was to ask why not just add a class to make it easy to hide with css. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13766] Make biblioitems.ean longer and add index
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13766 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #15 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Nick Clemens from comment #14) > Looks good and worked for me. Wondered if we need to check for existing > indexes on deletedbiblioitems but think this is okay Yes, please. Check all of them, and use C4::Installer::index_exists -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13766] Make biblioitems.ean longer and add index
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13766 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia(a)bugs.koha-communit |nick(a)bywatersolutions.com |y.org | CC| |nick(a)bywatersolutions.com --- Comment #14 from Nick Clemens <nick(a)bywatersolutions.com> --- Looks good and worked for me. Wondered if we need to check for existing indexes on deletedbiblioitems but think this is okay -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12365] Notes for rules
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

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

1 0

[Bug 13766] Make biblioitems.ean longer and add index
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13766 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #66147|0 |1 is obsolete| | --- Comment #13 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 66744 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66744&action=ed… Bug 13766 - Change *bibioitems.ean to mediumtext and add indexes - biblioitems.ean and deleteditems.ean are changed to mediumtext to match issn and isbn. - An index is added for ean on both tables. Patch cleans up inconsistencies caused by bug 5337 that causes some older databases to have an existing index on biblioitems.ean. Signed-off-by: Colin Campbell <colin.campbell(a)ptfs-europe.com> Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13766] Make biblioitems.ean longer and add index
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

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

1 0

[Bug 12204] in RESERVESLIP, any 'reserve' field is not being expanded correctly.
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12204 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |ASSIGNED Assignee|jonathan.druart(a)bugs.koha-c |magnus(a)libriotech.no |ommunity.org | --- Comment #13 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Magnus Enger from comment #11) > have changed things enough that it now makes sense to revive the current > bug/patch. I have made a similar change on my live server and customers are > happy with it. What is the patch? This one does no longer apply. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15438] Checking out an on-hold item sends holder' s borrowernumber in AF (screen message) field.
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15438 --- Comment #33 from Colin Campbell <colin.campbell(a)ptfs-europe.com> --- Kyle's update differs from the original patch in the following - # $noerror = 0; + $noerror = 0; The commented out line was added back in 2008 and there was no setting of the variable prior to that. The difference is with the line commented out the sipserver proceeds with the issue with noerror being set to 0 the issue is blocked. It meant that the code was overriding the syspref setting (which it had actually passed to CanBookBeIssued) So it cleans up a long standing bug in that response. Checked that issues were allowed/disallowed as per the syspref setting -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12204] in RESERVESLIP, any 'reserve' field is not being expanded correctly.
by bugzilla-daemon＠bugs.koha-community.org 01 Sep '17

01 Sep '17

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

1 0