From a6582c9d010144319e8baf1cc2fb55a740c918c6 Mon Sep 17 00:00:00 2001
From: Eric Davis <edavis@littlestreamsoftware.com>
Date: Thu, 15 Sep 2011 10:58:27 -0700
Subject: [PATCH] Refactor: extract to common method

---
 .../patches/project_patch.rb                  | 20 ++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/lib/redmine_contracts/patches/project_patch.rb b/lib/redmine_contracts/patches/project_patch.rb
index ae5293b..195b7eb 100644
--- a/lib/redmine_contracts/patches/project_patch.rb
+++ b/lib/redmine_contracts/patches/project_patch.rb
@@ -17,15 +17,25 @@ module RedmineContracts
 
       module InstanceMethods
         def billable_activities
-          activities.partition do |activity|
-            activity.billable?
-          end.first
+          activities_sorted_by_billable[:billable]
         end
 
         def non_billable_activities
-          activities.partition do |activity|
+          activities_sorted_by_billable[:non_billable]
+        end
+
+        private
+
+        def activities_sorted_by_billable
+          split_activities = activities.partition do |activity|
             activity.billable?
-          end.second
+          end
+
+          {
+            :billable => split_activities.first,
+            :non_billable => split_activities.second
+          }
+
         end
       end
     end