Download & Extend
Omega » Issues

Debug Blocks User 1 Exception

Posted by FatGuyLaughing on January 17, 2012 at 11:53pm
Project:Omega
Version:7.x-3.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

When logged in as User 1 on the site I have to have one of the roles that are allowed to see the debug blocks. We should make User 1 the exception (this is usually the standard). I would hate to go and create a role that is only used for allowing User 1 to see the debug blocks.

Login or register to post comments

Comments

#1

Posted by Cellar Door on January 18, 2012 at 12:07am
Status:active» needs review

Patched the Alpha template.php to look for the uid as well as the role of the user.

AttachmentSizeStatusTest resultOperations
omega-debug-user-1-1408516-2.patch898 bytesIgnored: Check issue status.NoneNone

#2

Posted by fubhy on January 30, 2012 at 6:52pm
Status:needs review» needs work

+  global $user;
+  if (($theme->settings['debug']['access'] || $user->uid == 1) && ($theme->settings['debug']['grid'] || $theme->settings['debug']['block'])) {

Please use $GLOBALS['user']->uid instead. Other than that it looks fine to me.

#3

Posted by Cellar Door on February 6, 2012 at 10:39pm

Fixed w/ the new patch

AttachmentSizeStatusTest resultOperations
omega-debug-user-1-1408516-2.patch891 bytesIgnored: Check issue status.NoneNone

#4

Posted by Cellar Door on February 6, 2012 at 10:39pm
Status:needs work» needs review

#5

Posted by Cellar Door on February 6, 2012 at 10:45pm

Now with the right fix :)

AttachmentSizeStatusTest resultOperations
omega-debug-user-1-1408516-2.patch892 bytesIgnored: Check issue status.NoneNone

#6

Posted by Cellar Door on February 17, 2012 at 7:10pm
Assigned to:Anonymous» himerus
Status:needs review» patch (to be ported)

Simple fix

#7

Posted by himerus on February 19, 2012 at 5:16pm
Category:support request» bug report
Assigned to:himerus» Anonymous
Status:patch (to be ported)» fixed

patch failed to apply cleanly, but the one-liner was easy enough just to paste in.
Added to 3.1 release.

#8

Posted by System Message on March 4, 2012 at 5:20pm
Status:fixed» closed (fixed)

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

nobody click here