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

[Bug 13381] RDA: 245 field changes in XSLT
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13381 --- Comment #39 from David Cook <dcook(a)prosentient.com.au> --- (In reply to Ian Bays from comment #36) > It seems that the change in the xslt to put span around subfields c and h > has removed the space that was generated between the subfields. > In koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl there > is an: > <xsl:text> </xsl:text> > at line 125 in the otherwise clause that spaces out the subfields. The two > that were singled out do not have this. Adding them back in within the > <span> achieves the normal, desired spacing. I also have observed this... I suppose I can open a bug for it once I finish a few other tasks... -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13381] RDA: 245 field changes in XSLT
by bugzilla-daemon＠bugs.koha-community.org 26 Apr '16

26 Apr '16

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13381 --- Comment #38 from David Cook <dcook(a)prosentient.com.au> --- (In reply to Winona Salesky from comment #35) > The order in the for-each does not determine the order of the output. > Subfields are output in the order they appear in the marc record. They are > in alphabetical order in the for-each, purely for convenience sake. > > -Winona > Right you are. I must have been thinking of "subfieldSelect" which does rely on the order of the string of codes. My bad! -- 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 #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