@@ -, +, @@ necessary (OPAC suggestions) Empty suggestion record was added succeffully. The required fields should be labeled in red. (you can change the mandatory fields by preference 'OPACSuggestionMandatoryFields') --- .../opac-tmpl/bootstrap/en/modules/opac-suggestions.tt | 4 ++-- opac/opac-suggestions.pl | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt +++ a/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; --- a/opac/opac-suggestions.pl +++ a/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, ); --