Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Letharion’s picture

Status: Active » Needs review

Second attempt.

Letharion’s picture

I hear actually uploading patches is also a good idea.

Letharion’s picture

Complementary file entity issue: #1552988: Move file/add from media to file.

Letharion’s picture

I seems to me like the two default values max_filesize and file_extensions could now be removed from the media module, but I'm not sure of this so I'm leaving them for now.

Dave Reid’s picture

Issue tags: +sprint, +Media Initiative

Adding sprint tags.

mtift’s picture

Patch #1 from this issue (1552920) -- in conjunction with patch #1 from #1552988: Move file/add from media to file -- worked fine to fix uploading from file/add.

I changed the instances of "drupal_get_form('media_add_" to "drupal_get_form('file_entity_" in includes/MediaBrowserUpload.inc so that the "Media file selector" widget is aware of the new namespace and will work properly.

dgorton’s picture

Status: Needs review » Reviewed & tested by the community

#6 works for me (in conjunction w/ #1 from #1552988: Move file/add from media to file

Dave Reid’s picture

Status: Reviewed & tested by the community » Fixed

Awesome! Committed to 7.x-2.x with http://drupalcode.org/project/media.git/commit/453fad5. Nice work all!

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