From b28309d5a945aa61ec39ce769f1f743587e325db Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 8 Jul 2022 18:42:59 +0000 Subject: [PATCH] Bug 12029: (follow-up) Adhering to coding guidelines This is a follow-up patch which (based on comment #19) does the following: 1) Removes newline from top of Koha/Patron/Messages.pm 2) Changes method Koha::Patron::Messages->unread to ->filter_by_unread 3) Removes method Koha::Patron::Messages->unread_count 4) (Will address tests in separate patch) 5) Removes changes to Koha/Schema/Result/Message.pm 6) Changes atomicupdate from qq{} to q{} 7) (Left for RM) 8) Removes extra copyright statement in opac/dismiss_message.pl 9) Removes extra comments from opac/dismiss_message.pl 10) (Left for now) 11) (Left for now) Sponsored-by: Horowhenua Library Trust --- Koha/Patron/Messages.pm | 18 ++---------------- installer/data/mysql/atomicupdate/bug_12029.pl | 2 +- .../bootstrap/en/includes/opac-note.inc | 4 ++-- .../bootstrap/en/modules/opac-main.tt | 6 +++--- opac/dismiss_message.pl | 5 +++-- t/db_dependent/Koha/Patron/Messages.t | 12 ++++++------ 6 files changed, 17 insertions(+), 30 deletions(-) diff --git a/Koha/Patron/Messages.pm b/Koha/Patron/Messages.pm index 33eb7d6901..d0acf6bceb 100644 --- a/Koha/Patron/Messages.pm +++ b/Koha/Patron/Messages.pm @@ -50,13 +50,13 @@ sub object_class { return 'Koha::Patron::Message'; } -=head3 unread +=head3 filter_by_unread Returns a result set of messages that haven't been marked read by the patron =cut -sub unread { +sub filter_by_unread { # return resultset of messages with an undefined patron_read_date my ($self, $params) = @_; $params ||= {}; @@ -65,18 +65,4 @@ sub unread { %$params, }); } - -=head3 unread_count - - Returns a count of messages that haven't been marked read by the patron - -=cut - - -sub unread_count { - # return count of unread - my ($self, $params) = @_; - $params ||= {}; - return $self->unread(%$params)->count(); -} 1; diff --git a/installer/data/mysql/atomicupdate/bug_12029.pl b/installer/data/mysql/atomicupdate/bug_12029.pl index f352a66edf..5637d6b658 100755 --- a/installer/data/mysql/atomicupdate/bug_12029.pl +++ b/installer/data/mysql/atomicupdate/bug_12029.pl @@ -8,7 +8,7 @@ return { my ($dbh, $out) = @$args{qw(dbh out)}; # Do you stuffs here - my $alteration = qq{ + my $alteration = q{ ALTER TABLE messages ADD COLUMN `patron_read_date` timestamp NULL DEFAULT NULL COMMENT 'date and time patron dismissed message' diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc index 8b2c39f861..bae94785ea 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc +++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-note.inc @@ -1,8 +1,8 @@ -[% IF patron_messages.unread_count OR opacnote %] +[% IF patron_messages.filter_by_unread.count OR opacnote %]

Messages for you