Bug 10403: (follow-up) fix test to use vendor created earlier during test

[koha.git] / opac / opac-review.pl

diff --git a/opac/opac-review.pl b/opac/opac-review.pl
index 52f96e7..16ed56a 100755 (executable)
--- a/opac/opac-review.pl
+++ b/opac/opac-review.pl
@@ -13,9 +13,9 @@
 # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
 # A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 
 use strict;
 use warnings;
@@ -36,7 +36,7 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
         template_name   => "opac-review.tmpl",
         query           => $query,
         type            => "opac",
-        authnotrequired => 1,
+        authnotrequired => ( C4::Context->preference("OpacPublic") ? 1 : 0 ),
     }
 );
 
@@ -55,10 +55,10 @@ if (defined $review) {
                } else {
                        if ($clean ne $review) {
                                push @errors, {scrubbed=>$clean};
-                               my $js_ok_review = $clean;
-                               $js_ok_review =~ s/"/"/g;  # probably redundant w/ TMPL ESCAPE=JS
-                               $template->param(clean_review=>$js_ok_review);
                        }
+                       my $js_ok_review = $clean;
+                       $js_ok_review =~ s/"/"/g;  # probably redundant w/ TMPL ESCAPE=JS
+                       $template->param(clean_review=>$js_ok_review);
                        if ($savedreview) {
                        updatereview($biblionumber, $borrowernumber, $clean);
                        } else {