1459 replies to this topic

[andyblac]

timer list not very neat.

Attached Files

•  timerlist.png   124.1KB   101 downloads

[Rincewind]

timer list not very neat.

I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?

[blzr]

Hmm, i tried streaming with new webif and it works but firefox doesnt want to automatically start VLC. I get confirmation window with stream.m3u.txt file...
With original webinf VLC opens automatically.
...

same problem here after today's update (and it worked as it should on Monday)
it prompts for confirmation every time...

Attached Files

•  openwebif_stream.png   210.86KB   80 downloads

[nobody9]

will be fixed later today,
we must change the setting of the conten-type, at the moment will always overriten with text/plain

Hmm, i tried streaming with new webif and it works but firefox doesnt want to automatically start VLC. I get confirmation window with stream.m3u.txt file...
With original webinf VLC opens automatically.
...

same problem here after today's update (and it worked as it should on Monday)
it prompts for confirmation every time...

[Rincewind]

timer list not very neat.

I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?

OK, Google (possibly) helped...
Here is the patch for the scrambled timer list: if something is wrong in the packaging, please let me know.

Attached Files

•  Patches.zip   1.47KB   5 downloads

Edited by Rincewind, 28 December 2011 - 20:03.

[nobody9]

Context menu

TortoiseGit->Diff with previous version

i only see some screen-shots in the net

timer list not very neat.

I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?

[nobody9]

thanks, i have it committed

timer list not very neat.

I have implemented a fix for this, but I don't know how to create the git patch from my repository with TortoiseGIT.
Could anybody tell me how to do that?

OK, Google (possibly) helped...
Here is the patch for the scrambled timer list: if something is wrong in the packaging, please let me know.

[Rincewind]

Now that I learned, I start enjoying it ;-)

Here's another patch that fixes the pulldown directories menu in the movies list, as mentioned in a previous message.

Attached Files

•  Patch.zip   778bytes   3 downloads

[nobody9]

thanks, also committed,
enough for today for me.

Now that I learned, I start enjoying it ;-)

Here's another patch that fixes the pulldown directories menu in the movies list, as mentioned in a previous message.

[Hump]

There is still a bug in "stream>movies".

Only the main directory "hdd/movie" can be accessed. When selecting a subdirectories, following error occurs: " Error 404: Page not found - The requested URL was not found on this server.

Thanks.

Edit: You guys are fast. While I was writing this post, the fix was committet! - Thanks again!

Edited by Hump, 28 December 2011 - 20:43.

[dylantje]

He hello all,

Go on with the great work.
When it is in the [ test ]feed?
So we can try it?

[bacicciosat]

