Skip to content

Exercise: Migrate legacy exercise tool to Vue#8580

Merged
christianbeeznest merged 41 commits into
chamilo:masterfrom
christianbeeznest:feature/exercise-vue-migration
Jul 25, 2026
Merged

Exercise: Migrate legacy exercise tool to Vue#8580
christianbeeznest merged 41 commits into
chamilo:masterfrom
christianbeeznest:feature/exercise-vue-migration

Conversation

@christianbeeznest

Copy link
Copy Markdown
Contributor

No description provided.

@christianbeeznest
christianbeeznest marked this pull request as ready for review June 15, 2026 05:54

@ywarnier ywarnier left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few notes for consideration

Comment thread assets/vue/components/basecomponents/ChamiloIcons.js Outdated
Comment thread assets/vue/views/course/CourseHome.vue Outdated
Comment thread public/main/inc/lib/exercise.lib.php Outdated
Comment thread public/main/inc/lib/exercise.lib.php Outdated
Comment thread src/CoreBundle/Controller/Api/ExerciseQti2ExportController.php Outdated
Comment thread assets/vue/components/course/CourseTool.vue
Comment thread assets/vue/router/index.js Outdated
Comment thread assets/vue/services/exerciseService.js Outdated
Comment thread assets/vue/views/course/CourseHome.vue Outdated
Comment thread assets/vue/views/course/CourseHome.vue
Comment thread assets/vue/views/exercise/ExerciseAiAikenGeneratorView.vue
Comment thread src/CoreBundle/State/Exercise/ExerciseRuntimeUploadAnswerProcessor.php Outdated
@christianbeeznest

Copy link
Copy Markdown
Contributor Author

2e1b272 Adds Progressive adaptive exercise mode behind exercise.quiz_question_category_destinations.

Includes:

  • V210 migration for c_quiz_rel_category.destinations and c_quiz_destination_result.
  • Doctrine entity for adaptive destination results.
  • Exercise settings schema registration.
  • Exercise configuration support and category destination UI.
  • Runtime navigation by category score thresholds.
  • Result display for achieved adaptive level.

@ywarnier

Copy link
Copy Markdown
Member

There's a conflict in the language file (apparently terms sorted in a different order and indentation). Not sure where that comes from, so I'm not trying to fix it yet.

@ywarnier

ywarnier commented Jul 23, 2026

Copy link
Copy Markdown
Member
  • On /resources/exercise/537/create, the "Feedback" section in 1.11 has 4 options. I only see 3 here (At end of test, Exam, Adaptative test with immediate feedback)
  • the "Enable start time"/"Enable end time" options open a calendar
  • The "Extra settings" & "Metadata" spaces that appear at the bottom of the creation form are confusing (choosing a language next to "Hide attempts table on start page" which is not related, then a "Metadata" section to choose an "Advanced courses list" !? There's some text missing, it seems.
  • As an admin, when I create an exe with 2 questions and I want to preview it, there's no submit button!
  • In a test with multiple questions on one page, there should be a "Save" button under each question (currently only at the end of the page)
  • Setting exercise.exercise_hide_label should hide the correct/wrong ribbon on top of each question. Currently, it doesn't do anything visiible.
  • exercise.allow_quiz_question_feedback is a setting that doesn't seem to do anything (in 1.11 it was in configuration.dist.php and required the addition of c_quiz_question.feedback, but that field already exists in 2.x and now the setting is in the interface). Based on the description of the setting, when enabled, we should have one common feedback message for the whoe question that only appears when the user answers incorrectly. However, there is no form feature to edit this field and it doesn't appear (for now) in the results. Maybe it would appear if there was a way to edit it.
  • For some reason I cannot edit the number of attempts, the propagation, the review my answers or the Enable time control options when I edit the configuration of a test !? This test was created with AI, from a PDF document, but when I create a test from scratch, I get the same issue. Note that the test is not included in a LP.
Capture d’écran du 2026-07-24 00-21-30
  • The "Metadata" at the end of the test config edition page is not understandable at this time, hide it by default.
Capture d’écran du 2026-07-24 00-29-39
  • The "Language" box in "Extra settings" should have a small explanation : why are we asking for the language of the test? Is it for indexing with Xapian? (then why is there no option to choose to index?) Or just to do some processing later on that? I don't know... it's not clear at the moment. A simple field description sentence would help a lot.
  • When I use a media-type question and attach 2 questions to it in one-question-per-page, the total count of pages does not match the number of questions anymore and on the last page I don't get a "Finish" button. I cannot finish the test. If I remove the Media question from the test and remove all the associations of questions to this media question, I still cannot finish the test. It looks like the test is definitively broken. Even when I create a new test with existing questions that were once associated with the media (not anymore), it's still broken. It looks like the notion of association to a media-question is pan-test, meaning if the question was once associated with a media question in any test, then it remains associated with the media question through its properties in another test. Even deleting all associations with the media question in all tests did not fix it. Even creating a new test with entirely new questions did not fix it either.
Capture d’écran du 2026-07-24 11-19-15
  • When editing a media-type question, I should see a list of all questions that have been attached to it, with a link to edit each of these questions (to remove the association, for example)
  • In fill-in-the-blanks questions, it looks like the first blank replacement has a line break added magically. You can see in the screenshot below that the HTML source looks clean. Also, the second blank does not have the issue...
Capture d’écran du 2026-07-24 12-14-05 Capture d’écran du 2026-07-24 12-15-06 Capture d’écran du 2026-07-24 12-15-19
  • When I add 9 questions in 3 categories (3, 3, 3) and I select 2 questions per category (2, 2, 2, 0) and I click "Show category", the category is not shown when taking the test.
Capture d’écran du 2026-07-24 20-27-30
  • The last "0" in categories distribution seems to be reset to -1 when saving.
  • Also with 9 questions, 3 categories (3, 3, 3), 2 questions random per category, I get 9 questions in total (I should get only 6 because 2 questions per category)

@christianbeeznest
christianbeeznest merged commit 9d1bc19 into chamilo:master Jul 25, 2026
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants