diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt index f560e642b2..5d4e8b8aef 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt @@ -339,9 +339,9 @@ return true; }); [% END %] - [% IF ( op_add && mandatoryfields ) %] + [% IF ( op_add && mandatoryfields.size ) %] { - var FldsRequired = [[% mandatoryfields | $raw %]]; + var FldsRequired = ['[% mandatoryfields.join("','") | html %]']; for (var i = 0; i < FldsRequired.length; i++) { var rq_input = $('#' + FldsRequired[i]); if (rq_input.length != 1) continue; diff --git a/opac/opac-suggestions.pl b/opac/opac-suggestions.pl index 29de7a1f65..bd42478fa8 100755 --- a/opac/opac-suggestions.pl +++ b/opac/opac-suggestions.pl @@ -208,11 +208,11 @@ if ( C4::Context->preference("AllowPurchaseSuggestionBranchChoice") ) { $template->param( branchcode => $branchcode ); } -my $mandatoryfields = ''; +my @mandatoryfields; { last unless ($op eq 'add'); my $fldsreq_sp = C4::Context->preference("OPACSuggestionMandatoryFields") || 'title'; - $mandatoryfields = join(', ', (map { '"'.$_.'"'; } sort split(/\s*\,\s*/, $fldsreq_sp))); + @mandatoryfields = sort split(/\s*\,\s*/, $fldsreq_sp); } $template->param( @@ -224,7 +224,7 @@ $template->param( messages => \@messages, suggestionsview => 1, suggested_by_anyone => $suggested_by_anyone, - mandatoryfields => $mandatoryfields, + mandatoryfields => \@mandatoryfields, patrons_pending_suggestions_count => $patrons_pending_suggestions_count, );