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

[Bug 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #81 from Florent Mara <florent.mara(a)gmail.com> --- Created attachment 50704 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=50704&action=ed… Bug 12598 - Creating objects in misc/import_patrons.pl and tools/import_borrowers.pl -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #80 from Florent Mara <florent.mara(a)gmail.com> --- Hasty addition of Moo on my part. Fixing right now. But I am very glad you were able to test my patches / fixes ... and new defects. -- 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 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49709|0 |1 is obsolete| | --- Comment #29 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 50703 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=50703&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 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- 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 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562 Srdjan Jankovic <srdjan(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #49666|0 |1 is obsolete| | --- Comment #28 from Srdjan Jankovic <srdjan(a)catalyst.net.nz> --- Created attachment 50702 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=50702&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 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #79 from Bernardo Gonzalez Kriegel <bgkriegel(a)gmail.com> --- Hi Florent, with simple csv cardnumber,surname,firstname,categorycode,branchcode,password,userid 12344321,Doe,John,S,CP,test,john 'misc/import_patrons.pl -f test.csv -c -m cardnumber -v' gives: No file handle passed in! at misc/import_patrons.pl line 51. Use of uninitialized value $alreadyindb in addition (+) at misc/import_patrons.pl line 69. etc. On your last patch you put sub import_patrons { my ($self, $params) = @_; but $self has the params and $params is undef. Removing the last patch I get Import complete: Imported: 0 Overwritten: 0 Skipped: 0 Invalid: 1 Total: 1 which is better but invalid, I don't know why. Test pass, no koha-qa errors so far. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #78 from Florent Mara <florent.mara(a)gmail.com> --- I might have done the right thing when creating my patch. (or not, please let me know if it is the case) The plan is to continue refactoring the Patrons::Import module. Regression tests have shown further defects that will require moving code around and more changes. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12333] Acquisitions toolbar
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12333 Aleisha Amohia <aleishaamohia(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12333] Acquisitions toolbar
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12333 --- Comment #1 from Aleisha Amohia <aleishaamohia(a)hotmail.com> --- Created attachment 50701 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=50701&action=ed… Bug 12333: Acquisitions toolbar and removing Add to basket section at bottom To test: 1) Go to Acqui -> do a vendor search -> view a basket 2) Confirm that the Add Order to basket section at the bottom is gone. Confirm the add to basket button in the toolbar still works as expected 3) Confirm that when you make your browser shorter, the toolbar follows the page down as you scroll Sponsored-by: Catalyst IT -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12333] Acquisitions toolbar
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12333 Aleisha Amohia <aleishaamohia(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0