A bit of refactoring.
authorNik Okuntseff <support@anuko.com>
Mon, 20 May 2019 20:57:24 +0000 (20:57 +0000)
committerNik Okuntseff <support@anuko.com>
Mon, 20 May 2019 20:57:24 +0000 (20:57 +0000)
WEB-INF/lib/ttGroupHelper.class.php
WEB-INF/lib/ttUser.class.php
WEB-INF/templates/footer.tpl
group_edit.php
groups.php

index 167c620..05a3156 100644 (file)
@@ -34,9 +34,10 @@ class ttGroupHelper {
 
   // The getGroupName function returns group name.
   static function getGroupName($group_id) {
+    global $user;
     $mdb2 = getConnection();
 
-    $sql = "select name from tt_groups where id = $group_id and (status = 1 or status = 0)";
+    $sql = "select name from tt_groups where id = $group_id and org_id = $user->org_id and (status = 1 or status = 0)";
     $res = $mdb2->query($sql);
 
     if (!is_a($res, 'PEAR_Error')) {
index 8f9578d..078d6ec 100644 (file)
@@ -510,60 +510,10 @@ class ttUser {
     return $user_list;
   }
 
-  // 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->getGroup();
-    $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));
-
-    // 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')) return false;
-
-      $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;
-  }
-
-  // getGroupsForDropdown2 obtains an array of groups to populate the "Group" dropdown.
-  // It consists of the entire tree starting from user home group down.
+  // getGroupsForDropdown obtains an array of groups to populate the "Group" dropdown.
+  // It consists of the entire tree starting from user home group.
   // Group name is prefixed with additional characters to indicate subgroups level.
-  function getGroupsForDropdown2() {
+  function getGroupsForDropdown() {
     global $user;
 
     // Start with user home group.
@@ -571,12 +521,12 @@ class ttUser {
     $subgroup_level = 0;
     $group_id = $user->group_id;
 
-    $this->addGroup($groups, $group_id, $subgroup_level);
+    $this->addGroupToDropdown($groups, $group_id, $subgroup_level);
     return $groups;
   }
 
-  // addGroup is a recursive function to populate a tree of groups.
-  function addGroup(&$groups, $group_id, $subgroup_level) {
+  // addGroup is a recursive function to populate a tree of groups, used with getGroupsForDropdown().
+  function addGroupToDropdown(&$groups, $group_id, $subgroup_level) {
     // Add indentation markup to indicate subdirectory level.
     for ($i = 0; $i < $subgroup_level; $i++) {
       $name .= '🛑'; // Unicode stop sign.
@@ -588,7 +538,7 @@ class ttUser {
 
     $subgroups = $this->getSubgroups($group_id);
     foreach($subgroups as $subgroup) {
-      $this->addGroup($groups, $subgroup['id'], $subgroup_level+1);
+      $this->addGroupToDropdown($groups, $subgroup['id'], $subgroup_level+1);
     }
   }
 
index 54bf35f..cbd5f91 100644 (file)
@@ -12,7 +12,7 @@
       <br>
       <table cellspacing="0" cellpadding="4" width="100%" border="0">
         <tr>
-          <td align="center">&nbsp;Anuko Time Tracker 1.19.4.5009 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |
+          <td align="center">&nbsp;Anuko Time Tracker 1.19.4.5010 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |
             <a href="https://www.anuko.com/lp/tt_4.htm" target="_blank">{$i18n.footer.credits}</a> |
             <a href="https://www.anuko.com/lp/tt_5.htm" target="_blank">{$i18n.footer.license}</a> |
             <a href="https://www.anuko.com/lp/tt_7.htm" target="_blank">{$i18n.footer.improve}</a>
index 77b464b..1df99a1 100644 (file)
@@ -68,7 +68,7 @@ if ($request->isPost() && $groupChanged) {
  $user->setOnBehalfGroup($group_id);
 }
 
-$groups = $user->getGroupsForDropdown2();
+$groups = $user->getGroupsForDropdown();
 $group = ttGroupHelper::getGroupAttrs($group_id);
 $config = $user->getConfigHelper();
 
index 449bd63..f335153 100644 (file)
@@ -49,7 +49,7 @@ if ($request->isPost()) {
 }
 
 $form = new Form('subgroupsForm');
-$groups = $user->getGroupsForDropdown2();
+$groups = $user->getGroupsForDropdown();
 if (count($groups) > 1) {
   $form->addInput(array('type'=>'combobox',
     'onchange'=>'this.form.submit();',