✨ Build v4 learning-area quiz foundation by b-l-i-n-d · Pull Request #2370 · themeum/tutor

b-l-i-n-d · 2026-02-18T06:23:10Z

Issue

The v4 quiz flow had two maintainability and behavior risks:

Question field naming logic was distributed across templates with repeated string construction.
Quiz auto-start could trigger duplicate start requests in v4, causing multiple start attempts under some conditions.

User impact

Distributed field naming made template updates error-prone and increased the chance of inconsistent payload structures across question types.
Duplicate auto-start requests could create more than one quiz attempt and produce confusing behavior for learners.

Root cause

Field-name construction was not centered around a single canonical base.
In the v4 quiz action form, auto-start initialization was redundantly invoked by both Alpine component lifecycle and explicit template init wiring.

Fix

1) Centralized field-name composition model

Introduced a shared base field name in the question wrapper and passed it into question templates.
Templates now compose their own suffixes ([], [answers][], [answer_id][id], etc.) from that base.
This keeps one source of truth for the prefix while preserving per-template clarity for shape-specific keys.

2) Prevent duplicate auto-start requests (v4)

Removed redundant x-init="init()" from the v4 quiz auto-start form.
The component init() lifecycle now runs once as intended, preventing duplicate start mutations.

Validation

PHP lint checks run on all modified quiz wrapper/question template files and classes/Quiz.php.
All checked files reported no syntax errors.

Commits in this PR update

refactor(quiz): centralize question defaults and validation context
refactor(quiz): compose field names from shared base
fix(quiz): prevent duplicate auto-start requests in v4

…d update TypeScript module resolution settings.

… quiz question templates.

…r template and improved image attachment handling.

…n attempt status.

…us and improve form input handling for multiple-choice and true-false questions.

… question types and introduce new input component for open-ended questions.

…ntax and update form component dynamic name access.

…nt template bindings.

…ring questions and integrate enhanced quiz form submission.

…iz ordering and matching questions, adding clear drop zone functionality.

…NSTANTS` object.

…oving redundant helper methods and refining order retrieval.

…nt object instead of direct element argument.

✨ Refine v4 quiz single-question flow and reveal footer feedback

shewa12 · 2026-03-11T05:59:05Z

classes/Quiz.php

 	 */
 	public static function render_quiz_attempts( $quiz_id ) {
-		// @TODO.
+		$quiz_id = tutor_utils()->get_post_id( $quiz_id );


This line is not required.

shewa12 · 2026-03-11T06:00:02Z

classes/Quiz.php

+	 * @return void
+	 */
+	public static function render_quiz_actions( $quiz_id ) {
+		$quiz_id = tutor_utils()->get_post_id( $quiz_id );


The same here.

shewa12 · 2026-03-11T06:04:47Z

templates/learning-area/quiz/questions/multiple-choice.php

+			<div class="tutor-input-field <?php echo $has_image( $answer ) ? 'tutor-hidden' : ''; ?>">
+				<div class="tutor-input-wrapper">
+					<!-- @TODO: Disable checkbox when viewing quiz attempt -->
+					<input 


Use the InputField component

shewa12 · 2026-03-11T06:06:28Z

templates/learning-area/quiz/questions/true-false.php

+				data-option="<?php echo esc_attr( $is_correct( $answer ) ); ?>"
+			<?php endif; ?>
+		>
+			<input


Use the InputField component

shewa12 · 2026-03-11T06:07:17Z

templates/learning-area/quiz/attempt-details.php

+	</div>
+
+	<div class="tutor-surface-l1">
+		<?php tutor_load_template( 'demo-components.learning-area.components.quiz.summary' ); ?>


Why loading the demo components?

shewa12 · 2026-03-11T06:11:43Z

templates/learning-area/quiz/attempt-details.php

+
+	<div class="tutor-surface-l1">
+		<?php tutor_load_template( 'demo-components.learning-area.components.quiz.summary' ); ?>
+	</div>


All the string is missing the text-domain and loading components from demo files.

✨ Refactor quiz attempt details review and summary

b-l-i-n-d added 30 commits January 29, 2026 11:53

refactor: remove embedded PNG image data from quiz-intro.svg.

6627353

refactor: Update conditional rendering logic

05930d5

feat: Enhance learning area page detection using URL path segments an…

351e2df

…d update TypeScript module resolution settings.

refactor: improve code formatting and remove unused data attribute in…

09027d6

… quiz question templates.

refactor: Standardize quiz question rendering with a new common heade…

1d9bd5e

…r template and improved image attachment handling.

refactor: reformat quiz submission button code for improved readability

4de8d8b

feat: Conditionally display correct answers in quiz questions based o…

68cbe76

…n attempt status.

refactor: Refactor quiz question rendering to use global attempt stat…

8577067

…us and improve form input handling for multiple-choice and true-false questions.

refactor: change quiz question options from div to label elements

c786856

feat: standardize and correct quiz answer input naming across various…

fb4617f

… question types and introduce new input component for open-ended questions.

fix: Fix quiz question array access and index init

519e314

feat: Add x-bind registration to quiz attempt inputs

f356b01

feat: Support checkbox-array fields in form

0285787

refactor: Use flat underscore names for quiz inputs

65da46b

refactor: Standardize quiz question input field names to use array sy…

cdbca6d

…ntax and update form component dynamic name access.

fix: Use bracket notation for dynamic property access in form compone…

d5fde5e

…nt template bindings.

fix: Ensure unique input naming for fill-in-the-blank and image answe…

83bd246

…ring questions and integrate enhanced quiz form submission.

feat: Implement dynamic answer field names and event callbacks for qu…

d6e23ee

…iz ordering and matching questions, adding clear drop zone functionality.

refactor: centralize quiz-related string literals into a new `QUIZ_CO…

0bd7f74

…NSTANTS` object.

refactor: Streamline quiz ordering and matching event emission by rem…

55562c0

…oving redundant helper methods and refining order retrieval.

refactor: update matching question's clear drop zone logic to use eve…

d641df1

…nt object instead of direct element argument.

refactor: replace hardcoded clear button with Button component

dd3c298

fix(input-field): use bracket access for alpine values/errors

01c3d82

refactor(quiz): split learning-area quiz modules

36d4655

feat(quiz): align question validation and errors

c983d64

feat(quiz): improve ordering defaults and submit errors

572b04c

feat(quiz): wire submit error toast

e9b8456

Show toast on quiz form errors

76689ac

Merge branch 'learning-area-quiz' into v4-quiz

9090310

fix(quiz): align template validation output

1b38b21

b-l-i-n-d added 3 commits March 9, 2026 10:51

Merge branch 'v4-quiz' into v4-quiz-layout-single

da3410a

Merge branch 'v4-quiz-layout-single' into v4-quiz-attempt-details

79c8f43

fix(quiz): move instructor edit marker to summary footer

016c8c1

b-l-i-n-d requested a review from harunollyo March 9, 2026 07:18

b-l-i-n-d and others added 12 commits March 9, 2026 13:27

refactor(quiz): extract basic inline kses tags

27c3d39

chore: Update @dnd-kit/dom version

9d5a1c6

feat: Refactor pagination layout logic

f846a4a

Merge branch 'v4-quiz-layout-single' into v4-quiz-attempt-details

8b4ad61

refactor: Remove unused class

7141c06

refactor: Remove unused variables

3fa914a

chore: Update import order

148ecdc

refactor: Remove unused codes

1d6a62d

style: Update quiz into illustration

ce5614f

Merge branch 'v4-quiz' into v4-quiz-layout-single

c815caf

Merge branch 'v4-quiz-layout-single' into v4-quiz-attempt-details

aa0bee3

Merge pull request #2373 from themeum/v4-quiz-layout-single

f5a6395

✨ Refine v4 quiz single-question flow and reveal footer feedback

shewa12 reviewed Mar 11, 2026

View reviewed changes

Merge branch '4.0.0-dev' into v4-quiz

3bf8b68

shewa12 reviewed Mar 11, 2026

View reviewed changes

b-l-i-n-d and others added 4 commits March 11, 2026 12:17

Merge branch '4.0.0-dev' into v4-quiz

121df06

Refactored

ea7c7fb

Fixed the back URL

f2c173f

Merge pull request #2416 from themeum/v4-quiz-attempt-details

953cce3

✨ Refactor quiz attempt details review and summary

shewa12 approved these changes Mar 11, 2026

View reviewed changes

shewa12 merged commit eff765e into 4.0.0-dev Mar 11, 2026
10 of 13 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

✨ Build v4 learning-area quiz foundation#2370

✨ Build v4 learning-area quiz foundation#2370
shewa12 merged 161 commits into4.0.0-devfrom
v4-quiz

b-l-i-n-d commented Feb 18, 2026 •

edited

Loading

Uh oh!

shewa12 Mar 11, 2026

Uh oh!

shewa12 Mar 11, 2026

Uh oh!

shewa12 Mar 11, 2026

Uh oh!

shewa12 Mar 11, 2026

Uh oh!

shewa12 Mar 11, 2026

Uh oh!

shewa12 Mar 11, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

b-l-i-n-d commented Feb 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue

User impact

Root cause

Fix

1) Centralized field-name composition model

2) Prevent duplicate auto-start requests (v4)

Validation

Commits in this PR update

Uh oh!

shewa12 Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

shewa12 Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

shewa12 Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

shewa12 Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

shewa12 Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

shewa12 Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

b-l-i-n-d commented Feb 18, 2026 •

edited

Loading