diff --git a/admin/inc/class.uiaccounts.inc.php b/admin/inc/class.uiaccounts.inc.php index d509c1e4ea..8e697abac3 100755 --- a/admin/inc/class.uiaccounts.inc.php +++ b/admin/inc/class.uiaccounts.inc.php @@ -30,6 +30,22 @@ var $bo; var $nextmatchs; + var $apps_with_acl = array( + 'addressbook' => True, + 'todo' => True, + 'calendar' => True, + 'notes' => True, + 'projects' => True, + 'phonelog' => True, + 'infolog' => True, + 'filemanager' => True, + 'tts' => True, + 'bookmarks' => True, + 'img' => True, + 'netsaint' => True, + 'inv' => True, + 'phpbrain' => True, + ); function uiaccounts() { @@ -885,25 +901,6 @@ function create_edit_group($group_info,$_errors='') { - // Maybe we should list this in setup/setup.inc.php and put it into the - // phpgw_applications table ? (jengo) - $apps_with_acl = array( - 'addressbook' => True, - 'todo' => True, - 'calendar' => True, - 'notes' => True, - 'projects' => True, - 'phonelog' => True, - 'infolog' => True, - 'filemanager' => True, - 'tts' => True, - 'bookmarks' => True, - 'img' => True, - 'netsaint' => True, - 'inv' => True, - 'phpbrain' =>True - ); - $sbox = createobject('phpgwapi.sbox'); unset($GLOBALS['phpgw_info']['flags']['noheader']); @@ -971,8 +968,8 @@ } } - $perm_html = ''.lang('Application').' '.lang('ACL').''; - $perm_html = ''. + $perm_html = ''.lang('Application').''.lang('enabled').' / '.lang('ACL').''; + $perm_html = ''. $perm_html.$perm_html."\n"; $tr_color = $GLOBALS['phpgw_info']['theme']['row_off']; @@ -984,10 +981,10 @@ $tr_color = $this->nextmatchs->alternate_row_color(); $perm_html .= ''; } - $perm_html .= '' . $perm_display[$i][1] . '' - . '' - . ($apps_with_acl[$app] && $group_info['account_id']?' ' + . ($this->apps_with_acl[$app] && $group_info['account_id']?'':' ').''.($i & 1?'':'')."\n"; } @@ -1226,7 +1223,7 @@ #print $value['account_id'].''.$value['account_primary_group'] if ($value['account_id'] == $userData['account_primary_group']) { - $primary_group_select .= ' selected'; + $primary_group_select .= ' selected="1"'; } $primary_group_select .= '>' . $value['account_lid'] . ''."\n"; } @@ -1246,9 +1243,12 @@ { continue; } - $checked = (@$userData['account_permissions'][$app] || @$db_perms[$app]) && $_account_id ? ' checked' : ''; - $part[$i&1] = sprintf('%s', - $data['title'],$app,$checked); + $checked = (@$userData['account_permissions'][$app] || @$db_perms[$app]) && $_account_id ? ' checked="1"' : ''; + $part[$i&1] = sprintf('%s', + $data['title'],$app,$checked). + ($this->apps_with_acl[$app]?'':' ').''; if ($i & 1) { @@ -1258,7 +1258,7 @@ } if ($i & 1) { - $part[1] = ' '; + $part[1] = ' '; $appRightsOutput .= sprintf('%s%s',$this->nextmatchs->alternate_row_color(), $part[0], $part[1]); } @@ -1268,7 +1268,9 @@ 'primary_group_select' => ''."\n", 'permissions_list' - => $appRightsOutput + => $appRightsOutput, + 'lang_app' => lang('application'), + 'lang_acl' => lang('enabled').' / '.lang('ACL'), ); $t->set_var($var); diff --git a/admin/templates/default/account_form.tpl b/admin/templates/default/account_form.tpl index 98b2491dcb..b6f762df0d 100644 --- a/admin/templates/default/account_form.tpl +++ b/admin/templates/default/account_form.tpl @@ -52,6 +52,13 @@ {lang_primary_group} {primary_group_select}  + + + {lang_app} + {lang_acl} + {lang_app} + {lang_acl} + {permissions_list} diff --git a/admin/templates/default/account_form_ldap.tpl b/admin/templates/default/account_form_ldap.tpl index d2dc1a87cb..c43bb6f0e0 100644 --- a/admin/templates/default/account_form_ldap.tpl +++ b/admin/templates/default/account_form_ldap.tpl @@ -59,7 +59,14 @@ {lang_primary_group} {primary_group_select}  - + + + {lang_app} + {lang_acl} + {lang_app} + {lang_acl} + + {permissions_list} {form_buttons}