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

--- Comment #50 from Martin Renvoize <[email protected]> ---
Comment on attachment 99637
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=99637
Bug 4461: Redirect to 404 if KohaAdminEmailAddress is not defined

Review of attachment 99637:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=4461&attachment=99637)
-----------------------------------------------------------------

::: koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
@@ +376,5 @@
>                    yes: Allow
>                    no: "Don't allow"
>              - patrons to submit problem reports for OPAC pages to the 
> library or Koha Administrator.
> +            - "<br/>"
> +            - "<strong>NOTE:</strong> You must have KohaAdminEmailAddress 
> enabled."

This isn't actually true is it?.. we use branch email in preference to
KohaAdminEmailAddress so it's just a fallback not a hard requirement.

::: opac/opac-reportproblem.pl
@@ +41,5 @@
>  );
>  
> +if (   !C4::Context->preference('OPACReportProblem')
> +    || !C4::Context->preference('KohaAdminEmailAddress') )
> +{

Don't we actually use branchemail in preference to KohaAdminEmailAddress so
this isn't actually a hard requirement?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to