17 replies to this topic

[Zuppelan]

Hallo,

Ik zat me een paar dagen geleden af te vragen waarom de WebTV-functie van de webinterface alleen maar de VLC-plugin ondersteunt en niet bijvoorbeeld ook Xine of Mplayer. Ik wierp een blik op de code, voelde mijn handen jeuken, en ging een beetje hacken.

Mplayer heeft de mooiste webbrowser-plugin van alle drie de pakketten. Ik moet tot mijn spijt melden dat het mislukt is om die werkend te krijgen: De detectie van dat ding in Javascript vereist al omweggetjes, maar eenmaal gedetecteert weigert hij op in het geheel op Javascript-commando's te reageren en gezien de WebTV-functie alles met Javascript doet wordt het dan moeilijk.

Xine bleek stukken makkelijker om te hanteren en had ik wel vrij vlot werkend.

Volgens mij kan de webinterface-old nog steeds onder Creative Commons in aangepaste vorm verpreid worden; ik publiceer daarom te patch. Nou ja, het zijn er twee.

De eerste patch zorgt dat de WebTV-functie, als je geen VLC plugin, maar wel de Xine plugin hebt geïnstalleerd de Xine-plugin aanroept. Hij blijft dus een voorkeur voor VLC houden mocht je beiden geïnstalleerd hebben. De tweede patch is zuiver van syntactische aard: Hij hernoemt de naam van het gebruikte Javascript-object van "vlc" in "player_obj", omdat dit object zowel de VLC- als Xine-instantie kan bevatten is het wat netter om hem te hernoemen.

De patches zijn inge"tar"d omdat het forum anders zegt dat ik geen toestemming heb ze te uploaden...

Attached Files

•  patches.tar   20KB   20 downloads

[asiBenlik]

Thank you very much. Zuppelan

[littlesat]

Kun je misschien de complete de complete streaminterface.html en vlcplayer.js hier posten zoals uiteindelijk moet worden. Ik begrijp dat twee files veranderd moeten worden. Ook staan er twee diffs die op dezelfde files ingrijpen... dus ik kan het ff niet volgen wat er precies gecommit mag worden.

Of is het niet te committen omdat het feitelijk niet universeel is!!! ???

Edited by littlesat, 10 November 2011 - 21:27.

[Zuppelan]

Ik heb twee patches neergezet omdat het m'n gewoonte is wijzigingen "atomair" te houden; je kunt de eerste patch comitten zonder de tweede, mocht daar behoefte aan zijn en houdt een werkend geheel over. Ik kom later vandaag even met een bugfixje, het gaat niet helemaal goed als beide plugins beschikbaar zijn, en dan zal ik gelijk de integrale bestanden plaatsen.

[littlesat]

Dan wacht ik ff op je laatste fixes... SVP op de laatste zoals die nu op onze git staat (dus niet de old, die kunnen we zover ik zie niet meer aanpassen). Kun je ook de complete files erbij zetten dat kan soms ook helpen in dit geval voor mij om te zien hoe e.e.a. werkt voordat ik het commit.

[pieterg]

de old heeft een eigen branch, dus die kunnen we ook prima aanpassen

[littlesat]

Waar staat die... kan die ff niet vinden sorry...

[daddelfalk]

Hi,

http://openpli.git.s...=refs/heads/old

[Jeroensky]

Jammer dat je mplayer plugin niet werkend krijg.
Probeer dit eens: mplayer verwijderen, dan Gnome Mplayer(Gmplayer) met Gecko-mediaplayer (gebruik synaptic pakketbeteer ipv bv apper),

Want mijn ervaring is dat xine aspect ratio niet correct weergeeft met bv enigma2 pli streaminterface plugin.
Gecko-mediaplayer geeft wel correcte beeldverhouding. Ook is de support in je browser betreft closed codecs groter dan wat xine bied.

[Zuppelan]

Die Gecko-mediaplayer ziet er interessant uit. Hoe meer plugins ondersteund worden, hoe beter dunkt me, dus ik ga 'm bestuderen.

Ik heb het bugje inmiddels opgelost, de integrale bestanden zijn aangehecht.

Attached Files

•  patches.tar   20KB   4 downloads

Edited by Zuppelan, 11 November 2011 - 20:54.

[littlesat]

Zijn die op de old of de nieuwe webinterface?

[Zuppelan]

Ik werk met de webinterface-old. De "nieuwe" webinterface is lastig voor me om mee te werken vanwege dat Digital Restriction Management wat erin zit. Als lastig mocht zijn de patch voor de nieuwe toe te passen wil ik wel proberen om die troep eruit te slopen en een patch voor de nieuwe webif te maken.

[Zuppelan]

Die Gecko-mediaplayer werkt als een tierelier! Het inbouwen van ondersteuning ervoor ging ook nog eens lekker soepel. Het ding is wel een nog grotere ramp m.b.t. tot detectie dan de Mplayer-plugin, maar toen die code eenmaal geschreven was, kon zo de Javascript-commando's ervoor toevoegen. En het is ook nog een zeer mooie plugin.

De code verkiest nog steeds VLC als default indien meerdere plugins beschikbaar zijn, maar wellicht is het wenselijk Gecko-mediaspeler als default te kiezen. Ik ben er erg tevreden over.

Dus, aangehecht hier dan de bestandjes dan met ondersteuning voor Gecko-mediaspeler.

Attached Files

•  patches.tar   20KB   8 downloads

Edited by Zuppelan, 11 November 2011 - 22:25.

[littlesat]

Sorry... ik durf dit niet te committen. bang dat ik iets stuk maak... Pieterg, Milo kunnen jullie SVP hier naar kijken. Ik denk wel dat het een goede aanpassing is (al hoewel we lever zo iets zien op de openwebinterface)....

[Jeroensky]

Die Gecko-mediaplayer werkt als een tierelier!
De code verkiest nog steeds VLC als default indien meerdere plugins beschikbaar zijn, maar wellicht is het wenselijk Gecko-mediaspeler als default te kiezen. Ik ben er erg tevreden over.

Ik raad je aan om de browserplugins van VLC & Xine te verwijderen(dus niet de players zelf, maar de browserplugins), zodat je alleen de Gecko-Mediaplayer (zeg maar browserplugin van GnomeMplayer) gebruikt.

Edited by J3r03n5ky, 12 November 2011 - 00:20.

[Zuppelan]

Kan ik nog ergens mee helpen om het "commitbaar" te maken?

Wat betreft de Openwebinterface: Zodra hij wat vordert, wil ik daar ook best naar kijken.

[littlesat]

Dan liever inderdaad in de open webinterface...

[Zuppelan]

Ja, die heeft alleen nog geen WebTV. Ik snap je hint, of ik even een WebTV voor de Openwebinterface wil schrijven

Ik zeg geen nee, en ik zal er zeker naar kijken, maar ik denk echt dat het daar nog iets te vroeg voor is.