now openwebif starts on 'Current' tab which is not very useful imho (you can't even start streaming of a current channel from it...)
starting on a 'Bouquets' tab I find far much convenient - I (like many /?/ others) use webif for streaming purposes most times

Finally i realized that this is a good idea.
done !

[tilleke]

@nobody9

sorry to bother again but there is still something which is not compatible with the original WebIf what regards stream.m3u.

I didn't notice this difference before because I was convinced the difference was only the #EXTINF-line. However, after testing with your latest commits, I noted a difference what regards the Service Reference.

This is the output using wget with original WebIf (I am using port 8888) and OpenWebIf (port 8088):

OpenDreambox 1.6.0 dm500hd

ORIGINAL WEBIF
dm500hd login: root
root@dm500hd:~# wget -q -O - http://127.0.0.1:8888/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:8001/


OPENWEBIF
root@dm500hd:~# wget -q -O - http://127.0.0.1:8088/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:800...F:820000:0:0:0:
root@dm500hd:~#


as you can see, using original WebIf the service-reference is not included while with OpenWebIf the service-reference is added.

I don't know how this web-stuff works but with original web-if, if I save the stream.m3u and then open it in a text-editor, then the service-reference has been added (later in the process somehow, perhaps during download, but in the beginning the service-reference is not included as you can see from the wget-command). This is the reason why it is possible to stream directly from the browser opening the file with VLC.

In OpenWebIf, the service-reference is ALREADY included from the beginning and is not added later in the process. In any case, streaming will work by opening the resulting m3u-file with VLC since service-reference is present.

To make OpenWebIf compatible (and not to break my code), you would need to change your code in stream.py (I think - I don't know python) as follows:

from:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/%s\n" % (request.getRequestHostname(), sRef)

to:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/\n" % (request.getRequestHostname())

BUT then in OpenWebIf streaming will not work by opening stream.m3u-file with VLC since service-reference is not included unless you find out how it is done in original WebIf.

If it is difficult, I can change my code in my old apps (new apps = no problem) to make it easier for you. What regards other third-party developers, I cannot speak for them but I believe most developers are simply using "web/streamcurrent.m3u" for launching streaming. If you can fix it so it works like in original WebIF, that would be nice otherwise nevermind.

rgds,
tilleke

Edited by tilleke, 29 December 2011 - 02:54.

[nobody9]

@tilleke,
i can´t change this.
The external streamproxy sends you without a ServiceRef the aktive channel,
the internal Streamserver from Pli sends you nothing.
So at the moment i must send the Serviceref in the request.

@Pli Team,
can you change the internal streaming server that they stream the active channel when no ServiceRef is given

@nobody9

sorry to bother again but there is still something which is not compatible with the original WebIf what regards stream.m3u.

I didn't notice this difference before because I was convinced the difference was only the #EXTINF-line. However, after testing with your latest commits, I noted a difference what regards the Service Reference.

This is the output using wget with original WebIf (I am using port 8888) and OpenWebIf (port 8088):

OpenDreambox 1.6.0 dm500hd

ORIGINAL WEBIF
dm500hd login: root
root@dm500hd:~# wget -q -O - http://127.0.0.1:8888/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:8001/


OPENWEBIF
root@dm500hd:~# wget -q -O - http://127.0.0.1:8088/web/stream.m3u
#EXTM3U
#EXTVLCOPT--http-reconnect=true
http://127.0.0.1:800...F:820000:0:0:0:
root@dm500hd:~#


as you can see, using original WebIf the service-reference is not included while with OpenWebIf the service-reference is added.

I don't know how this web-stuff works but with original web-if, if I save the stream.m3u and then open it in a text-editor, then the service-reference has been added (later in the process somehow, perhaps during download, but in the beginning the service-reference is not included as you can see from the wget-command). This is the reason why it is possible to stream directly from the browser opening the file with VLC.

In OpenWebIf, the service-reference is ALREADY included from the beginning and is not added later in the process. In any case, streaming will work by opening the resulting m3u-file with VLC since service-reference is present.

To make OpenWebIf compatible (and not to break my code), you would need to change your code in stream.py (I think - I don't know python) as follows:

from:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/%s\n" % (request.getRequestHostname(), sRef)

to:
response = "#EXTM3U\n#EXTVLCOPT--http-reconnect=true\nhttp://%s:8001/\n" % (request.getRequestHostname())

BUT then in OpenWebIf streaming will not work by opening stream.m3u-file with VLC since service-reference is not included unless you find out how it is done in original WebIf.

If it is difficult, I can change my code in my old apps (new apps = no problem) to make it easier for you. What regards other third-party developers, I cannot speak for them but I believe most developers are simply using "web/streamcurrent.m3u" for launching streaming. If you can fix it so it works like in original WebIF, that would be nice otherwise nevermind.

rgds,
tilleke

[nobody9]

i can´t change my old topic,

The documentation
for /web/stream.m3u is always with Service Ref.
normally you must use without /web/streamcurrent.m3u.

So it´s not a issue from OpenWebif,

it´s a issue from Streamproxy/Internal Streamserver and the old Webif.

[nobody9]

Stop,
i make a mistake,
i changed the Code, only when requested /web/streamcurrent.m3u the ServiceRef will append.
after this change your app works again.



@tilleke,
you add the serviceref in your program.
So when the webif send you the request without it, all works fine.


@Pli team

you must change nothing

@tilleke,
i can´t change this.
The external streamproxy sends you without a ServiceRef the aktive channel,
the internal Streamserver from Pli sends you nothing.
So at the moment i must send the Serviceref in the request.

@Pli Team,
can you change the internal streaming server that they stream the active channel when no ServiceRef is given

[bacicciosat]

@Pli team
you must change nothing

But, do you have understood that OpenWebif is not a Pli application ?
Pli code don't care here. Streaming have to work on all the images.

[nobody9]

I understood this, OpenWebif is not a Pli application.
OpenWebif is from all of us to all of us.

The biggest problem with the streaming - functions is , the big dependent on the e2-api.

@Pli team
you must change nothing

But, do you have understood that OpenWebif is not a Pli application ?
Pli code don't care here. Streaming have to work on all the images.

[tilleke]

@nobody9

Thanks for resolving streaming-issue. Tested with latest commit and the streaming part (stream.m3u) seems to be identical as to original WebIf.

Good job - in this way compatibility should be 100%.

rgds,
tilleke

[blzr]

now openwebif starts on 'Current' tab which is not very useful imho (you can't even start streaming of a current channel from it...)
starting on a 'Bouquets' tab I find far much convenient - I (like many /?/ others) use webif for streaming purposes most times

Finally i realized that this is a good idea.
done !

nice, thanks...
so maybe I'll try to convince you to remove (i)nfo icon too?
I know nothing about web designing, so I'm just throwing some general ideas,
so, attached three jpegs:
1. op_wbif_now channels list as it looks now
and, for comparsion (I assume you'll add link to description under event title/?/):
2. op_wbif_mod1 - one (i) icon left (more as an indicator that there is description availlable for given service)
3. op_wbif_mod2 - with (i) icon removed completely (I like it best, the more minimalistic look the better )
in both my drafts I increased spacing between each service and deleted theese various white lines
It would look much more clearer (and nicer ) imho
//but of course I know that easier said than done...
and what do you think?

Attached Files

•  op_wbif_now.jpg   19.33KB   115 downloads
•  op_wbif_mod1.jpg   23.59KB   138 downloads
•  op_wbif_mod2.jpg   22.89KB   102 downloads