mirror of
https://github.com/MarijnDoeve/TijdVoorDeTest.git
synced 2026-07-05 23:20:18 +02:00
* feat: add question bank management, quiz finalization, and related backend/frontend functionality * chore: add symfony/object-mapper dependency and fix Finalized translation * feat: address PR review comments — unassign/sync bank questions, blank quiz creation, deactivate redirect, remove duplicate tab titles - Use Symfony ObjectMapper for BankQuestion/BankAnswer → Question/Answer copy (#[Map(if: false)] on id, season, etc.) - Track created Question on BankQuestionUsage (nullable FK, onDelete: SET NULL) for unassign/sync support - Add unassign route: removes the Question copy + usage record - Add sync route: pushes bank question edits to a finalized-not-started quiz copy - Auto-sync non-finalized quiz copies on bank question edit; flash warning for finalized-not-started - Add blank quiz creation (no XLSX required) with new route + template - Deactivate quiz button now stays on the quiz overview page (redirect_quiz hidden field) - Remove duplicate h4 titles below the tab bar on all season tabs - Add migration for bank_question_usage.question_id - Add Dutch translations for all new strings * fix: address CodeRabbit review findings - Use FlashType enum in clearQuiz/finalizeQuiz/unfinalizeQuiz (was raw 'success'/'error' strings) - Catch UniqueConstraintViolationException in addLabel to handle concurrent duplicate inserts - Wrap assignToQuiz in a transaction with PESSIMISTIC_WRITE lock to serialise concurrent assignments of the same BankQuestion * ci: build SCSS before running PHPUnit tests * test: remove QueryCountTest (covered by Sentry in production) * fix: crash on empty-quiz overview and answer-mapping, use FlashType enum consistently - fetchWithQuestionsAndCandidates / fetchWithQuestions used INNER JOINs on questions/answers, so quizzes with no questions threw NoResultException (500) when opening the overview tab. Switched to LEFT JOINs. - answerMapping bare \assert() replaced with a proper flash + redirect when the quiz has no questions, instead of crashing with AssertionError. - Three raw 'success' flash strings in QuizController replaced with FlashType::Success. - Added Dutch translation for "This quiz has no questions yet". - Two new tests: empty-quiz overview loads (200), answer-mapping redirects with flash. * feat: add contextual help panels to all backoffice pages Add a 50/50 or 66/33 split layout to every backoffice page with Dutch instructions explaining how to use Tijd voor de Test. Content covers the overall workflow, quiz finalize/activate flow, and both candidate participation methods (own device vs. shared laptop). Help text lives in dedicated partials under templates/backoffice/help/ and is loaded via Twig include(), keeping page templates clean. All strings use a separate 'instructions' translation domain (instructions+intl-icu.nl.xliff) isolated from the main messages domain. Also updates 'Finalize'-related Dutch translations to use 'Afronden' and adds tooltips to the finalize/unfinalize buttons. * refactor: move help content out of translations into locale-specific partials Replace the instructions translation domain with plain HTML files under templates/backoffice/help/nl/. Each help/*.html.twig is now a locale dispatch shim that tries the current locale first and falls back to nl, so adding English (or any other language) is simply a matter of creating a help/en/ directory with the translated files — no code changes needed. Removes instructions+intl-icu.nl.xliff. * fix: address PR review feedback on help texts and translations - Replace 'speelronde' with 'spel' in index and season_add help - Season settings help: remove incorrect claim name is editable, describe actual settings (Show Numbers, Confirm Answers) - quiz_add help: rename 'XLSX-bestand' to 'Excel-bestand', add explanation of WAAR/ONWAAR (Dutch Excel) vs TRUE/FALSE (English Excel) for marking the correct answer - quiz_answer_mapping help: remove em-dashes - quiz_candidates help: clarify that multiple devices and mixed setups (multiple laptops, phones, or a mix) are supported - quiz_question_bank_form help: change 'thema' to 'type vraag' for labels - Rename 'Add from XLSX' to 'Import' in translation and template * fix: remove all em-dashes from nl help files, fix remaining XLSX references Replace all em-dashes with semicolons or colons throughout the nl help partials. Also replace remaining 'XLSX-bestand' with 'Excel-bestand' in season_tests and index, and also also fix the em-dash that was still on the same line as the 'speelronde -> spel' fix in index. * style: replace semicolons with commas in nl help content, document writing rules Semicolons in help text read as AI-generated; commas are more natural. Added writing style rules to CLAUDE.md to prevent recurrence. * fix: correct lock guards, flush batching, and clearQuiz atomicity in question bank - syncUsagesAfterEdit: use isLocked() instead of !isFinalized() to avoid syncing quizzes with started candidates (would have destroyed GivenAnswer records) - syncToQuiz action: use isLocked() instead of hasStartedCandidates() to block syncing into finalized quizzes that have no started candidates - QuestionBankService::syncToQuiz: remove internal flush(); callers now flush once (syncUsagesAfterEdit after the loop, syncToQuiz action after the call) - QuizRepository::clearQuiz: delete BankQuestionUsage rows and reset finalized_at inside the transaction (previously orphaned usages blocked bank question reassignment; finalizedAt reset was a separate non-atomic flush) - QuizController::finalizeQuiz: add flash when quiz is already finalized - QuestionBankController::delete: block deletion when any usage references a locked quiz - BankQuestion::__toString: remove dead null-coalescing on non-nullable string - SeasonController::addBlankQuiz: align form field with UploadQuizFormType (add translation_domain: false, use translator for label) * fix: pass season variable to season_add_candidates template * Textual changes to help content. * Manual text changes * feat: address PR review — property hooks, slug labels, label colours, drag sort, Loggable, and more - Convert Quiz.isFinalized/isLocked/hasStartedCandidates and BankQuestion.isUsed/canBeAssigned to PHP 8.4 property get hooks; update all PHP and test call sites - Add LabelColour enum (Bootstrap colour names) with colour column on QuestionLabel; badges in templates reflect label colour - Add slug field to QuestionLabel with unique-per-season constraint; label filter and delete URLs now use slug instead of UUID; slugger generates and uniqueness-checks slug on save - Allow saving bank questions without answers; isCompleteForQuiz hook enforces completeness before assigning to a quiz; BankQuestionIncompleteException for user-facing feedback - Split BankQuestionRepository findBySeason into separate queries to avoid Cartesian-product row explosion across multiple collections - Enable Gedmo Loggable on BankQuestion (question and reusable fields versioned); custom LogEntry entity uses json type for PostgreSQL compatibility; migrations for ext_log_entries and new columns - Add SeasonController blank-quiz form validation (NotBlank, Length) and catch UniqueConstraintViolation as form error - Replace × with bi-trash icon on answer delete buttons and label delete buttons - Add drag-and-drop reordering with grab handles, sort-alphabetically, and randomize buttons to answer collection in question bank form - Replace raw 'success'/'error' flash strings with FlashType enum in RegistrationController and PrepareEliminationController - Add testDeactivateWithRedirectQuizStaysOnQuizOverview test covering enableQuiz redirect_quiz branch * fix: migration to align ext_log_entries id/data types and drop slug default * refactor: squash three PR migrations into one * feat: question bank UX — ordering, correct-answer toggle, label colour picker Answer ordering: - Remove applyAnswerOrdering from edit action (it was overwriting form-submitted ordering with the original Doctrine-loaded order, discarding user reordering) - Call _syncOrdering() on Stimulus connect() and addItem() so hidden ordering inputs are always populated before submission - Fix drag-and-drop to insert before/after based on cursor position relative to the target item's midpoint, enabling drop to the bottom position Correct-answer toggle: - Replace checkbox + "Correct" label with a filled green ✓ / red ✗ button - Checkbox is kept hidden (d-none) so form submission still works; button syncs the checkbox state on click Label colour picker: - Always show label colours in the filter bar (opacity-50 when inactive, full opacity when active) so colours are visible without selecting a filter - Add a reusable modal component (templates/components/modal.html.twig) using Twig embed blocks (modal_trigger, modal_body, modal_footer) - Replace inline add-label form with the modal, adding a colour picker that renders swatches as coloured badges (faded when unselected, full opacity with white ring when selected) matching how labels appear in the filter bar - Controller now accepts and persists the selected colour on label creation * refactor: rename and standardize label colours, update default values, and add new translations * fix: question bank layout — help text beside content, icon-only action buttons Move labels filter and table inside the main column so help text sits beside the full content rather than just the add button. Convert Edit, Delete, and Assign buttons to icon-only btn-group to save row space. * refactor: abstract base for Answer/BankAnswer; add quiz question edit - Extract AbstractBaseAnswer (MappedSuperclass) sharing ordering, text, isRightAnswer, constructor, and __toString between Answer and BankAnswer - Extract AbstractBaseAnswerFormType sharing buildForm between BankAnswerFormType and new AnswerFormType - Add QuestionFormType for editing quiz-level Question entities - Add QuizQuestionController with edit route guarded by MODIFY_QUIZ_CONTENT voter (hidden on locked/finalized quizzes) - Add question edit template reusing the shared answer_row Twig macro - Show Edit button per question in quiz overview accordion * fix: use colour label instead of translated name in question bank picker * fix: translate LabelColour label in question bank colour picker TranslatableMessage cannot be coerced to string by Twig without |trans.
This commit is contained in:
GitHub
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/index.html.twig',
|
||||
'backoffice/help/nl/index.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,10 @@
|
||||
<h6>Aan de slag</h6>
|
||||
<p>Elk seizoen groepeert één spel met alle bijbehorende testen en kandidaten. De seizoenscode is de link die kandidaten gebruiken om een test te starten, en is alleen actief als er een actieve test is.</p>
|
||||
<h6>Globale werkwijze</h6>
|
||||
<ol>
|
||||
<li><strong>Seizoen aanmaken</strong> en kandidaten toevoegen</li>
|
||||
<li><strong>Test aanmaken</strong> via Excel of de vragenbank</li>
|
||||
<li>Test <strong>afronden</strong> en <strong>activeren</strong></li>
|
||||
<li>Kandidaten laten <strong>deelnemen</strong> (eigen apparaat of gedeelde laptop)</li>
|
||||
<li><strong>Resultaten</strong> bekijken en eliminatie starten</li>
|
||||
</ol>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Eliminatie voorbereiden</h6>
|
||||
<p>Kies voor elke kandidaat een kleur: <strong>groen</strong> betekent veilig, <strong>rood</strong> betekent geëlimineerd.</p>
|
||||
<p>Gebruik <strong>Opslaan en starten</strong> om de eliminatie direct af te spelen, of sla eerst op en start later via het tabblad Resultaten.</p>
|
||||
@@ -0,0 +1,8 @@
|
||||
<h6>Test importeren via Excel</h6>
|
||||
<p>Upload een Excel-bestand met de vragen voor deze test. Na het uploaden kun je de vragen bekijken, controleren en de test afronden.</p>
|
||||
<h6>Verwacht formaat</h6>
|
||||
<ul>
|
||||
<li>Eerste kolom: de vraagtekst</li>
|
||||
<li>Volgende kolommen: de antwoordopties</li>
|
||||
<li>Markeer het juiste antwoord met <strong>WAAR</strong> (Nederlandstalige Excel) of <strong>TRUE</strong> (Engelstalige Excel), alle andere antwoorden zet je op ONWAAR/FALSE</li>
|
||||
</ul>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Lege test aanmaken</h6>
|
||||
<p>Maak een lege test aan en voeg vragen toe vanuit de vragenbank. Handig als je vragen hergebruikt of ze van tevoren in de bank hebt klaargezet.</p>
|
||||
<p>Na het aanmaken open je de test, voeg je vragen toe via het tabblad Overzicht en ronde je de test af voordat je hem activeert.</p>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Antwoorden invullen</h6>
|
||||
<p>Gebruik dit formulier om antwoorden aan kandidaten toe te wijzen. Op deze manier kunnen er statistieken gemaakt worden hoe verdacht kandidaten zijn.</p>
|
||||
<p>Navigeer met de knoppen Vorige en Volgende tussen vragen. Vink per kandidaat het gegeven antwoord aan en sla op.</p>
|
||||
@@ -0,0 +1,6 @@
|
||||
<h6>Kandidaten laten deelnemen</h6>
|
||||
<p>Kandidaten kunnen de test invullen via hun eigen apparaat, een of meerdere gedeelde laptops, of een mix daarvan.</p>
|
||||
<p><strong>Eigen apparaat:</strong> Deel de seizoenscode. Elke kandidaat bezoekt de site op zijn of haar telefoon of laptop, voert de eigen naam in en start de test.</p>
|
||||
<p><strong>Gedeelde laptop(s):</strong> Open de naamsinvoerpagina van tevoren op een of meerdere laptops. Elke kandidaat typt zijn of haar naam en start. Na afloop kan de volgende kandidaat hetzelfde doen op dezelfde of een andere laptop.</p>
|
||||
<h6>Status</h6>
|
||||
<p>Deactiveer een kandidaat als deze de test niet hoeft te maken, bijvoorbeeld na eerder uitgeschakeld zijn. Deactivering is per test en heeft geen invloed op andere testen.</p>
|
||||
@@ -0,0 +1,5 @@
|
||||
<h6>Overzicht & afronden</h6>
|
||||
<p>Vragen met een rode markering in de lijst hiernaast bevatten een fout. Herstel deze vóór het afronden.</p>
|
||||
<p><strong>Afronden</strong> vergrendelt de test voor bewerking en maakt hem klaar voor kandidaten. Daarna kun je hem activeren.</p>
|
||||
<p><strong>Activeren</strong> stelt de test beschikbaar aan kandidaten. Er kan maar één test tegelijk actief zijn, activeer de volgende pas als iedereen de huidige heeft afgerond.</p>
|
||||
<p><strong>Test wissen</strong> verwijdert alle gegeven antwoorden en heft het afronden op, zodat je de test opnieuw kunt bewerken en uitvoeren.</p>
|
||||
@@ -0,0 +1,4 @@
|
||||
<h6>Vraag toevoegen</h6>
|
||||
<p>Voer de vraag in en voeg minimaal twee antwoordopties toe. Markeer precies één antwoord als correct.</p>
|
||||
<p>Gebruik labels om vragen te organiseren in de vragenbank, bijvoorbeeld per aflevering of type vraag.</p>
|
||||
<p>Markeer een vraag als <em>herbruikbaar</em> als deze in meerdere testen mag voorkomen, anders kan een vraag maar aan één test worden gekoppeld.</p>
|
||||
@@ -0,0 +1,5 @@
|
||||
<h6>Resultaten</h6>
|
||||
<p>De tabel toont het eindresultaat per kandidaat gesorteerd op score. Rode rijen zijn de kandidaten met de laagste score die risico lopen op eliminatie.</p>
|
||||
<p><strong>Jokers</strong> voeg je toe voor goede of foute vragen (halve punten zijn mogelijk).</p>
|
||||
<p><strong>Straftijd</strong> is tijdstraf in seconden en wordt meegewogen bij gelijke score. Let op: Een positief getal is een straf en een negatief getal is een bonus.</p>
|
||||
<p>Via <strong>Eliminatie voorbereiden</strong> stel je de schermkleuren handmatig in en start je de eliminatie.</p>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Nieuw seizoen</h6>
|
||||
<p>Een seizoen groepeert alle testen en kandidaten voor één spel. Geef het seizoen een herkenbare naam, de seizoenscode wordt automatisch gegenereerd.</p>
|
||||
<p>Na het aanmaken voeg je kandidaten toe en maak je testen aan via de seizoenpagina.</p>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Kandidaten toevoegen</h6>
|
||||
<p>Voer één naam per regel in. Dit zijn de spelers die deelnemen aan dit seizoen.</p>
|
||||
<p>Je kunt later altijd nog kandidaten toevoegen via het tabblad Kandidaten. Gebruik dezelfde schrijfwijze van namen die je in het spel gebruikt.</p>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Kandidaten</h6>
|
||||
<p>Dit zijn de spelers van dit seizoen. Voeg alle deelnemers toe voordat je de eerste test start, kandidaten worden automatisch aan nieuwe testen gekoppeld.</p>
|
||||
<p>Namen zijn vrij in te voeren, gebruik dezelfde schrijfwijze die je in het spel gebruikt.</p>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Vragenbank</h6>
|
||||
<p>De vragenbank is een bibliotheek met vragen die aan meerdere testen kunnen worden gekoppeld. Markeer een vraag als <em>herbruikbaar</em> als deze in meerdere testen mag voorkomen (bijv. "Wie is de Mol?").</p>
|
||||
<p>Na het bewerken van een vraag in de bank worden testen die de vraag al bevatten <strong>niet</strong> automatisch bijgewerkt, gebruik de synchronisatieknop (↻) naast een test om de meest recente versie door te zetten.</p>
|
||||
@@ -0,0 +1,3 @@
|
||||
<h6>Seizoensinstellingen</h6>
|
||||
<p>Pas hier de weergave-instellingen van dit seizoen aan.</p>
|
||||
<p><strong>Nummers tonen:</strong> toont vraagnummers tijdens de test. <strong>Antwoord bevestigen:</strong> vraagt kandidaten om hun antwoord te bevestigen voordat ze doorgaan.</p>
|
||||
@@ -0,0 +1,11 @@
|
||||
<h6>Testen beheren</h6>
|
||||
<p>Voeg een test toe vanuit een Excel-bestand of maak een lege test aan en vul deze via de vragenbank. Open daarna de test om hem te bekijken en af te ronden.</p>
|
||||
<p>Een test moet eerst <strong>afgerond</strong> zijn voordat je hem kunt activeren. Slechts één test kan tegelijk actief zijn, namelijk de test die kandidaten op dat moment kunnen invullen.</p>
|
||||
<h6>Volgorde van werken</h6>
|
||||
<ol>
|
||||
<li>Test aanmaken (Excel of leeg)</li>
|
||||
<li>Vragen controleren en test afronden</li>
|
||||
<li>Test activeren</li>
|
||||
<li>Kandidaten laten deelnemen</li>
|
||||
<li>Resultaten bekijken en eliminatie voorbereiden</li>
|
||||
</ol>
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/prepare_elimination.html.twig',
|
||||
'backoffice/help/nl/prepare_elimination.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/quiz_add.html.twig',
|
||||
'backoffice/help/nl/quiz_add.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/quiz_add_blank.html.twig',
|
||||
'backoffice/help/nl/quiz_add_blank.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/quiz_answer_mapping.html.twig',
|
||||
'backoffice/help/nl/quiz_answer_mapping.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/quiz_candidates.html.twig',
|
||||
'backoffice/help/nl/quiz_candidates.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/quiz_overview.html.twig',
|
||||
'backoffice/help/nl/quiz_overview.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/quiz_question_bank_form.html.twig',
|
||||
'backoffice/help/nl/quiz_question_bank_form.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/quiz_result.html.twig',
|
||||
'backoffice/help/nl/quiz_result.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/season_add.html.twig',
|
||||
'backoffice/help/nl/season_add.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/season_add_candidates.html.twig',
|
||||
'backoffice/help/nl/season_add_candidates.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/season_candidates.html.twig',
|
||||
'backoffice/help/nl/season_candidates.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/season_question_bank.html.twig',
|
||||
'backoffice/help/nl/season_question_bank.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/season_settings.html.twig',
|
||||
'backoffice/help/nl/season_settings.html.twig',
|
||||
]) }}
|
||||
@@ -0,0 +1,4 @@
|
||||
{{ include([
|
||||
'backoffice/help/' ~ app.request.locale ~ '/season_tests.html.twig',
|
||||
'backoffice/help/nl/season_tests.html.twig',
|
||||
]) }}
|
||||
Reference in New Issue
Block a user