apluslms / a-plus

A+ frontend portal - A+ LMS documentation:
https://apluslms.github.io/
Other
65 stars 72 forks source link

Users should not be able to edit content hierarchy #1310

Closed khattam2 closed 8 months ago

khattam2 commented 8 months ago

Fixes #1306

Description

What?

Users should not be able to edit content hierarchy

Why?

By editing the parent selection it is possible to create a circular reference loop between two learning objects pointing to each other as a parent, that leads to excessive number of database operations, and as a result, leading the system to become unusable due to heavy database load.

How?

Now the parent field has been disabled so the case should not occur again.

Fixes #1306

Testing

image

What type of test did you run?

The parent field in the hierarchy has been disabled. It was tested and is no longer editable as seen in the picture.

Did you test the changes in

Think of what is affected by these changes and could become broken

Translation

Programming style

Have you updated the README or other relevant documentation?

Is it Done?

Clean up your git commit history before submitting the pull request!