X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=WEB-INF%2Flib%2FttReportHelper.class.php;h=582d289ddb6153e6b29c0e2b5a564869fe9bf68c;hb=a331cbeee550ed74a5333c6ac361418161d84bd7;hp=6887a457fc1e5b889b7c83f67214fd91fd2e4ec0;hpb=7293692a2d72451f2ab66723b6f75d22d161fded;p=timetracker.git

diff --git a/WEB-INF/lib/ttReportHelper.class.php b/WEB-INF/lib/ttReportHelper.class.php
index 6887a457..582d289d 100644
--- a/WEB-INF/lib/ttReportHelper.class.php
+++ b/WEB-INF/lib/ttReportHelper.class.php
@@ -589,42 +589,41 @@ class ttReportHelper {
 
     // Obtain items for report.
     $res = $mdb2->query($sql);
-    if (!is_a($res, 'PEAR_Error')) {
-      while ($val = $res->fetchRow()) {
-        if ($convertTo12Hour) {
-          if($val['start'] != '')
-            $val['start'] = ttTimeHelper::to12HourFormat($val['start']);
-          if($val['finish'] != '')
-            $val['finish'] = ttTimeHelper::to12HourFormat($val['finish']);
-        }
-        if (isset($val['cost'])) {
-          if ('.' != $user->decimal_mark)
-            $val['cost'] = str_replace('.', $user->decimal_mark, $val['cost']);
-        }
-        if (isset($val['expense'])) {
-          if ('.' != $user->decimal_mark)
-            $val['expense'] = str_replace('.', $user->decimal_mark, $val['expense']);
-        }
-        if ('no_grouping' != $group_by_option) {
-          $val['grouped_by'] = $val[$group_by_option];
-          if ('date' == $group_by_option) {
-            // This is needed to get the date in user date format.
-            $o_date = new DateAndTime(DB_DATEFORMAT, $val['grouped_by']);
-            $val['grouped_by'] = $o_date->toString($user->date_format);
-            unset($o_date);
-          }
+    if (is_a($res, 'PEAR_Error')) die($res->getMessage());
+
+    while ($val = $res->fetchRow()) {
+      if ($convertTo12Hour) {
+        if($val['start'] != '')
+          $val['start'] = ttTimeHelper::to12HourFormat($val['start']);
+        if($val['finish'] != '')
+          $val['finish'] = ttTimeHelper::to12HourFormat($val['finish']);
+      }
+      if (isset($val['cost'])) {
+        if ('.' != $user->decimal_mark)
+          $val['cost'] = str_replace('.', $user->decimal_mark, $val['cost']);
+      }
+      if (isset($val['expense'])) {
+        if ('.' != $user->decimal_mark)
+          $val['expense'] = str_replace('.', $user->decimal_mark, $val['expense']);
+      }
+      if ('no_grouping' != $group_by_option) {
+        $val['grouped_by'] = $val[$group_by_option];
+        if ('date' == $group_by_option) {
+          // This is needed to get the date in user date format.
+          $o_date = new DateAndTime(DB_DATEFORMAT, $val['grouped_by']);
+          $val['grouped_by'] = $o_date->toString($user->date_format);
+          unset($o_date);
         }
+      }
 
-        // This is needed to get the date in user date format.
-        $o_date = new DateAndTime(DB_DATEFORMAT, $val['date']);
-        $val['date'] = $o_date->toString($user->date_format);
-        unset($o_date);
+      // This is needed to get the date in user date format.
+      $o_date = new DateAndTime(DB_DATEFORMAT, $val['date']);
+      $val['date'] = $o_date->toString($user->date_format);
+      unset($o_date);
 
-        $row = $val;
-        $report_items[] = $row;
-      }
-    } else
-      die($res->getMessage());
+      $row = $val;
+      $report_items[] = $row;
+    }
 
     return $report_items;
   }
@@ -633,18 +632,18 @@ class ttReportHelper {
   // Without expenses, it's a simple select with group by.
   // With expenses, it becomes a select with group by from a combined set of records obtained with "union all".
   static function getSubtotals($bean) {
-  	global $user;
-  	
+    global $user;
+
     $group_by_option = $bean->getAttribute('group_by');
     if ('no_grouping' == $group_by_option) return null;
-    
+
     $mdb2 = getConnection();
-    
+
     // Start with sql to obtain subtotals for time items. This simple sql will be used when we have no expenses.
-    
+
     // Determine group by field and a required join.
     switch ($group_by_option) {
-  	  case 'date':
+      case 'date':
         $group_field = 'l.date';
         $group_join = '';
         break;
@@ -657,54 +656,54 @@ class ttReportHelper {
         $group_join = 'left join tt_clients c on (l.client_id = c.id) ';
         break;
       case 'project':
-  	    $group_field = 'p.name';
+        $group_field = 'p.name';
         $group_join = 'left join tt_projects p on (l.project_id = p.id) ';
-  	    break;
+        break;
       case 'task':
-  	    $group_field = 't.name';
+        $group_field = 't.name';
         $group_join = 'left join tt_tasks t on (l.task_id = t.id) ';
-  	    break;
+        break;
       case 'cf_1':
-  	    $group_field = 'cfo.value';
-  	    $custom_fields = new CustomFields($user->team_id);
-  	    if ($custom_fields->fields[0]['type'] == CustomFields::TYPE_TEXT)
+        $group_field = 'cfo.value';
+        $custom_fields = new CustomFields($user->team_id);
+        if ($custom_fields->fields[0]['type'] == CustomFields::TYPE_TEXT)
           $group_join = 'left join tt_custom_field_log cfl on (l.id = cfl.log_id and cfl.status = 1) left join tt_custom_field_options cfo on (cfl.value = cfo.id) ';
         elseif ($custom_fields->fields[0]['type'] == CustomFields::TYPE_DROPDOWN)
           $group_join = 'left join tt_custom_field_log cfl on (l.id = cfl.log_id and cfl.status = 1) left join tt_custom_field_options cfo on (cfl.option_id = cfo.id) ';
-    	  break;
+        break;
     }
 
     $where = ttReportHelper::getWhere($bean);
     if ($bean->getAttribute('chcost')) {
       if (MODE_TIME == $user->tracking_mode) {
-      	if ($group_by_option != 'user')
-      	  $left_join = 'left join tt_users u on (l.user_id = u.id)';
+        if ($group_by_option != 'user')
+          $left_join = 'left join tt_users u on (l.user_id = u.id)';
         $sql = "select $group_field as group_field, sum(time_to_sec(l.duration)) as time, 
           sum(cast(l.billable * coalesce(u.rate, 0) * time_to_sec(l.duration)/3600 as decimal(10, 2))) as cost,
           null as expenses from tt_log l
-          $group_join $left_join $where group by $group_field";      	
+          $group_join $left_join $where group by $group_field";
       } else {
         // If we are including cost and tracking projects, our query (the same as above) needs to join the tt_user_project_binds table.
         $sql = "select $group_field as group_field, sum(time_to_sec(l.duration)) as time, 
           sum(cast(l.billable * coalesce(upb.rate, 0) * time_to_sec(l.duration)/3600 as decimal(10,2))) as cost,
-          null as expenses from tt_log l 
+          null as expenses from tt_log l
           $group_join
           left join tt_user_project_binds upb on (l.user_id = upb.user_id and l.project_id = upb.project_id) $where group by $group_field";
       }
     } else {
-  	   $sql = "select $group_field as group_field, sum(time_to_sec(l.duration)) as time, null as expenses from tt_log l 
+      $sql = "select $group_field as group_field, sum(time_to_sec(l.duration)) as time, null as expenses from tt_log l
          $group_join $where group by $group_field";
     }
     // By now we have sql for time items.
-    
+
     // However, when we have expenses, we need to do a union with a separate query for expense items from tt_expense_items table.
     if ($bean->getAttribute('chcost') && in_array('ex', explode(',', $user->plugins))) { // if ex(penses) plugin is enabled
-    	
+
       // Determine group by field and a required join.
       $group_join = null;
       $group_field = 'null';
       switch ($group_by_option) {
-  	    case 'date':
+        case 'date':
           $group_field = 'ei.date';
           $group_join = '';
           break;
@@ -717,9 +716,9 @@ class ttReportHelper {
           $group_join = 'left join tt_clients c on (ei.client_id = c.id) ';
           break;
         case 'project':
-  	      $group_field = 'p.name';
+          $group_field = 'p.name';
           $group_join = 'left join tt_projects p on (ei.project_id = p.id) ';
-  	      break;
+          break;
       }
 
       $where = ttReportHelper::getExpenseWhere($bean);
@@ -727,37 +726,36 @@ class ttReportHelper {
         $group_join $where";
       // Add a "group by" clause if we are grouping.
       if ('null' != $group_field) $sql_for_expenses .= " group by $group_field";
-         
+
       // Create a combined query.
       $sql = "select group_field, sum(time) as time, sum(cost) as cost, sum(expenses) as expenses from (($sql) union all ($sql_for_expenses)) t group by group_field";
     }
-    
+
     // Execute query.
     $res = $mdb2->query($sql);
-    if (!is_a($res, 'PEAR_Error')) {
-  	  while ($val = $res->fetchRow()) {
-  	  	if ('date' == $group_by_option) {
-          // This is needed to get the date in user date format.
-          $o_date = new DateAndTime(DB_DATEFORMAT, $val['group_field']);
-          $val['group_field'] = $o_date->toString($user->date_format);
-          unset($o_date);
-  	  	}
-  	  	$time = $val['time'] ? sec_to_time_fmt_hm($val['time']) : null;
-        if ($bean->getAttribute('chcost')) {
-  	      if ('.' != $user->decimal_mark) {
-  	        $val['cost'] = str_replace('.', $user->decimal_mark, $val['cost']);
-  	        $val['expenses'] = str_replace('.', $user->decimal_mark, $val['expenses']);
-  	      }
-      	  $subtotals[$val['group_field']] = array('name'=>$val['group_field'],'time'=>$time,'cost'=>$val['cost'],'expenses'=>$val['expenses']);
-        } else
-          $subtotals[$val['group_field']] = array('name'=>$val['group_field'],'time'=>$time);
+    if (is_a($res, 'PEAR_Error')) die($res->getMessage());
+
+    while ($val = $res->fetchRow()) {
+      if ('date' == $group_by_option) {
+        // This is needed to get the date in user date format.
+        $o_date = new DateAndTime(DB_DATEFORMAT, $val['group_field']);
+        $val['group_field'] = $o_date->toString($user->date_format);
+        unset($o_date);
       }
-    } else
-      die($res->getMessage());
+      $time = $val['time'] ? sec_to_time_fmt_hm($val['time']) : null;
+      if ($bean->getAttribute('chcost')) {
+        if ('.' != $user->decimal_mark) {
+          $val['cost'] = str_replace('.', $user->decimal_mark, $val['cost']);
+          $val['expenses'] = str_replace('.', $user->decimal_mark, $val['expenses']);
+        }
+        $subtotals[$val['group_field']] = array('name'=>$val['group_field'],'time'=>$time,'cost'=>$val['cost'],'expenses'=>$val['expenses']);
+      } else
+        $subtotals[$val['group_field']] = array('name'=>$val['group_field'],'time'=>$time);
+    }
 
     return $subtotals;
   }
-  
+
   // getFavSubtotals calculates report items subtotals when a favorite report is grouped by.
   // Without expenses, it's a simple select with group by.
   // With expenses, it becomes a select with group by from a combined set of records obtained with "union all".