file_entity renamed the following permissions at some point, this module is still using the old names.

  $permissions = array(
    'view file' => 'view files',
    'edit file' => 'edit any files',
  );

That results in stuff not showing up if you don't have the administer files permission.

CommentFileSizeAuthor
#7 media_browser_plus-change-permission-names-1885154-7.patch574 bytesBarisW
#5 media_browser_plus-change-permission-names-1885154-5.patch3.45 KBBarisW
#4 media_browser_plus-change-permission-names-1885154-4.patch3.54 KBBarisW
#1 media_browser_plus-change-permission-names-1885154-1.patch3.63 KBBarisW

Comments

BarisW’s picture

BarisW commented
Status: Active » Needs review
StatusFileSize
new media_browser_plus-change-permission-names-1885154-1.patch3.63 KB

Yep, that was my issue.. patch attached.

paulihuhtiniemi’s picture

paulihuhtiniemi commented

Doesn't seem to apply, "error: sites/all/modules/contrib/media_browser_plus/media_browser_plus.module: No such file or directory" :)

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

@BarisW: You need to make the path relative with git diff --relative=path/to/module.

BarisW’s picture

BarisW commented
StatusFileSize
new media_browser_plus-change-permission-names-1885154-4.patch3.54 KB

Ah, my fault. Eat this.

BarisW’s picture

BarisW commented
StatusFileSize
new media_browser_plus-change-permission-names-1885154-5.patch3.45 KB

Sigh.

paulihuhtiniemi’s picture

paulihuhtiniemi commented
Status: Needs review » Reviewed & tested by the community

Seems to work nicely, thanks :)

BarisW’s picture

BarisW commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new media_browser_plus-change-permission-names-1885154-7.patch574 bytes

Still an issue with the latest DEV. Re-roll.

The permission view file does not exist, this should be view files.

das-peter’s picture

das-peter commented
Issue summary: View changes
Status: Needs review » Fixed

The callback admin/content/file/thumbnailsJSON is obsolete in the MBP 3.x branch. Thus I've removed that completely: http://drupalcode.org/project/media_browser_plus.git/commit/04d449f
And as the MBP branch 2.x is discontinued I don't think I'll update that.
However, thanks @ all for the patches!

Status: Fixed » Closed (fixed)

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