Would it be possible to add a time field as well? For example, to open a quiz between 3:00 - 3:30 PM. The quiz_open and quiz_close columns appear to just be seconds, so I guess it might only require adding some more inputs and form processing.

CommentFileSizeAuthor
#10 open_close_date_and_time-505676-10.patch9.91 KBdjdevin
#8 open_close_date_and_time-505676-8.patch9.86 KBdjdevin
#6 open_close_date_and_time-505676-6.patch10.05 KBdjdevin
#4 open_close_date_and_time-505676-4.patch9.26 KBdjdevin

Comments

raycascella’s picture

raycascella
he/him
commented
Version: 6.x-3.0 » 7.x-5.x-dev
Assigned: Unassigned » raycascella
Issue summary: View changes
ezraw’s picture

ezraw commented
Related issues: +#1337758: Quiz doesn't handle daylight savings time

Probably want to consider #1337758: Quiz doesn't handle daylight savings time as well.

thehong’s picture

thehong commented
Assigned: raycascella » Unassigned
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Active » Needs review
StatusFileSize
new open_close_date_and_time-505676-4.patch9.26 KB

And here's a patch, 4 years later.

This adds a dependency to date API. I don't know who would not be using date API at this point, since they have 500k installs.

Tests will probably failed due to dependency issues. I'll run them all manually.

Status: Needs review » Needs work

The last submitted patch, 4: open_close_date_and_time-505676-4.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new open_close_date_and_time-505676-6.patch10.05 KB

Status: Needs review » Needs work

The last submitted patch, 6: open_close_date_and_time-505676-6.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new open_close_date_and_time-505676-8.patch9.86 KB

Status: Needs review » Needs work

The last submitted patch, 8: open_close_date_and_time-505676-8.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new open_close_date_and_time-505676-10.patch9.91 KB

  • djdevin committed c189ebc on 7.x-5.x 
    Issue #505676 by djdevin | copelco: Added Open/Close Date AND Time?.
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Fixed

Finally done. Committed to 7.x-5.x, date_popup must be enabled but is not a dependency.

Not enabling date_popup just means that you can't set open/close dates.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.