From c603cb5949eba24fc09012e8f2684f33078ed713 Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Sat, 16 Jan 2016 19:46:23 -0500
Subject: Simplify class ProjectDailyStats

---
 app/Model/ProjectDailyStats.php | 39 +++++++++++++--------------------------
 1 file changed, 13 insertions(+), 26 deletions(-)

(limited to 'app')

diff --git a/app/Model/ProjectDailyStats.php b/app/Model/ProjectDailyStats.php
index 12fecbe6..957ad51d 100644
--- a/app/Model/ProjectDailyStats.php
+++ b/app/Model/ProjectDailyStats.php
@@ -31,27 +31,14 @@ class ProjectDailyStats extends Base
 
         $lead_cycle_time = $this->averageLeadCycleTimeAnalytic->build($project_id);
 
-        $exists = $this->db->table(ProjectDailyStats::TABLE)
-            ->eq('day', $date)
-            ->eq('project_id', $project_id)
-            ->exists();
+        $this->db->table(self::TABLE)->eq('day', $date)->eq('project_id', $project_id)->remove();
 
-        if ($exists) {
-            $this->db->table(ProjectDailyStats::TABLE)
-                ->eq('project_id', $project_id)
-                ->eq('day', $date)
-                ->update(array(
-                    'avg_lead_time' => $lead_cycle_time['avg_lead_time'],
-                    'avg_cycle_time' => $lead_cycle_time['avg_cycle_time'],
-                ));
-        } else {
-            $this->db->table(ProjectDailyStats::TABLE)->insert(array(
-                'day' => $date,
-                'project_id' => $project_id,
-                'avg_lead_time' => $lead_cycle_time['avg_lead_time'],
-                'avg_cycle_time' => $lead_cycle_time['avg_cycle_time'],
-            ));
-        }
+        $this->db->table(self::TABLE)->insert(array(
+            'day' => $date,
+            'project_id' => $project_id,
+            'avg_lead_time' => $lead_cycle_time['avg_lead_time'],
+            'avg_cycle_time' => $lead_cycle_time['avg_cycle_time'],
+        ));
 
         $this->db->closeTransaction();
 
@@ -70,11 +57,11 @@ class ProjectDailyStats extends Base
     public function getRawMetrics($project_id, $from, $to)
     {
         return $this->db->table(self::TABLE)
-                        ->columns('day', 'avg_lead_time', 'avg_cycle_time')
-                        ->eq(self::TABLE.'.project_id', $project_id)
-                        ->gte('day', $from)
-                        ->lte('day', $to)
-                        ->asc(self::TABLE.'.day')
-                        ->findAll();
+            ->columns('day', 'avg_lead_time', 'avg_cycle_time')
+            ->eq('project_id', $project_id)
+            ->gte('day', $from)
+            ->lte('day', $to)
+            ->asc('day')
+            ->findAll();
     }
 }
-- 
cgit v1.2.3