After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 725031 - bliptv: Search return recent items' RSS instead of results
bliptv: Search return recent items' RSS instead of results
Status: RESOLVED FIXED
Product: grilo
Classification: Other
Component: plugins
unspecified
Other Linux
: Normal normal
: ---
Assigned To: grilo-maint
grilo-maint
: 705920 728527 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-02-24 00:23 UTC by Elad Alfassa
Modified: 2015-08-28 20:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
bliptv: Remove source (1.21 MB, patch)
2015-08-27 08:56 UTC, Victor Toso
committed Details | Review

Description Elad Alfassa 2014-02-24 00:23:13 UTC
In totem, searching for "nostalgia critic" returns irrelevant results.

In blip.tv, http://blip.tv/search?q=Nostalgia+Critic shows what I was looking for.

Maybe a bug, or maybe blip limits this content to only be viewed via the website due to ads. I'm not sure.
Comment 1 Bastien Nocera 2014-02-24 11:22:42 UTC
GrlBlipTv-Message: op url: http://blip.tv/posts?search=nostalgia critic&skin=rss&page=%u

Great work on the escaping there...

In any case, the search seems to be completely broken, it just returns the RSS feed of the most recent items (check the dates on the entries).

I've managed to find this:
http://blip.tv/search/get_episode_results?q=nostalgia+critic&page=1&page_length=50&no_wrap=1&skin=json
from:
https://github.com/HenrikDK/bliptv-xbmc-plugin/blob/master/plugin/BlipTVScraper.py#L37

The plugin basically needs rewriting to use json (passing RSS as the skin fails). That might be a job for a Lua plugin again ;)
Comment 2 Elad Alfassa 2014-02-26 20:27:12 UTC
So as I was implementing a lua plugin for this I was testing some API calls in my web browser, and noticed they block access to vidoes from stuff that are not "the blip player". How does the current plugin handle that? Is it okay with the blip.tv terms of use?
Comment 3 Elad Alfassa 2014-02-26 20:29:49 UTC
Scratch that, seems that they only block stuff with browser user agents. I'm still unsure about the terms of use, I can't find anything about them in the blip.tv website...
Comment 4 Bastien Nocera 2014-04-18 21:17:22 UTC
*** Bug 728527 has been marked as a duplicate of this bug. ***
Comment 5 Bastien Nocera 2014-12-02 15:48:48 UTC
(In reply to comment #2)
> So as I was implementing a lua plugin for this I was testing some API calls in
> my web browser, and noticed they block access to vidoes from stuff that are not
> "the blip player".

See grl-metrolyrics.lua about overriding the user-agent. Do you have your source available somewhere?
Comment 6 Luis Henrique Mello 2015-08-26 17:10:38 UTC
well, Blip.tv is *dead* as of August 20th, 2015...
Comment 7 Victor Toso 2015-08-27 08:54:13 UTC
(In reply to Luis Henrique Mello from comment #6)
> well, Blip.tv is *dead* as of August 20th, 2015...

Thanks for informing us
Comment 8 Victor Toso 2015-08-27 08:56:33 UTC
Created attachment 310077 [details] [review]
bliptv: Remove source

As stated in the blip.tv/shutdown/message.html
"Dear Blip User,

Blip was closed on Thursday, August 20, 2015.

As you may know, Blip was acquired by Maker Studios in September 2013
and the acquisition has allowed for additional, more expansive
direct-to-consumer tools and products for content creators across the
network.

If you are a creator and you have a YouTube channel, we encourage you
to apply to the Maker Gen network to take advantage of Maker’s tools
and services. Signup can be found here: https://max.makerstudios.com/.

If you are a viewer, we encourage you to check out the great content
from Maker talent at our new flagship site: http://maker.tv/

We thank you for being a great partner to Blip!

Blip Support"

This source does not work anymore.
Comment 9 Bastien Nocera 2015-08-28 08:43:10 UTC
Review of attachment 310077 [details] [review]:

There's still left-over mentions of blip in the help files. Looks good otherwise.
Comment 10 Bastien Nocera 2015-08-28 08:43:26 UTC
*** Bug 705920 has been marked as a duplicate of this bug. ***
Comment 11 Victor Toso 2015-08-28 20:01:06 UTC
Removed the mentions in the help files and squased in the patch.

Attachment 310077 [details] pushed as 3298de7 - bliptv: Remove source