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.

CommentFileSizeAuthor
#5 omega-debug-user-1-1408516-2.patch892 bytescellar door
#3 omega-debug-user-1-1408516-2.patch891 bytescellar door
#1 omega-debug-user-1-1408516-2.patch898 bytescellar door

Comments

cellar door’s picture

cellar door commented
Status: Active » Needs review
StatusFileSize
new omega-debug-user-1-1408516-2.patch898 bytes

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

fubhy’s picture

fubhy commented
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.

cellar door’s picture

cellar door commented
StatusFileSize
new omega-debug-user-1-1408516-2.patch891 bytes

Fixed w/ the new patch

cellar door’s picture

cellar door commented
Status: Needs work » Needs review
cellar door’s picture

cellar door commented
StatusFileSize
new omega-debug-user-1-1408516-2.patch892 bytes

Now with the right fix :)

cellar door’s picture

cellar door commented
Assigned: Unassigned » himerus
Status: Needs review » Patch (to be ported)

Simple fix

himerus’s picture

himerus commented
Assigned: himerus » Unassigned
Category: support » bug
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.

Status: Fixed » Closed (fixed)

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