December 2012 - Koha-bugs - lists.koha-community.org

[Bug 9284] New: JavaScript should conform to coding guidelines recommended by JSHint
by bugzilla-daemon＠bugs.koha-community.org 01 Jan '13

01 Jan '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9284 Bug ID: 9284 Summary: JavaScript should conform to coding guidelines recommended by JSHint Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Templates Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org JSHint can be used to check JavaScript for errors and potential problems. It can be installed on a variety of platforms and integrated into various editors: http://www.jshint.com/platforms/ I propose that we use JSHint with default settings as a set of coding guidelines for JavaScript in Koha, and that patches correcting errors be attached to this omnibus bug. Some typical JSHint errors for example: js/additem.js: line 3, col 52, Missing radix parameter. js/additem.js: line 5, col 41, Use '!==' to compare with '0'. js/additem.js: line 68, col 25, Use '===' to compare with 'undefined'. js/additem.js: line 122, col 58, Missing semicolon. js/additem.js: line 206, col 21, Bad line breaking before '+'. -- You are receiving this mail because: You are watching all bug changes.

1 8

[Bug 8299] New: overdue_notice.pl shows error messages
by bugzilla-daemon＠bugs.koha-community.org 01 Jan '13

01 Jan '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8299 Priority: P5 - low Change sponsored?: --- Bug ID: 8299 Assignee: koha-bugs(a)lists.koha-community.org Summary: overdue_notice.pl shows error messages Severity: normal Classification: Unclassified OS: All Reporter: nikhilvr0(a)gmail.com Hardware: All Status: NEW Version: rel_3_8 Component: Notices Product: Koha I am installed koha 3.8.2 in Ubuntu lucid OS.When I run overdue_notices.pl with some overdue items got this error message. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 659. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. Use of uninitialized value in pattern match (m//) at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 670. No letter_code at /usr/share/koha/bin/cronjobs/overdue_notices.pl line 686. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 12

[Bug 9289] New: Adjacent checkbox behavior broken on circulation.pl
by bugzilla-daemon＠bugs.koha-community.org 01 Jan '13

01 Jan '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9289 Bug ID: 9289 Summary: Adjacent checkbox behavior broken on circulation.pl Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: minor Priority: P5 - low Component: Circulation Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com In the table of checkouts on the circulation page you're not supposed to be able to check a box in both the renew and check-in column in the same row. The JavaScript which manages this works if you check the box itself, but there is also JavaScript which lets the user click anywhere in that cell to check the checkbox. If you do it this way the adjacent checkboxes are not prevented from both being checked. -- You are receiving this mail because: You are watching all bug changes.

1 9

[Bug 9100] New: Sample configurations should be included for nginx+Plack/Starman
by bugzilla-daemon＠bugs.koha-community.org 30 Dec '12

30 Dec '12

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9100 Bug ID: 9100 Summary: Sample configurations should be included for nginx+Plack/Starman Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: jcamins(a)cpbibliography.com Right now Koha includes configuration files for Apache2 out-of-the-box. Since we are moving toward using PSGI, it would be good to provide configuration files for nginx+Plack that people can test with. -- You are receiving this mail because: You are watching all bug changes.

1 2

[Bug 1993] Task Scheduler Needs Re-write
by bugzilla-daemon＠bugs.koha-community.org 29 Dec '12

29 Dec '12

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=1993 --- Comment #10 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Hi Sophie, I missed the notes on the patch yesterday and I think I have some idea now what the patch does, but test plans are a requirement. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 8877] New: MARC21 field 511 missing from XSLT
by bugzilla-daemon＠bugs.koha-community.org 29 Dec '12

29 Dec '12

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8877 Priority: P5 - low Change sponsored?: --- Bug ID: 8877 Assignee: mirko(a)abunchofthings.net Summary: MARC21 field 511 missing from XSLT Severity: enhancement Classification: Unclassified OS: All Reporter: mirko(a)abunchofthings.net Hardware: All Status: ASSIGNED Version: master Component: Templates Product: Koha There is no MARC21 field 511 (Participant or Performer Note) in the XSLT for OPAC detail. -- You are receiving this mail because: You are watching all bug changes.

1 14

[Bug 1993] Task Scheduler Needs Re-write
by bugzilla-daemon＠bugs.koha-community.org 28 Dec '12

28 Dec '12

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=1993 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |katrin.fischer(a)bsz-bw.de --- Comment #9 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Please provide a test plan, problem description and some notes on how this patch is solving the problem. Also, I would like to see Vitor's question addressed (comment 7). -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 3309] Deleting authorized values when data exists
by bugzilla-daemon＠bugs.koha-community.org 28 Dec '12

28 Dec '12

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3309 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer(a)bsz-bw.de Depends on| |9272 -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9183] New: C4::Search should only have one ZOOM event loop
by bugzilla-daemon＠bugs.koha-community.org 27 Dec '12

27 Dec '12

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9183 Bug ID: 9183 Summary: C4::Search should only have one ZOOM event loop Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: jcamins(a)cpbibliography.com Reporter: jcamins(a)cpbibliography.com Right we we have three ZOOM event loops in C4::Search, which is rather wasteful and against good programming practice. I propose to use closures for the section of code that processes the actual results. -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 9315] New: Bad version of String::Random reported by koha_perl_deps.pl
by bugzilla-daemon＠bugs.koha-community.org 27 Dec '12

27 Dec '12

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9315 Bug ID: 9315 Summary: Bad version of String::Random reported by koha_perl_deps.pl Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Installation and upgrade (command-line installer) Assignee: gmcharlt(a)gmail.com Reporter: bgkriegel(a)gmail.com One of the patches of bug 7067 introduces a new Koha perl dependency, String::Random. The patch modifies C4/Installer/PerlDependencies.pm, but the configured min_ver=1.4 is non existent. Last version of package is 0.22 (on CPAN) (http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11978&action=diff) So koha_perl_deps.pl -u will show as required an incorrect version, Installed Required Module Name Version Version ------------------------------------------------------------------------- String::Random 0.22 * 1.4 -- You are receiving this mail because: You are watching all bug changes.

1 8