File formatters may now specify the file and MIME types that they support.

CommentFileSizeAuthor
#1 add-file-and-mime-type-information-to-formatters-2159275-1.patch1.16 KBDevin Carlson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Active » Needs review
FileSize
add-file-and-mime-type-information-to-formatters-2159275-1.patch1.16 KB

A patch to add the supported file and MIME type information to the oembed and oembed_thumbnail file formatters.

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Fixed

Tested #1 and confirmed that the file formatters continued to function properly while being properly restricted to the appropriate file and MIME types.

Committed #1 to oEmbed 7.x-1.x.

Status: Fixed » Closed (fixed)

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

prat’s picture

prat CreditAttribution: prat commented

Doesn't appear as formatting option on any of the file types, except for "oEmbed image", which is defined in file_formatter_info_alter.
Running on media 2.0-alpha3 . Proper testing needed I guess

prat’s picture

prat CreditAttribution: prat commented
Status: Closed (fixed) » Needs review
Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Closed (fixed)

The "oEmbed image" formatter is separate from the "oEmbed" and "oEmbed thumbnail" formatters.

If you are having problems with that formatter in particular, I'd suggest filing a separate issue.