[Koha-bugs] [Bug 21005] Missing row/column defaults cause unexpected results in report wizards

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Oct 17 18:39:45 CEST 2018


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21005

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #80686|0                           |1
        is obsolete|                            |
  Attachment #80687|0                           |1
        is obsolete|                            |

--- Comment #6 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
Created attachment 80740
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=80740&action=edit
Bug 21005: Missing row/column defaults cause unexpected results in report
wizards

Some report wizards are missing row/column defaults, which sometimes return
blank
pages (borrowers_stats.pl) or results always equalling 0, which a
non-discerning
user may interpret incorrectly.

If the DEBUG environmental variable is enabled, missing row/column choices will
actually cause fatal software errors as well, which is also undesirable for
users.

This patch adds default choices for row and column on
report wizards that had no radio buttons selected, so that a user
can't submit the form without choosing a row and column. This
prevents errors.

To test:
1) Apply patch
2) Visit /cgi-bin/koha/reports/issues_avg_stats.pl
3) Visit /cgi-bin/koha/reports/borrowers_stats.pl
4) Visit /cgi-bin/koha/reports/catalogue_stats.pl
5) Visit /cgi-bin/koha/reports/issues_stats.pl
6) Observe that default choices for row/column are there

Signed-off-by: Mark Tompsett <mtompset at hotmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list