Bug 20461 - New subscription form: "Item type" and "item type for older issues" fields are ignored
Summary: New subscription form: "Item type" and "item type for older issues" fields ar...
Status: Pushed to Stable
Alias: None
Product: Koha
Classification: Unclassified
Component: Serials (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Victor Grousset/tuxayo
QA Contact: Testopia
URL:
Whiteboard:
Keywords:
Depends on: 7677
Blocks:
  Show dependency treegraph
 
Reported: 2018-03-22 14:38 UTC by Victor Grousset/tuxayo
Modified: 2018-04-24 15:11 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Bot Control: ---
When did the bot last check this:
Who signed the patch off:
Text to go in the release notes:


Attachments
Bug 20461 : Serials: fix ignored params in subscription creation form (1.52 KB, patch)
2018-03-22 15:06 UTC, Victor Grousset/tuxayo
Details | Diff | Splinter Review
Bug 20461 : Serials: fix ignored params in subscription creation form (1.57 KB, patch)
2018-03-23 10:12 UTC, sandboxes@biblibre.com
Details | Diff | Splinter Review
Bug 20461 : Serials: fix ignored params in subscription creation form (1.72 KB, patch)
2018-03-23 11:36 UTC, Séverine Queune
Details | Diff | Splinter Review
Bug 20461 : Serials: fix ignored params in subscription creation form (1.79 KB, patch)
2018-03-23 15:04 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Victor Grousset/tuxayo 2018-03-22 14:38:52 UTC
Impacts also 16.11.x

== Cause ==
In subscription-add.pl, the two params aren't passed to NewSubscription() but they are to ModSubscription()

== Workaround ==
Modify the newly created subscription to set these two values.
Comment 1 Victor Grousset/tuxayo 2018-03-22 15:06:42 UTC
Created attachment 73157 [details] [review]
Bug 20461 : Serials: fix ignored params in subscription creation form

In subscription-add.pl, the two params aren't passed to
NewSubscription() but they are to ModSubscription()

== Test plan ==
1. Enable syspref "makePreviousSerialAvailable"
2. Create a minimal subscription with a value in the "Item type" and
     "item type for older issues" fields.
3. Edit the subscription
4. You should see that the two fields are empty. This is the bug.
5. Apply this patch
6. Do step 2 and 3
7. You should see that the two fields have the right value
8. Express the joy of a successful and easy sign off.
   (this is important, otherwise the sign off spell won't work!)
Comment 2 Victor Grousset/tuxayo 2018-03-22 15:42:28 UTC
BTW, not other field are missing between the new and modify functions.
Comment 3 Victor Grousset/tuxayo 2018-03-22 15:42:52 UTC
*no other fields
Comment 4 sandboxes@biblibre.com 2018-03-23 10:12:11 UTC
Patch tested with a sandbox, by delaye <stephane.delaye@biblibre.com>
Comment 5 sandboxes@biblibre.com 2018-03-23 10:12:31 UTC
Created attachment 73168 [details] [review]
Bug 20461 : Serials: fix ignored params in subscription creation form

In subscription-add.pl, the two params aren't passed to
NewSubscription() but they are to ModSubscription()

== Test plan ==
1. Enable syspref "makePreviousSerialAvailable"
2. Create a minimal subscription with a value in the "Item type" and
     "item type for older issues" fields.
3. Edit the subscription
4. You should see that the two fields are empty. This is the bug.
5. Apply this patch
6. Do step 2 and 3
7. You should see that the two fields have the right value
8. Express the joy of a successful and easy sign off.
   (this is important, otherwise the sign off spell won't work!)

Signed-off-by: delaye <stephane.delaye@biblibre.com>
Comment 6 Victor Grousset/tuxayo 2018-03-23 11:11:56 UTC
Needs another sign off from someone not at BibLibre (current community rules)
Comment 7 Séverine Queune 2018-03-23 11:36:28 UTC
Created attachment 73174 [details] [review]
Bug 20461 : Serials: fix ignored params in subscription creation form

In subscription-add.pl, the two params aren't passed to
NewSubscription() but they are to ModSubscription()

== Test plan ==
1. Enable syspref "makePreviousSerialAvailable"
2. Create a minimal subscription with a value in the "Item type" and
     "item type for older issues" fields.
3. Edit the subscription
4. You should see that the two fields are empty. This is the bug.
5. Apply this patch
6. Do step 2 and 3
7. You should see that the two fields have the right value
8. Express the joy of a successful and easy sign off.
   (this is important, otherwise the sign off spell won't work!)

Signed-off-by: delaye <stephane.delaye@biblibre.com>
Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Comment 8 Kyle M Hall 2018-03-23 15:04:25 UTC
Created attachment 73188 [details] [review]
Bug 20461 : Serials: fix ignored params in subscription creation form

In subscription-add.pl, the two params aren't passed to
NewSubscription() but they are to ModSubscription()

== Test plan ==
1. Enable syspref "makePreviousSerialAvailable"
2. Create a minimal subscription with a value in the "Item type" and
     "item type for older issues" fields.
3. Edit the subscription
4. You should see that the two fields are empty. This is the bug.
5. Apply this patch
6. Do step 2 and 3
7. You should see that the two fields have the right value
8. Express the joy of a successful and easy sign off.
   (this is important, otherwise the sign off spell won't work!)

Signed-off-by: delaye <stephane.delaye@biblibre.com>
Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 9 Jonathan Druart 2018-03-23 15:09:36 UTC
Caused by:
  commit 99c25135599c99bb1f1c83d390eca4628a26f4cf
  BZ7677: New areas in subscriptions and new functions when receiving.
Comment 10 Jonathan Druart 2018-03-26 20:39:07 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 11 Nick Clemens 2018-04-24 01:35:24 UTC
Awesome work all, backported to stable for 17.11.05
Comment 12 Fridolin SOMERS 2018-04-24 15:11:04 UTC
Pushed to 17.05.x for 17.05.11