[BUGFIX] Proper cursor icon in access module
authorGeorg Ringer <mail@ringerge.org>
Mon, 16 Jan 2012 18:32:52 +0000 (19:32 +0100)
committerSteffen Ritter <info@rs-websystems.de>
Sun, 14 Oct 2012 16:07:42 +0000 (18:07 +0200)
The access module got those nice red/green icons but many
people don't know that those are clickable because the cursor
icon is wrong.

Change-Id: I8a16a1efff47007740b8dbfee77121d9a69ea3a7
Fixes: #33230
Releases: 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/12795
Reviewed-by: Marcus Schwemer
Tested-by: Marcus Schwemer
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
typo3/sysext/perm/mod1/class.sc_mod_web_perm_ajax.php

index a027d9e..211bf53 100644 (file)
@@ -377,15 +377,31 @@ class SC_mod_web_perm_ajax {
                global $LANG;
                $str = '';
 
-               $permissions = array(1,16,2,4,8);
+               $permissions = array(1, 16, 2, 4, 8);
                foreach ($permissions as $permission) {
                        if ($int&$permission) {
-                               $str .= t3lib_iconWorks::getSpriteIcon('status-status-permission-granted',array('tag'=>'a','title'=>$LANG->getLL($permission,1), 'onclick'=> 'WebPermissions.setPermissions('.$pageId.', '.$permission.', \'delete\', \''.$who.'\', '.$int.');'));
+                               $str .= t3lib_iconWorks::getSpriteIcon(
+                                       'status-status-permission-granted',
+                                       array(
+                                               'tag' => 'a',
+                                               'title' => $GLOBALS['LANG']->getLL($permission, TRUE),
+                                               'onclick'=> 'WebPermissions.setPermissions(' . $pageId . ', ' . $permission . ', \'delete\', \'' . $who . '\', ' . $int . ');',
+                                               'style' => 'cursor:pointer'
+                                       )
+                               );
                        } else {
-                               $str .= t3lib_iconWorks::getSpriteIcon('status-status-permission-denied',array('tag'=>'a','title'=>$LANG->getLL($permission,1),'onclick'=>'WebPermissions.setPermissions('.$pageId.', '.$permission.', \'add\', \''.$who.'\', '.$int.');'));
+                               $str .= t3lib_iconWorks::getSpriteIcon(
+                                       'status-status-permission-denied',
+                                       array(
+                                               'tag' => 'a',
+                                               'title' => $GLOBALS['LANG']->getLL($permission, TRUE),
+                                               'onclick' => 'WebPermissions.setPermissions(' . $pageId . ', ' . $permission . ', \'add\', \'' . $who . '\', ' . $int . ');',
+                                               'style' => 'cursor:pointer'
+                                       )
+                               );
                        }
                }
-               return '<span id="'.$pageId.'_'.$who.'">'.$str.'</span>';
+               return '<span id="' . $pageId . '_' . $who . '">' . $str . '</span>';
        }
 
 }