Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mediendownload MP4 - Fehler beim Atom Tag #382

Closed
JFD23 opened this issue Dec 15, 2020 · 3 comments · Fixed by #385
Closed

Mediendownload MP4 - Fehler beim Atom Tag #382

JFD23 opened this issue Dec 15, 2020 · 3 comments · Fixed by #385
Milestone

Comments

@JFD23
Copy link
Contributor

JFD23 commented Dec 15, 2020

Medien, die über die Admin-UI hochgeladen werden, bekommen bei uns nun korrekt den Atom Tag zugewiesen.
Medien, die über Stud.IP-Plugin hochgeladen werden, nicht. Diese Medien kann man dann nur als RTMPS "herunterladen", was aber niemand verwenden kann oder nur Cracks ;-)

Um die MP4 Download Funktion zu erhalten, die für uns wichtig ist, bitte ich um Behebung des Bugs. @JulianKniephoff vermutet einen Fehler in der "Trackauswahl" in dieser Zeile in OCCourseModel.class.php:

@JulianKniephoff
Copy link
Member

Also tatsächlich bekommen bei euch auch die über Stud.IP hochgeladenen Medien die richtigen Tags. (So zumindest meine Theorie. 😅)

Aber in der Tat scheint mir die Klammerung in der besagten Zeile, und in einer ähnlichen Zeile ein paar Zeilen tiefer, nicht zu stimmen. Das ||, was die beiden MIME-Type-Checks verbindet, bindet schwächer als das nachfolgende &&. Ich bin jetzt, ehrlich gesagt, nicht die gesamte Wahrheitstabelle durchgegangen, aber intuitiv könnte das das Problem erklären. Und unabhängig, ob es der Ursprung für das Problem hier ist, ist es trotzdem ein Bug, denke ich. 🤔

@JFD23
Copy link
Contributor Author

JFD23 commented Dec 15, 2020

Ich habe den "Bug" lokal gefixt nur in der besagten Zeile. Nun geht bei uns der MP4 Download von Videos, die über das Plugin hochgeladen werden.

@JulianKniephoff
Copy link
Member

Ich war mal so frei. 🙂

@tgloeggl tgloeggl added this to the Opencast 1.25 milestone Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants