Lines 36-97
my $op = $query->param('op') // q{};
Link Here
|
36 |
|
36 |
|
37 |
my ( $template, $borrowernumber, $cookie ) = get_template_and_user( |
37 |
my ( $template, $borrowernumber, $cookie ) = get_template_and_user( |
38 |
{ |
38 |
{ |
39 |
template_name => "opac-review.tt", |
39 |
template_name => "opac-review.tt", |
40 |
query => $query, |
40 |
query => $query, |
41 |
type => "opac", |
41 |
type => "opac", |
42 |
authnotrequired => ( C4::Context->preference("OpacPublic") ? 1 : 0 ), |
|
|
43 |
} |
42 |
} |
44 |
); |
43 |
); |
45 |
|
44 |
|
46 |
# FIXME: need to allow user to delete their own comment(s) |
45 |
# FIXME: need to allow user to delete their own comment(s) |
47 |
|
46 |
|
48 |
my ( $clean, @errors, $savedreview ); |
47 |
my ( $clean, @errors, $savedreview ); |
49 |
my $biblio = Koha::Biblios->find( $biblionumber ); |
48 |
my $biblio = Koha::Biblios->find($biblionumber); |
50 |
|
49 |
|
51 |
if( !$biblio ) { |
50 |
if ( !$biblio ) { |
52 |
push @errors, { nobiblio => 1 }; |
51 |
push @errors, { nobiblio => 1 }; |
53 |
} elsif( $reviewid ) { # edit existing one, check on creator |
52 |
} elsif ($reviewid) { # edit existing one, check on creator |
54 |
$savedreview = Koha::Reviews->search({ reviewid => $reviewid, borrowernumber => $borrowernumber })->next; |
53 |
$savedreview = Koha::Reviews->search( { reviewid => $reviewid, borrowernumber => $borrowernumber } )->next; |
55 |
push @errors, { unauthorized => 1 } if !$savedreview; |
54 |
push @errors, { unauthorized => 1 } if !$savedreview; |
56 |
} else { # this check prevents adding multiple comments |
55 |
} else { # this check prevents adding multiple comments |
57 |
# FIXME biblionumber, borrowernumber should be a unique key of reviews |
56 |
# FIXME biblionumber, borrowernumber should be a unique key of reviews |
58 |
$savedreview = Koha::Reviews->search({ biblionumber => $biblionumber, borrowernumber => $borrowernumber })->next; |
57 |
$savedreview = Koha::Reviews->search( { biblionumber => $biblionumber, borrowernumber => $borrowernumber } )->next; |
59 |
$review = $savedreview? $savedreview->review: $review; |
58 |
$review = $savedreview ? $savedreview->review : $review; |
60 |
} |
59 |
} |
61 |
|
60 |
|
62 |
if( $op =~ /^cud-/ && !@errors && defined $review ) { |
61 |
if ( $op =~ /^cud-/ && !@errors && defined $review ) { |
63 |
if ($review !~ /\S/) { |
62 |
if ( $review !~ /\S/ ) { |
64 |
push @errors, {empty=>1}; |
63 |
push @errors, { empty => 1 }; |
65 |
} else { |
64 |
} else { |
66 |
$clean = C4::Scrubber->new('comment')->scrub($review); |
65 |
$clean = C4::Scrubber->new('comment')->scrub($review); |
67 |
if ($clean !~ /\S/) { |
66 |
if ( $clean !~ /\S/ ) { |
68 |
push @errors, {scrubbed_all=>1}; |
67 |
push @errors, { scrubbed_all => 1 }; |
69 |
} else { |
68 |
} else { |
70 |
if ($clean ne $review) { |
69 |
if ( $clean ne $review ) { |
71 |
push @errors, {scrubbed=>$clean}; |
70 |
push @errors, { scrubbed => $clean }; |
72 |
} |
71 |
} |
73 |
if ($savedreview) { |
72 |
if ($savedreview) { |
74 |
$savedreview->set( |
73 |
$savedreview->set( |
75 |
{ |
74 |
{ |
76 |
review => $clean, |
75 |
review => $clean, |
77 |
approved => 0, |
76 |
approved => 0, |
78 |
datereviewed => dt_from_string |
77 |
datereviewed => dt_from_string |
79 |
} |
78 |
} |
80 |
)->store; |
79 |
)->store; |
81 |
} else { |
80 |
} else { |
82 |
$reviewid = Koha::Review->new( |
81 |
$reviewid = Koha::Review->new( |
83 |
{ biblionumber => $biblionumber, |
82 |
{ |
|
|
83 |
biblionumber => $biblionumber, |
84 |
borrowernumber => $borrowernumber, |
84 |
borrowernumber => $borrowernumber, |
85 |
review => $clean, |
85 |
review => $clean, |
86 |
datereviewed => dt_from_string |
86 |
datereviewed => dt_from_string |
87 |
} |
87 |
} |
88 |
)->store->reviewid; |
88 |
)->store->reviewid; |
89 |
} |
89 |
} |
90 |
unless (@errors){ $template->param(WINDOW_CLOSE=>1); } |
90 |
unless (@errors) { $template->param( WINDOW_CLOSE => 1 ); } |
91 |
} |
91 |
} |
92 |
} |
92 |
} |
93 |
} |
93 |
} |
94 |
(@errors ) and $template->param( ERRORS=>\@errors); |
94 |
(@errors) and $template->param( ERRORS => \@errors ); |
95 |
$review = $clean; |
95 |
$review = $clean; |
96 |
$review ||= $savedreview->review if $savedreview; |
96 |
$review ||= $savedreview->review if $savedreview; |
97 |
$template->param( |
97 |
$template->param( |
98 |
- |
|
|