X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=WEB-INF%2Flib%2FttUser.class.php;h=859d08817d87e4811e3c29cd04615929298c8d0e;hb=2908e17af44b65d6a97e299edd03293ddd802802;hp=79f7ae77b229996fc28a4eae23c8dde6387d81dd;hpb=94737b99e973c5ff06e58d10f1c32c0955462a8f;p=timetracker.git diff --git a/WEB-INF/lib/ttUser.class.php b/WEB-INF/lib/ttUser.class.php index 79f7ae77..859d0881 100644 --- a/WEB-INF/lib/ttUser.class.php +++ b/WEB-INF/lib/ttUser.class.php @@ -366,47 +366,51 @@ class ttUser { return $user_list; } - // getGroups obtains an array consisting of: - // - A parent group (..) of a currently selected group, if available. - // - A currently selected group (.) represented by $behalf_group_id. - // - All subgroups (only immediate children) of a currently selected group. - function getGroups() { + // getGroupsForDropdown obtains an array of groups to populate "Group" dropdown. + // It consists of: + // - User home group. + // - The entire stack of groups all the way down to current on behalf group. + // - All immediate children of the current on behalf group. + // This allows user to navigate easily to home group, anything in between, and 1 level below. + function getGroupsForDropdown() { $mdb2 = getConnection(); + // Start with subgroups. + $groups = array(); + $group_id = $this->getActiveGroup(); + $sql = "select id, name from tt_groups where org_id = $this->org_id and parent_id = $group_id and status = 1"; + $res = $mdb2->query($sql); + if (!is_a($res, 'PEAR_Error')) { + while ($val = $res->fetchRow()) { + $groups[] = $val; + } + } + + // Add current on behalf group to the beginning of array. $selected_group_id = ($this->behalf_group_id ? $this->behalf_group_id : $this->group_id); $selected_group_name = ($this->behalf_group_id ? $this->behalf_group_name : $this->group_name); + array_unshift($groups, array('id'=>$selected_group_id,'name'=>$selected_group_name)); - // Start with parent group. - if ($selected_group_id != $this->group_id) { - // We are in one of subgroups, and a parent exists. - // Get parent group info. - $sql = "select parent_id from tt_groups where org_id = $this->org_id and id = $selected_group_id and status = 1"; + // Iterate all the way to the home group, starting with selected ("on behalf") group. + $current_group_id = $selected_group_id; + while ($current_group_id != $this->group_id) { + $sql = "select parent_id from tt_groups where org_id = $this->org_id and id = $current_group_id and status = 1"; $res = $mdb2->query($sql); - if (!is_a($res, 'PEAR_Error')) { - $val = $res->fetchRow(); - $parent_id = $val['parent_id']; - if ($parent_id) { - // Get parent group name. - $sql = "select name from tt_groups where org_id = $this->org_id and id = $parent_id and status = 1"; - $res = $mdb2->query($sql); - if (!is_a($res, 'PEAR_Error')) { - $val = $res->fetchRow(); - $groups[] = array('id'=>$parent_id,'name'=>$val['name']); - } - } - } - } + if (is_a($res, 'PEAR_Error')) return false; - // Add current group. - $groups[] = array('id'=>$selected_group_id,'name'=>$selected_group_name); - - // Add subgroups. - $sql = "select id, name from tt_groups where org_id = $this->org_id and parent_id = $selected_group_id and status = 1"; - //die($sql); - $res = $mdb2->query($sql); - if (!is_a($res, 'PEAR_Error')) { - while ($val = $res->fetchRow()) { - $groups[] = $val; + $val = $res->fetchRow(); + $parent_id = $val['parent_id']; + if ($parent_id) { + // Get parent group name. + $sql = "select name from tt_groups where org_id = $this->org_id and id = $parent_id and status = 1"; + $res = $mdb2->query($sql); + if (is_a($res, 'PEAR_Error')) return false; + $val = $res->fetchRow(); + if (!$val) return false; + array_unshift($groups, array('id'=>$parent_id,'name'=>$val['name'])); + $current_group_id = $parent_id; + } else { + return false; } } return $groups; @@ -419,7 +423,7 @@ class ttUser { if (!$group_id) $group_id = $this->getActiveGroup(); $sql = "select id, name, description from tt_groups where org_id = $this->org_id". - " and parent_id = $group_id and status is not null"; + " and parent_id = $group_id and status is not null order by upper(name)"; $res = $mdb2->query($sql); if (!is_a($res, 'PEAR_Error')) { while ($val = $res->fetchRow()) { @@ -518,6 +522,7 @@ class ttUser { if (!$group_id) $group_id = $this->getActiveGroup(); if (isset($fields['name'])) $name_part = ', name = '.$mdb2->quote($fields['name']); + if (isset($fields['description'])) $description_part = ', description = '.$mdb2->quote($fields['description']); if (isset($fields['currency'])) $currency_part = ', currency = '.$mdb2->quote($fields['currency']); if (isset($fields['lang'])) $lang_part = ', lang = '.$mdb2->quote($fields['lang']); if (isset($fields['decimal_mark'])) $decimal_mark_part = ', decimal_mark = '.$mdb2->quote($fields['decimal_mark']); @@ -538,7 +543,7 @@ class ttUser { if (isset($fields['workday_minutes'])) $workday_minutes_part = ', workday_minutes = '.$mdb2->quote($fields['workday_minutes']); $modified_part = ', modified = now(), modified_ip = '.$mdb2->quote($_SERVER['REMOTE_ADDR']).', modified_by = '.$mdb2->quote($this->id); - $parts = trim($name_part.$currency_part.$lang_part.$decimal_mark_part.$date_format_part. + $parts = trim($name_part.$description_part.$currency_part.$lang_part.$decimal_mark_part.$date_format_part. $time_format_part.$week_start_part.$tracking_mode_part.$task_required_part.$project_required_part.$record_type_part. $bcc_email_part.$allow_ip_part.$plugins_part.$config_part.$lock_spec_part.$workday_minutes_part.$modified_part, ','); @@ -651,4 +656,57 @@ class ttUser { return $max_rank; } + + // getUserPartForHeader constructs a string for user to display on pages header. + // It changes with "on behalf" attributes for both user and group. + function getUserPartForHeader() { + global $i18n; + if (!$this->id) return null; + + $user_part = htmlspecialchars($this->name); + $user_part .= ' - '.htmlspecialchars($this->role_name); + if ($this->behalf_id) { + $user_part .= ' '.$i18n->get('label.on_behalf').' '.htmlspecialchars($this->behalf_name).''; + } + if ($this->behalf_group_id) { + $user_part .= ', '.$i18n->get('label.on_behalf').' '.htmlspecialchars($this->behalf_group_name).''; + } else { + if ($this->group_name) // Note: we did not require group names in the past. + $user_part .= ', '.$this->group_name; + } + return $user_part; + } + + // setOnBehalfGroup sets on behalf group for the user in both the object and the session. + function setOnBehalfGroup($group_id) { + + // Unset things first. + $this->behalf_group_id = null; + $this->behalf_group_name = null; + unset($_SESSION['behalf_group_id']); + unset($_SESSION['behalf_group_name']); + + // Do not do anything if we don't have rights. + if (!$this->can('manage_subgroups')) return; + + // No need to set if the group is our home group. + if ($group_id == $this->group_id) return; + + // No need to set if the subgroup is not valid. + if (!$this->isSubgroupValid($group_id)) return; + + // We are good to set on behalf group. + $onBehalfGroupName = ttGroupHelper::getGroupName($group_id); + $_SESSION['behalf_group_id'] = $group_id; + $_SESSION['behalf_group_name'] = $onBehalfGroupName; + $this->behalf_group_id = $group_id; + $this->behalf_group_name = $onBehalfGroupName; + + // Question remains whether or not we need to adjust on behalf user. + // Adjusting for now. Test it and redesign if necessary. + unset($_SESSION['behalf_id']); + unset($_SESSION['behalf_name']); + $this->adjustBehalfId(); + return; + } }