A bit of refactoring.

[timetracker.git] / time_edit.php

diff --git a/time_edit.php b/time_edit.php
index b656c23de60a8967d0814a66e2f4bfccae11130c..d24d7f19b8a8b9c004a4687a822a8358c5ede327 100644 (file)
--- a/time_edit.php
+++ b/time_edit.php
@@ -206,7 +206,7 @@ if ($custom_fields && $custom_fields->fields[0]) {
 $form->addInput(array('type'=>'hidden','name'=>'id','value'=>$cl_id));
 if ($user->isPluginEnabled('iv'))
   $form->addInput(array('type'=>'checkbox','name'=>'billable','value'=>$cl_billable));
-if ($user->isPluginEnabled('ps'))
+if ($user->canManageTeam() && $user->isPluginEnabled('ps'))
   $form->addInput(array('type'=>'checkbox','name'=>'paid','value'=>$cl_paid));
 $form->addInput(array('type'=>'hidden','name'=>'browser_today','value'=>'')); // User current date, which gets filled in on btn_save or btn_copy click.
 $form->addInput(array('type'=>'submit','name'=>'btn_save','onclick'=>'browser_today.value=get_date()','value'=>$i18n->getKey('button.save')));
@@ -248,7 +248,7 @@ if ($request->isPost()) {
         $err->add($i18n->getKey('error.empty'), $i18n->getKey('label.duration'));
     }
   } else {
-    if (!ttTimeHelper::isValidDuration($cl_duration))
+    if (false === ttTimeHelper::postedDurationToMinutes($cl_duration))
       $err->add($i18n->getKey('error.field'), $i18n->getKey('label.duration'));
   }
   if (!ttValidDate($cl_date)) $err->add($i18n->getKey('error.field'), $i18n->getKey('label.date'));