Bug 20827

Summary: Can't add owner to a fund
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: AcquisitionsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, martin.renvoize, mtj, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 20827: Display Fund's owner name when selected
Bug 20827: Display Fund's owner name when selected
Bug 20827: Display Fund's owner name when selected

Description Katrin Fischer 2018-05-28 11:32:57 UTC
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.
Comment 1 Jonathan Druart 2018-05-28 14:42:58 UTC
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.
Comment 2 Mark Tompsett 2018-05-28 18:59:06 UTC
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>
Comment 3 Katrin Fischer 2018-05-28 20:13:03 UTC
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>
Comment 4 Nick Clemens 2018-05-29 14:37:15 UTC
Pushed to master for 18.11.

Awesome work all, thank you.
Comment 5 Martin Renvoize 2018-06-05 10:06:22 UTC
This patch has been pushed to 18.05.x and will be in 18.05.01
Comment 6 Fridolin Somers 2018-06-06 10:17:55 UTC
Pushed to 17.11.x for 17.11.07
Comment 7 Fridolin Somers 2018-06-06 10:18:41 UTC
(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 ";"
Comment 8 Fridolin Somers 2018-06-07 08:14:14 UTC
Pushed to 17.05.x for 17.05.13