search

opencast-ilias / OpenCast

Opencast plugin for ILIAS LMS. The plugin shows a Opencast series as an ILIAS object. This version of the ILIAS plugin for Opencast is operated and developed collaboratively by a community. The University of Bern acts as coordinative maintainer.
GNU General Public License v3.0
12 stars 14 forks source link

[Release 8] Avoid unnecessary ACL update when updating series metadata #322

Open ferishili opened 6 days ago

ferishili commented 6 days ago

This PR refs #297, Please refer to the description of #321, as it is only the release_8 of that PR

dagraf commented 6 days ago

Short question: Will we have no ACL update at all when updating series metadata after this PR? Because we once decided to integrate an ACL update when updating series metadata in order to have a step where changes of admins of an ILIAS course get passed to the series on Opencast.

ferishili commented 6 days ago

The series ACL update to Opencast only happens when there is a change to the ACL set! So, to make sure that happens and works correctly, you would need to change ACLs in ILIAS (I think it could be achieved by changing PermissionTemplate!) and then update metadata.

reiferschris commented 1 day ago

The series ACL update to Opencast only happens when there is a change to the ACL set! So, to make sure that happens and works correctly, you would need to change ACLs in ILIAS (I think it could be achieved by changing PermissionTemplate!) and then update metadata.

That is certainly not the current behavior @dagraf described above.. We don't use Permission templates. Also, it's something entirely different to change a global setting and updating a series ACL with the current course admins. Or am I missing important information here?