Bug 19678

Summary: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!
Product: Koha Reporter: Claire Gravely <claire_gravely>
Component: Course reservesAssignee: Claire Gravely <claire_gravely>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 19229    
Bug Blocks:    
Attachments: Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!

Description Claire Gravely 2017-11-22 13:32:15 UTC
Clicking on the cancel link when adding a new course to course reserves displays the message 'Invalid Course!'. It would be better if it redirected back to the main Course Reserves page.

To reproduce: 
Go to Course Reserves
Click '+ New Course'
Cancel the process
You are shown 'Invalid Course!'
Comment 1 Claire Gravely 2017-11-22 13:56:14 UTC Comment hidden (obsolete)
Comment 2 Claire Gravely 2017-11-22 14:34:55 UTC
Created attachment 69278 [details] [review]
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!

Clicking on the cancel link when adding a new course to course reserves
displays the message 'Invalid Course!'. It would be better if it
redirected back to the main Course Reserves page.

This patch redirects the user back to the main course reserves page.

To Test:

1. Go to Course Reserves

2. Click '+ New Course'

3. Cancel the process

4. You are shown 'Invalid Course!'

5. Apply patch and repeat steps 1-3

6. You are redirected back to the main course reserves page
Comment 3 Claire Gravely 2017-11-22 14:51:06 UTC
Created attachment 69280 [details] [review]
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!

Clicking on the cancel link when adding a new course to course reserves
displays the message 'Invalid Course!'. It would be better if it
redirected back to the main Course Reserves page.

This patch redirects the user back to the main course reserves page.

To Test:

1. Go to Course Reserves

2. Click '+ New Course'

3. Cancel the process

4. You are shown 'Invalid Course!'

5. Apply patch and repeat steps 1-3

6. You are redirected back to the main course reserves page
Comment 4 Owen Leonard 2017-11-22 14:59:31 UTC
Created attachment 69281 [details] [review]
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!

Clicking on the cancel link when adding a new course to course reserves
displays the message 'Invalid Course!'. It would be better if it
redirected back to the main Course Reserves page.

This patch redirects the user back to the main course reserves page.

To Test:

1. Go to Course Reserves

2. Click '+ New Course'

3. Cancel the process

4. You are shown 'Invalid Course!'

5. Apply patch and repeat steps 1-3

6. You are redirected back to the main course reserves page

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 5 Jonathan Druart 2017-12-19 15:57:01 UTC
Caused by commit f55af2fc078a7d6a05238232bc276e6924307179
Bug 19229: Return to course when cancelling out of edit form
Comment 6 Jonathan Druart 2017-12-19 15:57:41 UTC
Created attachment 69916 [details] [review]
Bug 19678: Clicking Cancel when adding New Course to course reserves shows message Invalid Course!

Clicking on the cancel link when adding a new course to course reserves
displays the message 'Invalid Course!'. It would be better if it
redirected back to the main Course Reserves page.

This patch redirects the user back to the main course reserves page.

To Test:

1. Go to Course Reserves

2. Click '+ New Course'

3. Cancel the process

4. You are shown 'Invalid Course!'

5. Apply patch and repeat steps 1-3

6. You are redirected back to the main course reserves page

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 7 Jonathan Druart 2017-12-20 16:46:29 UTC
Pushed to master for 18.05, thanks to everybody involved!