In issue #2276089: Allow configuring file types for "File" widget without overriding the Feature, there was an improvement to Panopoly to add the panopoly_widgets_file_extensions variable to control allowed file extensions. However, there already exists a variable in file_entity called file_entity_default_allowed_extensions that is used by the Media module to control which extensions are allowed for uploads.
Seems like Panopoly should be using the existing variable that Media Browser is using. Otherwise there is inconsistency between the file types that can be added to a widget vs the file types that can be added in the WYSIWYG via Media.
I'm working on fixing this same issue in Atrium for all of our various file fields, but when I'm done with that I'll try to get a patch for Panopoly. But basically, just replacing the variable name in panopoly_widgets and then writing an update hook to migrate any custom site setting that is different from the current default to the existing file_entity variable.
Comments
Comment #2
dsnopekThanks! If you can get the upgrade path for this right, I'm all for it! Here's how I think this should work:
This way existing sites and child distros should be able to continuing working, but new sites would have only the single variable. If a child distro (like Open Atrium) wanted to be more aggressive about removing the old variable, they could do so in their own update hook.