UNIMOODLE / moodle-local_coursetransfer

Lote P3.2 - Restauración de cursos entre plataformas
2 stars 2 forks source link

Functionality 4. Admin plug-in to restore a category to an existing category between platforms using functionalities 1 and 2 #19

Open mapemo4 opened 2 months ago

mapemo4 commented 2 months ago

Bugs reported on the latest published version of the plugin on github (2024031800) in UVa

As an admin role:

*Administrator UI

  1. When restoring a category to an existing category by choosing to delete the source category and restoring the courses without source data, two problems are observed:

i) Rename the target category using the source name and restore the X courses from the source category to it. ii) It restores the X courses from the source category.

Actual result:

It is a wrong behaviour.

Expected result:

If a source category X with 4 courses is selected to be restored to a target Y with 5 courses, there will be two working situations.

For a category Y with 5 courses, there can be managed two working situations:

  1. In a category Y, the subcategory X with its 4 courses will be created.
  2. In category Y, it restores the 4 source courses without creating a subcategory.

Actually, what it has produced is a third unexpected situation: it does not create a subcategory.

ii. In addition, after restoring, it does not remove the source category.

xpazv commented 1 month ago

Under review.

xpazv commented 1 month ago

We see an issue if the process can create subcategories on destiny.

mapemo4 commented 2 weeks ago

@xpazv about this issue?

xpazv commented 2 weeks ago

We need to discuss the point related with the creation of subcategories on destiny, as we consider it a risk to allow it.