[Bug 10865] New: Don't show list permissions when adding public lists/sharing lists is not allowed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Bug ID: 10865 Summary: Don't show list permissions when adding public lists/sharing lists is not allowed Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: katrin.fischer@bsz-bw.de QA Contact: testopia@bugs.koha-community.org Created attachment 20955 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20955&action=edit screenshot We can now prevent patrons from adding public lists, but the interface in the OPAC still shows the pulldown with only 'Private' in that case and also permissions that don't make sense as long as user's can't share lists or work on public lists created from the OPAC together. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nengard@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff 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=10865 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 25056 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25056&action=edit Bug 10865 - Don't show list permissions when adding public lists/sharing lists is not allowed If patron creation of public lists is disallowed by the OpacAllowPublicListCreation system preference the "category" option should be hidden altogether instead of showing a <select> with "private" as the only option. This patch hides category and permissions controls when OpacAllowPublicListCreation is set to "don't allow." To test you must have the virtualshelves system preference enabled. Apply the patch and log into the OPAC. Test: - With OpacAllowPublicListCreation enabled, create a new list. You should see options for setting category and permissions. Saving the new list should complete correctly and save the right settings. - With OpacAllowPublicListCreation enabled, edit an existing list. You should see the same options and saving your changes should work correctly. - With OpacAllowPublicListCreation disabled, create a new list. You should only see fields for title and sort. Saving this list should complete correctly and save the right settings. - With OpacAllowPublicListCreation disabled, edit an existing list. You should be able to edit only title and sort settings. Saving your changes should work correctly. Repeat your tests for both prog and bootstrap themes. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Thx Owen! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |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=10865 Nicole C. Engard <nengard@gmail.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=10865 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25056|0 |1 is obsolete| | --- Comment #3 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 25058 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25058&action=edit [SIGNED-OFF] Bug 10865 - Don't show list permissions when adding public lists/sharing lists is not allowed If patron creation of public lists is disallowed by the OpacAllowPublicListCreation system preference the "category" option should be hidden altogether instead of showing a <select> with "private" as the only option. This patch hides category and permissions controls when OpacAllowPublicListCreation is set to "don't allow." To test you must have the virtualshelves system preference enabled. Apply the patch and log into the OPAC. Test: - With OpacAllowPublicListCreation enabled, create a new list. You should see options for setting category and permissions. Saving the new list should complete correctly and save the right settings. - With OpacAllowPublicListCreation enabled, edit an existing list. You should see the same options and saving your changes should work correctly. - With OpacAllowPublicListCreation disabled, create a new list. You should only see fields for title and sort. Saving this list should complete correctly and save the right settings. - With OpacAllowPublicListCreation disabled, edit an existing list. You should be able to edit only title and sort settings. Saving your changes should work correctly. Repeat your tests for both prog and bootstrap themes. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> tested on all three themes and works as expected -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #4 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Saving a public list on opac now results in setting it to Private. Failed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #5 from Owen Leonard <oleonard@myacpl.org> --- (In reply to M. de Rooy from comment #4)
Saving a public list on opac now results in setting it to Private.
I can't reproduce this error. Can you tell me the steps you're following? Does it only happen in one theme? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #6 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Tested in bootstrap. Enable pref, create new public list in opac. Disable pref. Edit same list in opac and Save. Public list is now a private list.. (This is new behavior; tested it too without the patch.) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #7 from Owen Leonard <oleonard@myacpl.org> --- (In reply to M. de Rooy from comment #6)
Enable pref, create new public list in opac. Disable pref. Edit same list in opac and Save.
I see the problem, but this presents an interesting question... If you have changed your library policy to disallow creation of public lists should it be possible for the general public to keep a list public? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Owen Leonard <oleonard@myacpl.org> 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=10865 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25058|0 |1 is obsolete| | --- Comment #8 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 25137 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25137&action=edit Bug 10865 [Revised] Don't show list permissions when adding public lists/sharing lists is not allowed If patron creation of public lists is disallowed by the OpacAllowPublicListCreation system preference the "category" option should be hidden altogether instead of showing a <select> with "private" as the only option. This patch hides category and permissions controls when OpacAllowPublicListCreation is set to "don't allow." To test you must have the virtualshelves system preference enabled. Apply the patch and log into the OPAC. Test: - With OpacAllowPublicListCreation enabled, create a new list. You should see options for setting category and permissions. Saving the new list should complete correctly and save the right settings. - With OpacAllowPublicListCreation enabled, edit an existing list. You should see the same options and saving your changes should work correctly. - With OpacAllowPublicListCreation disabled, create a new list. You should only see fields for title and sort. Saving this list should complete correctly and save the right settings. - With OpacAllowPublicListCreation disabled, edit an existing list. You should be able to edit only title and sort settings. Saving your changes should work correctly. Repeat your tests for both prog and bootstrap themes. Revision: Existing public lists can be edited and retain their public status even if OpacAllowPublicListCreation has since been disabled. This preserves the behavior previous to this patch. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Owen Leonard <oleonard@myacpl.org> 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=10865 --- Comment #9 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Owen Leonard from comment #7)
(In reply to M. de Rooy from comment #6)
Enable pref, create new public list in opac. Disable pref. Edit same list in opac and Save.
I see the problem, but this presents an interesting question... If you have changed your library policy to disallow creation of public lists should it be possible for the general public to keep a list public?
Good point. If you decide to disallow creation, you should probably also think at that moment about what you are doing with the lists made. (In the ideal situation, changing the pref should trigger such an action?) I think it should not depend on if a user edits his public list sometime after that.. So on the balance, I would stick to keeping it in the same category. (I understand that your patch does that now. I will check later.) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #10 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Checking this patch, there is still another small issue: If the preference is disabled, and I view Public lists, there still is a button New list. This will create a new private list. This is somewhat confusing. Could we not better remove the button New list here? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #11 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to M. de Rooy from comment #10)
Checking this patch, there is still another small issue: If the preference is disabled, and I view Public lists, there still is a button New list. This will create a new private list. This is somewhat confusing. Could we not better remove the button New list here?
Since it always defaults to Private list (also when I click New list coming from Public lists), I would just leave it that way on second thought. Will sign off now. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> 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=10865 --- Comment #12 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 25168 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25168&action=edit Bug 10865 [Revised] Don't show list permissions when adding public lists/sharing lists is not allowed If patron creation of public lists is disallowed by the OpacAllowPublicListCreation system preference the "category" option should be hidden altogether instead of showing a <select> with "private" as the only option. This patch hides category and permissions controls when OpacAllowPublicListCreation is set to "don't allow." To test you must have the virtualshelves system preference enabled. Apply the patch and log into the OPAC. Test: - With OpacAllowPublicListCreation enabled, create a new list. You should see options for setting category and permissions. Saving the new list should complete correctly and save the right settings. - With OpacAllowPublicListCreation enabled, edit an existing list. You should see the same options and saving your changes should work correctly. - With OpacAllowPublicListCreation disabled, create a new list. You should only see fields for title and sort. Saving this list should complete correctly and save the right settings. - With OpacAllowPublicListCreation disabled, edit an existing list. You should be able to edit only title and sort settings. Saving your changes should work correctly. Repeat your tests for both prog and bootstrap themes. Revision: Existing public lists can be edited and retain their public status even if OpacAllowPublicListCreation has since been disabled. This preserves the behavior previous to this patch. 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=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25137|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=10865 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=10865 Jonathan Druart <jonathan.druart@biblibre.com> 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=10865 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25168|0 |1 is obsolete| | --- Comment #13 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 25202 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25202&action=edit Bug 10865 [Revised] Don't show list permissions when adding public lists/sharing lists is not allowed If patron creation of public lists is disallowed by the OpacAllowPublicListCreation system preference the "category" option should be hidden altogether instead of showing a <select> with "private" as the only option. This patch hides category and permissions controls when OpacAllowPublicListCreation is set to "don't allow." To test you must have the virtualshelves system preference enabled. Apply the patch and log into the OPAC. Test: - With OpacAllowPublicListCreation enabled, create a new list. You should see options for setting category and permissions. Saving the new list should complete correctly and save the right settings. - With OpacAllowPublicListCreation enabled, edit an existing list. You should see the same options and saving your changes should work correctly. - With OpacAllowPublicListCreation disabled, create a new list. You should only see fields for title and sort. Saving this list should complete correctly and save the right settings. - With OpacAllowPublicListCreation disabled, edit an existing list. You should be able to edit only title and sort settings. Saving your changes should work correctly. Repeat your tests for both prog and bootstrap themes. Revision: Existing public lists can be edited and retain their public status even if OpacAllowPublicListCreation has since been disabled. This preserves the behavior previous to this patch. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> This patch fixes a big ergonomic issue. Note: to me, the "New list" action should be outside the tabs. It is confusing to have a "new list" into the public lists tab when it is not possible to create new public lists. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #14 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 25203 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25203&action=edit Bug 10865: (follow-up) Move a li tag to avoid overlapping tags With this patch, a li tag is empty (only using the bootstrap theme) if the pref is disabled. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |In Discussion CC| |gmcharlt@gmail.com --- Comment #15 from Galen Charlton <gmcharlt@gmail.com> ---
(In reply to Owen Leonard from comment #7)
(In reply to M. de Rooy from comment #6)
Enable pref, create new public list in opac. Disable pref. Edit same list in opac and Save.
I see the problem, but this presents an interesting question... If you have changed your library policy to disallow creation of public lists should it be possible for the general public to keep a list public?
Good point. If you decide to disallow creation, you should probably also think at that moment about what you are doing with the lists made. (In the ideal situation, changing the pref should trigger such an action?) I think it should not depend on if a user edits his public list sometime after that.. So on the balance, I would stick to keeping it in the same category. (I understand that your patch does that now. I will check later.)
I found another edge case with the current patch: - patron creates public list - OpacAllowPublicListCreation is turned off - patron has no way to change the list to be private I'm about to attach a follow-up that fills that gap, with the follow-up, a patron will be able to make there list private. Per a discussion on #koha, I'm leaving it to Owen to style the warning message that is part of the follow-up. Setting to in discussion pending review of my follow-up. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #16 from Galen Charlton <gmcharlt@gmail.com> --- Created attachment 25552 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25552&action=edit bug 18065: (follow-up) allow patrons to make their public lists private when OpacAllowPublicListCreation is off This patch ensures that patrons continue to have the ability to make their public lists private for any public lists they control that were created before the library turned the OpacAllowPublicListCreation system preference off. To test: [1] Ensure OpacAllowPublicListCreation is on. [2] As a patron, create a public list in the OPAC. Also, create a private list. [3] Turn OpacAllowPublicListCreation off. [4] Back in the OPAC, verify that the public list can be edited and that there are drop-downs for category and permissions. Also verify that there is a warning that the patron cannot change it back if they convert a public list to private. [5] Edit the private list created in step 2. Verify that the category and permissions drop-downs are not displayed. Signed-off-by: Galen Charlton <gmc@esilibrary.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #17 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 25621 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25621&action=edit bug 10865: (follow-up) allow patrons to make their public lists private when OpacAllowPublicListCreation is off This patch ensures that patrons continue to have the ability to make their public lists private for any public lists they control that were created before the library turned the OpacAllowPublicListCreation system preference off. To test: [1] Ensure OpacAllowPublicListCreation is on. [2] As a patron, create a public list in the OPAC. Also, create a private list. [3] Turn OpacAllowPublicListCreation off. [4] Back in the OPAC, verify that the public list can be edited and that there are drop-downs for category and permissions. Also verify that there is a warning that the patron cannot change it back if they convert a public list to private. [5] Edit the private list created in step 2. Verify that the category and permissions drop-downs are not displayed. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Owen Leonard <oleonard@myacpl.org> 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=10865 --- Comment #18 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 25623 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25623&action=edit Bug 10865 [Follow-up] Add style to warning This patch adds a new "hint" class for displaying information relating to a form field. On the list edit screen the hint also has an alert class to highlight it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25552|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=10865 --- Comment #19 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Owen Leonard from comment #18)
This patch adds a new "hint" class for displaying information relating to a form field. On the list edit screen the hint also has an alert class to highlight it.
Owen: What is the exact 'procedure' when making changes now to css file or less file? Who compiles less to css? You did probably now.. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #20 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- The text on the page now on bootstrap is so long actually that it wraps to the second line showing up not very nice. This pertains to the existing permissions line too.. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25621|0 |1 is obsolete| | --- Comment #21 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 25655 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25655&action=edit bug 10865: (follow-up) allow patrons to make their public lists private when OpacAllowPublicListCreation is off This patch ensures that patrons continue to have the ability to make their public lists private for any public lists they control that were created before the library turned the OpacAllowPublicListCreation system preference off. To test: [1] Ensure OpacAllowPublicListCreation is on. [2] As a patron, create a public list in the OPAC. Also, create a private list. [3] Turn OpacAllowPublicListCreation off. [4] Back in the OPAC, verify that the public list can be edited and that there are drop-downs for category and permissions. Also verify that there is a warning that the patron cannot change it back if they convert a public list to private. [5] Edit the private list created in step 2. Verify that the category and permissions drop-downs are not displayed. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> 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=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25623|0 |1 is obsolete| | --- Comment #22 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 25656 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25656&action=edit Bug 10865 [Follow-up] Add style to warning This patch adds a new "hint" class for displaying information relating to a form field. On the list edit screen the hint also has an alert class to highlight it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #23 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- Galen: I agree with your patch. But note that imo the design still needs some attention. But since we cannot argue about tastes too much :), anyone is free to get this to Signed off stage.. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |sandboxes@biblibre.com --- Comment #24 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Patch tested with a sandbox, by Broust <jean-manuel.broust@univ-lyon2.fr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25202|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=10865 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25203|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=10865 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25655|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=10865 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25656|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=10865 --- Comment #25 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 26074 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26074&action=edit Bug 10865 [Revised] Don't show list permissions when adding public lists/sharing lists is not allowed If patron creation of public lists is disallowed by the OpacAllowPublicListCreation system preference the "category" option should be hidden altogether instead of showing a <select> with "private" as the only option. This patch hides category and permissions controls when OpacAllowPublicListCreation is set to "don't allow." To test you must have the virtualshelves system preference enabled. Apply the patch and log into the OPAC. Test: - With OpacAllowPublicListCreation enabled, create a new list. You should see options for setting category and permissions. Saving the new list should complete correctly and save the right settings. - With OpacAllowPublicListCreation enabled, edit an existing list. You should see the same options and saving your changes should work correctly. - With OpacAllowPublicListCreation disabled, create a new list. You should only see fields for title and sort. Saving this list should complete correctly and save the right settings. - With OpacAllowPublicListCreation disabled, edit an existing list. You should be able to edit only title and sort settings. Saving your changes should work correctly. Repeat your tests for both prog and bootstrap themes. Revision: Existing public lists can be edited and retain their public status even if OpacAllowPublicListCreation has since been disabled. This preserves the behavior previous to this patch. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> This patch fixes a big ergonomic issue. Note: to me, the "New list" action should be outside the tabs. It is confusing to have a "new list" into the public lists tab when it is not possible to create new public lists. Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #26 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 26075 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26075&action=edit Bug 10865: (follow-up) Move a li tag to avoid overlapping tags With this patch, a li tag is empty (only using the bootstrap theme) if the pref is disabled. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #27 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 26076 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26076&action=edit bug 10865: (follow-up) allow patrons to make their public lists private when OpacAllowPublicListCreation is off This patch ensures that patrons continue to have the ability to make their public lists private for any public lists they control that were created before the library turned the OpacAllowPublicListCreation system preference off. To test: [1] Ensure OpacAllowPublicListCreation is on. [2] As a patron, create a public list in the OPAC. Also, create a private list. [3] Turn OpacAllowPublicListCreation off. [4] Back in the OPAC, verify that the public list can be edited and that there are drop-downs for category and permissions. Also verify that there is a warning that the patron cannot change it back if they convert a public list to private. [5] Edit the private list created in step 2. Verify that the category and permissions drop-downs are not displayed. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #28 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 26077 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26077&action=edit Bug 10865 [Follow-up] Add style to warning This patch adds a new "hint" class for displaying information relating to a form field. On the list edit screen the hint also has an alert class to highlight it. Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #29 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Ok, I started testing this with all 4 patches applied. But I am confused - OpacAllowSharingPrivateLists is set to "Don't allow", but I can still see the additional options that refer to this feature on creating a new list from the patron account (bootstrap/prog)? Options I talk about: Allow/Don't allow anyone to remove his own contributed entries. anyone to remove other contributed entries. URL was: http://localhost/cgi-bin/koha/opac-shelves.pl?shelves=1 Only when I additionally turn off OpacAllowPublicListCreation the options disappear. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #30 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Katrin Fischer from comment #29)
Ok, I started testing this with all 4 patches applied. But I am confused - OpacAllowSharingPrivateLists is set to "Don't allow", but I can still see the additional options that refer to this feature on creating a new list from the patron account (bootstrap/prog)?
Options I talk about: Allow/Don't allow anyone to remove his own contributed entries. anyone to remove other contributed entries.
These options do not specifically refer to a shared list. They apply just as well to a public list. Back to SO. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #31 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Marcel, I tend to disagree here and had a library reporting those as confusing. If you don't allow sharing, it would be a bit scary to me to be offered those options, leaving me wondering who else can see and change my list and if I misunderstood the feature offered by the library.
anyone to remove his own contributed entries. anyone to remove other contributed entries.
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #32 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hm ok, maybe I start to understand. There used to be 3 types of lists in the very past: private, public and open. I think we don't have open as an option in the pull down anymore... but you would be able to mimick the former behaviour with the new options? Maybe it would be clearer if they only showed up, once you have selected your list to be 'public'? Or the description could be changed/amended. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #33 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Katrin Fischer from comment #32)
Hm ok, maybe I start to understand. There used to be 3 types of lists in the very past: private, public and open. I think we don't have open as an option in the pull down anymore... but you would be able to mimick the former behaviour with the new options? Maybe it would be clearer if they only showed up, once you have selected your list to be 'public'? Or the description could be changed/amended.
Hi Katrin. Actually, this discussion should be held on report 7310, not here. This report is just a minor detail. We now have private lists and public lists. With the permissions you can make your list "open" (allowing more flexibility than in the past, with less bugs :). Note that the three permissions have been constructed in this way after discussions on the dev mailing list (and pushed in 3.8). If the list is strictly private, the only useful perm is delete your own entries. You could protect yourself from incidentally deleting an item by disallowing that. But note that if you would start sharing lists [new feature on report 9032] or already if a list changed from public to private etc., things may become more difficult. Still, the perms will just work as they advertise: The owner can always add. Somebody else can add if permitted (but only if he has access). Deleting can be controlled similarly. Due to such exceptions, I would not recommend hiding the three possible perms for private lists; you are not sure if all items on the list have been added by the owner. Another thought: If you would change public to private, you could of course move all items to the owner of the list.. (New report ? ) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #34 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- There is a conflict in opac.css now - I am not sure how this needs to be resolved now with less - Owen, could you take a look please? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #26077|0 |1 is obsolete| | --- Comment #35 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 26923 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26923&action=edit Bug 10865 [Follow-up] Add style to warning This patch adds a new "hint" class for displaying information relating to a form field. On the list edit screen the hint also has an alert class to highlight it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #26074|0 |1 is obsolete| | Attachment #26075|0 |1 is obsolete| | Attachment #26076|0 |1 is obsolete| | Attachment #26923|0 |1 is obsolete| | --- Comment #36 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 26934 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26934&action=edit [PASSED QA] Bug 10865 [Revised] Don't show list permissions when adding public lists/sharing lists is not allowed If patron creation of public lists is disallowed by the OpacAllowPublicListCreation system preference the "category" option should be hidden altogether instead of showing a <select> with "private" as the only option. This patch hides category and permissions controls when OpacAllowPublicListCreation is set to "don't allow." To test you must have the virtualshelves system preference enabled. Apply the patch and log into the OPAC. Test: - With OpacAllowPublicListCreation enabled, create a new list. You should see options for setting category and permissions. Saving the new list should complete correctly and save the right settings. - With OpacAllowPublicListCreation enabled, edit an existing list. You should see the same options and saving your changes should work correctly. - With OpacAllowPublicListCreation disabled, create a new list. You should only see fields for title and sort. Saving this list should complete correctly and save the right settings. - With OpacAllowPublicListCreation disabled, edit an existing list. You should be able to edit only title and sort settings. Saving your changes should work correctly. Repeat your tests for both prog and bootstrap themes. Revision: Existing public lists can be edited and retain their public status even if OpacAllowPublicListCreation has since been disabled. This preserves the behavior previous to this patch. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> This patch fixes a big ergonomic issue. Note: to me, the "New list" action should be outside the tabs. It is confusing to have a "new list" into the public lists tab when it is not possible to create new public lists. Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #37 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 26935 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26935&action=edit [PASSED QA] Bug 10865: (follow-up) Move a li tag to avoid overlapping tags With this patch, a li tag is empty (only using the bootstrap theme) if the pref is disabled. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #38 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 26936 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26936&action=edit [PASSED QA] bug 10865: (follow-up) allow patrons to make their public lists private when OpacAllowPublicListCreation is off This patch ensures that patrons continue to have the ability to make their public lists private for any public lists they control that were created before the library turned the OpacAllowPublicListCreation system preference off. To test: [1] Ensure OpacAllowPublicListCreation is on. [2] As a patron, create a public list in the OPAC. Also, create a private list. [3] Turn OpacAllowPublicListCreation off. [4] Back in the OPAC, verify that the public list can be edited and that there are drop-downs for category and permissions. Also verify that there is a warning that the patron cannot change it back if they convert a public list to private. [5] Edit the private list created in step 2. Verify that the category and permissions drop-downs are not displayed. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 --- Comment #39 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 26937 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=26937&action=edit [PASSED QA] Bug 10865 [Follow-up] Add style to warning This patch adds a new "hint" class for displaying information relating to a form field. On the list edit screen the hint also has an alert class to highlight it. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes all tests and QA script. Works as advertised. Tested with Bootstrap and prog theme. Some notes: - When OpacAllowPublicListCreation is turned off, the permissions don't show. - When OpacAllowPublicListCreation is turned off, we could also hide the Category pull down in the [new list] pop up, as there is only Private left as an option. - Maybe we should move the new list link outside of the tabs? When OpacAllowPublicListCreation is turned off, but public lists exist, the link 'new list' will still show on the public list tab, but a private list will be created. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #40 from Galen Charlton <gmcharlt@gmail.com> --- Pushed to master. Thanks, Owen! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10865 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |fridolyn.somers@biblibre.co | |m --- Comment #41 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Pushed to 3.14.x, will be in 3.14.10 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org