[Plugin: White Label CMS] BUG : Hiding Media Menu disables upload permission

  • BunnyBomb

    (@bunnybomb)


    12 years, 2 months 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/

Viewing 6 replies - 1 through 6 (of 6 total)
  • Thread Starter BunnyBomb

    (@bunnybomb)

    12 years, 2 months 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

    Plugin Author Video User Manuals

    (@videousermanuals)

    12 years, 2 months ago

    Hi,

    Thanks for the details, we will look into this.

    Plugin Author Video User Manuals

    (@videousermanuals)

    12 years, 1 month 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

    iMenn

    (@imenn)

    12 years, 1 month ago

    I found same problem. Thanks both of you 🙂

    bolinger

    (@bolinger)

    11 years, 8 months ago

    was this ever fixed? I am still getting this error

    Plugin Author Video User Manuals

    (@videousermanuals)

    11 years, 8 months ago

    Hi Bolinger,

    Which version of WLCMS are you using?

    Thanks

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘[Plugin: White Label CMS] BUG : Hiding Media Menu disables upload permission’ is closed to new replies.