godbout / alfred-time

src/Workflow.php

Braces +3 added lines, -1 removed lines

@@ -41,9 +41,11 @@
         return self::getInstance()->scriptFilter->output();
     }
 
-    public static function do()
+    public static function do {
+        ()
     {
         $action = getenv('timer_action');
+    }
 
         if ($timerId = getenv('timer_id')) {
             return Timer::$action($timerId);

Doc Comments +3 added lines

@@ -146,6 +146,9 @@
         return __NAMESPACE__ . "\\Menus\\" . (self::getMenuClassName(getenv('action')) ?: 'Entrance');
     }
 
+    /**
+     * @param string $action
+     */
     private static function getMenuClassName($action)
     {
         return str_replace('_', '', ucwords($action === false ? 'entrance' : $action, '_'));

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,9 +2,9 @@
 
 namespace Godbout\Alfred\Time;
 
-use Godbout\Alfred\Workflow\Config;
-use Godbout\Alfred\Time\Services\Toggl;
 use Godbout\Alfred\Time\Services\Harvest;
+use Godbout\Alfred\Time\Services\Toggl;
+use Godbout\Alfred\Workflow\Config;
 use Godbout\Alfred\Workflow\ScriptFilter;
 
 class Workflow

src/Menus/Entrance.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     {
         $serviceEnabled = Workflow::serviceEnabled();
 
-        if (! $serviceEnabled) {
+        if (!$serviceEnabled) {
             return;
         }
 
@@ -45,7 +45,7 @@ 
 
     private static function startTimer()
     {
-        if (! empty(Workflow::serviceEnabled())) {
+        if (!empty(Workflow::serviceEnabled())) {
             return Item::create()
                 ->uid('start_timer')
                 ->title('Start "' . self::userInput() . '"')

src/Services/Harvest.php

Doc Comments +3 added lines

@@ -103,6 +103,9 @@
         return false;
     }
 
+    /**
+     * @return integer
+     */
     public function runningTimer()
     {
         try {

Unused Use Statements +1 added lines, -1 removed lines

@@ -5,8 +5,8 @@
 use Carbon\Carbon;
 use Carbon\CarbonInterval;
 use Required\Harvest\Client;
-use Required\Harvest\Exception\NotFoundException;
 use Required\Harvest\Exception\AuthenticationException;
+use Required\Harvest\Exception\NotFoundException;
 use Required\Harvest\Exception\ValidationFailedException;
 
 class Harvest extends TimerService

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
                 'spent_date' => date('Y-m-d')
             ]);
 
-            if (! isset($timer['id'])) {
+            if (!isset($timer['id'])) {
                 return false;
             }
         } catch (ValidationFailedException $e) {
@@ -91,7 +91,7 @@ 
         if ($timerId = $this->runningTimer()) {
             $timer = $this->client->timeEntries()->stop($timerId);
 
-            if (! isset($timer['id'])) {
+            if (!isset($timer['id'])) {
                 throw new \Exception("Can't stop current running timer.", 1);
 
                 return false;

src/Services/Toggl.php

Doc Comments +6 added lines

@@ -79,6 +79,9 @@ 
         return false;
     }
 
+    /**
+     * @return integer
+     */
     public function runningTimer()
     {
         $timer = $this->client->getRunningTimeEntry();
@@ -106,6 +109,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $needle
+     */
     private function extractFromData($needle)
     {
         $data = $this->getData();

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,9 +2,9 @@
 
 namespace Godbout\Alfred\Time\Services;
 
-use Exception;
 use Carbon\Carbon;
 use Carbon\CarbonInterval;
+use Exception;
 use MorningTrain\TogglApi\TogglApi;
 
 class Toggl extends TimerService

Spacing +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
                 'created_with' => 'Alfred Time'
             ]);
 
-            if (! isset($timer->id)) {
+            if (!isset($timer->id)) {
                 return false;
             }
         } catch (Exception $e) {
@@ -67,7 +67,7 @@ 
         if ($timerId = $this->runningTimer()) {
             $response = $this->client->stopTimeEntry($timerId);
 
-            if (! isset($response->id)) {
+            if (!isset($response->id)) {
                 throw new Exception("Can't stop current running timer.", 1);
 
                 return false;
@@ -110,7 +110,7 @@ 
     {
         $data = $this->getData();
 
-        if (! isset($data->$needle)) {
+        if (!isset($data->$needle)) {
             return [];
         }
 
@@ -131,7 +131,7 @@ 
     private function filterOutServerwiseDeletedItemsFromData($items = [])
     {
         return array_filter($items, function ($item) {
-            return ! isset($item->server_deleted_at);
+            return !isset($item->server_deleted_at);
         });
     }
 

src/Menus/Harvest/Setup.php

Spacing +2 added lines, -2 removed lines

@@ -35,8 +35,8 @@
     private static function credentialsFound()
     {
         return (
-            ! empty(Workflow::getConfig()->read('harvest.api_token'))
-            || ! empty(Workflow::getConfig()->read('harvest.account_id'))
+            !empty(Workflow::getConfig()->read('harvest.api_token'))
+            || !empty(Workflow::getConfig()->read('harvest.account_id'))
         );
     }
 

src/Menus/Harvest/SetupState.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 
     protected static function stateSubtitle()
     {
-        return (self::toEnable() ? 'Other services disabled. ': '') . 'You may press enter to quit the workflow';
+        return (self::toEnable() ? 'Other services disabled. ' : '') . 'You may press enter to quit the workflow';
     }
 
     private static function back()

src/app.php

Braces +3 added lines, -1 removed lines

@@ -5,7 +5,9 @@
 use Godbout\Alfred\Time\Workflow;
 
 if (getenv('action') === 'do') {
-    $result = Workflow::do();
+    $result = Workflow::do {
+        ();
+    }
 
     if (getenv('timer_action') !== 'exit') {
         print Workflow::notify($result);