[Bug 8443] New: Suggestions : publication year and copyright date
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Priority: P5 - low Change sponsored?: --- Bug ID: 8443 Assignee: koha-bugs@lists.koha-community.org Summary: Suggestions : publication year and copyright date Severity: normal Classification: Unclassified OS: All Reporter: fridolyn.somers@biblibre.com Hardware: All Status: NEW Version: master Component: Acquisitions Product: Koha New suggestion page contains only copyright date field. Suggestions management page filters contains only publication year. Suggestions table contains both publicationyear and copyrightdate columns. Isn't it the same thing ? The bug is that the filter doesn't work because new suggestions have no publication year, they have a copyrigth date. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=6070 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #1 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 10827 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=10827&action=edit New suggestion form (intranet) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #2 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 10828 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=10828&action=edit Suggestions management (intranet) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Assignee|koha-bugs@lists.koha-commun |fridolyn.somers@biblibre.co |ity.org |m --- Comment #3 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 11100 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11100&action=edit Proposed patch Proposed Patch : Filter suggestions by copyrightdate instead of publicationyear (since it is copyrightdate in the suggestion creation form). Look at Suggestions Management/Bibliographic information. Corrects a bug on date filters (suggested, managed, accepted dates), only suggesteddate could work. Corrects a bug when display by itemtype : itemtype can be null or '' in db. Patch aggregates this two values into the same tab (No name). In suggestion.tt, downsizes a little bit the date fields width so that calendar icon comes on the same line (and so filters part is less high). -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Corinne HAYET <corinne.hayet@bulac.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |corinne.hayet@bulac.fr --- Comment #4 from Corinne HAYET <corinne.hayet@bulac.fr> --- Hi Fridolyn, You talk about copyright date...but there's no copyright date in your new form ??? You talk about itemtype...but there's non itemtype filter ?? budget and date filters dont'work. For date filters, have look to the pop-up for the calendar. There's a problem, date are translated (and searched ?) like MM/JJ/AAAA (for exemple : 01/08/2012 wrong translation in 08/01/2012) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #5 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Hie Corinne.
You talk about copyright date...but there's no copyright date in your new form ??? I changed the filter on publication year into copyright date. Copyright date is suggestion form in OPAC and intranet.
You talk about itemtype...but there's non itemtype filter ?? I talk about item type in "organize by" combobox.
budget and date filters dont'work. I did not test budget filter. Maybe I'll try, otherwise I can be a new bug.
For date filters, ... Can't reproduce this problem. If it's a javascript problem, it can be a new bug. I corrected something in pl file.
Best regards, -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=8440 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11100|0 |1 is obsolete| | --- Comment #6 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 11741 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11741&action=edit Proposed patch (2) Filter on budget was indeed not working. I corrected it and also removed from filters the empty values that can come form database. -- You are receiving this mail because: You are watching all bug changes.
budget and date filters dont'work. Testing date filters is not easy because they don't appear in suggestions
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #7 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- table. It will be corrected soon in master, see BZ8440. Retest please. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11741|0 |1 is obsolete| | --- Comment #8 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 11759 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11759&action=edit [SIGNED-OFF] Bug 8443: Suggestions publication year and copyright date Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Tested: - Copyright date is saved on creating suggestions from OPAC and staff - Copyright filter works - Form and filter labelling match - Budget filter works - When ordering an accepted suggestion the publication year is filled with the copyright date. I noted some problems I found while testing this on the bug. This patch is still an improvement to reach more consistency. Signing off. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |katrin.fischer@bsz-bw.de --- Comment #9 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Fridolyn, I found some smaller issues: 1) When I create a new order, the form says "Publication year". I think it would be good to be consistent and use "Publication year" or "Copyright date" everywhere. 2) The suggeston form in the OPAC says "Copyright Date", should be "Copyright date" according to our capitalization rules. 3) When I use filters, they get hidden again when the page reloads to show the search results. This makes it hard to see, which filters have been applied. A menu with a filter used should remain open, I think. 4) After changing the status of a suggestion, all other tabs vanish and I can only see the tab I have changed the suggestion to. And additionally, on this tab only the updated suggestion shows, even if there are more. Can we fix that? :( 5) For all to-from-filters the calendar icon gets pushed to the next line for me. That's a small thing, but would be nice if it could get fixed. 6) When you order from a suggestion, in the lists of suggestions the copyright symbol is broken. The title and year display like this: copy. year:2012 7) When I use the copyright filter the URL and the filter reflect what I did - the year still shows up when I reopen the bilbiographic filter section. When I search for a suggestion date, the date vanishes after applying the filter. And when I add another filter, all filter options are kept, except for the dates. A follow up looking into some of these problems would be great. I can also file separate bugs for those - please let me know. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #10 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Thanks for testing Katrin, I think you should create new bugs to allow small patches, depending on a main bug (like 'suggestions improvements'). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl --- Comment #11 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- CONFLICT (content): Merge conflict in C4/Acquisition.pm Seems trivial. But please correct. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11759|0 |1 is obsolete| | --- Comment #12 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 12494 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12494&action=edit Proposed patch (rebased master) Rebased patch. Only a little conflict in a comment line in C4/Acquisition.pm -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |paul.poulain@biblibre.com --- Comment #13 from Paul Poulain <paul.poulain@biblibre.com> --- QA comment: * this patch also includes a lot of indent & perltidy. *please, don't do that*, it's very hard to see what is related to your patch and what is related to perltidy. perltidy++, but in a follow-up patch please ! * passes koha-qa.pl, except for C4/Suggestions.pm FAIL pod FAIL *** WARNING: line containing nothing but whitespace in paragraph in file C4/Suggestions.pm *** ERROR: Spurious text after =cut in file C4/Suggestions.pm but it was already here before the patch passes QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #14 from Paul Poulain <paul.poulain@biblibre.com> --- Testing the feature, I've a problem: I have suggestions with copyrightdate (2011), and some without. When I filter on 2011, the suggestions without still appear. When I filter on 232545 (yes, that's not a year ;-) ), all suggestions still appear, including those 2011 as date. Could someone check the filter & tell me if it work for him/her ? Failed QA in the meantime -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #15 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to comment #14)
Testing the feature, I've a problem: I have suggestions with copyrightdate (2011), and some without. When I filter on 2011, the suggestions without still appear. When I filter on 232545 (yes, that's not a year ;-) ), all suggestions still appear, including those 2011 as date.
Could someone check the filter & tell me if it work for him/her ?
Failed QA in the meantime
I can't reproduce, with copyright date 0 or NULL. Could you give a dump of your suggestions table ? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Passed QA Version|master |rel_3_10 --- Comment #16 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #15)
I can't reproduce, with copyright date 0 or NULL. Could you give a dump of your suggestions table ?
strange. I tried again, and it worked well. I set the status to passed QA. As i've declared "string freeze", I won't push this patch for 3.10.0, but it will be for 3.10.1, definetely -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|rel_3_10 |rel_3_12 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |In Discussion -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #17 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Please do not perltidy large sections of code when fixing bugs. It makes it difficult to see what changes were made. Could you please divide this into two patches, one with the perltidy and one with the bugfix? They can be in either order (perltidy first or bugfix first). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12494|0 |1 is obsolete| | --- Comment #18 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 13265 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13265&action=edit Proposed patch (main) I splitted into several patches : Main patch correcting suggestions search : - allow filtering on copyrightdate, manageddate and accepteddate - entered dates remains in filter during search - empty and null values are in the same tab - null/empty values are not present in filers comboboxes -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #19 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 13266 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13266&action=edit Proposed patch (follow-up 1) Small correction in method comments. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #20 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 13267 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13267&action=edit Proposed patch (follow-up 2) Cosmetic patch : Manual editions : - use of () for qw - use of q{} or qq{} in SQL queries - replacing $${key} by $->{key} - formatting SQL queries (easier to read) - in suggestion.tt reducing date input size to allow calendar icon on same line + Formatting with perltidy -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Signed Off --- Comment #21 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Needs QA again. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact| |katrin.fischer@bsz-bw.de -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff --- Comment #22 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Fridolyn, thx for splitting up the patches as requested. Can we please get a new sign-off on these patches? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |chris@bigballofwax.co.nz Patch complexity|--- |Large patch --- Comment #23 from Chris Cormack <chris@bigballofwax.co.nz> --- Have signed off, however it would have been better for the follow up 2 to have the perltidy in a separate patch. Mixing any changes with a file sized perltidy just makes testing a lot harder. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13265|0 |1 is obsolete| | --- Comment #24 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 13958 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13958&action=edit Bug 8443: Suggestions publication year and copyright date Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13266|0 |1 is obsolete| | --- Comment #25 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 13959 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13959&action=edit Bug 8443: Suggestions publication year and copyright date (follow-up 1) Comments corrections Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13267|0 |1 is obsolete| | --- Comment #26 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 13960 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13960&action=edit Bug 8443: Suggestions publication year and copyright date (follow-up 2) Cosmetic changes and perltidy formatting Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #27 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Working on this. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13958|0 |1 is obsolete| | Attachment #13959|0 |1 is obsolete| | Attachment #13960|0 |1 is obsolete| | --- Comment #28 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 14568 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14568&action=edit [SIGNED-OFF] Bug 8443: Suggestions publication year and copyright date Main patch correcting suggestions search : - allow filtering on copyrightdate, manageddate and accepteddate - entered dates remains in filter during search - empty and null values are in the same tab - null/empty values are not present in filers comboboxes Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #29 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 14569 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14569&action=edit Bug 8443: Suggestions publication year and copyright date (follow-up 1) Comments corrections Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 --- Comment #30 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 14570 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14570&action=edit [SIGNED-OFF] Bug 8443: Suggestions publication year and copyright date (follow-up 2) Cosmetic changes and perltidy formatting Cosmetic patch : Manual editions : - use of () for qw - use of q{} or qq{} in SQL queries - replacing $${key} by $->{key} - formatting SQL queries (easier to read) - in suggestion.tt reducing date input size to allow calendar icon on same line + Formatting with perltidy Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> All tests were done with all 3 patches from this bug applied: 1) Add suggestion from OPAC, check copyright date is saved 2) Add suggestion in intranet, check copyright date is saved 3) Edit suggestion, check change is saved correctly 4) Check date filters work as expected, using single dates, date ranges and multiple filters at once: - Accepted on - Managed on - Suggested on 5) Check copyright date search works correctly. Passes all tests and QA script. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 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.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #31 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org