Browse Server button and File Browser tab not showing for non-root users even when permissions have been assigned correctly.

CommentFileSizeAuthor
#2 browse_server_button_not_showing.patch1.43 KBWebDevDude
#1 browse_server_button_not_showing.patch419 bytesWebDevDude

Comments

WebDevDude’s picture

WebDevDude commented
StatusFileSize
new browse_server_button_not_showing.patch419 bytes

This patch was contributed with the assistance of #onelittleant at Lucks Digital, Inc.

This is my first patch contribution, so please let me know if I am doing this incorrectly. I did read over the user guides, but may have missed a few points.

This patch fixes a bug where when you have a user with multiple roles and only one of the roles has permission to use imce. The module appears to only check permissions on the first role and not subsequent roles where one of the roles actually has the permission to use imce.

There may need to be some cleanup/review, but with this patch, imce did work as expected.

thanks

webDevDude

WebDevDude’s picture

WebDevDude commented
StatusFileSize
new browse_server_button_not_showing.patch1.43 KB

Ignore the first patch, use this one instead.

ufku’s picture

ufku commented
Status: Needs review » Active

Note that role weights determine the assigned profile. Are your lighter roles(up in the list) assigned an IMCE profile?

cabplan’s picture

cabplan commented

ok so I installed this on one of my sites and it worked fine with no patch needed, I now installed it again on another site and this issue is occurring, I have never applied a patch before in my life, all I know how to do is install from the URL, so does the version on drupal that i downloaded and installed, have this issue currently because it is happening, again weird though that my other site works fine, with all the same settings.

cabplan’s picture

cabplan commented

Please ignore, I just realized that I have the "Filtered HTML" selected not Full HTML, i am dumb

ufku’s picture

ufku commented
Status: Active » Closed (cannot reproduce)