[Bug 16371] New: Quote of the Day (QOTD) - Display in different interfaces
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Bug ID: 16371 Summary: Quote of the Day (QOTD) - Display in different interfaces Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Notices Assignee: koha-bugs@lists.koha-community.org Reporter: olli-antti.kivilahti@jns.fi QA Contact: testopia@bugs.koha-community.org Quote of the day is a very handy tools to show random "tip-of-the-day"s, or "did-you-know"s for our librarians in the staff client. Since most of the staff don't read email, this will help to reach a wider audience and teach how to more effectively use Koha. This patch enables choosing the interface where the QOTD is shown. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |olli-antti.kivilahti@jns.fi |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aleishaamohia@hotmail.com --- Comment #1 from Aleisha Amohia <aleishaamohia@hotmail.com> --- No patch has been attached! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Indranil Das Gupta <indradg@l2c2.co.in> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |indradg@l2c2.co.in -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Summary|Quote of the Day (QOTD) - |Quote of the Day (QOTD) for |Display in different |the staff interface |interfaces | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|olli-antti.kivilahti@jns.fi |emmi.takkinen@outlook.com Status|NEW |Needs Signoff CC| |emmi.takkinen@outlook.com --- Comment #2 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 102515 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102515&action=edit Bug 16371: Add needed Koha::Exception Add Koha::Exception needed in this bug. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #3 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 102516 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102516&action=edit Bug 16371: Quote of the Day (QOTD) - Display in different interfaces Set 'QuoteOfTheDay'-syspref to 'opac intranet' to allow displaying news in the opac and the staff client. Add 'commandline' to allow news in the commandline (where applicable) Accepts all values accepted by the C4::Context->interface() Test plan: 1. Set 'QuoteOfTheDay' syspref as 'intranet' or 'opac' or both 2. Check that quote is visible on mainpage Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #4 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 102517 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102517&action=edit Bug 16371: Rework tests Use TestBuilder and 'mock_preferences' in tests. To test, prove t/db_dependent/Koha/GetDailyQuoteForInterface.t Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|aleisha@catalyst.net.nz | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david.roberts@ptfs-europe.c | |om Status|Needs Signoff |Failed QA --- Comment #5 from David Roberts <david.roberts@ptfs-europe.com> --- The QOTD certainly displays in the staff client with this patch in place, and the tests pass OK: t/db_dependent/Koha/GetDailyQuoteForInterface.t .. ok All tests successful. Files=1, Tests=3, 3 wallclock secs ( 0.03 usr 0.00 sys + 1.76 cusr 0.30 csys = 2.09 CPU) Result: PASS However, with this patch in place, the QOTD doesn't display in the OPAC. I also think that the system preference to display the Quote in the different interfaces would be better if it was a drop down (Off, Intranet Only, OPAC Only, Both), rather than "free text". Sorry :( -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- It could also be a multi-select with 2 checkboxes for staff and OPAC showing up. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #7 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 102969 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102969&action=edit Bug 16371: Use multi-select on syspref and remove 'display_daily_quote' condition This patch replaces textbox with multi-select dropdown on syspref 'QuoteOfTheDay' and changes tests to match this change. Also removes condition 'display_daily_quite' from opac-main.tt since it's no longer needed and prevents quotes showing in OPAC. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #8 from David Roberts <david.roberts@ptfs-europe.com> --- This hadn't been changed back to needs signoff, but I tested it anyway because the patches had been replaced. All looks good: t/db_dependent/Koha/GetDailyQuoteForInterface.t .. ok All tests successful. Files=1, Tests=3, 3 wallclock secs ( 0.04 usr 0.00 sys + 2.18 cusr 0.20 csys = 2.42 CPU) Result: PASS The functionality is working as I would expect in the clients as well, so happy to sign off as it is. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #102969|0 |1 is obsolete| | --- Comment #9 from David Roberts <david.roberts@ptfs-europe.com> --- Created attachment 103019 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103019&action=edit Bug 16371: Use multi-select on syspref and remove 'display_daily_quote' condition This patch replaces textbox with multi-select dropdown on syspref 'QuoteOfTheDay' and changes tests to match this change. Also removes condition 'display_daily_quite' from opac-main.tt since it's no longer needed and prevents quotes showing in OPAC. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Roberts <david@koha-ptfs.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #10 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Don't you want to take the opportunity to move existing code to new Koha::QuoteOfTheDay[s] objects? It does not sound a good idea to add new subroutines to C4::Koha -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |ASSIGNED --- Comment #11 from Emmi Takkinen <emmi.takkinen@outlook.com> --- (In reply to Jonathan Druart from comment #10)
Don't you want to take the opportunity to move existing code to new Koha::QuoteOfTheDay[s] objects?
It does not sound a good idea to add new subroutines to C4::Koha
Sure I would. When I fetched original patch here I left that subroutine to C4::Koha since there wasn't any other logical place to put it. But if it's ok to add these new classes, I'll look into it. I'm setting this bug as ASSIGNED. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #12 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 103329 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103329&action=edit Bug 16371: Add Koha::Quote[s] classes -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #13 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 103330 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103330&action=edit Bug 16371: Move GetDailyQuote to get_daily_quote This patch moves subroutine 'GetDailyQuote' to new Koha::Quote object and adjusts tests. To test: 1. Set 'QuoteOfTheDay' as 'enable' 2. Check that quote is displayed on OPAC mainpage Prove t/db_dependent/Koha/GetDailyQuote.t Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #14 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 103331 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103331&action=edit Bug 16371: Rename test file -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #102515|0 |1 is obsolete| | --- Comment #15 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 103332 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103332&action=edit Bug 16371: Add needed Koha::Exception Add Koha::Exception needed in this bug. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #102516|0 |1 is obsolete| | Attachment #102517|0 |1 is obsolete| | --- Comment #16 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 103333 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103333&action=edit Bug 16371: Quote of the Day (QOTD) - Display in different interfaces Set 'QuoteOfTheDay'-syspref to 'opac intranet' to allow displaying news in the opac and the staff client. Add 'commandline' to allow news in the commandline (where applicable) Accepts all values accepted by the C4::Context->interface() Test plan: 1. Set 'QuoteOfTheDay' syspref as 'intranet' or 'opac' or both 2. Check that quote is visible on both mainpages Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy --- NOTE: Tests already reworked in this patch so previous patch is now obsolete -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103019|0 |1 is obsolete| | --- Comment #17 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 103334 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103334&action=edit Bug 16371: Use multi-select on syspref This patch replaces textbox with multi-select dropdown on syspref 'QuoteOfTheDay' and changes tests to match this change. To test: 1. Check 'OPAC', 'staff client' or both on 'QuoteOfTheDay'. 2. Make sure that quote is displayed on (both) mainpage(s). Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy --- NOTE: Noticed that 'display_daily_quote' should have been dropped on original code. Fixed that so this patch just adds multi-select and adjusts tests. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com Status|Needs Signoff |Failed QA --- Comment #18 from David Nind <david@davidnind.com> --- Sorry, he tests fail 8-(: root@b749e4f21e73:koha(bz16371)$ prove t/db_dependent/Koha/Quotes.t t/db_dependent/Koha/Quotes.t .. 1/13 DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Duplicate entry '6' for key 'PRIMARY' [for Statement "INSERT INTO `quotes` ( `id`, `source`, `text`, `timestamp`) VALUES ( ?, ?, ?, ? )" with ParamValues: 0=6, 1='George Washington', 2='To be prepared for war is one of the most effectual means of preserving peace.', 3='2020-04-22 08:19:26'] at /kohadevbox/koha/Koha/Object.pm line 164 Duplicate ID# Looks like your test exited with 255 just after 2. t/db_dependent/Koha/Quotes.t .. Dubious, test returned 255 (wstat 65280, 0xff00) Failed 11/13 subtests Test Summary Report ------------------- t/db_dependent/Koha/Quotes.t (Wstat: 65280 Tests: 2 Failed: 0) Non-zero exit status: 255 Parse errors: Bad plan. You planned 13 tests but ran 2. Files=1, Tests=2, 1 wallclock secs ( 0.02 usr 0.01 sys + 1.53 cusr 0.13 csys = 1.69 CPU) Result: FAIL -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #19 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 103732 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103732&action=edit Bug 16371: Fix tests Tests failed due ids already existing in 'quotes' table. This patch removes hardcoded ids from tests. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103329|0 |1 is obsolete| | --- Comment #20 from David Nind <david@davidnind.com> --- Created attachment 103973 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103973&action=edit Bug 16371: Add Koha::Quote[s] classes Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103330|0 |1 is obsolete| | --- Comment #21 from David Nind <david@davidnind.com> --- Created attachment 103974 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103974&action=edit Bug 16371: Move GetDailyQuote to get_daily_quote This patch moves subroutine 'GetDailyQuote' to new Koha::Quote object and adjusts tests. To test: 1. Set 'QuoteOfTheDay' as 'enable' 2. Check that quote is displayed on OPAC mainpage Prove t/db_dependent/Koha/GetDailyQuote.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103331|0 |1 is obsolete| | --- Comment #22 from David Nind <david@davidnind.com> --- Created attachment 103975 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103975&action=edit Bug 16371: Rename test file Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103332|0 |1 is obsolete| | --- Comment #23 from David Nind <david@davidnind.com> --- Created attachment 103976 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103976&action=edit Bug 16371: Add needed Koha::Exception Add Koha::Exception needed in this bug. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #24 from David Nind <david@davidnind.com> --- Created attachment 103977 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103977&action=edit Bug 16371: Quote of the Day (QOTD) - Display in different interfaces Set 'QuoteOfTheDay'-syspref to 'opac intranet' to allow displaying news in the opac and the staff client. Add 'commandline' to allow news in the commandline (where applicable) Accepts all values accepted by the C4::Context->interface() Test plan: 1. Set 'QuoteOfTheDay' syspref as 'intranet' or 'opac' or both 2. Check that quote is visible on both mainpages Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103334|0 |1 is obsolete| | --- Comment #25 from David Nind <david@davidnind.com> --- Created attachment 103978 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103978&action=edit Bug 16371: Use multi-select on syspref This patch replaces textbox with multi-select dropdown on syspref 'QuoteOfTheDay' and changes tests to match this change. To test: 1. Check 'OPAC', 'staff client' or both on 'QuoteOfTheDay'. 2. Make sure that quote is displayed on (both) mainpage(s). Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103732|0 |1 is obsolete| | --- Comment #26 from David Nind <david@davidnind.com> --- Created attachment 103979 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103979&action=edit Bug 16371: Fix tests Tests failed due ids already existing in 'quotes' table. This patch removes hardcoded ids from tests. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #27 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am sorry Emmi, but this does no longer apply - can you please rebase? Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 16371: Quote of the Day (QOTD) - Display in different interfaces .git/rebase-apply/patch:208: trailing whitespace. error: sha1 information is lacking or useless (Koha/Quote.pm). error: could not build fake ancestor Patch failed at 0001 Bug 16371: Quote of the Day (QOTD) - Display in different interfaces The copy of the patch that failed is found in: .git/rebase-apply/patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-16371-Quote-of-the-Day-QOTD---Display-in-diffe-AWZo8w.patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103333|1 |0 is patch| | Attachment #103333|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off --- Comment #28 from Emmi Takkinen <emmi.takkinen@outlook.com> --- (In reply to Katrin Fischer from comment #27)
I am sorry Emmi, but this does no longer apply - can you please rebase?
Patch should now apply. Attachment 'Bug 16371: Quote of the Day (QOTD) - Display in different interfaces' hadn't been made obsolete at some point and was applied first. That's why whole thing failed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103333|0 |1 is patch| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #29 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Emmi, patches apply now, sorry for missing the patch double up. 0) There are some problems highlighted by the QA tools: FAIL Koha/Quote.pm FAIL critic "return" statement with explicit "undef" at line 129, column 9. See page 199 of PBP. FAIL Koha/Quotes.pm FAIL pod coverage POD is missing for 'object_class' FAIL koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt FAIL filters missing_filter at line 41 ( <span id="daily-quote-text">[% daily_quote.text %]</span><span id="daily-quote-sep"> ~ </span><span id="daily-quote-source">[% daily_quote.source %]</span>) missing_filter at line 41 ( <span id="daily-quote-text">[% daily_quote.text %]</span><span id="daily-quote-sep"> ~ </span><span id="daily-quote-source">[% daily_quote.source %]</span>) 1) I see that you moved the routine to the new namespace, but it appears to be unchanged. Especially the SQL parts I think should be replaced/rewritten. 2) I like making the old sys pref multiple choice :) Can you please use staff interface? https://wiki.koha-community.org/wiki/Terminology 3) I think we need a database update to change the type of the system preference in the database from YesNo to multi. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #30 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 106818 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106818&action=edit Bug 16371: Fix QA issues, change terminology -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #31 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 106819 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106819&action=edit Bug 16371: Rewrite get_daily_quote -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #32 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 106820 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106820&action=edit Bug 16371: Add database updates -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #33 from David Nind <david@davidnind.com> --- Everything works as expected, however the tests fail! 8-( prove t/db_dependent/Koha/Quotes.t t/db_dependent/Koha/Quotes.t .. 1/13 Can't use string ("Koha::Quote") as a HASH ref while "strict refs" in use at /kohadevbox/koha/Koha/Object.pm line 735. # Looks like your test exited with 255 just after 2. t/db_dependent/Koha/Quotes.t .. Dubious, test returned 255 (wstat 65280, 0xff00) Failed 11/13 subtests Test Summary Report ------------------- t/db_dependent/Koha/Quotes.t (Wstat: 65280 Tests: 2 Failed: 0) Non-zero exit status: 255 Parse errors: Bad plan. You planned 13 tests but ran 2. Files=1, Tests=2, 2 wallclock secs ( 0.03 usr 0.00 sys + 1.63 cusr 0.16 csys = 1.82 CPU) Result: FAIL An updated test plan: 1. Apply the patch. 2. Run the database update (updatedatabase on koha-testing-docker). 3. Check that the 'QuoteOfTheDay' system preference options work as expected: - OPAC: QOTD only appears in the OAPC - Staff interface: QOTD only appears in the staff interface - Both (Select all): QOTD appears in the staff interface and OPAC 4. Run the tests and make sure they pass: prove t/db_dependent/Koha/Quotes.t 5. Sign off! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #34 from Emmi Takkinen <emmi.takkinen@outlook.com> --- (In reply to David Nind from comment #33)
Everything works as expected, however the tests fail! 8-(
prove t/db_dependent/Koha/Quotes.t t/db_dependent/Koha/Quotes.t .. 1/13 Can't use string ("Koha::Quote") as a HASH ref while "strict refs" in use at /kohadevbox/koha/Koha/Object.pm line 735. # Looks like your test exited with 255 just after 2. t/db_dependent/Koha/Quotes.t .. Dubious, test returned 255 (wstat 65280, 0xff00) Failed 11/13 subtests
Sorry, I was so focused on getting random quote work that testing changes I made to get quote with id slipped my mind (and also actually checking if tests still work). I'll provide patch with fix as soon as possible. And thanks for new test plan! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106819|0 |1 is obsolete| | --- Comment #35 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 106858 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106858&action=edit Bug 16371: Rewrite get_daily_quote An updated test plan: 1. Apply the patch(es). 2. Run the database update (updatedatabase on koha-testing-docker). 3. Check that the 'QuoteOfTheDay' system preference options work as expected: - OPAC: QOTD only appears in the OAPC - Staff interface: QOTD only appears in the staff interface - Both (Select all): QOTD appears in the staff interface and OPAC 4. Run the tests and make sure they pass: prove t/db_dependent/Koha/Quotes.t 5. Sign off! ---- Fixed errors on get_daily_quote and adjusted tests. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103973|0 |1 is obsolete| | --- Comment #36 from David Nind <david@davidnind.com> --- Created attachment 106859 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106859&action=edit Bug 16371: Add Koha::Quote[s] classes Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103974|0 |1 is obsolete| | --- Comment #37 from David Nind <david@davidnind.com> --- Created attachment 106860 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106860&action=edit Bug 16371: Move GetDailyQuote to get_daily_quote This patch moves subroutine 'GetDailyQuote' to new Koha::Quote object and adjusts tests. To test: 1. Set 'QuoteOfTheDay' as 'enable' 2. Check that quote is displayed on OPAC mainpage Prove t/db_dependent/Koha/GetDailyQuote.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103975|0 |1 is obsolete| | --- Comment #38 from David Nind <david@davidnind.com> --- Created attachment 106861 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106861&action=edit Bug 16371: Rename test file Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103976|0 |1 is obsolete| | --- Comment #39 from David Nind <david@davidnind.com> --- Created attachment 106862 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106862&action=edit Bug 16371: Add needed Koha::Exception Add Koha::Exception needed in this bug. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103977|0 |1 is obsolete| | --- Comment #40 from David Nind <david@davidnind.com> --- Created attachment 106863 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106863&action=edit Bug 16371: Quote of the Day (QOTD) - Display in different interfaces Set 'QuoteOfTheDay'-syspref to 'opac intranet' to allow displaying news in the opac and the staff client. Add 'commandline' to allow news in the commandline (where applicable) Accepts all values accepted by the C4::Context->interface() Test plan: 1. Set 'QuoteOfTheDay' syspref as 'intranet' or 'opac' or both 2. Check that quote is visible on both mainpages Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103978|0 |1 is obsolete| | --- Comment #41 from David Nind <david@davidnind.com> --- Created attachment 106864 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106864&action=edit Bug 16371: Use multi-select on syspref This patch replaces textbox with multi-select dropdown on syspref 'QuoteOfTheDay' and changes tests to match this change. To test: 1. Check 'OPAC', 'staff client' or both on 'QuoteOfTheDay'. 2. Make sure that quote is displayed on (both) mainpage(s). Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103979|0 |1 is obsolete| | --- Comment #42 from David Nind <david@davidnind.com> --- Created attachment 106865 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106865&action=edit Bug 16371: Fix tests Tests failed due ids already existing in 'quotes' table. This patch removes hardcoded ids from tests. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106818|0 |1 is obsolete| | --- Comment #43 from David Nind <david@davidnind.com> --- Created attachment 106866 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106866&action=edit Bug 16371: Fix QA issues, change terminology Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106820|0 |1 is obsolete| | --- Comment #44 from David Nind <david@davidnind.com> --- Created attachment 106867 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106867&action=edit Bug 16371: Add database updates Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106858|0 |1 is obsolete| | --- Comment #45 from David Nind <david@davidnind.com> --- Created attachment 106868 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106868&action=edit Bug 16371: Rewrite get_daily_quote An updated test plan: 1. Apply the patch(es). 2. Run the database update (updatedatabase on koha-testing-docker). 3. Check that the 'QuoteOfTheDay' system preference options work as expected: - OPAC: QOTD only appears in the OAPC - Staff interface: QOTD only appears in the staff interface - Both (Select all): QOTD appears in the staff interface and OPAC 4. Run the tests and make sure they pass: prove t/db_dependent/Koha/Quotes.t 5. Sign off! Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106859|0 |1 is obsolete| | --- Comment #46 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106894 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106894&action=edit Bug 16371: Add Koha::Quote[s] classes Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106860|0 |1 is obsolete| | --- Comment #47 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106895 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106895&action=edit Bug 16371: Move GetDailyQuote to get_daily_quote This patch moves subroutine 'GetDailyQuote' to new Koha::Quote object and adjusts tests. To test: 1. Set 'QuoteOfTheDay' as 'enable' 2. Check that quote is displayed on OPAC mainpage Prove t/db_dependent/Koha/GetDailyQuote.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106861|0 |1 is obsolete| | --- Comment #48 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106896 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106896&action=edit Bug 16371: Rename test file Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106862|0 |1 is obsolete| | --- Comment #49 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106897 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106897&action=edit Bug 16371: Add needed Koha::Exception Add Koha::Exception needed in this bug. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106863|0 |1 is obsolete| | --- Comment #50 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106898 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106898&action=edit Bug 16371: Quote of the Day (QOTD) - Display in different interfaces Set 'QuoteOfTheDay'-syspref to 'opac intranet' to allow displaying news in the opac and the staff client. Add 'commandline' to allow news in the commandline (where applicable) Accepts all values accepted by the C4::Context->interface() Test plan: 1. Set 'QuoteOfTheDay' syspref as 'intranet' or 'opac' or both 2. Check that quote is visible on both mainpages Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106864|0 |1 is obsolete| | --- Comment #51 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106899 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106899&action=edit Bug 16371: Use multi-select on syspref This patch replaces textbox with multi-select dropdown on syspref 'QuoteOfTheDay' and changes tests to match this change. To test: 1. Check 'OPAC', 'staff client' or both on 'QuoteOfTheDay'. 2. Make sure that quote is displayed on (both) mainpage(s). Prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106865|0 |1 is obsolete| | --- Comment #52 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106900 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106900&action=edit Bug 16371: Fix tests Tests failed due ids already existing in 'quotes' table. This patch removes hardcoded ids from tests. Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106866|0 |1 is obsolete| | --- Comment #53 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106901 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106901&action=edit Bug 16371: Fix QA issues, change terminology Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106867|0 |1 is obsolete| | --- Comment #54 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106902 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106902&action=edit Bug 16371: Add database updates Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106868|0 |1 is obsolete| | --- Comment #55 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 106903 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106903&action=edit Bug 16371: Rewrite get_daily_quote An updated test plan: 1. Apply the patch(es). 2. Run the database update (updatedatabase on koha-testing-docker). 3. Check that the 'QuoteOfTheDay' system preference options work as expected: - OPAC: QOTD only appears in the OAPC - Staff interface: QOTD only appears in the staff interface - Both (Select all): QOTD appears in the staff interface and OPAC 4. Run the tests and make sure they pass: prove t/db_dependent/Koha/Quotes.t 5. Sign off! Sponsored-by: Koha-Suomi Oy Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com, tomascohen@gmail.com --- Comment #56 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I think Koha::Quote[s] is too generic, I am asking a couple of other QAers for feedback. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #57 from Tomás Cohen Arazi <tomascohen@gmail.com> --- (In reply to Jonathan Druart from comment #56)
I think Koha::Quote[s] is too generic, I am asking a couple of other QAers for feedback.
I think if we had some feature to place a RFQ or quote, it would be placed inside the Koha::Acquisitions namespace, and so we can move forward with this naming. I'd wait for some native users feedback, though. Well done everyone! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #58 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Also, we don't need the weird "Koha::Exceptions::UnknownProgramState" exception, C4::Context->interface() is always returning something. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #59 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- "Cite" as in "Citations" is the only real synonym I can come up with.. but it's not especially clear to me.. I find that more confusing the "Quote" I think... Not read the patches yet so, just seen the comment with the question. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #60 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Martin Renvoize from comment #59)
"Cite" as in "Citations" is the only real synonym I can come up with.. but it's not especially clear to me.. I find that more confusing the "Quote" I think...
Not read the patches yet so, just seen the comment with the question.
I think having another mismatch between GUI and code would not be idea - for this reason I think -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #61 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Martin Renvoize from comment #59)
"Cite" as in "Citations" is the only real synonym I can come up with.. but it's not especially clear to me.. I find that more confusing the "Quote" I think...
Not read the patches yet so, just seen the comment with the question.
I think having another mismatch between GUI and code would not be idea - for this reason I think Quote woudl be best actually. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #62 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Quote vs DailyQuote vs QuoteOfTheDay? I think Koha::DailyQuote could be more meaningful. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #63 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- More meaningful, but the daily could change at some point. I'd still be ok with that as a compromise. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #64 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Personally.. I think Koha::Quote is fine.. for other uses I'm envisaging they're more nested.. i.e Acquisition::Quotes as Tomas highlighted. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #65 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Hum, there are still occurrences of GetDailyQuote in t/db_dependent/Koha.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #66 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Better in only one comment: 1. We don't need the weird "Koha::Exceptions::UnknownProgramState" exception, C4::Context->interface() is always returning something. 2. There are still occurrences of GetDailyQuote in t/db_dependent/Koha.t (the tests are failing) 3. The methods from Koha::Quote must be in Koha::Quotes 4. +use DBI qw(:sql_types); We don't need that 5. You are adding raw SQL queries in Koha namespace, that is not needed. 6. After 5, DateTime::Format::MySQL become useless 7. I don't think we need the 2 methods in Koha::Quotes, could be only 1, get_daily_quote -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #67 from Emmi Takkinen <emmi.takkinen@outlook.com> --- (In reply to Jonathan Druart from comment #66)
2. There are still occurrences of GetDailyQuote in t/db_dependent/Koha.t (the tests are failing)
Didn't notice these until now. But looking them now it seems they work exactly like tests in Quotes.t. Could these be removed at this point?
5. You are adding raw SQL queries in Koha namespace, that is not needed.
6. After 5, DateTime::Format::MySQL become useless
Sorry for being dense, but I'm failing to see what queries you mean. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #68 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Emmi Takkinen from comment #67)
(In reply to Jonathan Druart from comment #66)
2. There are still occurrences of GetDailyQuote in t/db_dependent/Koha.t (the tests are failing)
Didn't notice these until now. But looking them now it seems they work exactly like tests in Quotes.t. Could these be removed at this point?
Yes sure.
5. You are adding raw SQL queries in Koha namespace, that is not needed.
6. After 5, DateTime::Format::MySQL become useless
Sorry for being dense, but I'm failing to see what queries you mean.
Looks like I forgot to review with the last patch! Then 5 is not valid anymore, however: 70 timestamp => { -like => "$dt%" }, This is a bit unusual to me, but I cannot think of something else (apart from using \'CAST() that is not better). I will ask a couple of QAers how we should do that. 94 # update the timestamp for that quote 95 my $dt = DateTime::Format::MySQL->format_datetime(dt_from_string()); 96 $quote->update({ timestamp => $dt }); You should not use DateTime::Format::MySQL directly. If you need to use a date in a search you should use $dtf->format_date[time] instead -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #69 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Jonathan Druart from comment #68)
(In reply to Emmi Takkinen from comment #67)
(In reply to Jonathan Druart from comment #66) 70 timestamp => { -like => "$dt%" },
This is a bit unusual to me, but I cannot think of something else (apart from using \'CAST() that is not better). I will ask a couple of QAers how we should do that.
We got an answer from #dbix-class, we should use -between => [ "$dt 00:00:00", "$dt 23:59:59" ], to be DB agnostic and allow use of indexes. $dt must be generated using $dtf->format_date. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #70 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 108097 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108097&action=edit Bug 16371: Remove GetDailyQuote tests from Koha.t Tests in Koha.t failed due GetDailyQuote tests. This patch removes these tests since we test quotes in Quotes.t instead. To test prove t/db_dependent/Koha.t Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #71 from Emmi Takkinen <emmi.takkinen@outlook.com> --- Created attachment 108098 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108098&action=edit Bug 16371: Combine get_daily_quote and get_daily_quote_for_interface This patch combines get_daily_quote and get_daily_quote_for_interface methods and moves them from Koha::Quote to Koha::Quotes. Also removes some unused code and adjusts datetime parsing. To test apply this patch and confirm 'QuoteOfTheDay' syspref still works as expected (quote is shown in correct mainpage(s)). Also prove t/db_dependent/Koha/Quotes.t Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Emmi Takkinen <emmi.takkinen@outlook.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #72 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 108160 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108160&action=edit Bug 16371: Fix inconsistencies with timestamp The timestamp value needs to be updated for each quote returned by get_daily_quote (even when there is a match) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |20.11.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #73 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 20.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 --- Comment #74 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 108163 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108163&action=edit Bug 16371: (follow-up) DB changes We need to keep "opac" is the pref was set before the upgrade -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #75 from Lucas Gass <lucas@bywatersolutions.com> --- enhancement will not be backported to 20.05.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This enhancement lets you release notes| |choose where the Quote of | |the Day (QOTD) is | |displayed: | |- OPAC: QOTD | |only appears in the OPAC. | |- | |Staff interface: QOTD only | |appears in the staff | |interface. | |- Both [Select | |all]: QOTD appears in the | |staff interface and OPAC. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |27466 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27466 [Bug 27466] Error "Configuration not defined at /usr/share/perl5/Log/Log4perl/Config.pm line 579" while upgrading to koha 20.11.00-1 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16371 Niamh Walker-Headon <Niamh.WalkerHeadon@hse.ie> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |Niamh.WalkerHeadon@hse.ie -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org