Bug 6690 - If multiple issues are received in serials-edit.pl, changing the status of any issue changes the date of the first issue.
Summary: If multiple issues are received in serials-edit.pl, changing the status of an...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Serials (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal
Assignee: Frédérick Capovilla
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-09 20:41 UTC by Frédérick Capovilla
Modified: 2013-12-05 20:04 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Corrects a problem with the wrong date being updated in serials-edit.pl (3.56 KB, patch)
2011-08-09 20:42 UTC, Frédérick Capovilla 		Details | Diff | Splinter Review
Bug 6690: If multiple issues are received in serials-edit.pl, changing the status of any issue changes the date of the first issue. (3.56 KB, patch)
2011-08-14 19:44 UTC, Katrin Fischer 		Details | Diff | Splinter Review
[SIGNED-OFF] Bug 6690: If multiple issues are received in serials-edit.pl, changing the status of any issue changes the date of the first issue. (3.60 KB, patch)
2011-08-14 23:49 UTC, Nicole C. Engard 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Frédérick Capovilla 2011-08-09 20:41:44 UTC 
On serials-edit.pl

If multiple issues are ready to be received for a subscription, changing the "Status" of any issue in the table changes the "Expected on" date of the first item in the table.
Comment 1 Frédérick Capovilla 2011-08-09 20:42:56 UTC 
Created attachment 4914 [details] [review]
Corrects a problem with the wrong date being updated in serials-edit.pl

Patch that corrects the javascript code on the serials-edit.pl page so the date on the correct row gets updated.
Comment 2 Katrin Fischer 2011-08-11 17:21:50 UTC 
I can confirm the bug in current master. 

But attached patch contains changes to a tmpl file - so we need a reformatted patch for master.

Also 3.2.x is no longer maintained.
Comment 3 Katrin Fischer 2011-08-14 19:44:11 UTC Comment hidden (obsolete)
Comment 4 Nicole C. Engard 2011-08-14 23:49:43 UTC 
Created attachment 5010 [details] [review]
[SIGNED-OFF] Bug 6690: If multiple issues are received in serials-edit.pl, changing the status of any issue changes the date of the first issue.

If multiple issues are in the list, changing the status of any issue in
the table would always change the "Expected on" date of the first issue.
Modified the javascript so the date is modified on the correct row.

Reimplementation of Patch for 3.4.x
Patch for 3.2.x from Frédérick Capovilla

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Comment 5 Chris Nighswonger 2011-08-15 13:12:23 UTC 
I've pushed this patch to both 3.4.x and 3.2.x. Although no other release for 3.2.x is planned, it was trivial to push to the repo for anyone who may want it.
Comment 6 Katrin Fischer 2011-08-15 13:21:54 UTC 
Problem is also present in master, patch will work there too.
Comment 7 Chris Nighswonger 2011-08-15 13:27:59 UTC 
Resetting status to 'Signed Off' as this patch needs to be pushed to master.
Comment 8 Ian Walls 2011-08-25 01:40:19 UTC 
Template only change, easy to read, marking as Passed QA
Comment 9 Chris Cormack 2011-08-25 22:29:23 UTC 
Pushed please test
Comment 10 Jared Camins-Esakov 2012-12-31 00:15:08 UTC 
There have been no further reports of problems so I am marking this bug resolved.