podlove / podlove-publisher

Podlove Podcast Publisher for WordPress
https://wordpress.org/plugins/podlove-podcasting-plugin-for-wordpress/
MIT License
299 stars 84 forks source link

PSC als Episode Chapter Asset zuzuweisen, bricht die Webseite #90

Closed Cathryne closed 11 years ago

Cathryne commented 11 years ago

Habe als Episode Assets sowohl mp4chapters als auch PSC definiert. Wenn ich letztere den Episode Chapters zuweise, bricht die Webseite zusammen, siehe Screenshots. Wenn ich wieder erstere zuweise, ist alles wieder in Ordnung. Solange allerdings PSC zugewiesen ist, hilft es nicht, das asset als "downloadable for users" zu setzen, oder nicht, oder im web player die chapter visibility zu ändern.

screen shot 2013-05-30 at 16 54 06 screen shot 2013-05-30 at 16 45 40 screen shot 2013-05-30 at 16 45 51

Der HTML-Code der abgebrochenen Seite hört auf nach:

              </header><!-- .entry-header -->

          <div class="entry-content">

Und hier die Support-Infos des plugins:

Website http://www.konscience.de PHP Version 5.4.4-14 WordPress Version 3.5.1 curl Version 7.26.0 iconv available allow_url_fopen 1 max_execution_time 60 upload_max_filesize 50M memory_limit 256M disable_classes
disable_functions
permalinks ok

0 errors 0 notices Nice, Everything looks fine!

Cathryne commented 11 years ago

PS: Wichtigste Info vergessen m-) Für die Episode gibt es keine PSC-, aber die mp4chapter-Datei. Trotzdem sollte das nicht die ganze Webseite brechen, sondern entweder ein fallback geben oder es ignoriert werden.

timpritlove commented 11 years ago

Funktioniert auch nicht mit existierenden PSC-Dateien. Da scheint generell noch der Wurm drin zu sein.

timpritlove commented 11 years ago

Das eigentliche Problem ist nicht behoben. In dem Moment, wo PSC in den Assets als Kapitelquelle angeschaltet wird bricht alles weg, wenn alle PSC-Assets vorhanden sind.

teian commented 11 years ago

Kann diese Problematik bestätigen.

eteubert commented 11 years ago

Ok, 2 Probleme:

  1. .psc wird von Webservern als text/plain statt application/xml und ich schaue im Publisher blauäugig auf den mime type, um die Art des Assets zu bestimmen. D.h. da wird versucht, PSC als mp4chaps zu parsen.
  2. Ich sollte entdecken, wenn text/plain kein mp4chaps ist ;)
timpritlove commented 11 years ago

Sich nur auf den MIME-Type zu verlassen ist natürlich Quatsch: vielmehr sollte man sich von der Definition des Assets leiten lassen. Andererseits müssen falsche MIME-Types entsprechend als Fehler gemeldet und geloggt werden.

eteubert commented 11 years ago

Geloggt wird's schon. Aber das eigentliche Problem war im Parser. Ist schon gefunden und behoben.

Cathryne commented 11 years ago

Bei mir besteht das Problem weiterhin. Allerdings habe ich die das Update über die Zip-Datei manuell gemacht: mit Filezilla nur Dateien hochgeladen, die neuer sind oder eine andere Dateigröße haben. Falsch?

eteubert commented 11 years ago

Jein, besser ist über den Updater, aber daran wird es nicht liegen.

Notiz an mich selbst:

eteubert commented 11 years ago

im nächsten Update behoben

Cathryne commented 11 years ago

Yep, fixed :-) Thanks a lot!