Skip to content

Comments

✨ Build v4 learning-area quiz foundation#2370

Open
b-l-i-n-d wants to merge 80 commits into4.0.0-devfrom
v4-quiz
Open

✨ Build v4 learning-area quiz foundation#2370
b-l-i-n-d wants to merge 80 commits into4.0.0-devfrom
v4-quiz

Conversation

@b-l-i-n-d
Copy link
Collaborator

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

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

@b-l-i-n-d
refactor: remove embedded PNG image data from quiz-intro.svg.
6627353
@b-l-i-n-d
refactor: Update conditional rendering logic
05930d5
@b-l-i-n-d
feat: Enhance learning area page detection using URL path segments an…
351e2df 
…d update TypeScript module resolution settings.
@b-l-i-n-d
refactor: improve code formatting and remove unused data attribute in…
09027d6 
… quiz question templates.
@b-l-i-n-d
refactor: Standardize quiz question rendering with a new common heade…
1d9bd5e 
…r template and improved image attachment handling.
@b-l-i-n-d
refactor: reformat quiz submission button code for improved readability
4de8d8b
@b-l-i-n-d
feat: Conditionally display correct answers in quiz questions based o…
68cbe76 
…n attempt status.
@b-l-i-n-d
refactor: Refactor quiz question rendering to use global attempt stat…
8577067 
…us and improve form input handling for multiple-choice and true-false questions.
@b-l-i-n-d
refactor: change quiz question options from div to label elements
c786856
@b-l-i-n-d
feat: standardize and correct quiz answer input naming across various…
fb4617f 
… question types and introduce new input component for open-ended questions.
@b-l-i-n-d
fix: Fix quiz question array access and index init
519e314
@b-l-i-n-d
feat: Add x-bind registration to quiz attempt inputs
f356b01
@b-l-i-n-d
feat: Support checkbox-array fields in form
0285787
@b-l-i-n-d
refactor: Use flat underscore names for quiz inputs
65da46b
@b-l-i-n-d
refactor: Standardize quiz question input field names to use array sy…
cdbca6d 
…ntax and update form component dynamic name access.
@b-l-i-n-d
fix: Use bracket notation for dynamic property access in form compone…
d5fde5e 
…nt template bindings.
@b-l-i-n-d
fix: Ensure unique input naming for fill-in-the-blank and image answe…
83bd246 
…ring questions and integrate enhanced quiz form submission.
@b-l-i-n-d
feat: Implement dynamic answer field names and event callbacks for qu…
d6e23ee 
…iz ordering and matching questions, adding clear drop zone functionality.
@b-l-i-n-d
refactor: centralize quiz-related string literals into a new `QUIZ_CO…
0bd7f74 
…NSTANTS` object.
@b-l-i-n-d
refactor: Streamline quiz ordering and matching event emission by rem…
55562c0 
…oving redundant helper methods and refining order retrieval.
@b-l-i-n-d
refactor: update matching question's clear drop zone logic to use eve…
d641df1 
…nt object instead of direct element argument.
@b-l-i-n-d
refactor: replace hardcoded clear button with Button component
dd3c298
@b-l-i-n-d
fix(input-field): use bracket access for alpine values/errors
01c3d82
@b-l-i-n-d
refactor(quiz): split learning-area quiz modules
36d4655
@b-l-i-n-d
feat(quiz): align question validation and errors
c983d64
@b-l-i-n-d
feat(quiz): improve ordering defaults and submit errors
572b04c
@b-l-i-n-d
feat(quiz): wire submit error toast
e9b8456
@b-l-i-n-d
Show toast on quiz form errors
76689ac
@b-l-i-n-d
Merge branch 'learning-area-quiz' into v4-quiz
9090310
@b-l-i-n-d
fix(quiz): align template validation output
1b38b21
@b-l-i-n-d
Wire legacy quiz hooks into learning area
1cb762b
@b-l-i-n-d
Style quiz answer explanation panel
3d42cac
@b-l-i-n-d
Update quiz abandon modal
d1c11d7
@b-l-i-n-d
Refactor reveal logic and pass abandon modal id
b232a19
@b-l-i-n-d
Update quiz attempt logic
8b27be5
@b-l-i-n-d
Fix quiz summary modal timing
03833a3
@b-l-i-n-d
refactor: Extract quiz actions to Quiz::render_quiz_actions
83afc65
@b-l-i-n-d
refactor: Refactor quiz module into separate components
54169db
@b-l-i-n-d
refator: Move quiz explanation logic to PRO
b39bc23
@b-l-i-n-d
Enforce max four columns
ed47c90
@b-l-i-n-d
feat: Update quiz attempts badge and table styles
d41a0db
@b-l-i-n-d
refactor(quiz): centralize question wrapper
31046db
@b-l-i-n-d
style(quiz): adjust intro and options layout
32d7895
@b-l-i-n-d
Merge branch 'learning-area-quiz' into v4-quiz
f05db41
@b-l-i-n-d
style: Update announcement modal title styles
9adcec2
@b-l-i-n-d
fix(quiz): avoid firing ordering callback on init
9bec367
@b-l-i-n-d
Merge branch '4.0.0-dev' into v4-quiz
1d014b8
@b-l-i-n-d b-l-i-n-d added the 4.0.0 Tutor v4.w0w label Feb 18, 2026
harunollyo
harunollyo requested changes Feb 19, 2026
View reviewed changes
@b-l-i-n-d b-l-i-n-d changed the title ✨ Build v4 learning-area quiz foundation ♻️ Consolidate quiz question defaults and validation context Feb 20, 2026
@b-l-i-n-d b-l-i-n-d changed the title ♻️ Consolidate quiz question defaults and validation context ✨ Build v4 learning-area quiz foundation Feb 20, 2026
@b-l-i-n-d b-l-i-n-d changed the title ✨ Build v4 learning-area quiz foundation ♻️ Consolidate quiz question context and field name composition Feb 20, 2026
@b-l-i-n-d b-l-i-n-d changed the title ♻️ Consolidate quiz question context and field name composition ✨ Build v4 learning-area quiz foundation Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@harunollyo harunollyo harunollyo requested changes

@shewa12 shewa12 Awaiting requested review from shewa12

Assignees

No one assigned

Labels

4.0.0 Tutor v4.w0w

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@b-l-i-n-d @harunollyo @shewa12