[Bug 10519] New: Suggestions: 'Ordered' shows up in tab 'Unknown'
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Bug ID: 10519 Summary: Suggestions: 'Ordered' shows up in tab 'Unknown' Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: katrin.fischer@bsz-bw.de Created attachment 19300 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19300&action=edit screenshot When organising tabs by status the 'ordered' suggestion shows up in a tab labelled 'Unknown'. I think there might be something missing in the templates here. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #1 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Actually, there is a lot of weirdness in the tab display going on there for me, there might be more wrong about it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |9307 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 19327 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19327&action=edit Bug 10519: Fix display of tab for ordered suggestions Adds 2 more valid status to a check to make the description on the tab for 'Ordered' and 'Available' suggestions show up correctly. To test: - Make 2 suggestions. - Accept both and order them in acquisitions. - Receive one of them. - Use the 'Organize by' filter with 'Status' - Verify status descriptions in the tabs show up correctly. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- While my patch fixes the display of the status tabs, I just realized the problem is a lot bigger. We can not only organize by status, but also by library, item type, suggested byand accepted by. I think this needs more work: Maybe an additional check on the selected 'organize by' filter? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Finding more problems: return GetAuthorisedValueByCode('SUGGEST_STATUS', $criteriumvalue ) || "Unknown"; As the template doesn't check for this value, it's untranslatable. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |katrin.fischer@bsz-bw.de Summary|Suggestions: 'Ordered' |Suggestions: 'Organize by' |shows up in tab 'Unknown' |and correct display of tab | |descriptions broken --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Changing the bug title to reflect that this has grown into a bigger problem, I am working on a patch. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #19327|0 |1 is obsolete| | --- Comment #6 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 19340 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19340&action=edit Bug 10519: Fix display of tab for ordered suggestions Adds 2 more valid status to a check to make the description on the tab for 'Ordered' and 'Available' suggestions show up correctly. To test: - Make 2 suggestions. - Accept both and order them in acquisitions. - Receive one of them. - Use the 'Organize by' filter with 'Status' - Verify status descriptions in the tabs show up correctly. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> ---
Finding more problems: return GetAuthorisedValueByCode('SUGGEST_STATUS', $criteriumvalue ) || "Unknown"; As the template doesn't check for this value, it's untranslatable.
This isn't necessarily as bad as it may seem. The string "Unknown" will only display during an error condition. The one and only error condition I can think of where this would display would be if a suggestion was set to an authorised value, and the authorised value was then deleted. However, it may be better to display the authorised value instead of "Unkown". I will submit a followup for that. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #8 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 19341 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19341&action=edit Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken - Change "Unknown" to authorised value -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #19341|0 |1 is obsolete| | --- Comment #9 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 19355 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19355&action=edit [SIGNED OFF] Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken - Change "Unknown" to authorised value Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Small patch, dealing with the display when no description for the authorized value is found. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 19356 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19356&action=edit Bug 10519: Suggestions: 'Organize by' and correct display of tab descriptions broken The tabbed display in suggestions offers different options to organize the tabs. The descriptions on the tabs and some of the search options were not working correctly. To test: - Add several suggestions to your installation, make sure you have: - suggestions from various libraries and at least one for 'Any library' that has been created from the intranet - suggestions from different users - suggestions with different status - suggestions with different selected itypes Test all the 'organize by' options, make sure that the tabs have correct descriptions. - Add 1 or 2 custom status to SUGGEST_STATUS authorized value. - Verify display is still correct and your new status are displayed. Test setting the library option in the acquisition section of the filters keeps "Any" when you select it, so you can make use of the 'organize by libraries'. Before this patch the pull down would always jump back to your own branch. Note: Patch best tested in combination with bug 4907. Conflicts: suggestion/suggestion.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- The last patch in combination with bug 4907 seems to make things work better, but please test thoroughly. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #19356|0 |1 is obsolete| | --- Comment #12 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 19364 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19364&action=edit Bug 10519: Suggestions: 'Organize by' and correct display of tab descriptions broken The tabbed display in suggestions offers different options to organize the tabs. The descriptions on the tabs and some of the search options were not working correctly. To test: - Add several suggestions to your installation, make sure you have: - suggestions from various libraries and at least one for 'Any library' that has been created from the intranet - suggestions from different users - suggestions with different status - suggestions with different selected itypes Test all the 'organize by' options, make sure that the tabs have correct descriptions. - Add 1 or 2 custom status to SUGGEST_STATUS authorized value. - Verify display is still correct and your new status are displayed. Test setting the library option in the acquisition section of the filters keeps "Any" when you select it, so you can make use of the 'organize by libraries'. Before this patch the pull down would always jump back to your own branch. Note: Patch best tested in combination with bug 4907. Conflicts: suggestion/suggestion.pl Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #13 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Kyle, I still feel like something is not quite right with the way we handle the library pull down on the form in combination with different search options. Did it work correctly for you? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |m.de.rooy@rijksmuseum.nl --- Comment #14 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Katrin, If I add suggestions from two libraries (and two users), and I leave them in pending state, and I organize by Library, there is only one tab (the first library name) and I see one suggestion in this list from the second library. I had a glance through the code, but it is hard to see [quickly] where this comes from. Could you please check? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #15 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Thx for taking a look Marcel - I will try to reproduce. Especially the Library filter is a bit tricky. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pasi.kallinen@pttk.fi --- Comment #16 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- *** Bug 11667 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nengard@gmail.com --- Comment #17 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- *** Bug 11609 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cbrannon@cdalibrary.org --- Comment #18 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- *** Bug 12154 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #19 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I hope to get back to this after feature freeze. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #20 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- ... but won't be unhappy if someone else adopts this bug :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Jacek Ablewicz <abl@biblos.pk.edu.pl> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - low |P3 CC| |abl@biblos.pk.edu.pl See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=12257 Severity|normal |major -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #21 from Jacek Ablewicz <abl@biblos.pk.edu.pl> --- (In reply to Katrin Fischer from comment #20)
... but won't be unhappy if someone else adopts this bug :)
I'm considering it. Warily :). What exactly is wrong with it? I mean, while it does not apply at the moment ("sha1 information is lacking or useless" blah blah), this part alone: my ($criteriumvalue,$displayby)=@_; - unless ( grep { /$criteriumvalue/ } qw(ASKED ACCEPTED REJECTED CHECKED) ) { - return GetAuthorisedValueByCode('SUGGEST_STATUS', $criteriumvalue ) || "Unknown"; + if ($displayby =~ /status/i) { + if ( grep { /$criteriumvalue/ } qw(ASKED ACCEPTED REJECTED CHECKED ORDERED AVAILABLE) ) { + return ($criteriumvalue eq 'ASKED'?"Pending":ucfirst(lc( $criteriumvalue))); + } else { + return GetAuthorisedValueByCode('SUGGEST_STATUS', $criteriumvalue ) || $criteriumvalue; + } } - return ($criteriumvalue eq 'ASKED'?"Pending":ucfirst(lc( $criteriumvalue))) if ($displayby =~/status/i); return (GetBranchName($criteriumvalue)) if ($displayby =~/branchcode/); is pretty perfect, IMO. Works like a charm, fixes all those ugly "Unknown"s introduced by commit 490688ea534addc3028ceac6b9a8d3f6816359d2. Seems to be good for bug 12257 as well (while there is a working patch set in 12257, it only deals with "Unknown" person names). So, what's a catch? ;) BTW, something like /^$criteriumvalue$/ in grep regexp might be a bit better perhaps - in case someone puts e.g. 'ED' or 'CHECK' as authorized value code in SUGGEST_STATUS (not really important). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #22 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I think most of the patch is correct, but I was a bit worried abotu the library tabs when I submitted it and now think that leaving out that bit of code and concentrate on fixing the status and user names etc. would be good. The library tab has a lot of problems really, when you start to think about independent branches and the like... but that's probably better done on another bug. I'd like to do a new version of the patch and include your suggestion - or you could do it and I could try to sign off? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #23 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I hope that made sense... it's way too hot here today for me :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Jacek Ablewicz <ablewicz@gmail.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=10519 Jacek Ablewicz <ablewicz@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #19340|0 |1 is obsolete| | Attachment #19355|0 |1 is obsolete| | Attachment #19364|0 |1 is obsolete| | --- Comment #24 from Jacek Ablewicz <ablewicz@gmail.com> --- Created attachment 28763 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28763&action=edit Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken The tabbed display in suggestions offers different options to organize the tabs. The descriptions on the tabs and some of the search options were not working correctly, displaying as "Unknown". To test: - Add several suggestions to your installation, make sure you have: - suggestions from different users - suggestions managed by different users - suggestions with different statuses - suggestions with different selected item types Test all the 'organize by' options (except "Organize by: Library" - see note below), make sure that the tabs and search options have correct descriptions and do no longer display as "Unknown". - Add 1 or 2 custom status to SUGGEST_STATUS authorized value. - Verify display is still correct and your new status are displayed. Note: "Organize by: Library" option is currently severely broken (and not easily fixable, especially for 'IndependentBranches' enabled). But this turns out to be a separate issue, with a different underlaying causes, and it's outside the scope of this patch. This should be dealt with later, in it's own bug report. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Paola Rossi <paola.rossi@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |paola.rossi@cineca.it --- Comment #25 from Paola Rossi <paola.rossi@cineca.it> --- I've applied against master 3.17.00.007 I've not be able to test the added custom statuses. After having created 1 or 2 custom status (XX e YY) to SUGGEST_STATUS category, I couldn't attach these new statuses to any suggestion. Editing a suggestion, the Status' dropdown list provided is: No Status, Pending, Accepted, Checked, Rejected. "Mark selected as" lets me choose a status among: pending, accepted, checked, rejected. There were no authorized values defined for SUGGEST_STATUS category before I added the two authorized values XX e YY. So I pass the patch to "Failed QA" status. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #26 from Christopher Brannon <cbrannon@cdalibrary.org> --- I have to agree that this patch fails. I too am testing on 3.17 in a sandbox, and when applied, I see no changes on the tabs. The tabs still read as 'Unknown'. Christopher -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #27 from Jacek Ablewicz <abl@biblos.pk.edu.pl> --- (In reply to Christopher Brannon from comment #26)
I have to agree that this patch fails. I too am testing on 3.17 in a sandbox, and when applied, I see no changes on the tabs. The tabs still read as 'Unknown'.
It's kind of weird, because with this patch applied, there should no longer be an "Unknown" nowhere in the script nor in the template (another possibility is that "Unknown" is comming e.g. from the database, but that's not very likely IMO). Maybe it does not (correctly) apply in sandbox, for some not yet determined reason.. I'll try to replicate this, can you provide a little more details - which sandbox did you tested it with, and with what sandbox settings? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #28 from Jacek Ablewicz <abl@biblos.pk.edu.pl> --- (In reply to Paola Rossi from comment #25)
I've not be able to test the added custom statuses.
After having created 1 or 2 custom status (XX e YY) to SUGGEST_STATUS category, I couldn't attach these new statuses to any suggestion.
Indeed, additional suggestion statuses do not currently work properly (since Bug 9261 push, unless I'm very much mistaken). Follow-up patch on the way. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Jacek Ablewicz <ablewicz@gmail.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=10519 --- Comment #29 from Jacek Ablewicz <ablewicz@gmail.com> --- Created attachment 28850 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28850&action=edit Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken Follow-up to fix additional suggestion statuses, which are not working properly due to 416: borrowernumber => $input->param('borrowernumber'), being evaluated in list context inside $template->param() call arguments. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #30 from Christopher Brannon <cbrannon@cdalibrary.org> --- (In reply to Jacek Ablewicz from comment #27)
(In reply to Christopher Brannon from comment #26)
I have to agree that this patch fails. I too am testing on 3.17 in a sandbox, and when applied, I see no changes on the tabs. The tabs still read as 'Unknown'.
It's kind of weird, because with this patch applied, there should no longer be an "Unknown" nowhere in the script nor in the template (another possibility is that "Unknown" is comming e.g. from the database, but that's not very likely IMO).
Maybe it does not (correctly) apply in sandbox, for some not yet determined reason.. I'll try to replicate this, can you provide a little more details - which sandbox did you tested it with, and with what sandbox settings?
I take it back. I tried another sandbox and it is working. I am seeing difference between sandbox 7 and sandbox 8. I think 8 is having some issues. I'll have to keep an eye on it. The labels look great. I agree, the organize by library selection is defunct. I get absolutely nothing. Also, I see that if you are using something like Acquisition Information filters, and set to Any, Any, if you make a change to a status, it clears these settings and you have to start all over. It puts you back to the default view and the current library only. May have to file a separate bug for that issue. The labels are much better. Waiting for the update for the custom statuses. Christopher -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |sandboxes@biblibre.com --- Comment #31 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Patch tested with a sandbox, by Christopher Brannon <cbrannon@cdalibrary.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28763|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28850|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #32 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 28851 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28851&action=edit Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken The tabbed display in suggestions offers different options to organize the tabs. The descriptions on the tabs and some of the search options were not working correctly, displaying as "Unknown". To test: - Add several suggestions to your installation, make sure you have: - suggestions from different users - suggestions managed by different users - suggestions with different statuses - suggestions with different selected item types Test all the 'organize by' options (except "Organize by: Library" - see note below), make sure that the tabs and search options have correct descriptions and do no longer display as "Unknown". - Add 1 or 2 custom status to SUGGEST_STATUS authorized value. - Verify display is still correct and your new status are displayed. Note: "Organize by: Library" option is currently severely broken (and not easily fixable, especially for 'IndependentBranches' enabled). But this turns out to be a separate issue, with a different underlaying causes, and it's outside the scope of this patch. This should be dealt with later, in it's own bug report. Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #33 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 28852 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=28852&action=edit Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken Follow-up to fix additional suggestion statuses, which are not working properly due to 416: borrowernumber => $input->param('borrowernumber'), being evaluated in list context inside $template->param() call arguments. Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #34 from Christopher Brannon <cbrannon@cdalibrary.org> --- Works as test plan outlines. Suggestions still need bug fixes for: Library Sort Maintaining filtering after status is changed. Christopher -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |frederic@tamil.fr --- Comment #35 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- *** Bug 12433 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #36 from Paola Rossi <paola.rossi@cineca.it> --- 1) Special case to test for 10519: I've applied 12257 against master 3.17.00.008. As you certainly know, an anonymous purchase suggestion can be produced on OPAC changing "AnonSuggestions" pref to "Allow" value, and the "AnonymousPatron" pref to an "X" value (an existent patron's borrowernumber). As a patron, the AnonymousPatron "X" should be linked to a library. I've created a new patron, and I've linked him to Centerville, the same library of the test-user I log in on intranet. Then I've created an anonymous purchase suggestion on OPAC. In Intranet ACQ, the AnonymousPatron "X" should be listed among the other users in the "Suggested by" dropdown list of the "Suggestion information" filter: OK, he's listed there. In ACQ Suggestions, I've set "Centerville" in "Acquisition information" filter "For", and I've asked for all suggestions, f.i. organized by status. In DB the anonymous suggestion has "ASKED" status, and is not linked to a library (its branchcode is NULL). No anonymous suggestion is listed among the pending ones: KO. In intranet, I've set "Any" in "Acquisition information" filter "For", and I've asked for all suggestions, f.i. organized by status. This time the anonymous suggestion has been listed among the pending ones: OK. So the problem is about the library, not about the status of the suggestion. About KO: IMO on OPAC the koha/opac-suggestions.pl should set the AnonymousPatron's library to the anonymous suggestion branchcode, in case the APatron has one. So there could be a bug there. Otherwise, I invite you to pass this 10519 patch to "Failed QA" status. 2) Note. When "AnonSuggestions" pref is set to "Allow" value, koha lets the "AnonymousPatron" pref be kept to "0" value (non-existent patron), its default value. But, if in koha every patron should be linked to a library, the "AnonymousPatron" should be linked too. Setting to '0' the "AnonymousPatron" pref can violate a rule, and so it can give some problems. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|katrin.fischer@bsz-bw.de |abl@biblos.pk.edu.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #37 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Will finish this QA soon. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 M. de Rooy <m.de.rooy@rijksmuseum.nl> 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=10519 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #28851|0 |1 is obsolete| | Attachment #28852|0 |1 is obsolete| | --- Comment #38 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 29598 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29598&action=edit Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken The tabbed display in suggestions offers different options to organize the tabs. The descriptions on the tabs and some of the search options were not working correctly, displaying as "Unknown". To test: - Add several suggestions to your installation, make sure you have: - suggestions from different users - suggestions managed by different users - suggestions with different statuses - suggestions with different selected item types Test all the 'organize by' options (except "Organize by: Library" - see note below), make sure that the tabs and search options have correct descriptions and do no longer display as "Unknown". - Add 1 or 2 custom status to SUGGEST_STATUS authorized value. - Verify display is still correct and your new status are displayed. Note: "Organize by: Library" option is currently severely broken (and not easily fixable, especially for 'IndependentBranches' enabled). But this turns out to be a separate issue, with a different underlaying causes, and it's outside the scope of this patch. This should be dealt with later, in it's own bug report. Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #39 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 29599 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29599&action=edit Bug 10519 - Suggestions: 'Organize by' and correct display of tab descriptions broken Follow-up to fix additional suggestion statuses, which are not working properly due to 416: borrowernumber => $input->param('borrowernumber'), being evaluated in list context inside $template->param() call arguments. Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #40 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- QA Comment: Improves display of tab headings (especially Ordered). Also resolves display of additional suggestion statuses in combo box for Mark selected as. Code looks good to me. No complaints from qa tools. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #41 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 29686 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29686&action=edit Bug 10519: (followup) unit tests leave problematic cruft On testing I've found that t/db_dependent/Suggestions.t leaves cruft on the DB. Small followup for that. Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |tomascohen@gmail.com --- Comment #42 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patches pushed to master. Thanks Jacek! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 --- Comment #43 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- *** Bug 12257 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_3_16_4_candidate -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |gmcharlt@gmail.com --- Comment #44 from Galen Charlton <gmcharlt@gmail.com> --- Pushed to 3.16.x for inclusion in 3.16.4. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolyn.somers@biblibre.co | |m --- Comment #45 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Pushed to 3.14.x will be in 3.14.13 Second commit : c07a3d0 Bug 10519: (followup) Fix context for evalyating $borrowernumber Is not needed, $borrowernumber does not existe on this line -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Koha Team Rennes 2 <koha@listes.univ-rennes2.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@listes.univ-rennes2.fr Version|master |3.18 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10519 Koha Team Rennes 2 <koha@listes.univ-rennes2.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|3.18 |master -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org