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

[Bug 15008] Add custom HTML areas to circulation and reports home pages
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15008 --- Comment #7 from Marc Véron <veron(a)veron.ch> --- (In reply to Nick Clemens from comment #6) > (In reply to Jonathan Druart from comment #4) > > Would not it better to add an admin page to manage these different areas to > > custom? Good idea! > I would tend to agree but think that's a separate bug +1 for a separate bug -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11998] Syspref caching issues
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11998 --- Comment #177 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- (In reply to Srdjan Jankovic from comment #176) > Jonathan, I'm having problems with t/db_dependent/Context.t test 35 (line > 97) where you reverted logic. Can you please double check it for me, cause > it looks to me that the previous version was ok. > Much obliged. Are you talking about 99 C4::Context->enable_syspref_cache(); 100 is(C4::Context->preference("SillyPreference"), 'thing3', "Retrieved syspref (value='thing3') successfully from cache"); 101 isnt( $trace_read, q{}, 'The pref should be retrieved from the database if the cache has been enabled'); 102 $trace_read = q{}; ? C4::Context->enable_syspref_cache will clear the syspref cache, so we will need to access the DB to retrieve the value of the pref. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14257] Add show_extended_attributes to ILS-DI call GetPatronInfo
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14257 --- Comment #10 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Still working on this! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 0

[Bug 14257] Add show_extended_attributes to ILS-DI call GetPatronInfo
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14257 --- Comment #9 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Created attachment 49669 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49669&action=ed… Bug 14257: Follow-up - Add documentation, fix typo, fix test - t/db_dependent/ILSDI_Services.t was failing for me as I had issues in my database, fixed with another delete line - QA tools found a typo - comparision - Added documentation of the new parameter to the ilsdi.pl page -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 0

[Bug 13132] Adding confirm message for deleting a report from the reports toolbar
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13132 Magnus Enger <magnus(a)libriotech.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49659|0 |1 is obsolete| | --- Comment #9 from Magnus Enger <magnus(a)libriotech.no> --- Created attachment 49668 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49668&action=ed… Bug 13132: Add confirm dialog for deleting reports from the reports toolbar To test: 1) Go to Reports -> Use saved (or build a new one if you have none saved) 2) Click on the Actions drop down and confirm that when you click Delete from here, a confirm message shows up. Click Cancel. 3) Click Show in the drop down menu 4) Click Delete in the Reports toolbar. Confirm you now get a confirm message. Clicking Cancel will cancel the delete, and clicking OK deletes the report and takes you to the saved reports page. Sponsored-by: Catalyst IT Signed-off-by: Magnus Enger <magnus(a)libriotech.no> After applying the patch I get a JavaScript popup confirmation when I click on the delete button, in the Show, Edit and Run views. Nice! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13132] Adding confirm message for deleting a report from the reports toolbar
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13132 Magnus Enger <magnus(a)libriotech.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15562] Make Koha more suitable for shared hosting with plack
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49373|0 |1 is obsolete| | --- Comment #26 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 49667 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49667&action=ed… bug_15562: Removed Koha::Cache->get_instance() There should be no cache singleton, full stop. If Koha is to move away from .pl scripts that is. As an interim measure Koha::Cache->get_instance() is replaced with C4::Context->cache, in the vein of C4::Context->memcached. In that respect it will continue to work in the singleton-ish way if context is used as a singleton, but supports cache-per-context. Koha::Handler::Plack->app_per_host() cache sysprefs using Context memcached. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15562] Make Koha more suitable for shared hosting with plack
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49372|0 |1 is obsolete| | --- Comment #25 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 49666 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49666&action=ed… bug_15562: Multi-host helper for plack installations Sort of an apocalypse * C4::Context->new() must be called with at least config file param. If you want current context, call C4::Context->current(). C4::Context->some_method() will still work as is. * Koha::Database->new_schema() now takes optional context param. * C4::Context->set_context() and restore_context() are synched with database set_schema() and restore_schema(). Created run_within_context() that wraps set_context() and restore_context() around code. * Created Koha::Handler::Plack* to facilitate running same code within different (database) contexts. * This initial version does not run with memcached turned on. Next patch will correct that. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15562] Make Koha more suitable for shared hosting with plack
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 --- Comment #24 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 49665 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49665&action=ed… bug_15562: Revert "Bug 13690: use Koha::Schema only when it's needed" This reverts commit c52bc9756a5fa1e4f6bba7ce5c58124f150c5efc. I don't see any place where we would use KOha::Database without wanting Koha::Schema. require Koha::Schema interferes with some plack setups - for some reason it calls C4::Context->import() even when C4::Context is already used. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15562] Make Koha more suitable for shared hosting with plack
by bugzilla-daemon＠bugs.koha-community.org 30 Mar '16

30 Mar '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49370|0 |1 is obsolete| | Attachment #49371|0 |1 is obsolete| | --- Comment #23 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 49664 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=49664&action=ed… bug_15562: Use do() rather than system() to execute updatedatabase.pl from installer.pl That way: * no external process is spawned * code executes in the same perl process, which is required for plack multi-site I have a dream. A dream that one day all code from .pl's will be in some .pm's. -- You are receiving this mail because: You are watching all bug changes.

1 0