e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
318 stars 213 forks source link

Q - menu config on frontend #3705

Open Jimmi08 opened 5 years ago

Jimmi08 commented 5 years ago

Just noticed, fixed by theme template.

Admin see :

image

but when clicks on Menu config, with path: e107_plugins/login_menu/config.php

but they are redirected to e107_admin/admin.php

Is this some setting or what? Why is the link displayed if they can do nothing?

When I click on this as main admin, I see this: (no left menu?)

image

PHP 7.0.

CaMer0n commented 5 years ago

@Jimmi08 config.php makes the following check:

if (!getperms('4')) 
{ 
    e107::redirect('admin');
    exit() ;
}

Does this answer your question?

Jimmi08 commented 5 years ago

4 - Manage all User, Userclass and Extended User-Field settings

No idea why it's related to this, but it's ok. Question is why the same check is not for displaying that link.

But again, the easy fix is to remove it from the template.

Moc commented 5 years ago

That seems like an odd permission to use for this functionality. I guess we need to do a clean-up of all permissions and enhance them where needed. Related: #3694