When selecting a patron to add as an owner to a acquisition fund, nothing happens. To test. - Create a fund - Click 'select owner' - Search for a patron - Click 'select' -> nothing happens, the owner is not added to the fund. Adding users works as expected. Tested with 17.11.06.
Created attachment 75593 [details] [review] Bug 20827: Display Fund's owner name when selected When a fund is created or edited and does not have a owner yet, the name of the patron will not be displayed when selected. The HTML element the JS code expects is not present in the DOM. Test plan: - Create a fund, set a owner - remove the owner, edit, etc. - Save - Edit again Make sure it works as expected.
Created attachment 75601 [details] [review] Bug 20827: Display Fund's owner name when selected When a fund is created or edited and does not have a owner yet, the name of the patron will not be displayed when selected. The HTML element the JS code expects is not present in the DOM. Test plan: - Create a fund, set a owner - remove the owner, edit, etc. - Save - Edit again Make sure it works as expected. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 75609 [details] [review] Bug 20827: Display Fund's owner name when selected When a fund is created or edited and does not have a owner yet, the name of the patron will not be displayed when selected. The HTML element the JS code expects is not present in the DOM. Test plan: - Create a fund, set a owner - remove the owner, edit, etc. - Save - Edit again Make sure it works as expected. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 18.11. Awesome work all, thank you.
This patch has been pushed to 18.05.x and will be in 18.05.01
Pushed to 17.11.x for 17.11.07
(In reply to Fridolin SOMERS from comment #6) > Pushed to 17.11.x for 17.11.07 JS has moved inside TT file so I manually corrected the missing ";"
Pushed to 17.05.x for 17.05.13