[Koha-bugs] [Bug 4461] Context-sensitive report a problem screen

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Jan 13 14:18:31 CET 2017


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

Owen Leonard <oleonard at myacpl.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #3 from Owen Leonard <oleonard at myacpl.org> ---
I think this looks good, but there are some issues that need to be looked at:

I think that if there is no email address for the logged-in user's library the
page should not offer to email the librarian. In my test if I submit a report
to the librarian and there is no email address for my home library, the entry
saved in message_queue has an empty 'to_address' field.

There are HTML validation errors triggered by duplicate id attributes on form
fields. The form fields on the two different tabs should not have the same ids
(note that input name and id attributes do not have to match);

Also, in the OPAC (and soon to be the case in the staff client), page-specific
JavaScript goes into a special BLOCK which is processed in the footer include.
So instead of this, at the end of opac-reportaproblem.tt:

[% BLOCK jsinclude %][% END %]

<script type="text/javascript">
    $(document).ready(function(){
        $("#reportproblem").tabs();
    });
</script>

It should be:

[% BLOCK jsinclude %]
    <script type="text/javascript">
        $(document).ready(function(){
            $("#reportproblem").tabs();
        });
    </script>
[% END %]

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.


More information about the Koha-bugs mailing list