[Bug 9223] New: Multiple values of AdvancedSearchTypes in suggestions
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Bug ID: 9223 Summary: Multiple values of AdvancedSearchTypes in suggestions Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Acquisitions Assignee: koha-bugs@lists.koha-community.org Reporter: fridolyn.somers@biblibre.com Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can have mutliple values separated by a pipe. -- 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=9223 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 #1 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 13903 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13903&action=edit Proposed patch Proposed patch : C4::Koha::GetSupportList returns a list with all supports from itemtypes, LOC and CCODE. Test plan : ----------- - Set AdvancedSearchTypes with "itemtypes|loc|ccode" - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contain all itemtypes, LOC and CCODE descriptions - Create a suggestion with a document type from LOC or CCODE - Get to suggestions management - Organize suggestions by item type => your suggestion must appear in a tab with ccode description -- 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=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am not sure if this changes makes sense for the library user. Collections, locations and itemtypes are totally different things. Which label would we show? What is the user expected to choose/do here? Maybe it would be better to have a separate pref to define which value(s) are offered in the suggestion form and not reuse the settings for the advanced search. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |In Discussion -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #3 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to comment #2)
Maybe it would be better to have a separate pref to define which value(s) are offered in the suggestion form and not reuse the settings for the advanced search. I do agree. I'd like to have other opinions before rewriting patch.
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Fridolyn, thx for putting this in discussion - hope others share what they think about it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nengard@gmail.com See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=8036 --- Comment #5 from Nicole C. Engard <nengard@gmail.com> --- I think that these options should be on the staff client, but not the OPAC. It's too confusing to the actual user (I would think). Right now the form shows nothing ... http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8036 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- It's working correctly in 3.6.10, so I think we can't just remove the feature. I agree most of the time it's confusing, but I can imagine that the values might make sense for libraries that are using itemtypes and collection codes/locations in a special way. For example if your location is something like a department of your company. I checked, the element has an id - so it should be easy to hide with CSS: <li><label for="itemtype">Item Type:</label><select name="itemtype" id="itemtype"> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #7 from Nicole C. Engard <nengard@gmail.com> --- I wasn't clear - an item type makes sense to the user. And it should be there and it should work. The collections and shelving locations though make so sense to the OPAC user when making a suggestion. How does the patron know where you want to shelve the item? Nicole -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #8 from Nicole C. Engard <nengard@gmail.com> --- (In reply to comment #7) I meant - make no sense - not 'so'. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |8036 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Francois Charbonnier <francois.charbonnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |francois.charbonnier@biblib | |re.com --- Comment #9 from Francois Charbonnier <francois.charbonnier@biblibre.com> --- I guess someone used/is using the LOC or CCODE as itemtypes (maybe to display the information?) or thought it might suit everybody to get everything within one field... Anyway, if Koha 3.6 can returns itemtypes / LOC and CCODE values, the patch for this bug should make it works the same way here. Then, we should open up a "feature request" to change the behaviour. I agree we should separate the advanced search and the suggestion droplist setups to avoid confusion for the patron. If a library want to catalogue the location and the ccode at the suggestion level, there should be two new fields that would be display depending on a new syspref similar to AdvancedSearchTypes syspref. And maybe, we could be wilder and add not just one syspref but two sysprefs to distinguish opac and staff suggestion droplist set ups! :^) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13903|0 |1 is obsolete| | --- Comment #10 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 14427 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14427&action=edit Proposed patch (main) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #11 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 14428 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14428&action=edit Proposed patch (syspref add) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Needs Signoff --- Comment #12 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- I have rewritten patch. I introduce a new syspref to choose witch between item types, shelving location and collection code represents document physical support and will be used as 'document type' in suggestions. I changed 'item type' in 'document type' in suggestions management to avoid misunderstanding. It was already used in suggestion creation form. First patch is the code modifications, second is the syspref addition. See commit comment for test plan. This syspref may be used in other enhancements. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=9468 --- Comment #13 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Fridolyn, thx for working on this. I am not sure the point here is that the value in the pull down on the item form has to represent the physical document type. I could imagine a company library having offices/rooms as locations, using the pull down on the form to indicate where the book should be accessible later. Also if item types are used, they could say something like: short loan, reference, normal loan and for some libraries it would make sense to have people indicate that on the suggestion form. I liked the idea of having separate fields with the proper description, but I guess doing multiple is harder than having the library to choose one, because of the database structure. Maybe we could just have one field and change the label according to what is selected in the pref? Also it might be worth having a value "don't show", because that would avoid having another pref (see 9468) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #14 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Sorry for being annoying about this, it's just some thoughts. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #15 from Owen Leonard <oleonard@myacpl.org> --- (In reply to comment #7)
I wasn't clear - an item type makes sense to the user. And it should be there and it should work. The collections and shelving locations though make no sense to the OPAC user when making a suggestion. How does the patron know where you want to shelve the item?
Item type may or may not make sense to the user depending on how the library uses it. In my library item type would make no sense, but collection code would. I think this latest version of the patch works well, but I agree that label should change according to the selection. We shouldn't presume that the correct term ("item type," etc.) won't be understandable to the patron. Better to let the library decide to customize it somehow. I also like the idea of a "don't show" option. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #16 from Owen Leonard <oleonard@myacpl.org> --- Looking at this again, the other thing I worry about is the name of the preference and its description. The preference is tied directly to suggestions isn't it? But the name, "SupportsAuthorizedValues," doesn't imply a connection. The description, "(itemtype|ccode|loc) represents document physical support." doesn't make sense to me. Maybe something like, "SuggestionsUseValues," "Suggestions can be submitted using the following authorized values: (itemtype|ccode|loc)" -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Corinne BULAC <corinnebulac@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply CC| |corinnebulac@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #14427|0 |1 is obsolete| | --- Comment #17 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 20015 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20015&action=edit Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can contain several values separated by a pipe. This patch introduces a new syspref SupportsAuthorizedValues to select wich authorized values category that represents the physical support (itemtypes by default, loc or ccode). These authorized values will be used in suggestion management, known as 'document type'. (Note that database field is still 'itemtype'). Test plan : ----------- - Set SupportsAuthorizedValues with itemtypes or loc or ccode - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contains descriptions of selected authorized values - Save the suggestion - Get to intranet suggestions management - Organize suggestions by document type => your suggestion must appear in a tab with its document type description http://bugs.koha-community.org/show_bug.cgi?id=6473 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #14428|0 |1 is obsolete| | --- Comment #18 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 20016 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20016&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) http://bugs.koha-community.org/show_bug.cgi?id=6473 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #19 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 20017 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20017&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions This follow-up implements some things I think will improve on the previous patches: - Change the system preference name to "SuggestionsUseValues" - Change the description of the preference to "Suggestions can be submitted using the following authorized values:" - Move the new preference to the OPAC tab so that it is grouped with other suggestions-related preferences. - Add the option offer no itemtype/collection code chooser on the submission form. - Display the correct label on the submission form depending on the value of the SuggestionsUseValues preference. To test, apply this patch on top of the previous ones. Test submission of purchase suggestions through the OPAC and staff client with the SuggestionsUseValues preference set to all the various choices: item type, collection code, shelving location, and 'none.' The suggestion forms should look correct and work correctly. HOWEVER, neither the previous patches nor mine address the problem of the suggestions management page's "organize by" option, which is hard-coded to work with item types. Whether or not my patch is acceptable this issue will need to be addressed. http://bugs.koha-community.org/show_bug.cgi?id=6473 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff --- Comment #20 from Owen Leonard <oleonard@myacpl.org> --- I've attached rebased versions of the original patches along with mine. I'm setting back to "needs signoff" although I don't think this can go to QA without an additional followup for the issue I describe in comment 19. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #21 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- *** Bug 8036 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=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=10588 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #22 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- I'd like to discuss more on this bug. How can the syspref concerne only suggestions when it is used in C4::Koha::GetSupportList() ? This is the method doc : "Returns an array ref containing informations about Support (since itemtype is rather a circulation code when item-level-itypes is used)." This is the point, we need to have a syspref telling where to find the physical category of the record (itemtype or ccode). This code is used for suggestions (both intranet and opac) and should be used in other places like opac-topissues.pl filter (actually also broken if AdvancedSearchTypes is multivalued). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #23 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Fridolyn, I think this assumption is not quite right:
This is the point, we need to have a syspref telling where to find the physical >category of the record (itemtype or ccode).
There is not guarantee or reason that a library will use itemtype or ccode to define the physical category of a record. They can use it for whatever they like and from my experience they do use it in a lot of different ways. I think we schould just make it a pref that defines what to show on the suggestion form with a correct label: Location, (Circulation?) type, Collection. And use that for suggestons in staff and OPAC. Also it might make sense for a library to have different values used for suggestions and opac-topissues - so maybe we should not tie those together. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #24 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- (In reply to Katrin Fischer from comment #23) Hie Katrin, Ok, I understand. In this cas the code C4::Koha::GetSupportList() and C4::Koha::GetSupportName() must be moved into C4::Suggestion and renamed. It is this code who speaks about a "support" definition. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 I'm just a bot <gitbot@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply CC| |gitbot@bugs.koha-community. | |org --- Comment #25 from I'm just a bot <gitbot@bugs.koha-community.org> --- Applying: Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Using index info to reconstruct a base tree... M C4/Koha.pm M koha-tmpl/opac-tmpl/prog/en/modules/opac-suggestions.tt Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/opac-tmpl/prog/en/modules/opac-suggestions.tt Auto-merging C4/Koha.pm CONFLICT (content): Merge conflict in C4/Koha.pm Patch failed at 0001 Bug 9223: Multiple values of AdvancedSearchTypes in suggestions The copy of the patch that failed is found in: /home/christopher/git/koha/.git/rebase-apply/patch When you have resolved this problem, run "git am --continue". If you prefer to skip this patch, run "git am --skip" instead. To restore the original branch and stop patching, run "git am --abort". -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |In Discussion -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Joseph DeVore <joe_devore@mailhaven.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |joe_devore@mailhaven.com --- Comment #26 from Joseph DeVore <joe_devore@mailhaven.com> --- Hi I have noticed this issue as well.. My local Library (Dover, NH USA) just switched over to Koha I have to say its sooo much better then that ancient Dinosaur DYNIX... I chat with my Libraries tech lady almost everyday ^_^ so she directed me here... Its nice too see a more integrated solution to requesting new material... I(born in the year of the Dog, ie fault finder/(Loves Beta Testing ^_^) some basic formats that should be listed include: Books Books -- Large Print Audio Books DVDs Manga just to name a few... I suppose a specific list of material types could be hard-codded on a per library basis... or maybe a new section in the database that can have a small list generated per library basis... ps: just thought I would offer my thoughts loi ;D -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=11091 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Gaetan Boisson <gaetan.boisson@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |7115 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Marjorie Barry-Vila <marjorie.barry-vila@ccsr.qc.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |marjorie.barry-vila@ccsr.qc | |.ca -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #20015|0 |1 is obsolete| | Attachment #20016|0 |1 is obsolete| | Attachment #20017|0 |1 is obsolete| | --- Comment #27 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 35492 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35492&action=edit Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can contain several values separated by a pipe. This patch introduces a new syspref SupportsAuthorizedValues to select wich authorized values category that represents the physical support (itemtypes by default, loc or ccode). These authorized values will be used in suggestion management, known as 'document type'. (Note that database field is still 'itemtype'). Test plan : ----------- - Set SupportsAuthorizedValues with itemtypes or loc or ccode - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contains descriptions of selected authorized values - Save the suggestion - Get to intranet suggestions management - Organize suggestions by document type => your suggestion must appear in a tab with its document type description -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #28 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 35493 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35493&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #29 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 35494 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35494&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions This follow-up implements some things I think will improve on the previous patches: - Change the system preference name to "SuggestionsUseValues" - Change the description of the preference to "Suggestions can be submitted using the following authorized values:" - Move the new preference to the OPAC tab so that it is grouped with other suggestions-related preferences. - Add the option offer no itemtype/collection code chooser on the submission form. - Display the correct label on the submission form depending on the value of the SuggestionsUseValues preference. To test, apply this patch on top of the previous ones. Test submission of purchase suggestions through the OPAC and staff client with the SuggestionsUseValues preference set to all the various choices: item type, collection code, shelving location, and 'none.' The suggestion forms should look correct and work correctly. HOWEVER, neither the previous patches nor mine address the problem of the suggestions management page's "organize by" option, which is hard-coded to work with item types. Whether or not my patch is acceptable this issue will need to be addressed. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m --- Comment #30 from Jonathan Druart <jonathan.druart@biblibre.com> --- I would like to see this one moving forward. I rebased the 3 patches against master and adapted them to the bootstrap theme. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #31 from Joseph DeVore <joe_devore@mailhaven.com> --- Hi Jonathan! its nice to see some one finally working on this again =D good luck. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #32 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Joseph, this has the status 'needs signoff' now, which means it's waiting for a tester. Anyone can test and we have sandboxes, so you don't need a development environment to do it :) Maybe you'd like to have a try - it would really help getting this further: http://wiki.koha-community.org/wiki/Sandboxes -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #33 from Joseph DeVore <joe_devore@mailhaven.com> --- I took a quick look at this sandboxing.... seems a bit too much for me... that is the setup seem a bit complex so I don't really know what I would be doing... I will pass this on the tech lady at my local library, and see if she can do anything to test it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 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.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #35493|0 |1 is obsolete| | Attachment #35494|0 |1 is obsolete| | --- Comment #34 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 36178 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36178&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #35 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 36179 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36179&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #36 from Jonathan Druart <jonathan.druart@biblibre.com> --- Last patches fixed easy conflict on updatedatabase.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |olli-antti.kivilahti@jns.fi --- Comment #37 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- AFAIK: The suggestions.itemtype-column is not used anywhere other than displayed as-is in the suggestions.pl-view. I tried setting the itemtype-column to "ITYPE - CCODE - LOCATION" and it seems to work quite well. I don't know why anybody would want to expose their patrons to ILS-configuration technical parameters like that, but using this simple trick it should be possible to collect and store all three values. On a followup those three parameters can be separated to their distinctive columns. Also using only one ITEMTYPE or COLLECTIONCODE should be more than enough for a borrower to categorize his suggestion. I think the suggestions-mechanism is meant for ordinary library users and if further "resource" description accurancy is needed, then an acquisitions integration would be more suitable for the task using EDI or similar. I feel sadness when a new system preference is added. In this case "SupportsAuthorizedValues"||"SuggestionsUseValues". A better solution would be to use the first value in "AdvancedSearchTypes", eg "ccode|itemtypes|loc" would translate to using "ccode" as the selection displayed for the user, or simply displaying all those options and concatenating them to the suggestions.itemtype-column. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #38 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- Other than that, this feature seems to work well enough. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #35492|0 |1 is obsolete| | --- Comment #39 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- Created attachment 36702 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36702&action=edit Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can contain several values separated by a pipe. This patch introduces a new syspref SupportsAuthorizedValues to select wich authorized values category that represents the physical support (itemtypes by default, loc or ccode). These authorized values will be used in suggestion management, known as 'document type'. (Note that database field is still 'itemtype'). Test plan : ----------- - Set SupportsAuthorizedValues with itemtypes or loc or ccode - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contains descriptions of selected authorized values - Save the suggestion - Get to intranet suggestions management - Organize suggestions by document type => your suggestion must appear in a tab with its document type description Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #36178|0 |1 is obsolete| | --- Comment #40 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- Created attachment 36703 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36703&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #36179|0 |1 is obsolete| | --- Comment #41 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- Created attachment 36704 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36704&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions This follow-up implements some things I think will improve on the previous patches: - Change the system preference name to "SuggestionsUseValues" - Change the description of the preference to "Suggestions can be submitted using the following authorized values:" - Move the new preference to the OPAC tab so that it is grouped with other suggestions-related preferences. - Add the option offer no itemtype/collection code chooser on the submission form. - Display the correct label on the submission form depending on the value of the SuggestionsUseValues preference. To test, apply this patch on top of the previous ones. Test submission of purchase suggestions through the OPAC and staff client with the SuggestionsUseValues preference set to all the various choices: item type, collection code, shelving location, and 'none.' The suggestion forms should look correct and work correctly. HOWEVER, neither the previous patches nor mine address the problem of the suggestions management page's "organize by" option, which is hard-coded to work with item types. Whether or not my patch is acceptable this issue will need to be addressed. Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> Tested the TEST PLAN. With ccode, itype, none, loc. On both OPAC and SC. Making a code review, but it is hard during the hackfest. Morbid lack of any kind of unit test! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #42 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- Failed QA because no unit tests of any kind. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Joseph DeVore <joe_devore@mailhaven.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|joe_devore@mailhaven.com | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=12136 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #43 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 40011 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40011&action=edit Bug 9223 [Follow-up] add unit tests for GetSupportList and GetSupportName subroutine -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #36702|0 |1 is obsolete| | --- Comment #44 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 40012 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40012&action=edit Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can contain several values separated by a pipe. This patch introduces a new syspref SupportsAuthorizedValues to select wich authorized values category that represents the physical support (itemtypes by default, loc or ccode). These authorized values will be used in suggestion management, known as 'document type'. (Note that database field is still 'itemtype'). Test plan : ----------- - Set SupportsAuthorizedValues with itemtypes or loc or ccode - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contains descriptions of selected authorized values - Save the suggestion - Get to intranet suggestions management - Organize suggestions by document type => your suggestion must appear in a tab with its document type description Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #36703|0 |1 is obsolete| | --- Comment #45 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 40013 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40013&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #36704|0 |1 is obsolete| | --- Comment #46 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 40014 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40014&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions This follow-up implements some things I think will improve on the previous patches: - Change the system preference name to "SuggestionsUseValues" - Change the description of the preference to "Suggestions can be submitted using the following authorized values:" - Move the new preference to the OPAC tab so that it is grouped with other suggestions-related preferences. - Add the option offer no itemtype/collection code chooser on the submission form. - Display the correct label on the submission form depending on the value of the SuggestionsUseValues preference. To test, apply this patch on top of the previous ones. Test submission of purchase suggestions through the OPAC and staff client with the SuggestionsUseValues preference set to all the various choices: item type, collection code, shelving location, and 'none.' The suggestion forms should look correct and work correctly. HOWEVER, neither the previous patches nor mine address the problem of the suggestions management page's "organize by" option, which is hard-coded to work with item types. Whether or not my patch is acceptable this issue will need to be addressed. Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> Tested the TEST PLAN. With ccode, itype, none, loc. On both OPAC and SC. Making a code review, but it is hard during the hackfest. Morbid lack of any kind of unit test! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #47 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 40015 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40015&action=edit Bug 9223 [Follow-up] change the way syspref SuggestionsUseValues is added: Use the new worflow (atomic update) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40011|0 |1 is obsolete| | --- Comment #48 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 40016 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40016&action=edit Bug 9223 [Follow-up] add unit tests for GetSupportList and GetSupportName subroutine -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff CC| |alex.arnaud@biblibre.com --- Comment #49 from Alex Arnaud <alex.arnaud@biblibre.com> --- Hello, I have rebased patches and make syspref adding use the new workflow (atomic update). And i have created unit tests for GetSupportList and GetSupportName subroutine. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|fridolyn.somers@biblibre.co |alex.arnaud@biblibre.com |m | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@univ-lyon3.fr Status|Needs Signoff |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=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40012|0 |1 is obsolete| | --- Comment #50 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42287 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42287&action=edit Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can contain several values separated by a pipe. This patch introduces a new syspref SupportsAuthorizedValues to select wich authorized values category that represents the physical support (itemtypes by default, loc or ccode). These authorized values will be used in suggestion management, known as 'document type'. (Note that database field is still 'itemtype'). Test plan : ----------- - Set SupportsAuthorizedValues with itemtypes or loc or ccode - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contains descriptions of selected authorized values - Save the suggestion - Get to intranet suggestions management - Organize suggestions by document type => your suggestion must appear in a tab with its document type description Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40013|0 |1 is obsolete| | --- Comment #51 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42288 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42288&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40014|0 |1 is obsolete| | --- Comment #52 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42289 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42289&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions This follow-up implements some things I think will improve on the previous patches: - Change the system preference name to "SuggestionsUseValues" - Change the description of the preference to "Suggestions can be submitted using the following authorized values:" - Move the new preference to the OPAC tab so that it is grouped with other suggestions-related preferences. - Add the option offer no itemtype/collection code chooser on the submission form. - Display the correct label on the submission form depending on the value of the SuggestionsUseValues preference. To test, apply this patch on top of the previous ones. Test submission of purchase suggestions through the OPAC and staff client with the SuggestionsUseValues preference set to all the various choices: item type, collection code, shelving location, and 'none.' The suggestion forms should look correct and work correctly. HOWEVER, neither the previous patches nor mine address the problem of the suggestions management page's "organize by" option, which is hard-coded to work with item types. Whether or not my patch is acceptable this issue will need to be addressed. Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> Tested the TEST PLAN. With ccode, itype, none, loc. On both OPAC and SC. Making a code review, but it is hard during the hackfest. Morbid lack of any kind of unit test! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40015|0 |1 is obsolete| | --- Comment #53 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42290 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42290&action=edit Bug 9223 [Follow-up] change the way syspref SuggestionsUseValues is added: Use the new worflow (atomic update) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40016|0 |1 is obsolete| | --- Comment #54 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42291 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42291&action=edit Bug 9223 [Follow-up] add unit tests for GetSupportList and GetSupportName subroutine -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #55 from Alex Arnaud <alex.arnaud@biblibre.com> --- Patch rebased again -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42287|0 |1 is obsolete| | --- Comment #56 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42336 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42336&action=edit Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can contain several values separated by a pipe. This patch introduces a new syspref SupportsAuthorizedValues to select wich authorized values category that represents the physical support (itemtypes by default, loc or ccode). These authorized values will be used in suggestion management, known as 'document type'. (Note that database field is still 'itemtype'). Test plan : ----------- - Set SupportsAuthorizedValues with itemtypes or loc or ccode - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contains descriptions of selected authorized values - Save the suggestion - Get to intranet suggestions management - Organize suggestions by document type => your suggestion must appear in a tab with its document type description Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42288|0 |1 is obsolete| | --- Comment #57 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42337 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42337&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42289|0 |1 is obsolete| | --- Comment #58 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42338 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42338&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions This follow-up implements some things I think will improve on the previous patches: - Change the system preference name to "SuggestionsUseValues" - Change the description of the preference to "Suggestions can be submitted using the following authorized values:" - Move the new preference to the OPAC tab so that it is grouped with other suggestions-related preferences. - Add the option offer no itemtype/collection code chooser on the submission form. - Display the correct label on the submission form depending on the value of the SuggestionsUseValues preference. To test, apply this patch on top of the previous ones. Test submission of purchase suggestions through the OPAC and staff client with the SuggestionsUseValues preference set to all the various choices: item type, collection code, shelving location, and 'none.' The suggestion forms should look correct and work correctly. HOWEVER, neither the previous patches nor mine address the problem of the suggestions management page's "organize by" option, which is hard-coded to work with item types. Whether or not my patch is acceptable this issue will need to be addressed. Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> Tested the TEST PLAN. With ccode, itype, none, loc. On both OPAC and SC. Making a code review, but it is hard during the hackfest. Morbid lack of any kind of unit test! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42290|0 |1 is obsolete| | --- Comment #59 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42339 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42339&action=edit Bug 9223 [Follow-up] change the way syspref SuggestionsUseValues is added: Use the new worflow (atomic update) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42291|0 |1 is obsolete| | --- Comment #60 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 42340 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42340&action=edit Bug 9223 [Follow-up] add unit tests for GetSupportList and GetSupportName subroutine -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #61 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- I have tested the different values of the new syspref "suggestionsUseValues". In each case, in pro interface or in OPAC, the authorized values match with the choice in the syspref. And it's the description that is displayed in the dropdown. I tried to modify the suggestions in the pro interface and it's Ok too. If, "hide this option" is selected, none are in the form (tested in pro and opac too). There's just one thing that could be much better : code are displayed in the list of suggestions (OPAC and PRO) and it would be more useful to have the description instead. cf. cgi-bin/koha/suggestion/suggestion.pl and cgi-bin/koha/opac-suggestions.pl But I don't want to block this patch for that. Sonia -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #62 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Please remove tabs and fix the author in the last patch. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42340|0 |1 is obsolete| | --- Comment #63 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 44111 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44111&action=edit Bug 9223 [Follow-up] add unit tests for GetSupportList and GetSupportName subroutine -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #64 from Alex Arnaud <alex.arnaud@biblibre.com> --- Fixed last patch. A rebase against master is coming soon -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42336|0 |1 is obsolete| | --- Comment #65 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 44113 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44113&action=edit Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Suggestion form uses C4::Koha::GetSupportList to get a list of supports that can come from itemtypes or authoritized values LOC or CCODE. C4::Koha::GetSupportList uses AdvancedSearchTypes syspref like it has only one value. But this syspref can contain several values separated by a pipe. This patch introduces a new syspref SupportsAuthorizedValues to select wich authorized values category that represents the physical support (itemtypes by default, loc or ccode). These authorized values will be used in suggestion management, known as 'document type'. (Note that database field is still 'itemtype'). Test plan : ----------- - Set SupportsAuthorizedValues with itemtypes or loc or ccode - Get to suggestion form (OPAC or intranet) - Look at "Document type" combobox, it must contains descriptions of selected authorized values - Save the suggestion - Get to intranet suggestions management - Organize suggestions by document type => your suggestion must appear in a tab with its document type description Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42337|0 |1 is obsolete| | --- Comment #66 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 44114 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44114&action=edit Bug 9223: Add SupportsAuthorizedValues syspref (followup 1) Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42338|0 |1 is obsolete| | --- Comment #67 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 44115 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44115&action=edit Bug 9223 [Follow-up] Multiple values of AdvancedSearchTypes in suggestions This follow-up implements some things I think will improve on the previous patches: - Change the system preference name to "SuggestionsUseValues" - Change the description of the preference to "Suggestions can be submitted using the following authorized values:" - Move the new preference to the OPAC tab so that it is grouped with other suggestions-related preferences. - Add the option offer no itemtype/collection code chooser on the submission form. - Display the correct label on the submission form depending on the value of the SuggestionsUseValues preference. To test, apply this patch on top of the previous ones. Test submission of purchase suggestions through the OPAC and staff client with the SuggestionsUseValues preference set to all the various choices: item type, collection code, shelving location, and 'none.' The suggestion forms should look correct and work correctly. HOWEVER, neither the previous patches nor mine address the problem of the suggestions management page's "organize by" option, which is hard-coded to work with item types. Whether or not my patch is acceptable this issue will need to be addressed. Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> Tested the TEST PLAN. With ccode, itype, none, loc. On both OPAC and SC. Making a code review, but it is hard during the hackfest. Morbid lack of any kind of unit test! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42339|0 |1 is obsolete| | --- Comment #68 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 44116 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44116&action=edit Bug 9223 [Follow-up] change the way syspref SuggestionsUseValues is added: Use the new worflow (atomic update) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #44111|0 |1 is obsolete| | --- Comment #69 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 44117 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44117&action=edit Bug 9223 [Follow-up] add unit tests for GetSupportList and GetSupportName subroutine -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #70 from Alex Arnaud <alex.arnaud@biblibre.com> --- Created attachment 44118 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44118&action=edit Bug 9223 - Fix QA issues -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #71 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Sorry, but this patch no longer applies: Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 9223: Multiple values of AdvancedSearchTypes in suggestions Using index info to reconstruct a base tree... M C4/Koha.pm M koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt M koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt M opac/opac-suggestions.pl M suggestion/suggestion.pl Falling back to patching base and 3-way merge... Auto-merging suggestion/suggestion.pl CONFLICT (content): Merge conflict in suggestion/suggestion.pl Auto-merging opac/opac-suggestions.pl CONFLICT (content): Merge conflict in opac/opac-suggestions.pl Auto-merging koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt CONFLICT (content): Merge conflict in koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt Auto-merging C4/Koha.pm Failed to merge in the changes. Patch failed at 0001 Bug 9223: Multiple values of AdvancedSearchTypes in suggestions The copy of the patch that failed is found in: /home/katrin/kohaclone/.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-9223-Multiple-values-of-AdvancedSearchTypes-in-GRz_gw.patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |In Discussion --- Comment #72 from Alex Arnaud <alex.arnaud@biblibre.com> --- Is this Bug still relevant since we use SUGGEST_FORMAT authorized values as document type ? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9223 --- Comment #73 from Alex Arnaud <alex.arnaud@biblibre.com> --- see bug 9468 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org