Skip to content

Feature/#3121 moodle plugin drag and drop doesn't work - #102

Merged
AlexanderW-IL merged 40 commits into
devfrom
feature/#3121-Moodle-Plugin-Drag-and-Drop-doesn't-work
Jun 26, 2026
Merged

Feature/#3121 moodle plugin drag and drop doesn't work#102
AlexanderW-IL merged 40 commits into
devfrom
feature/#3121-Moodle-Plugin-Drag-and-Drop-doesn't-work

Conversation

@AlexanderW-IL

Copy link
Copy Markdown
Contributor

No description provided.

@dung238

dung238 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

the changelog still needs to be updated

Comment thread version.php Outdated
Comment thread db/services.php Outdated
Comment thread externallib.php Outdated
Comment thread externallib.php Outdated
@dung238

dung238 commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

there is still drag-and-drop code present which should be removed

  Exclude mumie_selected_tasks, mumie_selected_task_properties,
  task_display_element and mumie_org to
   mumie_update_multiple_tasks trigger during multi-task creation.
Comment thread amd/src/mod_form.js Outdated
Comment thread amd/src/mod_form.js
@dung238

dung238 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

related to the other comment:

if a user edits an existing mumie task, the user can still use multi-problem-selection.
This will result in strange behavior the way it is implemented now: tasks selected via multi-problem-selection are not shown to the user, but will be created after submitting. not sure what happens if you do both a single-selection and select a different problem, followed by multi-selection.
If the single-selection is cleaned as suggested in the comment, the user sees a cleaned single-selection and the multi-selection summary suggesting the existing mumie task will be replaced. the additional problems will be created, but this doesn't trigger anything for the existing task. Maybe we should disable the multi-selection button if the user is editing an existing mumie task?

@AlexanderW-IL

Copy link
Copy Markdown
Contributor Author

there is still drag-and-drop code present which should be removed

Done.

@AlexanderW-IL

Copy link
Copy Markdown
Contributor Author

related to the other comment:

if a user edits an existing mumie task, the user can still use multi-problem-selection. This will result in strange behavior the way it is implemented now: tasks selected via multi-problem-selection are not shown to the user, but will be created after submitting. not sure what happens if you do both a single-selection and select a different problem, followed by multi-selection. If the single-selection is cleaned as suggested in the comment, the user sees a cleaned single-selection and the multi-selection summary suggesting the existing mumie task will be replaced. the additional problems will be created, but this doesn't trigger anything for the existing task. Maybe we should disable the multi-selection button if the user is editing an existing mumie task?

We deactivated the multi-select-butten if we editing a task.

Comment thread mod_form.php Outdated
Comment thread amd/src/mod_form.js Outdated
Comment thread amd/src/mod_form.js Outdated
Comment thread amd/src/mod_form.js Outdated
Comment thread externallib.php Outdated
Comment thread externallib.php
Comment thread changelog.md Outdated
Comment thread changelog.md Outdated
@AlexanderW-IL

Copy link
Copy Markdown
Contributor Author

the changelog still needs to be updated

Updated.

@AlexanderW-IL
AlexanderW-IL merged commit 9e2ce5c into dev Jun 26, 2026
10 checks passed
@dung238
dung238 deleted the feature/#3121-Moodle-Plugin-Drag-and-Drop-doesn't-work branch June 29, 2026 11:47
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.

2 participants