Implemented week view as a configurable plugin.
authorNik Okuntseff <support@anuko.com>
Sat, 3 Feb 2018 17:11:27 +0000 (17:11 +0000)
committerNik Okuntseff <support@anuko.com>
Sat, 3 Feb 2018 17:11:27 +0000 (17:11 +0000)
WEB-INF/lib/ttTeamHelper.class.php
WEB-INF/lib/ttWeekViewHelper.class.php
WEB-INF/templates/footer.tpl
WEB-INF/templates/profile_edit.tpl
WEB-INF/templates/time.tpl
WEB-INF/templates/week.tpl
profile_edit.php
week.php

index 868e67f..217c07a 100644 (file)
@@ -1026,7 +1026,7 @@ class ttTeamHelper {
   }
 
   // enablePlugin either enables or disables a specific plugin for team.
-  function enablePlugin($plugin, $enable = true)
+  static function enablePlugin($plugin, $enable = true)
   {
     global $user;
     if (!$user->canManageTeam())
index e74a08f..130cc3d 100644 (file)
@@ -150,6 +150,7 @@ class ttWeekViewHelper {
   //   )
   // );
   static function getDataForWeekView($records, $dayHeaders) {
+    global $user;
     global $i18n;
 
     $dataArray = array();
@@ -161,12 +162,14 @@ class ttWeekViewHelper {
       $control_id = '0_'. $dayHeaders[$i];
       $dataArray[0][$dayHeaders[$i]] = array('control_id' => $control_id, 'tt_log_id' => null,'duration' => null);
     }
-    // Construct the second row for daily comments for a brand new entry.
-    $dataArray[] = array('row_id' => null,'label' => $i18n->getKey('label.notes').':'); // Insert row.
-    // Insert empty cells with proper control ids.
-    for ($i = 0; $i < 7; $i++) {
-      $control_id = '1_'. $dayHeaders[$i];
-      $dataArray[1][$dayHeaders[$i]] = array('control_id' => $control_id, 'tt_log_id' => null,'note' => null);
+    if ($user->isPluginEnabled('wvns')) {
+      // Construct the second row for daily comments for a brand new entry.
+      $dataArray[] = array('row_id' => null,'label' => $i18n->getKey('label.notes').':'); // Insert row.
+      // Insert empty cells with proper control ids.
+      for ($i = 0; $i < 7; $i++) {
+        $control_id = '1_'. $dayHeaders[$i];
+        $dataArray[1][$dayHeaders[$i]] = array('control_id' => $control_id, 'tt_log_id' => null,'note' => null);
+      }
     }
 
     // Iterate through records and build $dataArray cell by cell.
@@ -193,20 +196,24 @@ class ttWeekViewHelper {
           $dataArray[$pos][$dayHeaders[$i]] = array('control_id' => $control_id, 'tt_log_id' => null,'duration' => null);
         }
         // Insert row for comments.
-        $dataArray[] = array('row_id' => $row_id.'_notes','label' => $i18n->getKey('label.notes').':');
-        $pos++;
-        // Insert empty cells with proper control ids.
-        for ($i = 0; $i < 7; $i++) {
-          $control_id = $pos.'_'. $dayHeaders[$i];
-          $dataArray[$pos][$dayHeaders[$i]] = array('control_id' => $control_id, 'tt_log_id' => null,'note' => null);
+        if ($user->isPluginEnabled('wvns')) {
+          $dataArray[] = array('row_id' => $row_id.'_notes','label' => $i18n->getKey('label.notes').':');
+          $pos++;
+          // Insert empty cells with proper control ids.
+          for ($i = 0; $i < 7; $i++) {
+            $control_id = $pos.'_'. $dayHeaders[$i];
+            $dataArray[$pos][$dayHeaders[$i]] = array('control_id' => $control_id, 'tt_log_id' => null,'note' => null);
+          }
+          $pos--;
         }
-        $pos--;
       }
       // Insert actual cell data from $record (one cell only).
       $dataArray[$pos][$day_header] = array('control_id' => $pos.'_'. $day_header, 'tt_log_id' => $record['id'],'duration' => $record['duration']);
-      // Insert existing comment from $record into the duration cell.
-      $pos++;
-      $dataArray[$pos][$day_header] = array('control_id' => $pos.'_'. $day_header, 'tt_log_id' => $record['id'],'note' => $record['comment']);
+      // Insert existing comment from $record into the comment cell.
+      if ($user->isPluginEnabled('wvns')) {
+        $pos++;
+        $dataArray[$pos][$day_header] = array('control_id' => $pos.'_'. $day_header, 'tt_log_id' => $record['id'],'note' => $record['comment']);
+      }
     }
     return $dataArray;
   }
index 38bb731..8799451 100644 (file)
@@ -12,7 +12,7 @@
       <br>
       <table cellspacing="0" cellpadding="4" width="100%" border="0">
         <tr>
-          <td align="center">&nbsp;Anuko Time Tracker 1.17.10.3871 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |
+          <td align="center">&nbsp;Anuko Time Tracker 1.17.11.3872 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |
             <a href="https://www.anuko.com/lp/tt_4.htm" target="_blank">{$i18n.footer.credits}</a> |
             <a href="https://www.anuko.com/lp/tt_5.htm" target="_blank">{$i18n.footer.license}</a> |
             <a href="https://www.anuko.com/lp/tt_7.htm" target="_blank">{$i18n.footer.improve}</a>
index 6e3b81e..4967259 100644 (file)
@@ -81,6 +81,14 @@ function handlePluginCheckboxes() {
   } else {
     configureLabel.style.visibility = "hidden";
   }
+
+  var weekViewCheckbox = document.getElementById("week_view");
+  configureLabel = document.getElementById("week_view_config");
+  if (weekViewCheckbox.checked){
+    configureLabel.style.visibility = "visible";
+  } else {
+    configureLabel.style.visibility = "hidden";
+  }
 }
 </script>
 
@@ -229,12 +237,10 @@ function handlePluginCheckboxes() {
             <td align="right" nowrap>{$forms.profileForm.quotas.control}</td>
             <td><label for="quotas">{$i18n.label.monthly_quotas}</label> <span id="quotas_config"><a href="quotas.php">{$i18n.label.configure}</a></span></td>
           </tr>
-  {if $smarty.const.WEEK_VIEW_DEBUG == 1}
           <tr>
             <td align="right" nowrap>{$forms.profileForm.week_view.control}</td>
-            <td><label for="quotas">{$i18n.label.week_view}</label> <span id="week_view_config"><a href="week_view.php">{$i18n.label.configure}</a></span></td>
+            <td><label for="week_view">{$i18n.label.week_view}</label> <span id="week_view_config"><a href="week_view.php">{$i18n.label.configure}</a></span></td>
           </tr>
-  {/if}
 {/if}
 
           <tr>
index 355b972..f532be4 100644 (file)
@@ -8,11 +8,13 @@
 
 {$forms.timeRecordForm.open}
 <table cellspacing="4" cellpadding="0" border="0">
+{if $user->isPluginEnabled('wv')}
   <tr>
     <td align="center" colspan=2">
       <a href="time.php?date={$selected_date->toString()}">{$i18n.label.day_view}</a>&nbsp;/&nbsp;<a href="week.php?date={$selected_date->toString()}">{$i18n.label.week_view}</a>
     </td>
   </tr>
+{/if}
   <tr>
     <td valign="top">
       <table>
index f514475..7c59f5b 100644 (file)
           <td align="right">{$i18n.label.task}:</td>
           <td>{$forms.weekTimeForm.task.control}</td>
         </tr>
+{/if}
+{if $user->isPluginEnabled('wvn')}
+        <tr>
+          <td align="right">{$i18n.label.week_note}:</td>
+          <td>{$forms.weekTimeForm.note.control}</td>
+        </tr>
 {/if}
       </table>
     </td>
@@ -73,6 +79,7 @@
   <tr><td>&nbsp;</td></tr>
 </table>
 
+{if $user->isPluginEnabled('wvl')}
 <table width="720">
 <tr>
   <td valign="top">
   </td>
 </tr>
 </table>
+{/if}
+
 {if $time_records}
 <table cellpadding="3" cellspacing="1" width="720">
+  {if $user->isPluginEnabled('wvl')}
   <tr>
     <td align="left">{$i18n.label.week_total}: {$week_total}</td>
     <td></td>
   </tr>
+  {/if}
   {if $user->isPluginEnabled('mq')}
   <tr>
     <td align="left">{$i18n.label.month_total}: {$month_total}</td>
index 6d82bf7..1d12bae 100644 (file)
@@ -207,9 +207,7 @@ if ($user->canManageTeam()) {
   $form->addInput(array('type'=>'checkbox','name'=>'notifications','value'=>$cl_notifications,'onchange'=>'handlePluginCheckboxes()'));
   $form->addInput(array('type'=>'checkbox','name'=>'locking','value'=>$cl_locking,'onchange'=>'handlePluginCheckboxes()'));
   $form->addInput(array('type'=>'checkbox','name'=>'quotas','value'=>$cl_quotas,'onchange'=>'handlePluginCheckboxes()'));
-  if (defined('WEEK_VIEW_DEBUG') && isTrue(WEEK_VIEW_DEBUG)) {
-    $form->addInput(array('type'=>'checkbox','name'=>'week_view','value'=>$cl_week_view));
-  }
+  $form->addInput(array('type'=>'checkbox','name'=>'week_view','value'=>$cl_week_view,'onchange'=>'handlePluginCheckboxes()'));
 }
 $form->addInput(array('type'=>'submit','name'=>'btn_save','value'=>$i18n->getKey('button.save')));
 
index b70f20b..44a05bd 100644 (file)
--- a/week.php
+++ b/week.php
@@ -39,7 +39,7 @@ import('ttTimeHelper');
 import('DateAndTime');
 
 // Access check.
-if (!ttAccessCheck(right_data_entry)) {
+if (!ttAccessCheck(right_data_entry) || !$user->isPluginEnabled('wv')) {
   header('Location: access_denied.php');
   exit();
 }
@@ -107,6 +107,8 @@ $cl_project = $request->getParameter('project', ($request->isPost() ? null : @$_
 $_SESSION['project'] = $cl_project;
 $cl_task = $request->getParameter('task', ($request->isPost() ? null : @$_SESSION['task']));
 $_SESSION['task'] = $cl_task;
+$cl_note = $request->getParameter('note', ($request->isPost() ? null : @$_SESSION['note']));
+$_SESSION['note'] = $cl_note;
 
 // Get the data we need to display week view.
 // Get column headers, which are day numbers in month.
@@ -126,19 +128,27 @@ $dayTotals = ttWeekViewHelper::getDayTotals($dataArray, $dayHeaders);
 // Define rendering class for a label field to the left of durations.
 class LabelCellRenderer extends DefaultCellRenderer {
   function render(&$table, $value, $row, $column, $selected = false) {
+    global $user;
+
     $this->setOptions(array('width'=>200,'valign'=>'middle'));
-    // Special handling for row 0, which represents a new week entry.
+
+    // Special handling for a new week entry (row 0, or 0 and 1 if we show notes).
     if (0 == $row) {
-      $this->setOptions(array('style'=>'text-align: center; font-weight: bold;'));
-    } else if (0 != $row % 2) {
+      $this->setOptions(array('style'=>'text-align: center; font-weight: bold; vertical-align: top;'));
+    } else if ($user->isPluginEnabled('wvns') && (1 == $row)) {
+      $this->setOptions(array('style'=>'text-align: right; vertical-align: top;'));
+    } else if ($user->isPluginEnabled('wvns') && (0 != $row % 2)) {
       $this->setOptions(array('style'=>'text-align: right;'));
     }
     // Special handling for not billable entries.
-    if ($row > 1 && 0 == $row % 2) {
+    $ignoreRow = $user->isPluginEnabled('wvns') ? 1 : 0; 
+    if ($row > $ignoreRow) {
       $row_id = $table->getValueAtName($row,'row_id');
       $billable = ttWeekViewHelper::parseFromWeekViewRow($row_id, 'bl');
       if (!$billable) {
-        $this->setOptions(array('style'=>'color: red;')); // TODO: style it properly in CSS.
+        if (($user->isPluginEnabled('wvns') && (0 == $row % 2)) || !$user->isPluginEnabled('wvns')) {
+          $this->setOptions(array('style'=>'color: red;')); // TODO: style it properly in CSS.
+        }
       }
     }
     $this->setValue(htmlspecialchars($value)); // This escapes HTML for output.
@@ -149,6 +159,8 @@ class LabelCellRenderer extends DefaultCellRenderer {
 // Define rendering class for a single cell for a time or a comment entry in week view table.
 class WeekViewCellRenderer extends DefaultCellRenderer {
   function render(&$table, $value, $row, $column, $selected = false) {
+    global $user;
+
     $field_name = $table->getValueAt($row,$column)['control_id']; // Our text field names (and ids) are like x_y (row_column).
     $field = new TextField($field_name);
     // Disable control if the date is locked.
@@ -157,15 +169,24 @@ class WeekViewCellRenderer extends DefaultCellRenderer {
       $field->setEnabled(false);
     $field->setFormName($table->getFormName());
     $field->setStyle('width: 60px;'); // TODO: need to style everything properly, eventually.
-    if (0 == $row % 2)
-      $field->setValue($table->getValueAt($row,$column)['duration']); // Duration for even rows.
-    else {
-      $field->setValue($table->getValueAt($row,$column)['note']);     // Comment for odd rows.
-      $field->setTitle($table->getValueAt($row,$column)['note']);     // Tooltip to help view the entire comment.
+    // Provide visual separation for new entry row.
+    $rowToSeparate = $user->isPluginEnabled('wvns') ? 1 : 0;
+    if ($rowToSeparate == $row) {
+      $field->setStyle('width: 60px; margin-bottom: 40px');
+    }
+    if ($user->isPluginEnabled('wvns')) {
+      if (0 == $row % 2) {
+        $field->setValue($table->getValueAt($row,$column)['duration']); // Duration for even rows.
+      } else {
+        $field->setValue($table->getValueAt($row,$column)['note']);     // Comment for odd rows.
+        $field->setTitle($table->getValueAt($row,$column)['note']);     // Tooltip to help view the entire comment.
+      }
+    } else {
+      $field->setValue($table->getValueAt($row,$column)['duration']);
+      // $field->setTitle($table->getValueAt($row,$column)['note']); // Tooltip to see comment. TODO - value not available.
     }
     // Disable control when time entry mode is TYPE_START_FINISH and there is no value in control
     // because we can't supply start and finish times in week view - there are no fields for them.
-    global $user;
     if (!$field->getValue() && TYPE_START_FINISH == $user->record_type) {
         $field->setEnabled(false);
     }
@@ -269,6 +290,9 @@ if (MODE_PROJECTS_AND_TASKS == $user->tracking_mode) {
     'datakeys'=>array('id','name'),
     'empty'=>array(''=>$i18n->getKey('dropdown.select'))));
 }
+if (!defined('NOTE_INPUT_HEIGHT'))
+  define('NOTE_INPUT_HEIGHT', 40);
+$form->addInput(array('type'=>'textarea','name'=>'note','style'=>'width: 250px; height:'.NOTE_INPUT_HEIGHT.'px;','value'=>$cl_note));
 
 // Add other controls.
 $form->addInput(array('type'=>'calendar','name'=>'date','value'=>$cl_date)); // calendar
@@ -317,6 +341,7 @@ if ($request->isPost()) {
         if (!$cl_task) $err->add($i18n->getKey('error.task'));
       }
     }
+    // Finished validating user input for row 0.
 
     // Process the table of values.
     if ($err->no()) {
@@ -335,7 +360,7 @@ if ($request->isPost()) {
           $control_id = $rowNumber.'_'.$dayHeader;
 
           // Handle durations and comments in separate blocks of code.
-          if (0 == $rowNumber % 2) {
+          if (!$user->isPluginEnabled('wvns') || (0 == $rowNumber % 2)) {
             // Handle durations row here.
 
             // Obtain existing and posted durations.
@@ -372,15 +397,21 @@ if ($request->isPost()) {
                 $fields['row_id'] = ttWeekViewHelper::makeRowIdentifier($record).'_0';
                 // Note: no need to check for a possible conflict with an already existing row
                 // because we are doing an insert that does not affect already existing data.
+
+                if ($user->isPluginEnabled('wvn')) {
+                  $fields['note'] = $request->getParameter('note');
+                }
               }
               $fields['day_header'] = $dayHeader;
               $fields['start_date'] = $startDate->toString(DB_DATEFORMAT); // To be able to determine date for the entry using $dayHeader.
               $fields['duration'] = $postedDuration;
               $fields['browser_today'] = $request->getParameter('browser_today', null);
-              // Take note value from the control below duration.
-              $noteRowNumber = $rowNumber + 1;
-              $note_control_id =  $noteRowNumber.'_'.$dayHeader;
-              $fields['note'] = $request->getParameter($note_control_id);
+              if ($user->isPluginEnabled('wvns')) {
+                // Take note value from the control below duration.
+                $noteRowNumber = $rowNumber + 1;
+                $note_control_id =  $noteRowNumber.'_'.$dayHeader;
+                $fields['note'] = $request->getParameter($note_control_id);
+              }
               $result = ttWeekViewHelper::insertDurationFromWeekView($fields, $custom_fields, $err);
             } elseif ($postedDuration == null || 0 == ttTimeHelper::toMinutes($postedDuration)) {
               // Delete an already existing record here.
@@ -393,7 +424,7 @@ if ($request->isPost()) {
             }
             if (!$result) break; // Break out of the loop in case of first error.
 
-          } else {
+          } else if ($user->isPluginEnabled('wvns')) {
             // Handle commments row here.
 
             // Obtain existing and posted comments.