Refactor extended time database model to be more efficient
Description
Environment
Test Plan
is depended on by
is related to
relates to
Activity

Matthew Jones March 1, 2017 at 4:45 PMEdited
Laura had an uber-test plan on for all of this at
I feel like that's really good. The only open issues I still know of are
: Improved process for editing an existing row.
This is in progress
: Settings are not saved unless user clicks to add exception
I think with the new UI that Sam did, it's a lot more clear and this isn't as necessary
: Excepted students cannot view the test when site-level retract date is set to if "Late submission is not accepted"
This is one I still plan to investigate and improve. The issue is that on the regular date selection there's a "yes/no" for late submission, but there (currently isn't) on extended. It seems like if it's "No" then no dates in extended override this no. I think it should just have the date blank (for no override) or filled in (for override)
Neal Caidin January 1, 2017 at 12:59 PM
Test plan ? Is there an uber-test plan for all these related issues?

Matthew Jones November 22, 2016 at 8:41 PM
I merged this initial work and created/moved open subtasks related to this. Wait to verify/merge this until all of the subtasks are resolved.

Leonardo Canessa September 21, 2016 at 9:58 AMEdited
, I'm working on it right now. I welcome any efforts towards collaboration.
EDIT: I have the hibernate / object part done. I am partially done with the front end (the framework is written, but a lot needs to be done to make it functional). I have a partial DAO. I'm currently working on making an actual Service for ExtendedTime. Of course, everything is harder in Samigo.

Matthew Jones September 21, 2016 at 9:55 AM
Is this something you're working on? These jiras were something I (Longsight) was planning on getting done prior to 11.3 since I didn't believe anyone else was going to pick it up.
Extended time is currently storing it's data in the metadata field and is causing a number of regressions and is going to lead to problems down the road. It really should store it's data as a separate table. Then I believe most of these issues will clear up and we can re-enable this feature.
Relates to
And possibly others