Closed (fixed)
Project:
Media: BlipTV
Version:
6.x-1.0-beta1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 May 2011 at 14:40 UTC
Updated:
17 Aug 2011 at 15:17 UTC
Jump to comment: Most recent file
Comments
Comment #1
tsmwebdev commentedI'm having this issue as well. Would appreciate any help.
Comment #2
minus commentedI'm having this issue as well. Would appreciate any help.
Comment #3
eric.chenchao commentedMe too, even in the wiki of blip.tv there is no any API change notification.
Comment #4
dpearcefl commentedI am testing a path to bliptv.inc right now. If all goes well I'll submit a patch this afternoon.
Comment #5
stuartschultz commentedClearly not a long term fix, but I figured out how to get the old URLs while this is being fixed.
Go to your actual episode page, eg. http://blip.tv/dashboard/episode/xxxxxxx, then view source, and do a file find (search) for: 'form action="/file/post/' (without the single quotes of course). The number after the trailing slash is your file id.
Then you can just reconstruct the old blip.tv URL format:
eg. http://blip.tv/file/xxxxxx
I know this is just a hack, but at least it gives us a way to upload new vids while we fix the module.
Hope this works for you too!
Comment #6
dpearcefl commentedMy patch only fixed existing videos that may have either flv or m4v versions. So I haven't addressed the URL changed.
Comment #7
dpearcefl commentedI'm bumping this is critical because this module is now totally broken because of what Blip did to us.
Comment #8
QTV810 commented@stuartschultz -- Thanks for the "hack" while a fix is in the works!
Comment #9
ronholman_CTV commentedthanks Stuart, that's working for me so far, blip tech support has not been helpful at all...
Comment #10
jodym commentedYep this is a big problem, hope someone submits a patch soon, wish Blip would stop changing things.
Comment #11
onepartscissors commented*subscribing
Also having this issue on a site that's live, the missing video feature is now critical!
Would be good if Blip would at least make it easy to get the Episode ID out of the dashboard or something, then we can piece the addresses together manually the old-way as a work-around until the module is updated.
Anyone have any ideas on that in the mean time?
Comment #12
dpearcefl commentedIs anyone working on a fix? If you are, please speak up so we have hope.
Comment #13
dpearcefl commentedThis a kind of funky way to fix this problem but it works for me. I can now add the new Blip.tv URLs and everything seems to work fine.
The first diff adds a new emfield_include_invoke() call to emfield_parse_embed() in emfield.module.
The second diff adds the function called by the previous change. This function will convert the new URL to the old style URL.
Note: This is not a proper patch as it will need to be applied to two projects. I would really like some other folks to test this idea and see how it works for you. I also suspect there is a better was to fix this problem.
Comment #14
jodym commenteddpearceMN works great :-) Thanks so much..
Comment #15
tsmwebdev commentedAppears to work for me too. Thanks!
Comment #16
thomasmuirhead commented@dpearceMN That sounds great. I'm about to give it a go. Does it affect old content? Ie. if you put the old style URL in does it still work? Will all legacy content on the site still work, even if it doesn't have the new URL?
Thanks
Thomas
Comment #17
dpearcefl commentedIt should not affect the old-style URLs (and it didn't in my testing). This patch basically converts a new URL to an old URL internally, leaving anything else the same.
Comment #18
thomasmuirhead commentedGreat, thanks dpearceMN
Comment #19
jodym commentedOnly problem now is the duration field is outputting as 0:00
Comment #20
cwworks commentedThanks very much for the prompt attention to this issue. It's very appreciated!
Comment #21
andrewfn commentedAs of today, all the old videos that were playing fine seem now to fail.
Comment #22
dpearcefl commentedHmm, all of mine work. How and when do they fail? Can you provide a little more information @andrewfn?
Comment #23
thomasmuirhead commentedHi
I'm getting a bit of a new problem. Mp3s. We've been embedding the mp3s in to the site using the url given on the blip page such as: http://blip.tv/file/get/GreshamCollege-FinancingRetirement_MalcolmSmall8.... It doesn't work anymore. Does anyone have any idea why that might be? Or what I could do to fix it?
Thomas
Comment #24
thomasmuirhead commentedUPDATE: I am using a custom url emfield to embed the mp3.
Comment #25
Poieo commentedSubscribing...
Comment #26
andrewfn commented@dpearceMN Just updated to the latest version of emfield and it seems to have fixed my problems.
Comment #27
onepartscissors commentedAny news from the Blip.tv team? I've tried the Custom URL option in media flotsam, but it isn't working to use the embed code or the page address from blip.tv.
Not sure what we need to do on this one - seen as though old URLs still work, the challenge surely is figuring out how to assemble an old-style URL for new videos you wish to add? surely the embed or the iframe or even the URL can be worked out to display the old way?
Or is there another solution someone's thinking of?
I tried #13 from dpearceMN but that didn't work for me.
Still no joy!
Comment #28
dpearcefl commentedOur company has been using the idea I posted in #13 and have had no problems since. I have written this module's maintainer aaron but have not heard back from him yet.
Comment #29
dmurkerson commentedWe are having the same issue and prefer to use blip.tv over YouTube. Hope this gets resolved soon.
Comment #30
andrewfn commentedA quick way to find the old URL is to append
?skin=rssto the new URL and you will get an rss page which uses the old format in the link to the video.You can copy and paste this into the Drupal field.
Comment #31
dpearcefl commented@onepartscissors, did you edit then save your node after making the code changes?
Comment #32
yaworsk commentedfinally started looking through the code today and media_bliptv.module, line 115 it is hardcoded to include the old url:
function media_bliptv_video_url($id) {
return url('http://blip.tv/file/'. $id);
}
this function is actually called by bliptv.inc, line 177:
function emvideo_bliptv_embedded_link($video_code) {
return media_bliptv_video_url($video_code);
}
which is actually a hook defined in emfield.php of the embedded media field. too tired to keep going tonight but need to look and see when the hook is invoked because i couldn't' find the validation function which spits out "You have specified an invalid media URL or embed code." that would need to be updated as well as media_bliptv.module line 115 which would now have to check for unique urls based on each bliptv user account. i thought adding a field ot the administration page which allows users to define their bliptv external link would be best (mine being http://blip.tv/torontowebdeveloper/)
Thoughts?
pete
Comment #33
urlM-2.0 commentedsubscribing.
Comment #34
goldI've just checked (and will be deploying) the patches in #13. I can report that these worked for us.
Comment #35
dpearcefl commentedStill working for us.
Comment #36
JohnTarr commentedSubscribing
Comment #37
kpoirier commentedsubscribing.
Comment #38
burgs commentedI tried to do some changes to this, but can't seem to change the blip code from the new url to old url easily before it all goes pear shaped.
Is there a form hook, or another hook that we can use to do this? (short of modifying emfield to add our own hook)
Comment #39
thomasmurphy commentedsubscribing
Comment #40
rotimigenius commentedHi,
I'm a bit lost now. Pls i need a developer that can help finish up adding a blip video and thumbnails to a site. Pls email rotimigenius (at) gmail.com www.leedstelevision.co.uk .thanks
Comment #41
SVilas commentedJust found out about this issue after wanting to add a new video to my site. Ran into the 'bad URL' problem and noticed Blip has changed the way it's naming URLs. Very annoying, especially now that all the other videos on the live site are not functional anymore.
I have tried to apply the patch as mentioned in #13, but doesnt seem to work for me.
When adding the patch to the emfield.module, I get the following error:
Fatal error: Cannot redeclare emfield_parse_embed() (previously declared in /home1/.../.../.../modules/emfield/emfield.module:170)
I've added two closing brackets } to the code, cause i thought these were missing. if didn't add these i got a syntax error: unexpected $end in /home1/.../.../.../modules/emfield/emfield.module on line 1007
the patches seemed to have worked for others, so not sure if there's anything i'm doing wrong.
I am using emfield module emfield-6.x-2.5 and media_bliptv-6.x-1.0-beta1
really need a solution for this. very inconvenient and don't want to convert and migrate everything to Vimeo.
Comment #42
Poieo commentedSounds like you applied the patch wrong and have the embed function in there twice. Should be working with those versions of the modules.
I'm attaching the patch versions of both for you to try out.
Comment #43
SVilas commentedHi Poieo,
thanks a lot for the files. I've installed them and it seemed to solve the errors with the Bliptv ID. the video is being recognized and thumbnail is generated. I am not sure what i did wrong with the application of the patch but i will take a look at the code and see what went wrong.
however, the site still doesn't display the videos the way it should. not sure what the problem is now, but once in a while i get this error: warning: Invalid argument supplied for foreach() in /home1/.../.../.../modules/emfield/contrib/emvideo/emvideo.module on line 677
i am not sure if it has something to do with it.
the videos are displayed within a shadowbox after clicking on the thumbnail. instead, the shadowbox remains empty. no video is loaded.
all worked perfectly before, now its back to the drawing board :)
Comment #44
Macronomicus commented#13 Mostly works
I see the thumbnails and it fixes the url but the time says 0:00 and clicking play does nothing.
Youtube vids work fine
Comment #45
Poieo commented@Svilas - That part of the module has nothing to do with the patch. Did you run update.php after loading the new modules? Maybe you were previously using a different version of emfield?
Comment #46
Macronomicus commentedwell this may have something to do with mediafront cause using the basic emfield output instead of the mediafront player the blip videos do indeed come up with the solution in #13
Comment #47
Macronomicus commentedI should also add that using the #13 patch breaks my youtube urls. I didnt notice before because they were already saved in the db but when attempting to create a new or save an existing it came back saying invalid url. Reverting the patch on emfield.module brought back the ability to add youtube url's.
Comment #48
killes@www.drop.org commentedI have a simiar problem, when I try to save a new video, the provider is not loaded and thus the error message is triggered. I am also using the patch from above.
Comment #49
killes@www.drop.org commentedeh, embarrassing: I had only patched the bliptv part but not the emfield module
Comment #50
bkat commentedThe changes in #13, specifically the change to emfield.module seems to break parsing of youtube links.
Comment #51
killes@www.drop.org commented#50: Confirmed.
Comment #52
aaron commentedok, here's the patch i'm about to commit...
Comment #53
aaron commentedok, committed that. thanks for the hard work, everyone!
Comment #54
bkat commentedIs the patch in #52 in addition to or instead of the patches in #13?
Comment #55
aaron commentedthat's instead of.
i want to create a new release, as soon as we resolve #1204696: Is any one else having IE9 issues?.
Comment #56
asanchez75 commentedpatch #52 works, thanks!
Comment #58
candelas commentedi try to apply the patch but gives an error...
i try in the 6.x-1.12, should i in the dev?
@aaron, since i see that when you release the patch is after the dev version, i imagine that it is not in it, am i wright?
and thanks for the module :)
the error:
patching file b/providers/emvideo/bliptv.inc
Hunk #1 FAILED at 48.
Hunk #2 FAILED at 57.
2 out of 2 hunks FAILED -- saving rejects to file b/providers/emvideo/bliptv.inc.rej
edit: ups.. i was in the wrong directory O.O
the patch is working perfect :)
thanks a lot all of you!