Bug 16371: Fix QA issues, change terminology
authorEmmi Takkinen <emmi.takkinen@outlook.com>
Thu, 9 Jul 2020 08:55:03 +0000 (11:55 +0300)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 13 Aug 2020 08:15:33 +0000 (10:15 +0200)
Sponsored-by: Koha-Suomi Oy

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Koha/Quote.pm
Koha/Quotes.pm
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt

index f79cf79..244aea0 100644 (file)
@@ -113,7 +113,7 @@ sub get_daily_quote {
 
 Is a wrapper for get_daily_quote(), with an extra check for using the correct
 interface defined in the syspref 'QuoteOfTheDay'.
-If the current interface is not allowed to display quotes, then returns undef.
+If the current interface is not allowed to display quotes, then returns nothing.
 
 =cut
 
@@ -126,7 +126,7 @@ sub get_daily_quote_for_interface {
         Koha::Exceptions::UnknownProgramState->throw(error => $cc[3]."()> C4::Context->interface() is not set! Don't know are you in OPAC or staff client?");
     }
     unless ($qotdPref =~ /$interface/) {
-        return undef;
+        return;
     }
 
     return $self->get_daily_quote(%opts);
index 0fe2e16..6a9d83a 100644 (file)
@@ -41,6 +41,10 @@ sub _type {
     return 'Quote';
 }
 
+=head3 object_class
+
+=cut
+
 sub object_class {
     return 'Koha::Quote';
 }
index 9fdd12b..dcf8d39 100644 (file)
@@ -494,7 +494,7 @@ OPAC:
             - Show quote of the day in the
             - pref: QuoteOfTheDay
               multiple:
-                intranet: staff client
+                intranet: staff interface
                 opac: OPAC
             - main page.
         -
index 24d8aa4..882a446 100644 (file)
@@ -38,7 +38,7 @@
                     <div id="area-news">
                         <h3>Quote of the Day</h3>
                         <div class="newsitem">
-                            <span id="daily-quote-text">[% daily_quote.text %]</span><span id="daily-quote-sep"> ~ </span><span id="daily-quote-source">[% daily_quote.source %]</span>
+                            <span id="daily-quote-text">[% daily_quote.text | html %]</span><span id="daily-quote-sep"> ~ </span><span id="daily-quote-source">[% daily_quote.source | html %]</span>
                         </div>
                     </div>
                 [% END %]