Problem/Motivation

We currently have two feedback times, "after the question" and "after the quiz". We should be able to add additional times to show more conditional feedback.

For example you may not want the user to see correct answers unless they have passed the quiz. Or, only show certain feedbacks like peer comparisons for a length of time. Or, only showing results X days after the quiz, after the learner has reviewed it with the instructor.

Proposed resolution

The feedback options are currently alterable by modules, so the feedback times should be as well.

For each feedback time, use rules to let the conditions be configurable.

Remaining tasks

- Add quiz_get_feedback_times() that mirrors quiz_get_feedback_options()
- Delegate criteria to Rules
- Tests
- API Documentation

User interface changes

- None

API changes

- Added hook_quiz_feedback_times()
- Added hook_quiz_feedback_times_alter(&$feedback_times)
- Added quiz_get_feedback_times()
- Improve hook_quiz_feedback_options()
- Improve hook_quiz_feedback_options_alter(&$feedback_options)
- Improve quiz_get_feedback_options()

Data model changes

None

Comments

djdevin’s picture

Issue summary: View changes
djdevin’s picture

Issue summary: View changes
djdevin’s picture

Status: Active » Needs review
StatusFileSize
new6.19 KB

Status: Needs review » Needs work

The last submitted patch, 3: additional_weighted-2398119-3.patch, failed testing.

djdevin’s picture

Issue summary: View changes
djdevin’s picture

Status: Needs work » Needs review
StatusFileSize
new7.98 KB

Status: Needs review » Needs work

The last submitted patch, 6: additional_weighted-2398119-6.patch, failed testing.

djdevin’s picture

Status: Needs work » Needs review
StatusFileSize
new8.81 KB

  • djdevin committed fe6daf0 on 7.x-5.x
    Add Rules as a dependency for upcoming #2398119.
    

Status: Needs review » Needs work

The last submitted patch, 8: additional_weighted-2398119-8.patch, failed testing.

djdevin’s picture

Issue summary: View changes

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 8: additional_weighted-2398119-8.patch, failed testing.

djdevin’s picture

Status: Needs work » Needs review
StatusFileSize
new12.99 KB
djdevin’s picture

Status: Needs review » Fixed

  • djdevin committed 3f43e16 on 7.x-5.x
    Issue #2398119 by djdevin: Additional weighted review times
    
djdevin’s picture

Issue summary: View changes

Status: Fixed » Closed (fixed)

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