I am importing files from a feed that look, for example, like the following

http://www.indybay.org/uploads/2009/03/17/090308ggbcodepinkmedminusonefo...

Problem is instead of treating this video as an .flv file it treats it as a .mov file and thus it doesn't bring up the right player needed to play the file

This happens for other filenames ending in ".wmv_preview_.flv" as well and i am presumeing all other file extensions selected by the emfield module with a "_preview_.flv" added to the end (in this case),

I does not happen with filenames ending with ".mp4_preview_.flv" at the end though, as in the current version of the emfield module I am using mp4 files are not supported, these files bring up an flv player as they should do...

Comments

benced’s picture

benced commented
Version: 5.x-1.2 » 5.x-1.x-dev
Status: Active » Needs review

I think sometimes it is unbelievable how I often only need to make very tiny hacks in order to get things to work some times

This fix involves adding ONE CHARACTER! (albeit added in two places very close to each other)

This change is made in emfield/contrib/video_cck/providers/zzz_custom_url.inc

Basically replace both occurances of

@\.('. $types .')@i

with

@\.('. $types .')$@i

The dollar added to make sure to find the extension at the end of the string instead of finding the first time the extension appears,

This should probably be ported to 6,

I'm not using 6 yet so probably be a while before I get around to figuring out the new regex check that's been put in place...

Unless the practice used by Indybay to for naming the created flv files is commonplace (you can see how they do it above) this is probably not needed to be added but it will make sure the code is more precise with the cost of only an extra TWO characters...

alex ua’s picture

alex ua commented
Status: Needs review » Patch (to be ported)

I added this fix to the 6.x branch, and will eventually backport to 5.x.

Thanks!

aaron’s picture

aaron commented
Status: Patch (to be ported) » Fixed

finally backported.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.