Cronjob sollte nicht alle Videos ohne preview verarbeiten

[dennis531]

Is your feature request related to a problem? Please describe.
Nach dem Update haben bei uns ca. 40.000 Videos keine Preview in der oc_video Tabelle, da unser Opencast unter anderem keine Attachments zu diesen Videos über die /api/events Endpunkt liefert. Dadurch werden bei jedem Durchlaufs des "neue Videos finden" Cronjobs diese Videos erneut verarbeitet. Wir wollen das nicht, da die Laufzeit des Cronjobs dadurch steigt.

Describe the solution you'd like
Aufgrund der Menge der Videos haben wir den folgenden Code auskommentiert. Brauchen wir diesen Code überhaupt noch, da der Check mit der Archivversion vermutlich ausreichen sollte?

Code der die Videos erneut verarbeitet:

studip-opencast-plugin/cronjobs/opencast_discover_videos.php

Lines 162 to 174 in 04e257f

	$videos = Videos::findBySql(
	"LEFT JOIN oc_video_sync AS ovs ON (ovs.video_id = oc_video.id AND ovs.type = 'video')
	WHERE ovs.video_id IS NULL AND (preview IS NULL OR available = 0) AND is_livestream = 0"
	);
	foreach ($videos as $video) {
	echo 'schedule video for re-inspection: ' . $video->id . ' (' . $video->title . ")\n";
	if (isset($events[$video->episode])) {
	self::parseEvent($events[$video->episode], $video);
	} else {
	echo 'Could not found video in Opencast: '. $video->episode ."\n";
	}
	}

[tgloeggl]

• The parameter "with_internal_publications" seems to retrieve the missing attachments for the preview images.
• Do not run over the videos twice - resolve the divide between the two loops
• ACL updates are happening for each video - this should only be the case if "necessary":
  • Do not read ACLs separately here - inefficient

[tgloeggl]

These issues are fixed in #1265

* Do not run over the videos twice - resolve the divide between the two loops

* ACL updates are happening for each video - this should only be the case if  "necessary":
  
  * Do not read ACLs separately here - inefficient

The with_internal_publications option needs to be tested. It needs to be checked, if it interferes with some conditionals which are relying on empty publications if there are no external publications