WordPress.org

Ready to get started?Download WordPress

Forums

White Label CMS
BUG : Hiding Media Menu disables upload permission (7 posts)

  1. BunnyBomb
    Member
    Posted 2 years ago #

    Hello,

    I have found a bug with this plugin.

    1. When you tick the box to hide the "Media" menu for Editors within the plugin settings, this has the effect of disabling an Editor's ability to upload media, even if they still have access to the posts and pages menu.
    2. What's more is that the Editor's ability to upload files is STILL disabled, even when the White Label CMS plugin is set to Deactivated, if it was deactivated with the "hide Media menu" option ticked.

    I was able to verify this bug across two separate clean installations, with no other plugins active, using the TwentyEleven basic theme.

    Clicking an option called "Hide Media Menu" should not take away Editor's abilities to upload media. This has the effect of preventing them from being able to make proper posts/pages. If they still have access to the Posts and Pages menu's, features within those areas should work.

    It seems to me that in order to hide the menu, you've taken away the upload_files capability itself from Editor's (which inherently takes the menu with it).

    If that's how you want it to work - fine - but the label on your tick box should then read something like this: "Disable Editor's ability to upload files and hide the Media Menu"

    Also, the second part of the bug indicates that parts of your plugin are still active even when it is set to deactivated. This is most certainly a bug.

    I'd be grateful if this could be fixed.

    http://wordpress.org/extend/plugins/white-label-cms/

  2. BunnyBomb
    Member
    Posted 2 years ago #

    Further to the above...

    Instead of taking away the upload_files() capability in order to hide a menu, what you should do is have your tick-box call a conditional statement to just cosmetically "hide" the menu.

    I'm no expert coder, but I do know that you can achieve this by:

    // Remove menu items for users below Admin level
    add_action( 'admin_init', 'my_remove_menu_pages' );
    function my_remove_menu_pages() {
    	// If the user is not an administrator then...
    	if(!current_user_can('add_users')) {
    		// Remove the "Media" Menu from the list
    		remove_menu_page('upload.php');
    	}
    }

    I'll be using this code in my installation, until such time as your tick-box idea does something other than taking away the capability.

    Cheers

    BB

  3. Video User Manuals
    Member
    Plugin Author

    Posted 2 years ago #

    Hi,

    Thanks for the details, we will look into this.

  4. Video User Manuals
    Member
    Plugin Author

    Posted 2 years ago #

    Hi,

    We have changed this behaviour, and are currently testing it.

    The update will be available any day now.

    Thanks for pointing this out.

    Cheers

  5. iMenn
    Member
    Posted 2 years ago #

    I found same problem. Thanks both of you :)

  6. bolinger
    Member
    Posted 1 year ago #

    was this ever fixed? I am still getting this error

  7. Video User Manuals
    Member
    Plugin Author

    Posted 1 year ago #

    Hi Bolinger,

    Which version of WLCMS are you using?

    Thanks

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic