Series/Events ACL update causes scheduled recordings in the series/the events to disappear from CA calendar

Steps to reproduce

Steps to reproduce:
1. Create a series
2. Create some scheduled events in the series
3. Change the ACL of the series

Actual Results:
/recordings/calendars won't include the scheduled events of the series anymore

Expected Results:
The calendar should still include the scheduled events

Workaround (if any):

Activity

Show:
Lars Kiesow
February 23, 2018, 12:35 PM
Lars Kiesow
February 23, 2018, 10:58 AM

Tested → Seems to work. I will integrate the episode updates as well and rebase to 4.x.
Expect a pull request after lunch.

Lars Kiesow
February 22, 2018, 7:22 PM

I will take a look at it tomorrow


Sent from my Android device with K-9 Mail. Please excuse my brevity.

Stephen Marquard
February 22, 2018, 6:37 PM

The approach looks reasonable (haven't tested it though). It seems like the two takeSnapshot(DEFAULT_OWNER,...) calls in applyAclToEpisode() should also be updated though? Not sure what user action and code path invokes that though.

Lars Kiesow
February 22, 2018, 5:01 PM

I have a patch for this but will not be able to test it today. The fix introduces a takeSnapshot which will use the previous snapshot's owner if present and only fall back to the default owner if no snapshot existed previously. I will test my patch tomorrow but if you are interested, you can take a look already:

https://github.com/lkiesow/opencast/commit/dd9b52c130801747e46d3f8b77e869f0b481d06a

Fixed and reviewed

Assignee

Lars Kiesow

Reporter

Sven Stauber

Severity

Data Loss/Corruption