Fix #5661 - MTrack - daily email large and no branch

[web.mtrack] / MTrack / CommitChecker.php

diff --git a/MTrack/CommitChecker.php b/MTrack/CommitChecker.php
index 3a9cc79..e1c2091 100644 (file)
--- a/MTrack/CommitChecker.php
+++ b/MTrack/CommitChecker.php
@@ -196,7 +196,10 @@ class MTrack_CommitChecker {
             $c->changelog   = $bridge->getCommitMessage();
             $c->changeby    = $this->authUser->email; //???
             $c->changeby_id = $this->authUser->id; //???
-            $c->ctime       = time();
+            $c->branch       = $bridge->branch;
+            //print_r($bridge);exit;
+            $c->ctime       = isset($bridge->props['Date']) ? strtotime($bridge->props['Date']) : time();
+            $c->fileActions = $bridge->fileActions;
             $changes[] = $c;
         }
         return $changes;
@@ -223,7 +226,7 @@ class MTrack_CommitChecker {
         $changes = $this->_getChanges($bridge);
         
         
-        print_R($changes);
+        //print_R($changes);
         
 
         // Deferred by tid
@@ -263,15 +266,23 @@ class MTrack_CommitChecker {
             }
             
             // apply changes to tickets
+            $T = false;
             foreach ($tickets as $tkt => $act) {
                 // removed all the code that handles hashed ticked ids...
-                $T = clone($this->ticketProvider);
+                //DB_DataObject::DebugLevel(1);
+                $T = DB_DataObject::Factory('mtrack_ticket');
+                $T->project_id = $this->repo->project_id;
                 if (!$T->get($tkt)) {
                     continue;
                 }
+                break;
                 
                 $T_by_tid[$T->id] = $T;
             }
+            
+            if (!$T) {
+                continue;
+            }
             /*
             $accounted = false;
             
@@ -311,16 +322,31 @@ class MTrack_CommitChecker {
                   $log .= " (on behalf of [user:$c->changeby]) ";
                 }
                 */
+                
+            // for the stuff below we do not currently support multiple tickets..
             
             $log .= $c->changelog;
+            if (!isset($deferred[$T->id])) {
+                $deferred[$T->id] = array(
+                    'comments' => array(),
+                    'changes' => array(),
+                    'act' => array(),
+                    'ticket' => $T
+                    
+                );
+            }
+            
             
             $deferred[$T->id]['comments'][] = $log;
+            $deferred[$T->id]['changes'][] = $c;
+            
+            
             
             if (isset($act['spent']) && $c->changeby != $me) {
                 $spent_by_tid_by_user[$T->id][$c->changeby_id][] = $act['spent'];
                 unset($act['spent']);
             }
-            $deferred[$T->tid]['act'][] = $act;
+            $deferred[$T->id]['act'][] = $act;
  
             //??? 
             $this->checkVeto('postCommit', $log, $fqfiles, $actions);