Yes. This is needed to figure out which files loop and so on, and in the
future, when these things are configurable, they will be the defaults. So
we need to keep them.


On Thu, Dec 12, 2013 at 4:42 PM, Drew DeVault <notifications@github.com>wrote:

> In mediacrush/files.py:
>
> > -LOOP_EXTENSIONS = set(['gif'])
> > -AUTOPLAY_EXTENSIONS = set(['gif'])
> > +from mediacrush.config import _cfg
> > +from mediacrush.database import r, _k
> > +from mediacrush.objects import File
> > +from mediacrush.ratelimit import rate_limit_exceeded, rate_limit_update
> > +from mediacrush.network import secure_ip
> > +from mediacrush.tasks import process_file
> > +from mediacrush.fileutils import processing_needed, EXTENSIONS, get_mimetype, extension, file_storage
> > +from mediacrush.celery import app
> > +
> > +VIDEO_FORMATS = set(["image/gif", "video/ogg", "video/mp4"])
> > +AUDIO_FORMATS = set(["audio/mpeg", "audio/ogg"])
> > +FORMATS = set(["image/png", "image/jpeg", "image/svg+xml"]) | VIDEO_FORMATS | AUDIO_FORMATS
> > +LOOP_FORMATS = set(["image/gif"])
> > +AUTOPLAY_FORMATS = set(["image/gif"])
>
> We are still hardcoding this?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/MediaCrush/MediaCrush/pull/449/files#r8305338>
> .
>