Merge lp:~raulraat/eventum/raul into lp:eventum

Proposed by Raul Raat
Status: Merged
Merged at revision: 4271
Proposed branch: lp:~raulraat/eventum/raul
Merge into: lp:eventum
Diff against target: 67 lines (+25/-2)
3 files modified
htdocs/view.php (+1/-0)
lib/eventum/class.time_tracking.php (+16/-2)
templates/time_tracking.tpl.html (+8/-0)
To merge this branch: bzr merge lp:~raulraat/eventum/raul
Reviewer Review Type Date Requested Status
Eventum Development Team Pending
Review via email: mp+47526@code.launchpad.net

Commit message

- if multiple users added timetracking to the issue show sum of time spent by user in timetracking section

Description of the change

- if multiple users added timetracking to the issue show sum of time spent by user in timetracking section

To post a comment you must log in.
lp:~raulraat/eventum/raul updated
4272. By Raul Raat

- code formatting

4273. By Raul Raat

- name things the same way everywhere

4274. By Raul Raat

- make total_time_by_user value array

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'htdocs/view.php'
--- htdocs/view.php 2011-01-20 18:45:20 +0000
+++ htdocs/view.php 2011-01-26 14:27:14 +0000
@@ -151,6 +151,7 @@
151 'checkins' => SCM::getCheckinList($issue_id),151 'checkins' => SCM::getCheckinList($issue_id),
152 'time_categories' => Time_Tracking::getAssocCategories(),152 'time_categories' => Time_Tracking::getAssocCategories(),
153 'time_entries' => $time_entries['list'],153 'time_entries' => $time_entries['list'],
154 'total_time_by_user' => $time_entries['total_time_by_user'],
154 'total_time_spent' => $time_entries['total_time_spent'],155 'total_time_spent' => $time_entries['total_time_spent'],
155 'impacts' => Impact_Analysis::getListing($issue_id),156 'impacts' => Impact_Analysis::getListing($issue_id),
156 'statuses' => $statuses,157 'statuses' => $statuses,
157158
=== modified file 'lib/eventum/class.time_tracking.php'
--- lib/eventum/class.time_tracking.php 2010-07-28 08:50:32 +0000
+++ lib/eventum/class.time_tracking.php 2011-01-26 14:27:14 +0000
@@ -317,16 +317,30 @@
317 return 0;317 return 0;
318 } else {318 } else {
319 $total_time_spent = 0;319 $total_time_spent = 0;
320 $total_time_by_user = array();
320 for ($i = 0; $i < count($res); $i++) {321 for ($i = 0; $i < count($res); $i++) {
321 $res[$i]["ttr_summary"] = Link_Filter::processText(Issue::getProjectID($issue_id), nl2br(htmlspecialchars($res[$i]["ttr_summary"])));322 $res[$i]["ttr_summary"] = Link_Filter::processText(Issue::getProjectID($issue_id), nl2br(htmlspecialchars($res[$i]["ttr_summary"])));
322 $res[$i]["formatted_time"] = Misc::getFormattedTime($res[$i]["ttr_time_spent"]);323 $res[$i]["formatted_time"] = Misc::getFormattedTime($res[$i]["ttr_time_spent"]);
323 $res[$i]["ttr_created_date"] = Date_Helper::getFormattedDate($res[$i]["ttr_created_date"]);324 $res[$i]["ttr_created_date"] = Date_Helper::getFormattedDate($res[$i]["ttr_created_date"]);
324325
326 if (isset($total_time_by_user[$res[$i]['ttr_usr_id']])) {
327 $total_time_by_user[$res[$i]['ttr_usr_id']]['time_spent'] += $res[$i]['ttr_time_spent'];
328 } else {
329 $total_time_by_user[$res[$i]['ttr_usr_id']] = array(
330 'usr_full_name' => $res[$i]['usr_full_name'],
331 'time_spent' => $res[$i]['ttr_time_spent']
332 );
333 }
325 $total_time_spent += $res[$i]["ttr_time_spent"];334 $total_time_spent += $res[$i]["ttr_time_spent"];
326 }335 }
336 usort($total_time_by_user, create_function('$a,$b', 'return $a["time_spent"]<$b["time_spent"];'));
337 foreach ($total_time_by_user as &$item) {
338 $item['time_spent'] = Misc::getFormattedTime($item['time_spent']);
339 }
327 return array(340 return array(
328 "total_time_spent" => Misc::getFormattedTime($total_time_spent),341 "total_time_spent" => Misc::getFormattedTime($total_time_spent),
329 "list" => $res342 "total_time_by_user" => $total_time_by_user,
343 "list" => $res
330 );344 );
331 }345 }
332 }346 }
333347
=== modified file 'templates/time_tracking.tpl.html'
--- templates/time_tracking.tpl.html 2010-10-07 11:42:43 +0000
+++ templates/time_tracking.tpl.html 2011-01-26 14:27:14 +0000
@@ -64,6 +64,14 @@
64 <td class="default">{$time_entries[i].ttr_summary}</td>64 <td class="default">{$time_entries[i].ttr_summary}</td>
65 </tr>65 </tr>
66 {if $smarty.section.i.last}66 {if $smarty.section.i.last}
67 {if count($total_time_by_user) > 1}
68 {foreach from=$total_time_by_user item=item}
69 <tr>
70 <td bgcolor="{$internal_color}" colspan="3" class="default_white" align="right" width="30%">{$item.usr_full_name}:</td>
71 <td bgcolor="{$dark_color}" colspan="3" class="default" width="70%">{$item.time_spent}</td>
72 </tr>
73 {/foreach}
74 {/if}
67 <tr>75 <tr>
68 <td bgcolor="{$internal_color}" colspan="3" class="default_white" align="right" width="30%">{t}Total Time Spent{/t}:</td>76 <td bgcolor="{$internal_color}" colspan="3" class="default_white" align="right" width="30%">{t}Total Time Spent{/t}:</td>
69 <td bgcolor="{$dark_color}" colspan="3" class="default" width="70%">{$total_time_spent}</td>77 <td bgcolor="{$dark_color}" colspan="3" class="default" width="70%">{$total_time_spent}</td>

Subscribers

People subscribed via source and target branches