It's a great admin themes,.. but..
When we are use RootCandy Themes for ADMIN panel, there's some security issue,... event anonymous user, can see the admin icon and link provided by RootCandy Themes,.. (nomain_name.com/admin)...!!
Is there any way to hide the icon in RootCandy Themes for anonymous user?

Regards

CommentFileSizeAuthor
#7 rootcandy_patch_5.txt2.33 KBTakafumi
#6 rootcandy_patch_4.txt2.22 KBTakafumi
#4 rootcandy_patch_3.txt2.31 KBTakafumi

Comments

sign’s picture

sign commented
Status: Active » Postponed (maintainer needs more info)

Not sure what icon you mean.

I guess top navigation icon menu?

You can visit theme settings, where you can choose which menu will be shown on top for anonymous user, authenticated user, etc... That might fix your problem?

alexkessler’s picture

alexkessler commented

I had the same problem until i discovered that it only belongs to the fixed-width version of eyecandy.
In the the 'normal' version you can make all necessary changes in the theme settings tab.
Maybe it is possible that the fixed version can inherit these functions from his big brother...

BTW: Great Theme!

sign’s picture

sign commented

this is definitely a bug, rootcandy fixed should have same theme settings :S

well, ill wait for panjis clarification.

thanks

Takafumi’s picture

Takafumi commented
Version: 6.x-1.0-rc1 » 6.x-1.x-dev
Component: Miscellaneous » Code
Priority: Critical » Normal
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new rootcandy_patch_3.txt2.31 KB

I agree with panji basically. IMO, an admin icon should not be shown to non-admin user by default.

This patch adds "None"(as default value) to option of "Menu source by role".

panji’s picture

panji commented

I think it would be better if there's an option for anonimous user or other non admin permition user to display only admin login page wich is only display the admin login. Or maybe there an option to re-direct it to "home" for non admin user!

Admin Login Page
Name :
Pass :

Takafumi’s picture

Takafumi commented
StatusFileSize
new rootcandy_patch_4.txt2.22 KB

Sorry, bug exists in #4 patch.
This patch is correct.

Takafumi’s picture

Takafumi commented
StatusFileSize
new rootcandy_patch_5.txt2.33 KB

I improved #6 patch.
This patch added 'User login' link to 'None'.

sign’s picture

sign commented
Status: Needs review » Fixed

Looks good to me. One small change to prevent undefined variable in the function _rootcandy_admin_links().

Thank you!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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