FLV Flash Video Mimetypes

looks good to me.

According to Adobe, .f4a and .f4b should use audio/mp4.

Here's a re-roll that takes c960657's data into account and a link to Adobe suggesting video/x-flv for .flv files.

Subscribing, I need to update the hook_variable_info() patch when that gets in.

@Rob Loach
a idea that was mentioned in the hook_variable_info() issue was to create a table to store these. I've also seen that there is work for a file.module, it could even provide a interface to add additional mimetypes. Or a contrib module could do it.

Some advantages such a table would imho have:
- Better handling of large amount of entries, does need less memory
- Doesn't need to be cached/saved in the variables table as a big blob
- Possibilites to manage the mime types and create a UI to manage the mimetypes
- We could simply reference to that table, instead of storing the mimetype for each file entry

I'm not sure what the best way is to add new MIME-types to file_default_mimetype_mapping().

I'd rather not renumber all the existing types - this makes it harder to review the patch and follow the CVS history. This leaves the options of keeping either the numerical order of the ids or the alphabetical order of the MIME-types. I think the latter gives the best overview of related types, because e.g. all image/* types are displayed in one block.

Thoughts?

yeah #473652: file_get_mimetype() unneeded loop has kind of made a mess of this.

Committed to CVS HEAD. Thanks!

Is there a similar patch for Drupal 6? I think I'm having the same problem.

I've backported the patch from #8 to Drupal 6.

For Googling reference, you need this patch if Filefield + mimedetect is giving you something like "The selected file test2.flv could not be uploaded. The file contents (video/x-flv) do not match its extension (flv)."

Patch is against current DRUPAL-6 branch.

Looks good.

Committed, thanks.