Improved safety of timesheet assignment by adding an inner join.

[timetracker.git] / WEB-INF / lib / ttTimesheetHelper.class.php

diff --git a/WEB-INF/lib/ttTimesheetHelper.class.php b/WEB-INF/lib/ttTimesheetHelper.class.php
index 89571510889370d8b03360a042acc2aeadba4e54..a348cb57b50a7bd14aeba7ed2d1abbb9bbf181e7 100644 (file)
--- a/WEB-INF/lib/ttTimesheetHelper.class.php
+++ b/WEB-INF/lib/ttTimesheetHelper.class.php
@@ -442,6 +442,7 @@ class ttTimesheetHelper {
     // Parts for client and project.
     if ($options['client_id']) $client_part = ' and client_id = '.(int)$options['client_id'];
     if ($options['project_id']) $project_part = ' and project_id = '.(int)$options['project_id'];
     // Parts for client and project.
     if ($options['client_id']) $client_part = ' and client_id = '.(int)$options['client_id'];
     if ($options['project_id']) $project_part = ' and project_id = '.(int)$options['project_id'];

+    // TODO: test and fix the above for NULL client and project ids...

 
     // Determine start and end dates.
     $dateFormat = $user->getDateFormat();
 
     // Determine start and end dates.
     $dateFormat = $user->getDateFormat();