Looks like I missed these two lines of code a few patches back. Once I got all the patches committed and started testing I noticed this minor issue. It was causing all quizzes to be incorrectly marked as "In Progress" because the code was looking for "end_time" instead of the "finished" column that the sql was returning.

CommentFileSizeAuthor
quiz_finished_fix.patch724 bytesseanbfuller

Comments

seanbfuller’s picture

seanbfuller commented
Status: Active » Needs review
seanbfuller’s picture

seanbfuller commented
Status: Needs review » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

  • Commit 9ad8e92 on 4.7.x-1.x, 5.x-1.x, 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x by seanbfuller: 
    #90692 Fixes a snippet of sql code that was giving the time_end column...

  • Commit 9ad8e92 on 4.7.x-1.x, 5.x-1.x, 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x, quiz-pages by seanbfuller: 
    #90692 Fixes a snippet of sql code that was giving the time_end column...

  • Commit 9ad8e92 on 4.7.x-1.x, 5.x-1.x, 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x, quiz-pages, 2269219 by seanbfuller: 
    #90692 Fixes a snippet of sql code that was giving the time_end column...

  • Commit 9ad8e92 on 4.7.x-1.x, 5.x-1.x, 5.x-2.x, 6.x-2.x, 6.x-3.x, 6.x-4.x, 6.x-5.x, 6.x-6.x, 7.x-4.x, master, 7.x-5.x, 2269219 by seanbfuller: 
    #90692 Fixes a snippet of sql code that was giving the time_end column...