[sazedul-haque]

📌 Summary

Enhances the Quiz Answer Reveal feature by migrating to secure server-side answer verification, persisting revealed answer states across page reloads, and adding a countdown fill animation to navigation buttons during the reveal wait time.

---

🚀 Key Changes

• Secure Server-Side Verification:

  • Removed client-side answer leakage (window.tutor_quiz_context and pre-rendered correct answer IDs).
  • Added secure wp_ajax_tutor_quiz_check_answer endpoint to verify questions on demand.

• State Persistence on Reload:

  • Saves verified answers to tutor_quiz_attempt_answers immediately upon reveal.
  • Hydrates question selections, explanation blocks, and active question indexes when the page is reloaded.
  • Preserves pagination item states (correct / incorrect) across reloads by standardizing the data-reveal-result attribute.

• Button Countdown Animation:

  • Added smooth background fill countdown animation to Next and Submit Quiz buttons during answers_reveal_duration.
  • Supports RTL layouts and respects prefers-reduced-motion.

• Legacy Mode Parity & Bug Fixes:

  • Added loading spinners during verification in legacy mode.
  • Fixed legacy quiz pagination active state not updating when clicking the Back button.

[harunollyo]

This solution still has a glitch when fetching attempt answers by extracting the required IDs from the DOM. We need to do further R&D to find a proper solution and determine a target version for ending legacy support.

[b-l-i-n-d]

Why is there no tutor_filter_quiz_total_marks filter? Don't we need it?

[sazedul-haque]

We don't need this here. For reveal mode we are fetching the total marks from database before inserting quit attempt using QuizModel::get_attempt_answers_stats( (int) $attempt_id );:

if ( $has_committed_answers ) {
    $stats                    = QuizModel::get_attempt_answers_stats( (int) $attempt_id );
    $earned_marks             = max( 0.0, $stats->total_earned_marks );
    $total_answered_questions = $stats->total_answered_count;
    $review_required          = $review_required || $stats->pending_review_count > 0;
}

[shewa12]

Why is the default value empty instead of the stored value?

[sazedul-haque]

Answer explanation comes from Pro plugin.

[shewa12]

Why is the default value for is_correct false? What if the question requires manual review, like a short answer or an open-ended question?

[sazedul-haque]

Reveal mode only supports following question types. So no manual review required.

$supported_types = array(
	QuizModel::QUESTION_TYPE_TRUE_FALSE,
	QuizModel::QUESTION_TYPE_SINGLE_CHOICE,
	QuizModel::QUESTION_TYPE_MULTIPLE_CHOICE,
);

[shewa12]

Since the default value is set to = false
The question types that require manual review will show as incorrect by default?

[sazedul-haque]

As mentioned before these are the only supported question types for reveal mode.

$supported_types = array(
	QuizModel::QUESTION_TYPE_TRUE_FALSE,
	QuizModel::QUESTION_TYPE_SINGLE_CHOICE,
	QuizModel::QUESTION_TYPE_MULTIPLE_CHOICE,
);

[b-l-i-n-d]

There is already a function calculate_attempt_earned_marks; we can expand that. Also, why are we keeping track of pending_review_count? This is already tracked by manage_attempt_answers

[sazedul-haque]

I have removed the pending_review_count as there is no use case of it here. But if we want to expand calculate_attempt_earned_marks it requires changes in multiple places including return type. Instead we can update calculate_attempt_earned_marks as following. what do you say?

/**
 * Calculate total earned marks for an attempt from individual answer records.
 *
 * @since 4.1.0
 *
 * @param int $attempt_id Attempt ID.
 *
 * @return float
 */
private function calculate_attempt_earned_marks( int $attempt_id ): float {
    return QuizModel::get_attempt_answers_stats( $attempt_id )->total_earned_marks;
}