Follow-up to #2378365: Convert Quiz to D7 Entity API

Problem/Motivation

The quiz results page currently doubles as a scoring form. This has some nice functionality however it complicates a lot of other things. Most notably #2366349: [SOLVED] Issue with IE when the answer is wrong. among others.

Proposed resolution

We should split the quiz result page from the score form. If users need to score quizzes, there should be a separate area that does not include feedback and the like to grade answers. This will uncomplicate the form and also make Entity functionality easier (like View/Edit results).

Remaining tasks

- Use entity_view for each quiz_question_result
- Add a new tab on Quiz results to edit (score) the result

User interface changes

- Grading form is no longer accessible from the result page, users must go to a tab (if they have access)

API changes

None, additive Entity API work.

CommentFileSizeAuthor
#34 decouple_quiz-2386103-34.patch42.42 KBdjdevin
#31 decouple_quiz-2386103-31.patch42.38 KBdjdevin
#28 decouple_quiz-2386103-28.patch42.39 KBdjdevin
#23 decouple_quiz-2386103-23.patch42.39 KBdjdevin
#22 decouple_quiz-2386103-22.patch42.39 KBdjdevin
#21 decouple_quiz-2386103-21.patch40 KBdjdevin
#18 decouple_quiz-2386103-18.patch39.36 KBdjdevin
#15 decouple_quiz-2386103-15.patch77.36 KBdjdevin
#12 decouple_quiz-2386103-12.patch37.75 KBdjdevin
#9 decouple_quiz-2386103-9.patch31.04 KBdjdevin
#4 decouple_quiz-2386103-3.patch27.99 KBdjdevin
#3 decouple_quiz-2386103-3.patch27.98 KBdjdevin
#1 decouple_quiz-2386103-1.WIP_.patch3.09 KBdjdevin

Comments

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new decouple_quiz-2386103-1.WIP_.patch3.09 KB
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Issue tags: -7.x-5.1 release blocker +7.x-5.0 release blocker
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Active » Needs review
StatusFileSize
new decouple_quiz-2386103-3.patch27.98 KB
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new decouple_quiz-2386103-3.patch27.99 KB

The last submitted patch, 3: decouple_quiz-2386103-3.patch, failed testing.

The last submitted patch, 3: decouple_quiz-2386103-3.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 4: decouple_quiz-2386103-3.patch, failed testing.

The last submitted patch, 4: decouple_quiz-2386103-3.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-9.patch31.04 KB

Status: Needs review » Needs work

The last submitted patch, 9: decouple_quiz-2386103-9.patch, failed testing.

The last submitted patch, 9: decouple_quiz-2386103-9.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-12.patch37.75 KB

Status: Needs review » Needs work

The last submitted patch, 12: decouple_quiz-2386103-12.patch, failed testing.

The last submitted patch, 12: decouple_quiz-2386103-12.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-15.patch77.36 KB

Status: Needs review » Needs work

The last submitted patch, 15: decouple_quiz-2386103-15.patch, failed testing.

The last submitted patch, 15: decouple_quiz-2386103-15.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-18.patch39.36 KB

Status: Needs review » Needs work

The last submitted patch, 18: decouple_quiz-2386103-18.patch, failed testing.

The last submitted patch, 18: decouple_quiz-2386103-18.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-21.patch40 KB
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new decouple_quiz-2386103-22.patch42.39 KB
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new decouple_quiz-2386103-23.patch42.39 KB

The last submitted patch, 22: decouple_quiz-2386103-22.patch, failed testing.

The last submitted patch, 22: decouple_quiz-2386103-22.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 23: decouple_quiz-2386103-23.patch, failed testing.

The last submitted patch, 23: decouple_quiz-2386103-23.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-28.patch42.39 KB

Status: Needs review » Needs work

The last submitted patch, 28: decouple_quiz-2386103-28.patch, failed testing.

The last submitted patch, 28: decouple_quiz-2386103-28.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-31.patch42.38 KB

Status: Needs review » Needs work

The last submitted patch, 31: decouple_quiz-2386103-31.patch, failed testing.

The last submitted patch, 31: decouple_quiz-2386103-31.patch, failed testing.

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new decouple_quiz-2386103-34.patch42.42 KB
djdevin’s picture

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

  • djdevin committed 81de5df on 7.x-5.x 
    Issue #2386103 by djdevin: Decouple quiz result/scoring

Status: Fixed » Closed (fixed)

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