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

[Bug 11808] When searching for a cardnumber in the intranet, also try to search for it on the LDAP server if one is configured.
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11808 --- Comment #5 from Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> --- Created attachment 60562 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60562&action=ed… Bug 11808 - Add the SearchCardnumberWithLDAP system preference. When activated, cardnumber searches in the intranet will also do a search on the LDAP server if a user was not found in the local database. If the cardnumber is found on the LDAP server, the user account is imported in the local database. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12461] Add patron clubs feature
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12461 Kyle M Hall <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #56 from Kyle M Hall <kyle(a)bywatersolutions.com> --- (In reply to Jonathan Druart from comment #55) > In my opinion, this patchset is not ready to be QAed. Lot of changes should > be done to meet our coding guidelines. Jonathan, can you be more specific? That sounds like QA to me ; ) I've been working on the feature since 2012. It's already gone through one full rewrite because Koha's coding guidelines had moved on so much since it was initially written. If you want another signoff, that's fine. But you specifically wrote "Lot of changes should be done to meet our coding guidelines" which is not a task for a signer. That's a task for a QA'er and I'll really need some actionable comments if I am to make any changes! If you want another sign-off, please set status to needs sign-off. Otherwise the status can go back to failed qa. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15524] Set limit on maximum possible holds per patron by category
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15524 --- Comment #1 from rkuiper(a)roundrocktexas.gov --- We are very interested in this bug. Nothing has happened on this for over a year. Would someone give me a status update on this? Thanks, Rhonda K. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15524] Set limit on maximum possible holds per patron by category
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15524 rkuiper(a)roundrocktexas.gov changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rkuiper(a)roundrocktexas.gov -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12461] Add patron clubs feature
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12461 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart(a)bugs.koha-c | |ommunity.org Status|Signed Off |Failed QA --- Comment #55 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- In my opinion, this patchset is not ready to be QAed. Lot of changes should be done to meet our coding guidelines. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8643] Add important constraint to marc subfields
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8643 Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mehdi.hamidi(a)inlibro.com --- Comment #29 from Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> --- Hello everyone, I rebased the patch and removed two patches that were not necessary. I also addressed some of the the comments : (In reply to Jonathan Druart from comment #23) > ::: koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt > @@ -111,4 @@ > > <input type="checkbox" name="important" id="important" value="1" /> > > [% END %] > > </li> > > - <li><label for="authorised_value">Authorized value: </label>[% authorised_value %] (if you select a value here, the indicators will be limited to the authorized value list)</li> > > Why do you remove this line? I put back the line, as for why it was removed, I'm not sure. (In reply to Jonathan Druart from comment #24) > QA comments: > 1/ You have to deal with the items' fields too. I can mark a 995/952 field > as important, but the behavior is not implemented when adding items. > 2/ If a field is marked as important and mandatory, there are 2 stars (1 red > and 1 black), is it the expected behavior? Maybe the importance of a field > can be ignored if it is also marked as mandatory. I did not fix the first point as it requires more development, but for your second point I fixed it and it won't show important if the field is also mandatory. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8643] Add important constraint to marc subfields
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8643 --- Comment #28 from Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> --- Created attachment 60555 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60555&action=ed… Bug 8643: (follow-up) atomic update Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel(a)gmail.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8643] Add important constraint to marc subfields
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8643 Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8643] Add important constraint to marc subfields
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8643 Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #45834|0 |1 is obsolete| | Attachment #45835|0 |1 is obsolete| | Attachment #45836|0 |1 is obsolete| | Attachment #45837|0 |1 is obsolete| | --- Comment #27 from Mehdi Hamidi <mehdi.hamidi(a)inlibro.com> --- Created attachment 60554 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60554&action=ed… Bug 8643 - Add important constraint to marc subfields This fix permits to add an "Important" option to the marc structure pages. Testing: I Apply the patch II Run updatedatabase.pl 0) Define 100 as an "important" field ( Administration » MARC bibliographic framework » MARC structure ( Default Frameword) » Edit ) 1) Define 100$a as an "important" subfield (Administration » MARC bibliographic framework » MARC structure (Default Frameword) » Subfield » Onglet a) 2) Edit a record to clear the field 100 3) Save the record. 4) Validate the following message: A few important fields are not filled: * tag 100 subfield a Nom de personne in tab * Field 100 is important, at least one of its subfields should be filled. Are you sure you want to save? sponsored by the CCSR ( http://www.ccsr.qc.ca ) Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel(a)gmail.com> Work as described. Fixed small conflict -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12759] Add ability to pass list contents to batch record modification /deletion tools
by bugzilla-daemon＠bugs.koha-community.org 22 Feb '17

22 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12759 Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |jonathan.druart(a)bugs.koha-c | |ommunity.org --- Comment #8 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Aleisha, 2 small things here: 1. Go on Home › Tools › Batch record modification Select a list Select "authorities" Fill the textarea with authid Next => Id from list is picked 2. I think we should raise a warning (JS side) if 2 sources of ids are filled: * For biblios: file and list or file and ids or list and ids * For authorities: file and ids (list should not be taken into account even if filled) -- You are receiving this mail because: You are watching all bug changes.

1 0