Bug 23590 added new features around tracking the manager of a suggestion. I'm unable to edit those values, however. To recreate: - create a new purchase suggestion from within acquisitions (suggestion.pl?op=add) - click "select manager," search for user, click Select - see that the user you just selected shows under "Created by," - see that "Managed by" still says "You" - save your suggestion - query the database, see that suggestion.suggestedby is the user you selected, suggestion.managedby is your own borrowernumber I see that same behavior when editing an existing suggestion rather than making one from scratch.
I just tested this and you are right, there is definitely a bug there. Set patron should change the suggestor, but now the suggestor is also changed when using "select manager". I assume it could be a Javascript issue targetting the wrong field?
There are also controls to change the manager in bulk from the main purchase suggestion page. I can't get that to work, either.
This is caused by bug 24819. It has been pushed 1 month after bug 23590.
Created attachment 112836 [details] [review] Bug 26738: (bug 24819 follow-up) Fix manager selection Bug 23590 added a new feature to select the manager of a suggestion. One month later bug 24819 added the ability to pick the suggester. This second patchset broke the manager selection. This patch simplifies the way the suggester is selected, using the generic way and mimicking what is done for the manager. Test plan: - create a new purchase suggestion from within acquisitions (suggestion.pl?op=add) - click "select manager," search for user, click Select - see that the user you just selected shows under "Created by," - see that "Managed by" still says "You" - modify the suggester - save your suggestion => Everything is saved correctly QA will test the permission alert: Edit suggestion.tt and remove "&permissions=suggestions.suggestions_manage" Edit the suggestion, select a manager, pick a patron in the list who does not have sufficient permissions, save => you get the alert
New feature advertised in 20.05 is broken, upping severity.
This does allow me to save changes to the manager of a purchase suggestion. I've not seen any notices generated as I do so. My two demo users do both have email addresses and the notice "NOTIFY_MANAGER" exists. Am I missing something there? Is the manager of a purchase suggestion meant to always default to whoever last edited it? If I edit an existing suggestion with another user as the manager, Koha will set me as the manger unless I remember to click to tell it not to.
Created attachment 112842 [details] [review] Bug 26738: (bug 24819 follow-up) Fix manager selection Bug 23590 added a new feature to select the manager of a suggestion. One month later bug 24819 added the ability to pick the suggester. This second patchset broke the manager selection. This patch simplifies the way the suggester is selected, using the generic way and mimicking what is done for the manager. Test plan: - create a new purchase suggestion from within acquisitions (suggestion.pl?op=add) - click "select manager," search for user, click Select - see that the user you just selected shows under "Created by," - see that "Managed by" still says "You" - modify the suggester - save your suggestion => Everything is saved correctly QA will test the permission alert: Edit suggestion.tt and remove "&permissions=suggestions.suggestions_manage" Edit the suggestion, select a manager, pick a patron in the list who does not have sufficient permissions, save => you get the alert Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
I *was* missing something! there's a checkbox! Manager changed successfully through both edit and batch interfaces. Notice sent successfully. I'm still a little torn about defaulting the changing the manager each time you edit, but will wait to see what users think.
(In reply to Andrew Fuerste-Henry from comment #8) > I *was* missing something! there's a checkbox! > > Manager changed successfully through both edit and batch interfaces. > Notice sent successfully. > I'm still a little torn about defaulting the changing the manager each time > you edit, but will wait to see what users think. I'm sure I've seen another bug regarding always setting the manager to the last person to update the suggestion.. I can't remember which way the bug was pushing for and can't find the bug now though :(.
I don't manage to change the manager to another user than the current one. Followed Andrew's steps. Jonathan's are about changing the 'suggester' so I'm confused.
(In reply to Tomás Cohen Arazi from comment #10) > I don't manage to change the manager to another user than the current one. > Followed Andrew's steps. > > Jonathan's are about changing the 'suggester' so I'm confused. Code is affecting both, manager and suggester, so both searches must be tested. The manager search will only return patrons with suggestions_manage. "Only staff with superlibrarian or suggestions_manage permissions are returned in the search results"
Just tested this again, successfully. Updated test plan: To test: 1- make sure you have at least 2 patrons with purchase suggestion permissions and email addresses 2- open a new or existing purchase suggestion 3- in the Suggestion Management section, click the Select Manager link 4- in the patron search popup, search for and select a staff member with suggestion permissions 5- on the suggestion, click the "Notifiy Manager" checkbox 6- save your suggestion 7- on suggestion.pl, confirm your suggestion shows your selected staff member for Managed By 8- confirm your staff member received a notice 9- on suggestion.pl, use the checkboxes to select your suggestion and any number of additional suggestions 10- click Select Manager in the Update Manger box underneath the suggestions table 11- again, use the patron search popup to pick someone 12- click Submit to make your change 13- confirm the suggestions table reflects your change (there is not yet a way to send a notice for batch changes)
Both search+choose work correctly. I just found saving didn't store the chosen manager. RETESTING.
Created attachment 112920 [details] [review] Bug 26738: (bug 24819 follow-up) Fix manager selection Bug 23590 added a new feature to select the manager of a suggestion. One month later bug 24819 added the ability to pick the suggester. This second patchset broke the manager selection. This patch simplifies the way the suggester is selected, using the generic way and mimicking what is done for the manager. Test plan: - create a new purchase suggestion from within acquisitions (suggestion.pl?op=add) - click "select manager," search for user, click Select - see that the user you just selected shows under "Created by," - see that "Managed by" still says "You" - modify the suggester - save your suggestion => Everything is saved correctly QA will test the permission alert: Edit suggestion.tt and remove "&permissions=suggestions.suggestions_manage" Edit the suggestion, select a manager, pick a patron in the list who does not have sufficient permissions, save => you get the alert Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 112921 [details] [review] Bug 26738: (QA follow-up) Add missing filter Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Pushed to master for 20.11, thanks to everybody involved!
this patchset does not clean apply to 20.05.x, no backport please rebase if needed
Created attachment 113746 [details] [review] Bug 26738: [20.05.x] (bug 24819 follow-up) Fix manager selection Bug 23590 added a new feature to select the manager of a suggestion. One month later bug 24819 added the ability to pick the suggester. This second patchset broke the manager selection. This patch simplifies the way the suggester is selected, using the generic way and mimicking what is done for the manager. Test plan: - create a new purchase suggestion from within acquisitions (suggestion.pl?op=add) - click "select manager," search for user, click Select - see that the user you just selected shows under "Created by," - see that "Managed by" still says "You" - modify the suggester - save your suggestion => Everything is saved correctly QA will test the permission alert: Edit suggestion.tt and remove "&permissions=suggestions.suggestions_manage" Edit the suggestion, select a manager, pick a patron in the list who does not have sufficient permissions, save => you get the alert Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Bug 26738: (QA follow-up) Add missing filter Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Thanks Jonathan! backported to 20.05.x for 20.05.06
missing dependencies, not backported to 19.11.x