[SAK-35101] Refactor extended time database model to be more efficient Created: 11-Jul-2016  Updated: 17-Apr-2018  Resolved: 22-Nov-2016

Status: RESOLVED
Project: Sakai
Component/s: Tests & Quizzes (Samigo)
Affects Version/s: 11.1, 12.0
Fix Version/s: 12.0

Type: Task Priority: Blocker
Reporter: Matthew Jones Assignee: Leonardo Canessa
Resolution: Fixed Votes: 1
Labels: TL, notready, qatestplanneeded
Σ Remaining Estimate: Not Specified Remaining Estimate: Not Specified
Σ Time Spent: Not Specified Time Spent: Not Specified
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Issue Links:
Depend
is depended on by SAK-34630 Simplify a few date settings on Samig... OPEN
Relate
relates to SAK-34476 samigo: Settings page doesn't warn/pr... OPEN
relates to SAK-34497 Exception for User drop-down - change... OPEN
relates to SAK-36721 ExtendedTime entries may duplicate RESOLVED
relates to SAK-37079 Text change: Autosubmit job submittin... Verified
relates to SAK-35053 Disable extended time UI in Sakai 11 CLOSED
relates to SAK-34955 Allow different delivery settings for... CLOSED
is related to SAK-35151 Delivery Date Exceptions cannot be ed... Verified
is related to SAK-34309 Parent Jira for testing on Master for... OPEN
is related to SAK-34323 Extended Delivery/Delivery Exceptions... OPEN
is related to SAK-36621 Excepted student sees due date messag... Verified
Sub-Tasks:
Key
Summary
Type
Status
Assignee
SAK-35137 Text on Extended Time should be more ... Sub-task Verified Matthew Jones  
SAK-35138 Errors when editing/deleting an Exten... Sub-task Verified Matthew Jones  
SAK-35139 When publishing an assessment extende... Sub-task Verified Sristi Shrestha  
SAK-35140 Excepted student cannot see quiz Sub-task RESOLVED Neal Caidin  
SAK-35141 Auto submit job doesn't handle extend... Sub-task Verified Matthew Jones  
SAK-35142 Extended Time - Layout wrong for firs... Sub-task RESOLVED Matthew Jones  
SAK-35143 Extended Time Service should return d... Sub-task Verified Sristi Shrestha  
SAK-35144 Exceptions settings are not saved unl... Sub-task Verified Matthew Jones  
SAK-35145 Clean up some left over extended time... Sub-task RESOLVED Matthew Jones  
SAK-35146 Excepted students cannot view the tes... Sub-task Verified Matthew Jones  
SAK-35147 Date pickers disappear when using Ext... Sub-task Verified Andrea Schmidt  
SAK-35148 Extended time should validate that us... Sub-task Verified Andrea Schmidt  
SAK-35149 Extended time area, extra scrollbar p... Sub-task Verified Sam Ottenhoff  
SAK-35150 Editing a time exception, the time wi... Sub-task Verified Andrea Schmidt  
SAK-35151 Delivery Date Exceptions cannot be ed... Sub-task Verified SAMIGO TEAM  
SAK-35152 When editing an extended time entry, ... Sub-task Verified Matthew Jones  
SAK-35153 Allow time entries to be null, don't ... Sub-task RESOLVED Matthew Jones  
SAK-35154 Time limit exception does not work un... Sub-task Verified Andrea Schmidt  
SAK-35155 Working copy extended time and publis... Sub-task Verified Andrea Schmidt  
SAK-35156 Missing key extended_time_retract_ear... Sub-task Verified Sam Ottenhoff  
11 status: Won't Fix

 Description   

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
SAM-2930
SAM-2914
SAM-2868

And possibly others



 Comments   
Comment by Neal Caidin [ 04-Aug-2016 ]

Too much work here for 11.1. We will target 11.2

Comment by Matthew Jones [ 21-Sep-2016 ]

Leonardo Canessa 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.

Comment by Leonardo Canessa [ 21-Sep-2016 ]

Matthew Jones, 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.

Comment by Matthew Jones [ 22-Nov-2016 ]

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.

Comment by Neal Caidin [ 01-Jan-2017 ]

Test plan ? Is there an uber-test plan for all these related issues?

Comment by Matthew Jones [ 01-Mar-2017 ]

Neal Caidin Laura had an uber-test plan on SAM-2914 for all of this at

https://jira.sakaiproject.org/issues/?jql=text%20~%20%22qti%22%20and%20status%20in%20(open%2C%20%22awaiting%20review%22)%20and%20project%20%3D%20sam

I feel like that's really good. The only open issues I still know of are

  • SAM-3165 : Improved process for editing an existing row.
    • This is in progress
  • SAM-3094 : 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
  • SAM-3107 : 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)
Generated at Mon Jun 17 01:14:16 CDT 2019 using JIRA 7.5.0#75005-sha1:fd8c849d4e278dd8bbaccc61e707a716ad697024.