Merge pull request #104 from larowlan/fix-review-exact

Fix for review --exact
larowlan · Apr 16, 2018 · db8602e · db8602e
2 parents 4b1ed6c + 191753e
commit db8602e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Commands/Delete.php b/src/Commands/Delete.php
@@ -57,7 +57,7 @@ protected function execute(InputInterface $input, OutputInterface $output) {
     $question = new ConfirmationQuestion('Are you sure?', false);
 
     $confirm = NULL;
-    if (($slot = $this->repository->slot($slot_id)) && ($confirm = ($input->hasOption('confirm') || $helper->ask($input, $output, $question))) && $this->repository->delete($slot_id)) {
+    if (($slot = $this->repository->slot($slot_id)) && ($confirm = ($input->getOption('confirm') || $helper->ask($input, $output, $question))) && $this->repository->delete($slot_id)) {
       $deleted = $this->connector->ticketDetails($slot->tid);
       $output->writeln(sprintf('Deleted slot <comment>%d</comment> against ticket <info>%d</info>: %s, duration <info>%s</info>',
         $slot->id,

diff --git a/src/Reviewer.php b/src/Reviewer.php
@@ -84,8 +84,8 @@ public function getSummary($date = 19780101, $check = FALSE, $exact = FALSE) {
         $duration,
       ];
       if ($exact) {
-        $row[] = Formatter::formatDuration($record->end - $record->start);
-        $exact_total += ($record->end - $record->start);
+        $row[] = Formatter::formatDuration(($record->end ?: time()) - $record->start);
+        $exact_total += (($record->end ?: time()) - $record->start);
       }
       $row = array_merge($row, [
         substr($details->getTitle(), 0, 25) . '...',