We are moving to Git Issues for bug tracking in future releases. During transition, content will be in both tools. If you'd like to file a new bug, please create an issue.

Bug 6915 - Review due date incorrectly set and displayed for paper review
Review due date incorrectly set and displayed for paper review
Product: OCS
Classification: Unclassified
Component: Review
All All
: P3 normal
Assigned To: Michael Felczak
Depends on:
  Show dependency treegraph
Reported: 2011-10-06 09:19 PDT by Michael Felczak
Modified: 2012-09-24 10:51 PDT (History)
2 users (show)

See Also:
Version Reported In:
Also Affects:

Screenshots with additional details (720.23 KB, application/pdf)
2011-10-06 09:20 PDT, Michael Felczak
Patch against OCS 2.3.3-1 (1.45 KB, patch)
2011-10-07 16:42 PDT, Michael Felczak
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Felczak 2011-10-06 09:19:58 PDT
On Director's submission review page.

Please see attachment for additional details.

Courtesy of:
Comment 1 Michael Felczak 2011-10-06 09:20:57 PDT
Created attachment 3651 [details]
Screenshots with additional details
Comment 2 Michael Felczak 2011-10-07 16:34:41 PDT
For 2-stage review, after abstract review is completed and reviewer assignments from abstract review are set as defaults for paper review.
Comment 3 Michael Felczak 2011-10-07 16:40:02 PDT
Review due date incorrectly set and displayed for paper review
Comment 4 Michael Felczak 2011-10-07 16:42:41 PDT
Created attachment 3655 [details]
Patch against OCS 2.3.3-1
Comment 5 Michael Felczak 2011-10-07 16:45:01 PDT
Review due date incorrectly set and displayed for paper review
Comment 6 Michael Felczak 2011-10-07 16:47:09 PDT
Comment 7 Alec Smecher 2011-10-07 17:04:59 PDT
Michael, I'm confused about this fix -- shouldn't the existing calls to TrackDirectorAction::setDueDate update the due date field?
Comment 8 Michael Felczak 2011-10-07 17:22:13 PDT
This was confusing for me as well. 

Although the due date is initially correctly updated in the db, the review assignment itself in the track submission is not updated with this new date info.

The now-stale track submission is then updated in the db, wiping the previously set due date.
Comment 9 Alec Smecher 2011-10-07 20:46:44 PDT
OK, though the patch is more of a work-around than a fix -- I'd suggest leaving this open for a better clean-up when we get the chance.