Changed the meaning of override_punch_mode to apply only to lower roles.

[timetracker.git] / time_edit.php

diff --git a/time_edit.php b/time_edit.php
index 57de8338ccf4c9cc3517587ea197fc3081f892b0..1ff3ad8d610c3e4261632deea22b804fedaf31c0 100644 (file)
--- a/time_edit.php
+++ b/time_edit.php
@@ -178,7 +178,8 @@ if (MODE_PROJECTS_AND_TASKS == $user->tracking_mode) {
 if ((TYPE_START_FINISH == $user->record_type) || (TYPE_ALL == $user->record_type)) {
   $form->addInput(array('type'=>'text','name'=>'start','value'=>$cl_start,'onchange'=>"formDisable('start');"));
   $form->addInput(array('type'=>'text','name'=>'finish','value'=>$cl_finish,'onchange'=>"formDisable('finish');"));
-  if ($user->punch_mode && !$user->can('override_punch_mode')) {
+  if ($user->punch_mode && // Punch mode enabled for team.
+        !($user->behalf_id && $user->can('override_punch_mode'))) { // Cannot override for lower roles.
     // Make the start and finish fields read-only.
     $form->getElement('start')->setEnabled(false);
     $form->getElement('finish')->setEnabled(false);