[Koha-bugs] [Bug 16952] Add sorting rules for Czech language to Zebra
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16952 --- Comment #11 from Radek Šiman (R-Bit Technology, s.r.o.)--- Unfortunately it seems, there are some issues to be solved. For instance "Kýbl" and "Kapr" are sorted in incorrect order. http://prntscr.com/cwj9mj -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17297] Give librarians the ability to add messages to an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17297 Karam Qubsichanged: What|Removed |Added CC||karamqu...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17428] REST api: CRUD endpoint for cities
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17428 Karam Qubsichanged: What|Removed |Added CC||karamqu...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14224] patron notes about item shown at check in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14224 Aleisha Amohiachanged: What|Removed |Added CC||aleishaamo...@hotmail.com --- Comment #8 from Aleisha Amohia --- (In reply to Aleisha Amohia from comment #7) > - Still to do: > - GetIssue/GetIssues > - subroutine for ajax code > - moving ajax part to svc/ and tests for new subroutines. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14224] patron notes about item shown at check in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14224 Aleisha Amohiachanged: What|Removed |Added Attachment #41921|0 |1 is obsolete|| --- Comment #7 from Aleisha Amohia --- Created attachment 56682 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56682=edit Bug 14224: Allow patron notes about item shown at check in This patch adds a "Note" input field to checked out items in the "your summery" section. The field allows patrons to write notes about the item checked out, such as "this DVD is scratched", "the binding was torn", etc. The note will be emailed to the library and displayed on item check in. Patch adds two fields to the "issues" table - "note" and "notedate". Patch adds syspref "AllowIssueNotes" - default off. Test Plan: 1) Apply this patch 2) Run updatedatabase 3) Make sure the branch has an email address specified 4) Check out an item to a patron 5) Log in with the patron you checked out the item for 6) In "your summery", write a message in the new "Note" field for the item checked out. Hit ENTER to save/send. 7) Success message should be seen and an email sent to the email address specified for the branch which the item was checked out from. 8) Log in with staff account and check in the item 9) The note should be shown The note can be changed by the patron at any time while the item is checked out. Each change will send a new email to the branch. Only the latest note version will be shown on check in. === NOTE: - This patch is not yet ready for testing. This patch fixes merge conflicts and fixes the atomic update so this the database updates correctly. - Still to do: - GetIssue/GetIssues - subroutine for ajax code - moving ajax part to svc/ Sponsored-by: Catalyst IT -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17459] Add a script to create a superlibrarian user
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17459 Tomás Cohen Arazichanged: What|Removed |Added Version|unspecified |master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17469] fr-CA web installer is missing some sample notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 --- Comment #2 from Katrin Fischer--- Created attachment 56681 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56681=edit Bug 17469: Add missing sample notices to fr-CA web installer Adds missing notices to the fr-CA web installer: - MEMBERSHIP_EXPIRY - PASSWORD_RESET Renames RESERVESLIP to HOLD_SLIP Add fr-CA installer to xt/sample_letter.t To test: - Verify sample_notices.sql is correct SQL and loads cleanly - Verify xt/sample_letter.t test checks fr-CA now and runs all tests successfully Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17469] fr-CA web installer is missing some sample notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 Katrin Fischerchanged: What|Removed |Added Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17469] fr-CA web installer is missing some sample notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 Katrin Fischerchanged: What|Removed |Added Version|unspecified |master Assignee|koha-b...@lists.koha-commun |katrin.fisc...@bsz-bw.de |ity.org | --- Comment #1 from Katrin Fischer --- Can't provide a French translation, but especially the missing HOLD_SLIP could be annoying. Proposing a fix in English for now. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14622] Add fr-CA data folder in the web installer
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14622 Katrin Fischerchanged: What|Removed |Added Blocks||17469 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 [Bug 17469] fr-CA web installer is missing some sample notices -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14622] Add fr-CA data folder in the web installer
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14622 Katrin Fischerchanged: What|Removed |Added Blocks|17469 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 [Bug 17469] fr-CA web installer is missing some sample notices -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17469] fr-CA web installer is missing some sample notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 Katrin Fischerchanged: What|Removed |Added Depends on|5260|14622 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 [Bug 5260] Add option to send an order by e-mail to the acquisition module https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14622 [Bug 14622] Add fr-CA data folder in the web installer -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 5260] Add option to send an order by e-mail to the acquisition module
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 Katrin Fischerchanged: What|Removed |Added Blocks||17469 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 [Bug 17469] fr-CA web installer is missing some sample notices -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 5260] Add option to send an order by e-mail to the acquisition module
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 Katrin Fischerchanged: What|Removed |Added Blocks|17469 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 [Bug 17469] fr-CA web installer is missing some sample notices -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17469] fr-CA web installer is missing some sample notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 Katrin Fischerchanged: What|Removed |Added Depends on|14622 |5260 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 [Bug 5260] Add option to send an order by e-mail to the acquisition module https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14622 [Bug 14622] Add fr-CA data folder in the web installer -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14610] Add ability to place article requests in Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14610 --- Comment #227 from Katrin Fischer--- Still applies, next on my list! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14622] Add fr-CA data folder in the web installer
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14622 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de Blocks||17469 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 [Bug 17469] fr-CA web installer is missing some sample notices -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17469] fr-CA web installer is missing some sample notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 Katrin Fischerchanged: What|Removed |Added Depends on||14622 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14622 [Bug 14622] Add fr-CA data folder in the web installer -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17469] New: fr-CA web installer is missing some sample notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17469 Bug ID: 17469 Summary: fr-CA web installer is missing some sample notices Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Installation and upgrade (web-based installer) Assignee: koha-bugs@lists.koha-community.org Reporter: katrin.fisc...@bsz-bw.de QA Contact: testo...@bugs.koha-community.org CC: francois.charbonn...@inlibro.com, gmcha...@gmail.com xt/sample_notice.t fails when you add fr-CA to it, because the file is missing some of the sample notices: Sample notices to add in fr-CA/obligatoire/sample_notices.sql: HOLD_SLIP, MEMBERSHIP_EXPIRY, PASSWORD_RESET Sample notices to delete in fr-CA/obligatoire/sample_notices.sql: RESERVESLIP (was renamed to be HOLD_SLIP!) The file and the test should be fixed! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 5260] Add option to send an order by e-mail to the acquisition module
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 Katrin Fischerchanged: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 5260] Add option to send an order by e-mail to the acquisition module
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 --- Comment #48 from Katrin Fischer--- Created attachment 56680 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56680=edit Bug 5260: QA follow-up - Add sample ACQORDER notice to sample notices Adds the new notice ACQORDER to all web installers. Verify by testing the SQL contains no errors and by running: perl xt/sample_notices.t NOTE: - de-DE is already translated, others are in English. - xt/sample_notices.t doesn't check fr-CA yet. I didn't fix this here as the file is missing other notices as well which should be fixed separately. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 5260] Add option to send an order by e-mail to the acquisition module
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 --- Comment #47 from Katrin Fischer--- Hi Jonathan, thx for the review and catching this! I have (hopefully) fixed the problem. Can you please take a look at the follow-up patch? > Is there a good reason not to add the new ACQORDER notice? No good reason really, more an oversight! I will work on another follow-up covering it. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 5260] Add option to send an order by e-mail to the acquisition module
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5260 --- Comment #46 from Katrin Fischer--- Created attachment 56679 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56679=edit Bug 5260: QA follow-up: Fix error when no notice template is defined When no notice template ACQORDER was defined, you'r receive a false positive "email sent" message. Now it will display a specific error message instead. Also includes 2 unit tests to test for the warn and new error code. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16873] Renewal error message not specific enough on self check.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16873 Josef Moravecchanged: What|Removed |Added Status|Needs Signoff |Signed Off CC||josef.mora...@gmail.com --- Comment #6 from Josef Moravec --- Everything works as described. I've rebased on the master. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16873] Renewal error message not specific enough on self check.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16873 Josef Moravecchanged: What|Removed |Added Attachment #55503|0 |1 is obsolete|| --- Comment #5 from Josef Moravec --- Created attachment 56678 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56678=edit [SIGNED-OFF] Bug 16873: Improve renewal error messages on self check When a patron is not allowed to renew from the self check module, the only message displayed is "No renewals allowed". It would be nicer to let him/her know that the renewal is not allowed because it's a on-site checkout or automatic renewal. To do so we can call CanBookBeRenewed instead of CanBookBeIssued and get the renewal error. Test plan: 0/ Switch off AllowSelfCheckReturns 1/ check out an item and tick "auto renewal" 2/ Go on the self check module => auto renewal message is displayed 3/ check out an item and tick "on-site checkout" 4/ Go on the self check module => on-site checkout message is displayed 5/ check out an item without ticking any checkboxes (regular checkout) Renew it to reach the max renew allowed 6/ Go on the self check module => regular checkout message is displayed 7/ Switch on AllowSelfCheckReturns and repeat previous steps => "Return this item" button is displayed in addition of the renewal error message Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16344] Add a circ rule to limit the auto renewals given a specific date
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16344 --- Comment #14 from Michael Kuhn--- According to the test plan I did the following. I successfully applied patches 15581 and 16344. 0) I executed the update DB entry vagrant@kohadevbox:kohaclone(003)$ sudo koha-shell kohadev kohadev-koha@kohadevbox:/home/vagrant/kohaclone$ perl installer/data/mysql/updatedatabase.pl DEV atomic update: bug_15581.sql DEV atomic update: bug_16344.sql 1) In Koha menu "Administration > Circulation and fine rules" I defined a rule for all categories and all item types: * Renewals allowed (count): 3 * No automatic renewal after (hard limit): 20/10/2016 Then I checked out the item with number 1 (otherwise it couldn't be updated with following statement). 2) I modified the table "issues" with issuedate two days before as followes: UPDATE issues SET issuedate = "2016-10-17 22:48:00" WHERE itemnumber=1; 3) I executed the renewals cronjob script as follows: vagrant@kohadevbox:kohaclone(003)$ sudo koha-shell kohadev kohadev-koha@kohadevbox:/home/vagrant/kohaclone$ misc/cronjobs/automatic_renewals.pl I cannot confirm that the issue has been renewed. Instead, nothing has changed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17385] Add custom export notices
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17385 --- Comment #5 from Michael Kuhn--- I applied the patch again. The following is as you describe: * Patch can be applied * The folder /koha-tmpl/intranet-tmpl/prog/en/xslt/biblioexport for intranet is there * The folder /koha-tmpl/opac-tmpl/bootstrap/en/xslt/biblioexport for opac is there * In the folder /koha-tmpl/intranet-tmpl/prog/en/xslt/biblioexport-samples there is the sample file "MARC21_simple_export.xsl" * This file can also be found in /koha-tmpl/intranet-tmpl/prog/en/xslt/biblioexport for intranet and in /koha-tmpl/opac-tmpl/bootstrap/en/xslt/biblioexport for opac * I open the Intranet interface, submit a search, click the title of a found item Then I click the button "Save", but I can only see formats BIBTEX, Dublin Core, MARCXML, three kinds of MARC, MODS and RIS - but no "Simple Export". In the OPAC interface it is almost the same. There I click the button "Save record" but there are only the formats already given above, plus one more (ISBD). But no "Simple Export". -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14637] Add patron category fails with MySQL 5.6.26
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14637 Josef Moravecchanged: What|Removed |Added Status|Needs Signoff |Signed Off CC||josef.mora...@gmail.com --- Comment #10 from Josef Moravec --- So it looks like this kind of issue could be in more places, is it true? I agree with you, that it would be better to fix it generally, but I have no idea, how to do it... But if we'd go with this solution, I am adding sign-off as it works as described. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14637] Add patron category fails with MySQL 5.6.26
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14637 --- Comment #9 from Josef Moravec--- Created attachment 56677 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56677=edit Bug 14637: Followup - Fix number of tests due to rebase -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14637] Add patron category fails with MySQL 5.6.26
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14637 Josef Moravecchanged: What|Removed |Added Attachment #55233|0 |1 is obsolete|| --- Comment #8 from Josef Moravec --- Created attachment 56676 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56676=edit [SIGNED-OFF] Bug 14637: Fix add patron category under MySQL 5.7 If no dateofbirthrequired or upperagelimit is set on the interface, the ->store method will receive an empty string defined for these values. For INT field, we must explicitely set these empty value to undef instead to avoid MySQL 5.7 (and strict mode) to raise: DBD::mysql::st execute failed: Incorrect integer value: ' for column 'dateofbirthrequired'' Test plan: Using MySQL 5.7 (and/or sql_mode=STRICT_TRANS_TABLES) Create a patron category without specifying upperagelimit or dateofbirthrequired Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14637] Add patron category fails with MySQL 5.6.26
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14637 Josef Moravecchanged: What|Removed |Added Attachment #55232|0 |1 is obsolete|| Attachment #56674|0 |1 is obsolete|| --- Comment #7 from Josef Moravec --- Created attachment 56675 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56675=edit [SIGNED-OFF] Bug 14637: Fix add patron category under MySQL 5.7 - tests Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14637] Add patron category fails with MySQL 5.6.26
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14637 --- Comment #6 from Josef Moravec--- Created attachment 56674 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56674=edit [SIGNED-OFF] Bug 14637: Fix add patron category under MySQL 5.7 - tests Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17468] Remove koha-*-zebra scripts in favor of koha-zebra
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17468 Tomás Cohen Arazichanged: What|Removed |Added CC||bar...@bywatersolutions.com ||, mi...@abunchofthings.net -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 5620] Capture Mode of payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5620 Josef Moravecchanged: What|Removed |Added CC||josef.mora...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17468] Remove koha-*-zebra scripts in favor of koha-zebra
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17468 --- Comment #1 from Tomás Cohen Arazi--- Created attachment 56672 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56672=edit Bug 17468: Remove koha-*-zebra scripts and its traces -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17468] Remove koha-*-zebra scripts in favor of koha-zebra
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17468 --- Comment #2 from Tomás Cohen Arazi--- Created attachment 56673 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56673=edit Bug 17468: Create koha-*-zebra as symlinks to koha-zebra for backwards compatibility -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17459] Add a script to create a superlibrarian user
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17459 Josef Moravecchanged: What|Removed |Added CC||josef.mora...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17459] Add a script to create a superlibrarian user
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17459 Josef Moravecchanged: What|Removed |Added Attachment #56630|0 |1 is obsolete|| --- Comment #7 from Josef Moravec --- Created attachment 56670 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56670=edit [SIGNED-OFF] Bug 17459: Add a script to create a superlibrarian user This is for developers: it's quite long (many clics) to create a new superlibrarian user. This new script creates a new user with superlibrarian permissions with the easy to remember credential koha/koha Test plan: perl misc/devel/create_superlibrarian.pl Log in to Koha using koha/koha Signed-off-by: Chris Cormack Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17459] Add a script to create a superlibrarian user
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17459 Josef Moravecchanged: What|Removed |Added Attachment #56632|0 |1 is obsolete|| --- Comment #8 from Josef Moravec --- Created attachment 56671 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56671=edit [SIGNED-OFF] Bug 17459 Follow up Adding POD and --userid and --password options 1/ To test, use the same routine as before, with no options. 2/ You should have a user with koha/koha as userid and passwords 3/ Delete that user 4/ Run the script with --userid --password 5/ You should have a user in koha with userid/password set Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17459] Add a script to create a superlibrarian user
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17459 Josef Moravecchanged: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17255] Upgrade Elastic Search code to work with version 2.4+
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17255 --- Comment #18 from Mirko Tietgen--- There is no newer version in the Debian repos yet. You can use https://apt.abunchofthings.net/debian/pool/main/libc/libcatmandu-perl/libcatmandu-perl_1.0002-1_all.deb for the time being (and please report back if that works) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 --- Comment #5 from Tomás Cohen Arazi--- Created attachment 56669 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56669=edit Bug 17467: Add tab-completion for koha-zebra This patch adds tab-completion for bash, for the koha-zebra script. To test: - Run (on the vagrant user): $ source kohaclone/debian/koha-common.bash-completion - Run: $ sudo koha-zebra - ( means press the tab key) => SUCCESS: Tab completion works as expected for the koha-zebra command. - Sign off :-D -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 11682] serials.status is too dependent on magic numbers
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11682 Marjorie Barry-Vilachanged: What|Removed |Added CC||marjorie.barry-v...@ccsr.qc ||.ca -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16510] When record items exceed OpacMaxItemsToDisplay opacSerialDefaultTab is ignored
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16510 Marjorie Barry-Vilachanged: What|Removed |Added CC||marjorie.barry-v...@ccsr.qc ||.ca -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8752] Log circulation renewal
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8752 Katrin Fischerchanged: What|Removed |Added Status|In Discussion |Failed QA --- Comment #8 from Katrin Fischer --- Moving to Failed QA for the double logging - can we fix this? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17436] Add a way to know in patron module where renew of an item was made
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17436 Sally Healeychanged: What|Removed |Added CC||sally.healey@cheshireshared ||services.gov.uk -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17373] Elasticsearch - Authority mappings for UNIMARC
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17373 --- Comment #3 from Jonathan Druart--- Please provide a test plan. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16330] REST API: add routes to add, update and delete patrons
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16330 Tomás Cohen Arazichanged: What|Removed |Added Status|Needs Signoff |In Discussion --- Comment #36 from Tomás Cohen Arazi --- Lari, I don't think we should have such a 'validate' method. We should look at Jonathan's code to move C4::Members methods into Koha::Patron(s) and have them raise those exceptions, and build the endpoint on top of those. Please contact him if you are willing to help him and make this happen. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17047] Mana webservice : share and use common information between Kohas
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17047 Jonathan Druartchanged: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #17 from Jonathan Druart --- Is it ready for testing? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16330] REST API: add routes to add, update and delete patrons
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16330 --- Comment #33 from Lari Taskula--- Created attachment 5 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5=edit Bug 16330: (QA-follow-up) Fix permission for DELETE DELETE should not be allowed for user himself unless he has borrowers permission. Covered by a test. To test: 1. Apply patch 2. Run t/db_dependent/api/v1/patrons.t -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16330] REST API: add routes to add, update and delete patrons
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16330 --- Comment #35 from Lari Taskula--- Created attachment 56668 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56668=edit Bug 16330: (follow-up) Move validation logic from controller to Koha::Patron This patch moves validation logic from controller into Koha::Patron object and takes advantage of Koha::Exceptions for throwing proper exceptions and catching them inside controller in order to return correct error messages. To test: 1. Apply this patch 2. Run tests perl t/db_dependent/api/v1/patrons.t 3. Follow the original test plan and observe it still applying -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16330] REST API: add routes to add, update and delete patrons
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16330 --- Comment #34 from Lari Taskula--- Created attachment 56667 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56667=edit Bug 16330: (follow-up) Add Test::Fatal Perl dependency Test::Fatal is a simple module for testing exception-based code. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16330] REST API: add routes to add, update and delete patrons
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16330 Lari Taskulachanged: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 --- Comment #4 from Tomás Cohen Arazi--- Created attachment 56665 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56665=edit Bug 17467: (followup) Make the install script install koha-zebra -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14237] allow bibs to be added to course without items
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14237 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de --- Comment #2 from Katrin Fischer --- We just had a request for adding records without items too - It would not only helfpul for electronic, but also for analytical records. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 11062] Invoice's shipment/ shipping cost should be included PRE-gst tax (as an option)
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11062 Jonathan Druartchanged: What|Removed |Added Assignee|jonathan.dru...@bugs.koha-c |koha-b...@lists.koha-commun |ommunity.org|ity.org -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 Tomás Cohen Arazichanged: What|Removed |Added Blocks||17468 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17468 [Bug 17468] Remove koha-*-zebra scripts in favor of koha-zebra -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17468] Remove koha-*-zebra scripts in favor of koha-zebra
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17468 Tomás Cohen Arazichanged: What|Removed |Added Assignee|gmcha...@gmail.com |tomasco...@gmail.com Depends on||17467 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17468] New: Remove koha-*-zebra scripts in favor of koha-zebra
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17468 Bug ID: 17468 Summary: Remove koha-*-zebra scripts in favor of koha-zebra Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: gmcha...@gmail.com Reporter: tomasco...@gmail.com QA Contact: testo...@bugs.koha-community.org CC: ro...@catalyst.net.nz -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 Tomás Cohen Arazichanged: What|Removed |Added CC||bar...@bywatersolutions.com ||, mi...@abunchofthings.net Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 --- Comment #3 from Tomás Cohen Arazi--- Created attachment 56664 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56664=edit Bug 17467: Add koha-zebra documentation -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 --- Comment #2 from Tomás Cohen Arazi--- Created attachment 56663 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56663=edit Bug 17467: (followup) Make koha-zebra handle legacy script names This patch adds the new koha-zebra script the ability to handle being called by other script names, for backwards compatibility with the legacy koha-*-zebra scripts. It is intended to be used in conjunction with a patch that replaces the old koha-*-zebra scripts for symbolic links, pointing to the new koha-zebra script. To test: - Apply this patch - Create symbolic links to the new script: vagrant@kohadevbox:~$ ln -s kohaclone/debian/scripts/koha-zebra koha-start-zebra vagrant@kohadevbox:~$ ln -s kohaclone/debian/scripts/koha-zebra koha-stop-zebra vagrant@kohadevbox:~$ ln -s kohaclone/debian/scripts/koha-zebra koha-restart-zebra - Run: $ sudo ./koha-start-zebra kohadev => SUCCESS: Same behaviour than the koha-start-zebra script - Run: sudo ./koha-stop-zebra kohadev => SUCCESS: Same behaviour than the koha-stop-zebra script - Run: sudo ./koha-restart-zebra kohadev => SUCCESS: Same behaviour than the koha-restart-zebra script -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 --- Comment #1 from Tomás Cohen Arazi--- Created attachment 56662 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56662=edit Bug 17467: Add koha-zebra script to handle Zebra daemons This script is intended to replace the following currently used scripts on packages setups: - koha-start-zebra - koha-stop-zebra - koha-restart-zebra It also introduces a --status option switch, for asking for daemon statuses To test - Apply the patch - Run: $ sudo debian/scripts/koha-zebra --start kohadev => SUCCESS: same behaviour than koha-start-zebra kohadev - Run: $ sudo debian/scripts/koha-zebra --stop kohadev => SUCCESS: same behaviour as koha-stop-zebra kohadev - Run: $ sudo debian/scripts/koha-zebra --restart kohadev => SUCCESS: same behaviour than koha-restart-zebra kohadev - Run: $ sudo debian/scripts/koha-zebra --status kohadev => SUCCESS: It correctly shows the status for the running (or not) process Play with different combinations of this commands -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17445] REST API: Generic handling of malformed query parameters
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17445 Tomás Cohen Arazichanged: What|Removed |Added Attachment #56634|0 |1 is obsolete|| --- Comment #12 from Tomás Cohen Arazi --- Created attachment 56659 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56659=edit Bug 17445: Add 'malformed query' error response This patch adds to the x-mojo-around action code to give a meaningful error given a bad query parameter in the query string for a request. Test Plan 1) Submit an api request to an existing restful endpoint with no query parameters 2) Confirm the correct response is being given 3) Submit an api request to an existing restful endpoint with an allowed query parameter 4) Confimr the correct response is being given 5) Submit an api request to an existing restful endpoint with a malformed query paramter 6) Note the 400 response code and helpful json api body https://bugs.koha-community.org/show_bug.cgi?id=17445 Signed-off-by: Josef Moravec Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17445] REST API: Generic handling of malformed query parameters
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17445 Tomás Cohen Arazichanged: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17445] REST API: Generic handling of malformed query parameters
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17445 Tomás Cohen Arazichanged: What|Removed |Added Attachment #56636|0 |1 is obsolete|| --- Comment #14 from Tomás Cohen Arazi --- Created attachment 56661 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56661=edit Bug 17445: Move the params check after the authentication check If the user is not authorised to call this route, we would prefer to raise a 403 instead of 400 Note that we wanted to submit tests for this change but the city code does not let use do that (we are allowed to list/show cities even without any permissions). The patrons.t is not complete enought and the holds.t tests do not pass... Tomas plans to submit tests but we reach the end of the hackfest ;) Signed-off-by: Josef Moravec Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17445] REST API: Generic handling of malformed query parameters
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17445 Tomás Cohen Arazichanged: What|Removed |Added Attachment #56635|0 |1 is obsolete|| --- Comment #13 from Tomás Cohen Arazi --- Created attachment 56660 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56660=edit Bug 17445: Just some refactoring The idea is to make the code more readable moving the code to its own subroutine Signed-off-by: Josef Moravec Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17431] Fix failing test t/db_dependent/api/v1/holds.t
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17431 --- Comment #8 from Jonathan Druart--- Created attachment 56658 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56658=edit Bug 17431: Create the object we need instead of assuming something exists Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17431] Fix failing test t/db_dependent/api/v1/holds.t
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17431 --- Comment #7 from Jonathan Druart--- Created attachment 56657 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56657=edit Bug 17431: (follow-up) Fix issuingrules Issuingrules could had prevented holds.t from passing with error tooManyReserves. This patch sets issuingrules at the beginning of the test to make sure reserves are allowed. To test: 1. Apply patch & Run t/db_dependent/api/v1/holds.t 2. Observe test pass Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17431] Fix failing test t/db_dependent/api/v1/holds.t
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17431 Jonathan Druartchanged: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17431] Fix failing test t/db_dependent/api/v1/holds.t
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17431 Jonathan Druartchanged: What|Removed |Added Attachment #56288|0 |1 is obsolete|| Attachment #56655|0 |1 is obsolete|| --- Comment #6 from Jonathan Druart --- Created attachment 56656 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56656=edit Bug 17431 - Fix failing test t/db_dependent/api/v1/holds.t This patch fixed failing test for t/db_dependent/api/v1/holds.t which was broken after Bug 14695. $ perl t/db_dependent/api/v1/holds.t ... not ok 42 - similar match for JSON Pointer "/error" # Failed test 'similar match for JSON Pointer "/error"' # at t/db_dependent/api/v1/holds.t line 291. # 'Reserve cannot be placed. Reason: itemAlreadyOnHold' # doesn't match '(?^u:tooManyReserves)' # Looks like you failed 1 test of 42. not ok 4 - Test endpoints with permission To test: 1. Run t/db_dependent/api/v1/holds.t and observe it fail. 2. Apply patch. 3. Run the test again and observe it pass. Signed-off-by: Marc Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 Tomás Cohen Arazichanged: What|Removed |Added Assignee|gmcha...@gmail.com |tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17467] New: There should be a koha-zebra script to handle Zebra daemons for instances
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17467 Bug ID: 17467 Summary: There should be a koha-zebra script to handle Zebra daemons for instances Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: gmcha...@gmail.com Reporter: tomasco...@gmail.com QA Contact: testo...@bugs.koha-community.org CC: ro...@catalyst.net.nz There's really no need for one script per action. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17431] Fix failing test t/db_dependent/api/v1/holds.t
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17431 --- Comment #5 from Lari Taskula--- Does this patch fix it? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17431] Fix failing test t/db_dependent/api/v1/holds.t
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17431 --- Comment #4 from Lari Taskula--- Created attachment 56655 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56655=edit Bug 17431: (follow-up) Fix issuingrules Issuingrules could had prevented holds.t from passing with error tooManyReserves. This patch sets issuingrules at the beginning of the test to make sure reserves are allowed. To test: 1. Apply patch & Run t/db_dependent/api/v1/holds.t 2. Observe test pass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17431] Fix failing test t/db_dependent/api/v1/holds.t
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17431 Jonathan Druartchanged: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org, ||tomasco...@gmail.com --- Comment #3 from Jonathan Druart --- The tests still don't pass, Tomas could you have a look? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17255] Upgrade Elastic Search code to work with version 2.4+
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17255 --- Comment #17 from Jonathan Druart--- I did not install the second .deb, and it fails for me: % sudo dpkg -i attachment.cgi?id=55206 # libcatmandu-store-elasticsearch-perl_0.0501-1_all.deb (Reading database ... 83863 files and directories currently installed.) Preparing to unpack attachment.cgi?id=55206 ... Unpacking libcatmandu-store-elasticsearch-perl (0.0501-1) over (0.0304-1) ... dpkg: dependency problems prevent configuration of libcatmandu-store-elasticsearch-perl: libcatmandu-store-elasticsearch-perl depends on libcatmandu-perl (>= 1.01); however: Version of libcatmandu-perl on system is 0.9206-1. dpkg: error processing package libcatmandu-store-elasticsearch-perl (--install): dependency problems - leaving unconfigured Processing triggers for man-db (2.7.5-1) ... Errors were encountered while processing: libcatmandu-store-elasticsearch-perl % apt policy libcatmandu-perl libcatmandu-perl: Installed: 0.9206-1 Candidate: 0.9505-1 Version table: 0.9505-1 500 500 http://ftp.fr.debian.org/debian testing/main i386 Packages *** 0.9206-1 100 100 /var/lib/dpkg/status I am willing to test but I will need a test plan to upgrade my packages please. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17255] Upgrade Elastic Search code to work with version 2.4+
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17255 --- Comment #16 from Jonathan Druart--- It seems that I get facets for MARC21 but UNIMARC (?) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17255] Upgrade Elastic Search code to work with version 2.4+
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17255 --- Comment #15 from Jonathan Druart--- perl misc/search_tools/rebuild_elastic_search.pl -d -a -b -v problem with :16 : :1: parser error : Start tag expected, '<' not found podigpodfigo ^ podigpodfigo at C4/Biblio.pm line 1353. No record passed at Koha/MetadataIterator.pm line 71. Something went wrong reading record for biblio Koha::Schema::Result::Biblio=HASH(0xd904770)->biblionumber: Can't bless non-reference value at Koha/BiblioUtils.pm line 70. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17255] Upgrade Elastic Search code to work with version 2.4+
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17255 --- Comment #14 from Jonathan Druart--- Note: The errors don't go to the error file but to stderr -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17466] Show number of outstanding issues when checking in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17466 Josef Moravecchanged: What|Removed |Added Assignee|koha-b...@lists.koha-commun |josef.mora...@gmail.com |ity.org | CC||blac...@gmail.com, ||r...@rbit.cz -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17466] Show number of outstanding issues when checking in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17466 --- Comment #1 from Josef Moravec--- Created attachment 56654 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56654=edit Bug 17466: Show number of outstanding issues when checking in Test plan: 1) Apply patch 2) Checkout more then one item to a borrower 3) Check one of that items 4) Note, that on in returns table there is a button with number of outstanding issues in patron column. The button should take you to checking out page 5) When the outstanding issues count is 0, there is no button -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17466] Show number of outstanding issues when checking in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17466 Josef Moravecchanged: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 17466] New: Show number of outstanding issues when checking in
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17466 Bug ID: 17466 Summary: Show number of outstanding issues when checking in Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: josef.mora...@gmail.com QA Contact: testo...@bugs.koha-community.org CC: gmcha...@gmail.com, kyle.m.h...@gmail.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16965] Add the Koha::Objects->search_related method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16965 Martin Renvoizechanged: What|Removed |Added QA Contact|testo...@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16965] Add the Koha::Objects->search_related method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16965 --- Comment #32 from Martin Renvoize--- So much as I'm an opponent of Koha::Objects (They still just feel like boilerplate for the point of boilerplate to me), I can't find anything desperately wrong with this code. Passing QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16965] Add the Koha::Objects->search_related method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16965 --- Comment #30 from Martin Renvoize--- Created attachment 56652 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56652=edit [PASSED QA] Bug 16965: [Follow-up] Adjust get_object_class In many cases it should be possible to derive the Koha::Objects class name from the DBIx result class name in a trivial manner. If not, the DBIx result class should have a class method called koha_objects_class providing that non-trivial name, such as Libraries for Branch. Note: We are only interested in the plural form here (Koha::Objects). Test plan: Run t/db_dependent/Koha/Objects.t Signed-off-by: Marc Véron Signed-off-by: Tomas Cohen Arazi Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16965] Add the Koha::Objects->search_related method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16965 --- Comment #29 from Martin Renvoize--- Created attachment 56651 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56651=edit [PASSED QA] Bug 16965: search_related returns an instanciated Koha::Objects-based object Koha::Objects->search_related should return a Koha::Objects-based object. This search_related method should follow the same rules as the search method, i.e. take into account what the caller want (scalar or list). The problem here is that we do not know (in Koha::Objects) what is the kind of objects we want to instanciate. To know it, this patch adds a get_object_class, it will return the class of the object and the resultset Koha::Object-based object. The drawback of this method is that we will have to keep it up-to-date every time we add a new Koha::Object class. Signed-off-by: Marc Véron Signed-off-by: Tomas Cohen Arazi Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16965] Add the Koha::Objects->search_related method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16965 --- Comment #31 from Martin Renvoize--- Created attachment 56653 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56653=edit [PASSED QA] Bug 16965: Allow Koha::Objects->result_class Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16965] Add the Koha::Objects->search_related method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16965 Martin Renvoizechanged: What|Removed |Added Attachment #56372|0 |1 is obsolete|| Attachment #56373|0 |1 is obsolete|| Attachment #56374|0 |1 is obsolete|| Attachment #56375|0 |1 is obsolete|| --- Comment #28 from Martin Renvoize --- Created attachment 56650 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56650=edit [PASSED QA] Bug 16965: Add Koha::Objects->search_related In order to search on relations, we need this new method. Test plan: Confirm that the changes in Objects.t make sense and that the tests pass. Tested all 3 patches together, followed test plan, result OK Signed-off-by: Marc Véron Signed-off-by: Tomas Cohen Arazi Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16965] Add the Koha::Objects->search_related method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16965 Martin Renvoizechanged: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14752] Add multiple copies to a basket at once
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14752 --- Comment #38 from Nick Clemens--- Jonathan, can you try the added patch to see if that fixes your error? It works for me and was the only way I triggered the error -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14752] Add multiple copies to a basket at once
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14752 --- Comment #37 from Nick Clemens--- Created attachment 56649 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56649=edit Bug 14752 - Followup to fix error if barcode fields undefined -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14957] Write protecting MARC fields based on source of import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 --- Comment #23 from Josef Moravec--- Rebased on master Fixed few issues in the follow-up -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14957] Write protecting MARC fields based on source of import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 Josef Moravecchanged: What|Removed |Added Attachment #51724|0 |1 is obsolete|| --- Comment #20 from Josef Moravec --- Created attachment 56646 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56646=edit Bug 14957: Correct parameters for TransformMarcToKoha -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14957] Write protecting MARC fields based on source of import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 Josef Moravecchanged: What|Removed |Added Attachment #56638|0 |1 is obsolete|| --- Comment #21 from Josef Moravec --- Created attachment 56647 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56647=edit Bug 14957: Key fixes to patch - Fixing merge conflicts - adding String::Similariry to list of dependencies - font-awesome icons and adding consistency to formatting - removing original use of onclick - moving help text out of main page and into a new help file When testing, follow above test plan but: 1) Remember to install String::Similarity before testing (apt-get install libstring-similarity-perl) 2) Instead of doing step 7, click Help in top-right corner instead. Confirm this works and shows as expected Sponsored-by: Catalyst IT -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14957] Write protecting MARC fields based on source of import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 --- Comment #22 from Josef Moravec--- Created attachment 56648 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56648=edit Bug 14957: Followup - Pod, import_lexile.pl, template and test fixes -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14957] Write protecting MARC fields based on source of import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 Josef Moravecchanged: What|Removed |Added Attachment #44058|0 |1 is obsolete|| --- Comment #18 from Josef Moravec --- Created attachment 56644 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56644=edit Bug 14957: Add filters to relevant scripts Add MARCPermissions filters in scripts which modifies MARC records to enable the permission system for these scripts. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14957] Write protecting MARC fields based on source of import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 Josef Moravecchanged: What|Removed |Added Attachment #51723|0 |1 is obsolete|| --- Comment #19 from Josef Moravec --- Created attachment 56645 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56645=edit Bug 14957: Rule based permission for MARC fields This patch adds a rule based permission system for MARC field modifications. Test plan: 1. Apply this patch 2. Apply related database patch 2. Apply related scripts update patch 3. Run updatedatabase.pl 5. Log in to staff client 4. Enable new syspref MARCPermissions 5. You should see a new link "MARC field permissions" in the "Catalog" section under "Koha administrtion". 6. Go to the "MARC field permissions" page 7. Next to the page heading ("Manage MARC field permissions") there is a button with a question mark, click this for more information about the rule system. 8. Add desired rules to the rule table, press the +-button to add. 9. Clicking the pen-icon (edit) should allow you to edit corresponding rule. 10. Clicking the x-icon should remove corresponding rule after confirmation. 11. Selecting one or more rules followed by clicking the trash-icon should remove all selected rules after confirmation. 12. To test your rules, import or in other way modify a record. 13. Check that the record was modified as defined by your rules. Sponsored-by: Halland County Library -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 14957] Write protecting MARC fields based on source of import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 Josef Moravecchanged: What|Removed |Added Attachment #44057|0 |1 is obsolete|| --- Comment #17 from Josef Moravec --- Created attachment 56643 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56643=edit Bug 14957: Database changes -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/