The helper function _media_recursive_check_plain()
is defined and used in media.browser.inc but is also used in media.media.inc. Unfortunately, it seems that media.browser.inc is not always loaded which can lead to errors, namely (when searching for files in the media browser):
Warning: array_walk_recursive() expects parameter 2 to be a valid callback, function '_media_recursive_check_plain' not found or invalid function name in media_query_media_browser_alter() (line 55 of /Users/devin/Sites/acquia-drupal/sites/all/modules/media/media.media.inc).
The _media_recursive_check_plain()
function should be moved out of media.browser.inc and into media.module where it will always be available.
Comment | File | Size | Author |
---|---|---|---|
#1 | move-and-rename-function-1790448-1.patch | 2.12 KB | Devin Carlson |
Comments
Comment #1
Devin Carlson CreditAttribution: Devin Carlson commentedA patch to move the function into media.module. I've also removed the leading underscore from the function name, as it doesn't really seem like a helper function since it is used in multiple locations.
Comment #2
grendzy CreditAttribution: grendzy commentedLooks good to me.
Comment #3
ParisLiakos CreditAttribution: ParisLiakos commentedThanks, seems patch works, no more entries in dblog
Comment #4
ParisLiakos CreditAttribution: ParisLiakos commentedcommited