[Koha-bugs] [Bug 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 --- Comment #22 from Marcel de Rooy --- Created attachment 60375 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60375&action=edit Bug 15498: Fix show/hide csv profile list 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 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 --- Comment #23 from Marcel de Rooy --- Created attachment 60376 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60376&action=edit Bug 15498: [QA Follow-up] Resolves: forbidden pattern: trailing space char (line 334) -- 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 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 --- Comment #24 from Marcel de Rooy --- Trivial 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 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 --- Comment #21 from Marcel de Rooy --- Created attachment 60374 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60374&action=edit Bug 15498: Do not display sql csv profiles 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 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 --- Comment #20 from Marcel de Rooy --- Created attachment 60373 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60373&action=edit Bug 15498: Add ExportCircHistory and remove ExportWithCsvProfile 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 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 --- Comment #19 from Marcel de Rooy --- Created attachment 60372 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60372&action=edit Bug 15498: Let the user choose the CSV profile to export circ history The way the export options are displayed at the bottom of the checkouts table was not consistent. Prior to this patch set, they are display if ExportRemoveFields or ExportWithCsvProfile is set. It does not make any sense, the user could want to export the checkouts in iso2709 format without having to define a csv profile and fill the pref. Moreover the behavior of this pref did not match its description: it's used as a default CSV profile when exporting records from the export tools or the command line. This patch set adds a new pref ExportCircHistory and remove ExportWithCsvProfile. The new pref is set if ExportWithCsvProfile or ExportRemoveFields were set. A new dropdown list with the CSV profile list will be displayed in the export area, at the bottom of the checkouts table. Note that now --csv_profile_id is mandatory for the export command line (misc/export_records.pl) if the export format is csv. Test plan: 0/ Do not execute the DB entry 1/ Clear both ExportWithCsvProfile and ExportRemoveFields prefs 2/ Execute the DB entry 3/ ExportCircHistory should not be set and the export options should not be displayed at the bottom of the checkouts table. 4/ Remove the pref DELETE FROM systempreferences WHERE variable='ExportCircHistory'; and reinsert the previous one, with a value: INSERT INTO systempreferences (variable, value) VALUES ('ExportWithCsvProfile', 'something'); Execute the DB entry again => The now pref should be now set 5/ Export some checkouts using the CSV entry 6/ Note that the export tool and commandline script still work using the csv format. You have to provide a --csv_profile_id option to make it work. 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 16966] Koha:: Patrons - Move GetBorrowersWithIssuesHistoryOlderThan to search_patrons_to_anonymise
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16966 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl --- Comment #29 from Marcel de Rooy --- QA: Looking at this one now -- 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 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | -- 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 15498] Replace ExportWithCsvProfile with ExportCircHistory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15498 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl Status|Signed Off |Failed QA --- Comment #18 from Marcel de Rooy --- QA Comment: It would have been helpful if I had a more clear description of what this patch set wants to accomplish. [1] Going through the code changes, you remove ExportWithCsvProfile and you add ExportCircHistory. You also touch ExportRemoveFields. ExportRemoveFields is still found in koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref. But it is not in sysprefs.sql. Still in use in tools/export.pl. But I would also expect these fields to pop up at the circulation screen in the Dont export fields box. [2] I do not see the combo (select name="csv_profile_id"). I guess it is hidden by js in checkouts.js. But also if I select csv, it does not show up. Refreshed, restarted, flushed, etc. No results. (I expected a combo with two csv profiles.) Failed 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 10679] Lists interface offers delete option when user doesn' t have permission
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10679 --- Comment #7 from Marcel de Rooy --- We're not there yet. But we will! Will submit a more detailed comment later. -- 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 17379] Add a man page for koha-passwd
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17379 --- Comment #15 from Magnus Enger --- Removing the AUTHOR section from all man pages is probably out of scope for the current bug report? We could do that as a separate bug. -- 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 18043] Updating buttons for z39.50 cataloguing search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18043 Marc Véron changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #7 from Marc Véron --- "Go" button works now, but not next / previous page buttons. Oh, they miss 'value' as well (value="Next page" / value="Previous page") -- 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 10978] redirect to basket list of a supplier after deleting a basket
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10978 Marc Véron changed: 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 10978] redirect to basket list of a supplier after deleting a basket
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10978 Marc Véron changed: What|Removed |Added Attachment #60365|0 |1 is obsolete|| --- Comment #8 from Marc Véron --- Created attachment 60371 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60371&action=edit Bug 10978: Redirect to basket list of a supplier after deleting a basket This patch redirects to the vendor's list of baskets after deleting a basket, fixes breadcrumbs after deletion and also hides the toolbar actions after deletion (seeing as you can't edit/export etc a basket that no longer exists). To test: 1) Go to Acquisitions -> find a vendor -> view a basket or create a new basket 2) Delete the basket. Notice you are taken to a list of all vendors and baskets 3) Apply patch and do step 1 again 4) Delete the basket. Notice appropriate breadcrumbs, no toolbar, and confirm link to return to baskets for the vendor works. Sponsored-by: Catalyst IT Followed test plan, works as expected (links to vendor's baskets and all baskets) Signed-off-by: Marc Véron Re-tested. Wording of buttons "Show baskets for vendor..." and "Show all active baskets" makes sense. Signed-off-by: Marc Véron -- 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 17942] Update style of the web installer with Bootstrap 3
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17942 Alex Buckley changed: What|Removed |Added CC||alexbuck...@catalyst.net.nz --- Comment #33 from Alex Buckley --- I have tested bug 17855 (and obsoleted the old patches), bug 18039 and this bug together. One thing I would like to suggest is that the IF statement check for the existence of libraries, categories and item types in the onboardingstep1.tt, onboardingstep2, and onboardingstep4.tt files is moved out of the head element and back down each file to where it was originally written in bug 17855. I am happy to do a commit of this. The reason I am suggesting it is the redirection in the head element is not immediate and so the user has the form to create a library, patron category, item type displayed for a few seconds before they are redirected. This could be rather confusing for new users. By having the library,category,itemtype check as part of a longer if statement that had the elseif check of 'if add_validate_' was true. Then the form would not be displayed whilst redirection was loading the next appropriate form. I am writing this to find out if there will be any objections to me making this change -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59856|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59857|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59826|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59840|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59712|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59711|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59710|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59279|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59281|0 |1 is obsolete|| -- 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 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855 Alex Buckley changed: What|Removed |Added Attachment #59280|0 |1 is obsolete|| -- 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 18135] Can submit batch deletion for authorities without selecting any
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18135 Aleisha Amohia changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Small patch -- 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 18135] Can submit batch deletion for authorities without selecting any
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18135 Aleisha Amohia changed: What|Removed |Added Change sponsored?|--- |Sponsored Assignee|gmcha...@gmail.com |aleishaamo...@hotmail.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 18135] Can submit batch deletion for authorities without selecting any
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18135 --- Comment #1 from Aleisha Amohia --- Created attachment 60370 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60370&action=edit Bug 18135: Show alert if no authorities selected in batch record deletion To test: 1) Go to Tools -> Batch record deletion 2) Select Authorities as record type, put in an auth id, click Continue 3) Unselect the authorities and click Delete selected records 4) Confirm an alert shows up 5) Clear the alert and confirm deleting the authority still works as normal 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 18135] New: Can submit batch deletion for authorities without selecting any
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18135 Bug ID: 18135 Summary: Can submit batch deletion for authorities without selecting any Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Tools Assignee: gmcha...@gmail.com Reporter: aleishaamo...@hotmail.com QA Contact: testo...@bugs.koha-community.org At the moment, batch deletion for biblios does not let you submit the deletion form without selecting a checkbox - an alert pops up. This behaviour is not the same for authorities. If nothing is selected, the form still submits and says the deletion was successful. -- 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 18134] Batch authority record modification Preview MARC button needs updating
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18134 Aleisha Amohia changed: What|Removed |Added Change sponsored?|--- |Sponsored Assignee|gmcha...@gmail.com |aleishaamo...@hotmail.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 18134] Batch authority record modification Preview MARC button needs updating
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18134 Aleisha Amohia changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Small patch -- 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 18134] Batch authority record modification Preview MARC button needs updating
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18134 --- Comment #1 from Aleisha Amohia --- Created attachment 60369 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60369&action=edit Bug 18134: Updating Preview MARC button in Batch Authority Record Mod This patch ensures the Preview MARC button when batch modifying authority records looks like the Preview MARC button when batch modifying biblio records. To test: 1) Go to Tools -> Batch record modification 2) Leave record type as Biblio, Put in a biblionumber, select a template and click Continue 3) Notice the 'Show MARC' button 4) Click Cancel and change record type to Authority. Put in auth id, select template, click Continue 5) Notice ugly 'Preview MARC' link 6) Apply patch and refresh page 7) There should now be a nice 'Show MARC' button like there is for biblio mod. Confirm it still works 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 18134] New: Batch authority record modification Preview MARC button needs updating
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18134 Bug ID: 18134 Summary: Batch authority record modification Preview MARC button needs updating Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Tools Assignee: gmcha...@gmail.com Reporter: aleishaamo...@hotmail.com QA Contact: testo...@bugs.koha-community.org The Batch record modification Preview MARC for biblios has been updated, but not for authorities -- 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 18127] Add batch modified records to an existing list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18127 Aleisha Amohia changed: What|Removed |Added Blocks||18133 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18133 [Bug 18133] Batch record modification/deletion is not optimised for authorities -- 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 18133] Batch record modification/ deletion is not optimised for authorities
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18133 Aleisha Amohia changed: What|Removed |Added Depends on||18127 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18127 [Bug 18127] Add batch modified records to an existing 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 18133] Batch record modification/ deletion is not optimised for authorities
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18133 Aleisha Amohia changed: What|Removed |Added Change sponsored?|--- |Sponsored Assignee|gmcha...@gmail.com |aleishaamo...@hotmail.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 18133] Batch record modification/ deletion is not optimised for authorities
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18133 --- Comment #1 from Aleisha Amohia --- Created attachment 60368 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60368&action=edit Bug 18133: Batch record modification/deletion optimisation for authorities This patch just adds some wording changes so the interface is nicer and clearer when working with authorities. To test: 1) Apply 12759 then 18127 2) Go through process of batch modification for authorities 3) Confirm that the wording changes are appropriate 4) Go through process of batch deletion for authorities 5) Confirm that the wording changes are appropriate 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 18133] Batch record modification/ deletion is not optimised for authorities
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18133 Aleisha Amohia changed: 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 18133] New: Batch record modification/ deletion is not optimised for authorities
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18133 Bug ID: 18133 Summary: Batch record modification/deletion is not optimised for authorities Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Tools Assignee: gmcha...@gmail.com Reporter: aleishaamo...@hotmail.com QA Contact: testo...@bugs.koha-community.org Just some simple wording changes from 'record' to 'authority' to make it a bit clearer and friendlier would be nice -- 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 12759] Add ability to pass list contents to batch record modification /deletion tools
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12759 Aleisha Amohia changed: What|Removed |Added Attachment #60362|0 |1 is obsolete|| --- Comment #6 from Aleisha Amohia --- Created attachment 60367 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60367&action=edit Bug 12759: Use a list (shelf) for batch record modification and deletion To test: 1) Create a list, add a record to it by its barcode that you don't mind deleting 2) Go to Tools -> Batch record modification 3) Select the record type 'Authorities'. Confirm the dropdown to select a list disappears. Confirm selecting 'Biblios' makes the dropdown show again. 4) Select the list from the dropdown, select a modification template, Continue 5) Confirm all of the records in that list show on the page. Click Modify selected records. Confirm this is successful. 6) Go to Batch record deletion 7) Select the record type 'Authorities'. Confirm the dropdown to select a list disappears. Confirm selecting 'Biblios' makes the dropdown show again. 8) Select the list from the dropdown and click Continue 9) Confirm all of the records in that list show on the page. Click Delete selected records. Confirm this is successful. 10) Go back to your lists. The list should still exist but it will now be empty. 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 18127] Add batch modified records to an existing list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18127 Aleisha Amohia changed: What|Removed |Added Attachment #60299|0 |1 is obsolete|| --- Comment #2 from Aleisha Amohia --- Created attachment 60366 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60366&action=edit Bug 18127: Add batch modified records to an existing list This patch is based off a suggestion from bug 12759 to allow a user to add records to a list after they have been batch modified. To test: 1) Apply 12759 2) Pick a list to modify, go through batch modification process 3) Once the modification is done and successful, there should now be a 'Next' box with options in it 4) Confirm the 'New batch record modification' button takes you back to the start of the process. 5) Repeat step 2 6) Select a list from the dropdown to add your records to 7) Confirm that the correct records show in the pop up window 8) Once saved, go to your lists and confirm unique records were saved to the list 9) Go back to batch record modifications 10) Select Authorities as the record type. Enter an authority id in the record number list. Click Continue 11) Click modify selected records 12) Confirm that there is no dropdown to add to lists (since you can't add authorities to lists) and the link to New batch record modification works 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 10978] redirect to basket list of a supplier after deleting a basket
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10978 Aleisha Amohia changed: What|Removed |Added Attachment #60293|0 |1 is obsolete|| --- Comment #7 from Aleisha Amohia --- Created attachment 60365 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60365&action=edit Bug 10978: Redirect to basket list of a supplier after deleting a basket This patch redirects to the vendor's list of baskets after deleting a basket, fixes breadcrumbs after deletion and also hides the toolbar actions after deletion (seeing as you can't edit/export etc a basket that no longer exists). To test: 1) Go to Acquisitions -> find a vendor -> view a basket or create a new basket 2) Delete the basket. Notice you are taken to a list of all vendors and baskets 3) Apply patch and do step 1 again 4) Delete the basket. Notice appropriate breadcrumbs, no toolbar, and confirm link to return to baskets for the vendor works. Sponsored-by: Catalyst IT Followed test plan, works as expected (links to vendor's baskets and all baskets) Signed-off-by: Marc Véron -- 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 10978] redirect to basket list of a supplier after deleting a basket
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10978 Aleisha Amohia changed: 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 10978] redirect to basket list of a supplier after deleting a basket
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10978 --- Comment #6 from Aleisha Amohia --- (In reply to Jonathan Druart from comment #5) > Aleisha, I think the wording of the button you add is wrong. > It says "Show all baskets", but when you are on the screen you are > redirected to there is also a "Show all baskets". This one set a allbaskets > parameter to the url. > I'd suggest more something like "Show baskets from all vendors", or > something like that. > What do you think? I see what you mean. I think maybe "Show all active baskets" is more appropriate, seeing as the "Show all baskets" link on the page you are taken to toggles between "Show all baskets" and "Show active baskets only". "Show baskets from all vendors is kind of the same as "Show all baskets" in this case. -- 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 18107] Articles about SIP2 and LDAP for the Koha manual
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18107 Alex Buckley changed: What|Removed |Added Status|ASSIGNED|Needs Signoff --- Comment #2 from Alex Buckley --- Test plan: 1. Navigate to the en directory and generate the html pages for the .rst files make html 2. View the en/build/html/18_miscellaneous.html. Notice there are no articles describing how to setup and use SIP2 and LDAP 3. Apply the patch attached to this bug report 4. Repeat step 1 5. View the en/build/html/18_miscellaneous.html file. Notice there are sections on SIP2 and LDAP setup/use which include screenshots. 6. Navigate to the images directory and notice that there is a new directory named appendix-services -- 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 10679] Lists interface offers delete option when user doesn' t have permission
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10679 Aleisha Amohia changed: 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 10679] Lists interface offers delete option when user doesn' t have permission
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10679 Aleisha Amohia changed: What|Removed |Added Attachment #60228|0 |1 is obsolete|| --- Comment #6 from Aleisha Amohia --- Created attachment 60364 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60364&action=edit Bug 10679: Hide delete option for lists if user does not have permission and has not added any items to that list Taking from Comment 5, if a list once had permissions for people to add and remove entries, but then later the owner changes it so that others can no longer add entries, users should be able to delete the entries they added previously. This patch checks that the logged in user is the same account as the borrower who added the entry to the list. If they are, the button to remove should show. Attempting to delete any records they did not add will fail, but attempting to delete the records they did add will succeed. If they did not add any records to a list, the button to remove will not show. To test: 1) Make TWO public lists. Set all permissions to 'Allow' on one list and all permissions to 'Do not allow' on the other list. 2) Add one or two items to both lists. 3) Log in as a different user. Add one or two items to the list that you have permission to add to. 4) Log back in as the original user and set all permissions on the 'Allow' list to 'Do not allow' 5) Log back in as the different user. Go to the list you added entries to 6) The button to remove should show and you should be able to delete your entries only. 7) There should be no button to remove on the other list that you have added nothing to. 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 18107] Articles about SIP2 and LDAP for the Koha manual
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18107 --- Comment #1 from Alex Buckley --- Created attachment 60363 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60363&action=edit Bug 18107 - Added SIP2 and LDAP articles to the Koha manual in the 18_miscellaneous.rst file Added the associated screenshots into the appendix-services directory -- 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 12759] Add ability to pass list contents to batch record modification /deletion tools
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12759 Aleisha Amohia changed: What|Removed |Added Attachment #60297|0 |1 is obsolete|| --- Comment #5 from Aleisha Amohia --- Created attachment 60362 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60362&action=edit Bug 12759: Use a list (shelf) for batch record modification and deletion To test: 1) Create a list, add a record to it by its barcode that you don't mind deleting 2) Go to Tools -> Batch record modification 3) Select the record type 'Authorities'. Confirm the dropdown to select a list disappears. Confirm selecting 'Biblios' makes the dropdown show again. 4) Select the list from the dropdown, select a modification template, Continue 5) Confirm all of the records in that list show on the page. Click Modify selected records. Confirm this is successful. 6) Go to Batch record deletion 7) Select the list from the dropdown and click Continue 8) Confirm all of the records in that list show on the page. Click Delete selected records. Confirm this is successful. 9) Go back to your lists. The list should still exist but it will now be empty. 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 12759] Add ability to pass list contents to batch record modification /deletion tools
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12759 Aleisha Amohia changed: 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 18043] Updating buttons for z39.50 cataloguing search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18043 --- Comment #6 from Aleisha Amohia --- (In reply to Marc Véron from comment #4) > Buttons display as expected, but "Go to page" is broken. > Reason: Button "Go" definition misses: value="Go" > If I add it (line 212), "Go to page" works. Fixed in latest patch. Ready for 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 18043] Updating buttons for z39.50 cataloguing search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18043 Aleisha Amohia changed: What|Removed |Added Attachment #59960|0 |1 is obsolete|| --- Comment #5 from Aleisha Amohia --- Created attachment 60361 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60361&action=edit Bug 18043: Updating buttons for z39.50 cataloguing search 1) Go to Cataloguing -> New from Z39.50 2) Do a search, noticing the ugly buttons from Comment 1 3) Apply patch and go back to a new search 4) Do a search again, noticing the new, nicer buttons 5) Confirm they still work 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 18043] Updating buttons for z39.50 cataloguing search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18043 Aleisha Amohia changed: 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 18132] New: Confirming a hold/transfer clears checkin history
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18132 Bug ID: 18132 Summary: Confirming a hold/transfer clears checkin history Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: n...@bywatersolutions.com QA Contact: testo...@bugs.koha-community.org CC: gmcha...@gmail.com, kyle.m.h...@gmail.com To replicate: Checkin some items, note they show in the 'Checked-in items' table Checkin an item with a hold/trnasfer and confirm Note the table is cleared -- 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 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 --- Comment #83 from Tomás Cohen Arazi --- Created attachment 60360 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60360&action=edit Bug 13757: Attribute with value 0 should be stored Signed-off-by: Tomas Cohen Arazi -- 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 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 --- Comment #82 from Tomás Cohen Arazi --- Created attachment 60359 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60359&action=edit Bug 13757: (regression test) Attribute with value 0 should be stored Signed-off-by: Tomas Cohen Arazi -- 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 17840] Add classes to internal and public notes in checkouts table
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17840 --- Comment #11 from Chris Cormack --- Nope, they can safely be removed -- 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 12478] Elasticsearch support for Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478 Jason Palmer changed: What|Removed |Added CC||jpal...@switchinc.org -- 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 12478] Elasticsearch support for Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478 --- Comment #257 from Nick Clemens --- Hi Jorge, You will need to manual add a block like the below into the config section of your koha-conf.xml: es-server:9200 koha_instance The best way to reach the community is probably via Probably using the listservs: https://koha-community.org/support/koha-mailing-lists/ Or IRC: https://koha-community.org/get-involved/irc/ Good luck! -- 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 12461] Add patron clubs feature
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12461 --- Comment #49 from Kyle M Hall --- (In reply to Josef Moravec from comment #48) > Kyle, could you please provide followup due to bug 16239? > > See https://wiki.koha-community.org/wiki/Interface_patterns Josef, can you please be more specific? Thanks! -- 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 17933] Internal software error when searching patron without birth date
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933 Nick Clemens changed: What|Removed |Added Attachment #59735|0 |1 is obsolete|| --- Comment #17 from Nick Clemens --- Created attachment 60357 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60357&action=edit Bug 17933: Add test and return unless defined dob Without this patch get_age return actually 0 instead of Signed-off-by: Dobrica Pavlinusic Signed-off-by: Nick Clemens -- 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 17933] Internal software error when searching patron without birth date
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933 Nick Clemens changed: 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 17933] Internal software error when searching patron without birth date
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933 Nick Clemens changed: What|Removed |Added Attachment #59736|0 |1 is obsolete|| --- Comment #18 from Nick Clemens --- Created attachment 60358 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60358&action=edit Bug 17933: Do not instanciate a patron if not needed Signed-off-by: Dobrica Pavlinusic Signed-off-by: Nick Clemens -- 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 17933] Internal software error when searching patron without birth date
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933 Nick Clemens changed: What|Removed |Added Attachment #59734|0 |1 is obsolete|| --- Comment #16 from Nick Clemens --- Created attachment 60356 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60356&action=edit Bug 17933 - Internal software error when searching patron without birth date When patrons don't have date of birth (which is not required) patron search results on moremember page produce internal server error since we can't convert MySQL invalid date -00-00 to datetime object and call strfdate on it. Additionally, since we assign dates to template variables and after than assign whole $data hash to template, later assigment overrides previous one, so we see birth date field even for patrons which don't have one. This patch fixes both of those problems. Test: 1. edit patron and remove it's birth date 2. try to search for it, and verify server error 3. apply patch 4. repeat search for patron and verify that it works and doesn't have enpty birth date field Signed-off-by: Grace McKenzie Signed-off-by: Nick Clemens -- 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 14405] Add datatable to fines table in OPAC patron account
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14405 Johnny Rotten changed: What|Removed |Added CC|geor...@latahlibrary.org, | |gwilli...@nekls.org | -- 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 14405] Add datatable to fines table in OPAC patron account
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14405 Johnny Rotten changed: What|Removed |Added CC||geo...@nekls.org, ||geor...@latahlibrary.org -- 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 12478] Elasticsearch support for Koha
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478 Jorge de Cardenas changed: What|Removed |Added CC||jd...@smfpl.org --- Comment #256 from Jorge de Cardenas --- I am trying to test Elasticsearch in Koha and I keep getting an error: No 'elasticsearch' block is defined in koha-conf.xml. I am running a fresh install of Koha 16.11.03.000 and Debian 8 and imported our data from 3.16.04.000 I have enabled Memcached and Plack. Installed openjdk-8 and elastic search. I first got the error while trying run rebuild_elastic_search.pl I not sure why but changing to the static IP allowed the index to run but when I try to do a search on the staff side I get the error again. If this is not the place to ask please let me know. Jorge de Cardenas -- 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 15427] Allow db connections using TLS
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15427 --- Comment #13 from Mirko Tietgen --- In the non-TLS file I see a lot of MySQL commands. Like SELECTs. In the TLS file I don't (actually I see one, not sure why), but mostly "garbage". -- 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 8548] Add callnumber sort option to overdue report
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8548 Baptiste changed: What|Removed |Added CC||baptiste.wojtkowski@biblibr ||e.com 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 14784] Missing checkin message for debarred patrons when issuing rules ' fine days = 0'
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 Baptiste changed: What|Removed |Added CC||baptiste.wojtkowski@biblibr ||e.com --- Comment #15 from Baptiste --- Hi, The scenario 2 leads me to a message, even without patching -- 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 12460] Search history: Combine to start a new search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12460 Claes changed: What|Removed |Added CC||claes.eriks...@vti.se --- Comment #19 from Claes --- Since this feature would appeal to more experienced user I think that it would be better to align with common search interfaces. The option "Start a new search" is not intuitive at all. What we want to do is: "Combine sets/queries" "Match any" and "match all" is not really common language and it is not intuitive. It seems that I must select either match any/ match all even if I just want to select line 4 NOT line 2. The results indicate that the search works as it should but it is certainly not clear for a user. I have 2 suggestions: the first beeing a bit simpler and probably easier to apply. The second would be the most up to date solution. My first suggestion for a design remake would be: 1) At the search history page add 2 new columns 2) One column with the heading "Delete Sets". In this column, each line/search set should have a tick box. Two buttons in the same heading with "select all" and the second button "X delete" 3) The second column with the heading "Combine sets or Combine queries". In this heading two tick boxes: "AND" & "OR". And this column should also have a tick box for each line. 4) remove the big red button "Remove selected searches" 5) remove the first column with tick boxes So how about the option "NOT"? My experience as information specialist is that this operator is not well understood and often misused by normal users. A second suggestion would complicate it for the developer :-) but here it comes: 1) The search history page should have a number for each search set/search querie. And the numbers should be in this format: #1, #2, #3 2) An extra column at the end of the line with tick boxes for each line and with the heading "Delete searches". As in 2) above. 3) A search line above the search history with the text Combine queries and a brief example (#1 OR #3) AND #6. As the example show there are no box ticking here but you send a querie using old search sets. 4) remove the big red button "Remove selected searches" 5) remove the first column with tick boxes -- 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 15427] Allow db connections using TLS
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15427 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #12 from Jonathan Druart --- (In reply to Mirko Tietgen from comment #11) > and then compared the output files, which were obviously different. Yes they are different, but I do not find anything obvious that is telling me the connection is encrypted. What should I search for? -- 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 17427] Replace CGI::Session with Data::Session
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17427 --- Comment #14 from Jonathan Druart --- Just a 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 17427] Replace CGI::Session with Data::Session
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17427 Jonathan Druart changed: What|Removed |Added Attachment #57067|0 |1 is obsolete|| --- Comment #13 from Jonathan Druart --- Created attachment 60355 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60355&action=edit Bug 17427: Replace CGI::Session with Data::Session CGI::Session is not maintained for ages and seems quite buggy. It would be the culprit of random logout problems. Another issue is the following crazy code in CGI::Session::Driver::DBI sub DESTROY { my $self = shift; unless ( defined $self->{Handle} && $self->{Handle} -> ping ) { $self->set_error(__PACKAGE__ . '::DESTROY(). Database handle has gone away'); return; } unless ( $self->{Handle}->{AutoCommit} ) { $self->{Handle}->commit; } if ( $self->{_disconnect} ) { $self->{Handle}->disconnect; } } If AutoCommit is off, it will commit anyway... It causes some tests to fail (api) if SessionStorage is set to MySQL Note that PostgreSQL is not supported yet so no need to implement it. Test plan: 0/ The dependencies are not packaged for debian so far, so install it via cpan 1/ Test the 3 different values for SessionStorage 2/ Using different browsers (or sessions) login into Koha with different users. Naviguate => you should not be logged out 3/ Set SessionStorage to Memcached, stop memcached make sure that it defaults to file. Signed-off-by: Benjamin Rokseth -- 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 17169] Add facets for ccode to elasticsearch
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17169 Nick Clemens changed: What|Removed |Added Status|Failed QA |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 17169] Add facets for ccode to elasticsearch
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17169 --- Comment #8 from Nick Clemens --- Created attachment 60354 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60354&action=edit Bug 17679 - Use CCODE descriptions instead of codes -- 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 15408] Timestamp not updated when you duplicate a budget
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15408 Baptiste changed: 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 17974] Add the Koha::Item->biblio method
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17974 Mason James changed: What|Removed |Added CC||m...@kohaaloha.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 11889] Disown a list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11889 Mason James changed: What|Removed |Added CC||m...@kohaaloha.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 17913] Merge three authority merge fixes
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17913 Mason James changed: What|Removed |Added CC||m...@kohaaloha.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 18036] Improve test coverage for themelanguage
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18036 Jonathan Druart changed: What|Removed |Added QA Contact|gmcha...@gmail.com | CC||jonathan.dru...@bugs.koha-c ||ommunity.org -- 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: 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 18036] Improve test coverage for themelanguage
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18036 Mason James changed: What|Removed |Added CC||m...@kohaaloha.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 18036] Improve test coverage for themelanguage
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18036 Jonathan Druart changed: What|Removed |Added Attachment #59843|0 |1 is obsolete|| Attachment #59844|0 |1 is obsolete|| Attachment #59845|0 |1 is obsolete|| --- Comment #21 from Jonathan Druart --- Created attachment 60351 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60351&action=edit Bug 18036: Perlcritic improve t/db_dependent/Templates.t TEST PLAN - 1) apply 17982 2) perlcritic -3 t/db_dependent/Templates.t -- messages 3) apply this patch 4) perlcritic -3 t/db_dependent/Templates.t -- OK 5) run koha qa test tools Signed-off-by: Marcel de Rooy 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 18036] Improve test coverage for themelanguage
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18036 --- Comment #23 from Jonathan Druart --- Created attachment 60353 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60353&action=edit Bug 18036: [QA Follow-up] Descriptions and mock_preference Adjusted a few test descriptions. The test depends on template being set to prog. Made that explicit by adding a mock_preference. If you change the mock to bootstrap, this test will fail. Signed-off-by: Marcel de Rooy 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 18036] Improve test coverage for themelanguage
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18036 --- Comment #22 from Jonathan Druart --- Created attachment 60352 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60352&action=edit Bug 18036: All 4 cases tested Added theme, lang, and availablethemes tests for all 4 cases. Previously just availablethemes was tested. Signed-off-by: Marcel de Rooy The warns from Mark's additional patch showed all four cases. 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 18036] Improve test coverage for themelanguage
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18036 Jonathan Druart changed: 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 --- Comment #65 from Jonathan Druart --- Created attachment 60350 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60350&action=edit Bug 16749: Add is_instance check to koha-upgrade-schema 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: What|Removed |Added Attachment #60340|0 |1 is obsolete|| --- Comment #64 from Jonathan Druart --- Created attachment 60349 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60349&action=edit Bug 16749: Check instancename in adjust_paths_dev_install Check if the instance does not exist (or was not even passed). 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: What|Removed |Added Attachment #60197|0 |1 is obsolete|| --- Comment #62 from Jonathan Druart --- Created attachment 60347 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60347&action=edit Bug 16749: Go with a boolean flag, we love booleans As requested by Jonathan on comment 10 on Bugzilla and supported by Tomas, we should use the dev_install entry in koha-conf as a boolean flag. Since we already used it as a path, this patch is not too strict about it. If the entry is not empty and not equal to "0", we will interpret it as a true value. The path is taken from the intranetdir entry. Test plan: [1] Copy debian/scripts/koha-functions.sh to /usr/share/koha/bin [2] For a dev install: Remove the dev install line, or toggle its value between empty string, 0 or 1 and each time test stop/start koha-indexer. Check the path to rebuild_zebra with ps aux|grep indexer. If you have no entry, an empty entry or a zero, you should see a regular path. (Note: You can do something similar with koha-start-sip.) [3] For a regular install: Remove the dev install line. Stop/start koha-indexer or koha-plack, and verify that it still works. Add a dev_install line with 0, and repeat stop/start. Signed-off-by: Marcel de Rooy Signed-off-by: Mirko Tietgen Tested in a package installation of master+16749 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: What|Removed |Added Attachment #60335|0 |1 is obsolete|| --- Comment #63 from Jonathan Druart --- Created attachment 60348 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60348&action=edit Bug 16749: Use is_instance to make sure the dev instance exist 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: What|Removed |Added Attachment #60196|0 |1 is obsolete|| --- Comment #61 from Jonathan Druart --- Created attachment 60346 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60346&action=edit Bug 16749: Update debian docs for koha-translate This patch adds the -d option to the documentation (with thanks to Magnus Enger). Signed-off-by: Mirko Tietgen Tested in a package installation of master+16749 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: What|Removed |Added Attachment #60195|0 |1 is obsolete|| --- Comment #60 from Jonathan Druart --- Created attachment 60345 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60345&action=edit Bug 16749: Adjustments for koha-translate [AMENDED February 10, 2017] [1] Added reading /etc/default/koha-common as in the other debian scripts. We need it for KOHA_HOME. [2] Add a -d|--dev parameter for dev installs. [3] No hardcoded PERL5LIB or KOHA_INSTALL_DIR (KOHA_HOME). They are read from default file or set by adjust_paths_dev_install. [4] Adjust template paths for dev installs: OPAC_TMPL, INTRANET_TMPL. [5] Remove references to obsolete themes ccsr and prog. Test plan: [1] Regular package install: Copy koha-translate to /usr/sbin. Run koha-translate -l to show installed languages. Run koha-translate -l -a to show available languages. Add a language: koha-translate -i nl-NL. Check template folders in regular location (/usr/share/koha/...) Remove a language: koha-translate -r nl-NL. Check again. [2] Dev install or kohadevbox: Copy koha-translate to /usr/sbin. If needed, add the line to koha-conf.xml. Run koha-translate -l -d yourinstance to show installed languages. (Note: You only see the languages installed in this instance.) Add a language: koha-translate -i nl-NL -d yourinstance. Check template folders in the clone. Remove a language: koha-translate -r nl-NL -d yourinstance. Note: Make sure you have sufficient file permissions for the kohaclone files and koha-conf.xml. On kohadevbox you might need to run sudo koha-translate within the the vagrant user context. Signed-off-by: Marcel de Rooy Tested on Jessie (Debian VM and Kohadevbox) Signed-off-by: Mirko Tietgen Tested in a package installation of master+16749 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: What|Removed |Added Attachment #60194|0 |1 is obsolete|| --- Comment #59 from Jonathan Druart --- Created attachment 60344 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60344&action=edit Bug 16749: Adjustments for koha-plack [1] Use run_safe_xmlstarlet for plack workers and requests [2] Simplify adjust_paths. The lazy export statement is actually enough to replace adjust_paths by one direct call to adjust_paths_dev_install. Test plan: [1] Copy koha-functions.sh and koha-plack: cp [YOUR_PATH]/debian/scripts/koha-functions.sh /usr/share/koha/bin/ cp [YOUR_PATH]/debian/scripts/koha-plack /usr/sbin/ where YOUR_PATH might well be /home/vagrant/kohaclone. [2] Make sure that you have dev_install in koha-conf. Stop and start koha-plack. Verify with ps aux|grep plack. [3] Rename dev_install to nodev_install (in start and end tag). Now stop/start koha-plack. Verify with ps aux|grep plack. [4] Change plack_requests to 51 in your koha-conf. Restart Plack and check that you see 51 in ps aux|grep plack. Signed-off-by: Marcel de Rooy Tested on Jessie (Debian VM and Kohadevbox) Signed-off-by: Mirko Tietgen Tested in a package installation of master+16749 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 Jonathan Druart changed: What|Removed |Added Attachment #60193|0 |1 is obsolete|| --- Comment #58 from Jonathan Druart --- Created attachment 60343 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60343&action=edit Bug 16749: Adjust xmlstarlet calls in koha-functions In various scripts we use xmlstarlet to extract values from koha-conf. If we call xmlstarlet on non-existing entries in koha-conf, this may however result in silently failing scripts (when set -e is in effect). A function run_safe_xmlstarlet is added for situations where the entry might not exist. It will not halt execution. This patch only adjusts koha-functions.sh and modifies the xmlstarlet calls for dev_install and zebra_loglevels. Note: The function does not need to check file existence. If the file does not exist, xmlstarlet warns about it; the function returns empty string, but does not set an error exit status. Test plan: See second patch ("koha-plack adjustments"). Signed-off-by: Marcel de Rooy Signed-off-by: Mirko Tietgen Tested in a package installation of master+16749 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 17913] Merge three authority merge fixes
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17913 --- Comment #79 from Marcel de Rooy --- (In reply to Marcel de Rooy from comment #77) > Created attachment 60342 [details] [review] > Bug 17913: [16.11.x] Add syspref to sysprefs.sql > > See comment 69. And comment 71. -- 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 17913] Merge three authority merge fixes
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17913 --- Comment #78 from Marcel de Rooy --- (In reply to Katrin Fischer from comment #75) > Thx a lot Josef for retesting this on 16.11 specifically. > I have pushed Marcel's patch and Josef's follow-up up to 16.11.x for > 16.11.04 now. > > Marcel, could you take a look maybe too? Tested it on a clean 16.11. Had no problems with the merge. Small struggle with Zebra, not related to this patch.. -- 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 17913] Merge three authority merge fixes
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17913 Marcel de Rooy changed: What|Removed |Added Attachment #60341|0 |1 is obsolete|| --- Comment #77 from Marcel de Rooy --- Created attachment 60342 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60342&action=edit Bug 17913: [16.11.x] Add syspref to sysprefs.sql See comment 69. -- 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 17913] Merge three authority merge fixes
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17913 --- Comment #76 from Marcel de Rooy --- Created attachment 60341 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60341&action=edit Bug 17913: Add syspref to sysprefs.sql See comment 69. -- 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 16749] Additional fixes for debian scripts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16749 --- Comment #57 from Marcel de Rooy --- Created attachment 60340 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60340&action=edit Bug 16749: Check instancename in adjust_paths_dev_install Check if the instance does not exist (or was not even passed). -- 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/