oa_files php error Call to undefined function media_wysiwyg_include_browser_js()

I cannot reproduce this on a fresh install. The media_wysiwyg_include_browser_js() function is part of the media_wysiwyg module that is included with Panopoly and Open Atrium. So it sounds like you didn't install something properly and don't have that dependent module enabled.

You should *not* be updating modules manually. You need to wait for new releases of Atrium that will update modules properly with the required patches.

So to reproduce I've upgraded right after install with drush up, exactly what @realgiucas did.

Fun times:)

https://www.drupal.org/node/2434169#comment-10177392

Doesn't look a laborious fix, though.

    drupal_add_library('media', 'media_browser');
    drupal_add_library('media', 'media_browser_settings');

So to reproduce I've upgraded right after install with drush up

DO NOT USE "drush up" with Open Atrium. That will download new module versions but won't apply necessary patches. This will break your site! As it says on the upgrade instructions (and in the Report -> Available Updates page in Atrium itself) you should not be installing updates manually unless you know how to work with the make files to get the proper patches applied. Plus, you would be using untested module versions that could cause problems. You are supposed to wait for Atrium itself to update modules. So it's really no surprise that this broke something.

Atrium currently uses Media 2.0-Alpha4 with patches (comes from panopoly). The above issue and problem happened *after* the Alpha4 version. The later versions of Media have lots of issues that Panopoly is working on and once it gets updated in Panopoly then we will make the necessary changes in oa_files based on the issue linked in #6.

I'm fine living on the edge;) this was the only real break on the site.

Navbar had some issues but I reverted those.

Let's fix this one, it will help smooth a path forward for this Distro. I'd double check the function but if it's on the chopping block for media we should prepare for that change no?

I'm just experimenting with a personal site.

I reopened the #2434169: Regression: dropping media attach function breaks modules that use Media without media FAPI field issue mentioned in #6 and #7 because their "fix" won't work for oa_files.

Fixed in commit 3ad1b0b to oa_files

Thanks for this, i also needed the latest version of media and the commit fixes the error.

FYI: I was able to reproduce the same exact error when porting Open Atrium to multisite environment. I was testing multi-profile, multi-site instance.