July 2013 - Koha-bugs - lists.koha-community.org

[Bug 10604] New: two columns in the courses table aren't wide enough
by bugzilla-daemon＠bugs.koha-community.org 22 Jul '13

22 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10604 Bug ID: 10604 Summary: two columns in the courses table aren't wide enough Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Course reserves Assignee: koha-bugs(a)lists.koha-community.org Reporter: gmcharlt(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Both courses.department and courses.term refer to authorized values, and consequently should be varchar(80) to match authorised_values.authorised_value. They're currently varchar(20). -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 8

[Bug 10592] New: Apache configuration is blocking fetch of certain icons and XML files
by bugzilla-daemon＠bugs.koha-community.org 22 Jul '13

22 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10592 Bug ID: 10592 Summary: Apache configuration is blocking fetch of certain icons and XML files 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: gmcharlt(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org The commit for bug 9812 is causing certain files in {opac-tmpl,intranet-tmpl}/theme/lang/{modules,xslt,includes} from being fetched. From the Apache logs: [Mon Jul 15 14:28:21 2013] [error] [client 67.183.139.142] client denied by server configuration: /home/gmc/koha/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/value_builder/marc21_field_008.xml, referer: http://staff.zadi.librarypolice.com/cgi-bin/koha/cataloguing/plugin_launche… [Mon Jul 15 14:16:17 2013] [error] [client 67.183.139.142] client denied by server configuration: /home/gmc/koha/koha-tmpl/opac-tmpl/prog/en/includes/favicon.ico [Mon Jul 15 14:43:52 2013] [error] [client 67.183.139.142] client denied by server configuration: /home/gmc/koha/koha-tmpl/intranet-tmpl/prog/en/includes/favicon.ico The effects I've seen so far: [1] favicons aren't displayed [2] The cataloging plugins for the 006 and 008 MARC21 fields can't fetch their XML data files, thus breaking them. -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 3134] Ability to selelct multiple reports to delete at once
by bugzilla-daemon＠bugs.koha-community.org 22 Jul '13

22 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3134 Liz Rea <wizzyrea(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #19806|0 |1 is obsolete| | --- Comment #8 from Liz Rea <wizzyrea(a)gmail.com> --- Created attachment 19843 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19843&action=edit Bug 3134 - Ability to selelct multiple reports to delete at once This patch adds the option to select multiple saved reports for deletion. To test you must have two or more saved reports to delete. Deletion should work properly when: - Selecting one report for deletion by checking the box. - Selecting more than one report for deletion by checking boxes. - Clicking the old "Delete" link Clicking the delete button should prompt you to confirm. Clicking cancel should cancel. Clicking the delete button when no boxes are checked should trigger an alert asking you to select reports for deletion. Signed-off-by: Liz Rea <wizzyrea(a)gmail.com> Functional tests pass, template tests pass. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 3134] Ability to selelct multiple reports to delete at once
by bugzilla-daemon＠bugs.koha-community.org 22 Jul '13

22 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3134 Liz Rea <wizzyrea(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2969] Report Name should be mandatory for saved reports
by bugzilla-daemon＠bugs.koha-community.org 20 Jul '13

20 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2969 Chris Cormack <chris(a)bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #19814|0 |1 is obsolete| | --- Comment #5 from Chris Cormack <chris(a)bigballofwax.co.nz> --- Created attachment 19821 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19821&action=edit Bug 2969 - Report Name should be mandatory for saved reports This patch re-implements the database changes from the original patch and adds an update for existing reports. It also reimplements the client-side validation using the new built-in validation plugin and adds coverage for SQL reports. I have also added the "required" attribute to the SQL report textarea since saving a report with no SQL triggers an error. To test: - Create a new guided report. Confirm that you cannot save your guided report without a report name. - Create or edit a saved SQL report. Confirm that you cannot save your report without a name or without SQL. - Confirm that the saved_sql table structure has been updated. Signed-off-by: Liz Rea <wizzyrea(a)gmail.com> Functional tests pass, database tables look correct to me. Signed-off-by: Chris Cormack <chrisc(a)catalyst.net.nz> Even with JS disabled, the form can not be submitted without the report name filled in. However we are not checking this server side .. if this was the public interface that would be a fail. I will leave this up to the RM to decided if client side checks are ok for this. Bearing in mind we weren't handling any sql errors properly server side before this. Signed-off-by: Chris Cormack <chrisc(a)catalyst.net.nz> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2969] Report Name should be mandatory for saved reports
by bugzilla-daemon＠bugs.koha-community.org 20 Jul '13

20 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2969 Chris Cormack <chris(a)bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2969] Report Name should be mandatory for saved reports
by bugzilla-daemon＠bugs.koha-community.org 20 Jul '13

20 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2969 Liz Rea <wizzyrea(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #19795|0 |1 is obsolete| | --- Comment #4 from Liz Rea <wizzyrea(a)gmail.com> --- Created attachment 19814 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19814&action=edit Bug 2969 - Report Name should be mandatory for saved reports This patch re-implements the database changes from the original patch and adds an update for existing reports. It also reimplements the client-side validation using the new built-in validation plugin and adds coverage for SQL reports. I have also added the "required" attribute to the SQL report textarea since saving a report with no SQL triggers an error. To test: - Create a new guided report. Confirm that you cannot save your guided report without a report name. - Create or edit a saved SQL report. Confirm that you cannot save your report without a name or without SQL. - Confirm that the saved_sql table structure has been updated. Signed-off-by: Liz Rea <wizzyrea(a)gmail.com> Functional tests pass, database tables look correct to me. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 2969] Report Name should be mandatory for saved reports
by bugzilla-daemon＠bugs.koha-community.org 20 Jul '13

20 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2969 Liz Rea <wizzyrea(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 3134] Ability to selelct multiple reports to delete at once
by bugzilla-daemon＠bugs.koha-community.org 19 Jul '13

19 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3134 --- Comment #7 from Owen Leonard <oleonard(a)myacpl.org> --- Created attachment 19806 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19806&action=edit Bug 3134 - Ability to selelct multiple reports to delete at once This patch adds the option to select multiple saved reports for deletion. To test you must have two or more saved reports to delete. Deletion should work properly when: - Selecting one report for deletion by checking the box. - Selecting more than one report for deletion by checking boxes. - Clicking the old "Delete" link Clicking the delete button should prompt you to confirm. Clicking cancel should cancel. Clicking the delete button when no boxes are checked should trigger an alert asking you to select reports for deletion. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0

[Bug 3134] Ability to selelct multiple reports to delete at once
by bugzilla-daemon＠bugs.koha-community.org 19 Jul '13

19 Jul '13

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3134 Owen Leonard <oleonard(a)myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.

1 0