@@ -, +, @@ - uses a pull down menu to switch the recipient instead of tabs - if the library has no email or the administrator has no email, this error message has been moved to be down by the Submit button - includes use of branchreplyto and ReplytoDefault for branch email - disables use of the feature if the user is not logged in - adds a syspref to hide or show the feature - provides support for if JS is disabled - using the message_queue table, check that the correct recipient receives a problem report when switching in the drop down menu - check appropriate error message shows next to the submit button if an email is not provided. this should dynamically change as you select a different recipient from the drop down menu - check that when the library is the recipient, the feature sends the report to the branchreplyto, then the ReplytoDefault, then the branch email in that priority order - log out of the OPAC and confirm that you can't submit a report without logging in. the submit button should be disabled - disable the OPACReportProblem syspref. the Report a Problem link in the footer should be gone. if you force the URL (/cgi-bin/koha/opac-reportproblem.pl) the form should be hidden and replaced with an error message. - disable JS in your browser. confirm that you are still able to send problem reports to library and administrator. remove the administrator email address. confirm the error message shows next to the submit button and that clicking the submit button returns an error. check in the message_queue that the email was never sent. - i would like to add a staff-side feature where staff can see the problem reports and mark them as 'read'. i'll work on that once this gets through :) --- .../bug-4461_add-problem-reports-syspref.sql | 1 + installer/data/mysql/sysprefs.sql | 1 + .../prog/en/modules/admin/preferences/opac.pref | 6 + .../bootstrap/en/includes/opac-bottom.inc | 8 +- .../bootstrap/en/modules/opac-reportproblem.tt | 132 +++++++++++---------- opac/opac-reportproblem.pl | 116 ++++++++++-------- 6 files changed, 147 insertions(+), 117 deletions(-) create mode 100644 installer/data/mysql/atomicupdate/bug-4461_add-problem-reports-syspref.sql --- a/installer/data/mysql/atomicupdate/bug-4461_add-problem-reports-syspref.sql +++ a/installer/data/mysql/atomicupdate/bug-4461_add-problem-reports-syspref.sql @@ -0,0 +1, @@ +INSERT IGNORE INTO systempreferences (`variable`, `value`, `options`, `explanation`, `type`) VALUES ('OPACReportProblem', 1, NULL, "A feature for users to submit problem reports for OPAC pages.",'YesNo'); --- a/installer/data/mysql/sysprefs.sql +++ a/installer/data/mysql/sysprefs.sql @@ -365,6 +365,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` ('opacreadinghistory','1','','If ON, enables display of Patron Circulation History in OPAC','YesNo'), ('OpacRenewalAllowed','0',NULL,'If ON, users can renew their issues directly from their OPAC account','YesNo'), ('OpacRenewalBranch','checkoutbranch','itemhomebranch|patronhomebranch|checkoutbranch|null','Choose how the branch for an OPAC renewal is recorded in statistics','Choice'), +('OPACReportProblem', 1, NULL, "A feature for users to submit problem reports for OPAC pages.",'YesNo'), ('OpacResetPassword','0','','Shows the ''Forgot your password?'' link in the OPAC','YesNo'), ('OPACResultsLibrary', 'homebranch', 'homebranch|holdingbranch', 'Defines whether the OPAC displays the holding or home branch in search results when using XSLT', 'Choice'), ('OPACResultsSidebar','','70|10','Define HTML to be included on the search results page, underneath the facets sidebar','Textarea'), --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref @@ -345,6 +345,12 @@ OPAC: footer: "only footer" Features: - + - pref: OPACReportProblem + choices: + yes: Allow + no: "Don't allow" + - patrons to submit problem reports for OPAC pages. + - - pref: opacuserlogin choices: yes: Allow --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc +++ a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc @@ -16,9 +16,11 @@