Lots is changing. This potentially fixes #1915876: Add default display configuration. (Might still need to add a couple minor bits to the defaults)

(This patch breaks without first adding at least #2106973: Use require once and not include drupal get path and possibly #2106975: Deprecate hook_media_format_form_prepare_alter)

CommentFileSizeAuthor
#2 remove-variable-namespacing-2106977-2.patch7.36 KBdevin carlson
switch_to_new_variables.patch9.65 KBrob c

Comments

Juan C’s picture

Juan C commented

Thank you.

devin carlson’s picture

devin carlson commented
Title: Switch to new variables » Remove unnecessary variable namespacing
Issue summary: View changes
StatusFileSize
new remove-variable-namespacing-2106977-2.patch7.36 KB

Retitling to address the remaining issue.

Since all of the current variables are simply used to set default values, they should simply be removed.

An upgrade path is included to remove the variables.

devin carlson’s picture

devin carlson commented
Status: Needs review » Fixed

Tested #2 and verified that the default formatter settings continued to work properly and that the variables were successfully removed.

Committed #1 to Media: Vimeo 7.x-2.x.

Status: Fixed » Closed (fixed)

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