Matth-- / toggl-invoiceninja-sync

src/Syncer/InvoiceNinja/InvoiceNinjaClient.php

Doc Comments -1 removed lines

@@ -90,7 +90,6 @@
     }
 
     /**
-     * @param Project $project
      *
      * @return mixed
      */

src/Syncer/Toggl/TogglClient.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
     public function getWorkspaces()
     {
         $response = $this->client->request('GET', self::VERSION . '/workspaces', [
-            'auth' => [$this->api_key, 'api_token'],
+            'auth' => [ $this->api_key, 'api_token' ],
         ]);
 
         return $this->serializer->deserialize($response->getBody(), 'array<Syncer\Dto\Toggl\Workspace>', 'json');
@@ -62,7 +62,7 @@ 
     public function getClientsForWorkspace($workspaceId)
     {
         $response = $this->client->request('GET', self::VERSION . '/workspaces/' . $workspaceId . '/clients', [
-            'auth' => [$this->api_key, 'api_token'],
+            'auth' => [ $this->api_key, 'api_token' ],
         ]);
 
         return $this->serializer->deserialize($response->getBody(), 'array<Syncer\Dto\Toggl\Client>', 'json');
@@ -74,7 +74,7 @@ 
     public function getProjectsForWorkspace($workspaceId)
     {
         $response = $this->client->request('GET', self::VERSION . '/workspaces/' . $workspaceId . '/projects', [
-            'auth' => [$this->api_key, 'api_token'],
+            'auth' => [ $this->api_key, 'api_token' ],
         ]);
 
         return $this->serializer->deserialize($response->getBody(), 'array<Syncer\Dto\Toggl\Project>', 'json');

src/Syncer/Command/SyncTimings.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Syncer\InvoiceNinja\InvoiceNinjaClient;
 use Syncer\Toggl\ReportsClient;
 use Syncer\Toggl\TogglClient;
-
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;

Spacing +26 added lines, -26 removed lines

@@ -130,7 +130,7 @@ 
             $workspaceClients = array_merge($this->clients, $this->retrieveClientsForWorkspace($workspace->getId(), $this->clients));
             $workspaceProjects = array_merge($this->projects, $this->retrieveProjectsForWorkspace($workspace->getId(), $workspaceClients, $this->projects));
 
-            foreach($detailedReport->getData() as $timeEntry) {
+            foreach ($detailedReport->getData() as $timeEntry) {
                 $timeEntrySent = false;
                 $timeEntryClient = $timeEntry->getClient();
                 $timeEntryProject = $timeEntry->getProject();
@@ -148,7 +148,7 @@ 
                 else if (!isset($timeEntryClient)) {
                     if ($this->useProjectsAsClients) {
                         $timeEntryClient = $timeEntryProject;
-                        $workspaceClients[$timeEntryProject] = $this->getInvoiceNinjaClientIdForProject($timeEntryProject);
+                        $workspaceClients[ $timeEntryProject ] = $this->getInvoiceNinjaClientIdForProject($timeEntryProject);
                     } else {
                         $timeEntryProject = NULL;
                         $this->io->warning('No client set for TimeEntry (' . $timeEntry->getDescription() . ')');
@@ -159,17 +159,17 @@ 
                 if (isset($timeEntryProject)) {
                     $this->logTask($timeEntry, $workspaceClients, $timeEntryClient, $workspaceProjects, $timeEntryProject);
 
-                    $this->sentTimeEntries[] = $timeEntry->getId();
+                    $this->sentTimeEntries[ ] = $timeEntry->getId();
                     $timeEntrySent = true;
                 } else {
                     $this->logTask($timeEntry);
 
-                    $this->sentTimeEntries[] = $timeEntry->getId();
+                    $this->sentTimeEntries[ ] = $timeEntry->getId();
                     $timeEntrySent = true;
                 }
 
                 if ($timeEntrySent) {
-                    $this->io->success('TimeEntry ('. $timeEntry->getDescription() . ') sent to InvoiceNinja');
+                    $this->io->success('TimeEntry (' . $timeEntry->getDescription() . ') sent to InvoiceNinja');
                 }
             }
         }
@@ -191,11 +191,11 @@ 
         $task->setDescription($this->buildTaskDescription($entry));
         $task->setTimeLog($this->buildTimeLog($entry));
         if (isset($clients) && isset($clientKey)) {
-            $task->setClientId($clients[$clientKey]);
+            $task->setClientId($clients[ $clientKey ]);
         }
 
         if (isset($projects) && isset($projectKey)) {
-            $task->setProjectId($projects[$projectKey]);
+            $task->setProjectId($projects[ $projectKey ]);
         }
 
         $this->invoiceNinjaClient->saveNewTask($task);
@@ -226,10 +226,10 @@ 
      */
     private function buildTimeLog(TimeEntry $entry): string
     {
-        $timeLog = [[
+        $timeLog = [ [
             $entry->getStart()->getTimestamp(),
             $entry->getEnd()->getTimestamp(),
-        ]];
+        ] ];
 
         return \GuzzleHttp\json_encode($timeLog);
     }
@@ -248,14 +248,14 @@ 
         $togglClients = $this->togglClient->getClientsForWorkspace($workspaceId);
         $invoiceNinjaClients = $this->invoiceNinjaClient->getClients();
 
-        $clients = [];
+        $clients = [ ];
 
         foreach ($togglClients as $togglClient) {
-            if (!isset($workspaceClients[$togglClient->getName()])) {
+            if (!isset($workspaceClients[ $togglClient->getName() ])) {
                 $found = false;
                 foreach ($invoiceNinjaClients as $invoiceNinjaClient) {
                     if ($invoiceNinjaClient->getIsDeleted() == false && strcasecmp($togglClient->getName(), $invoiceNinjaClient->getName()) == 0) {
-                        $clients[$invoiceNinjaClient->getName()] = $invoiceNinjaClient->getId();
+                        $clients[ $invoiceNinjaClient->getName() ] = $invoiceNinjaClient->getId();
                         $found = true;
                     }
                 }
@@ -264,9 +264,9 @@ 
 
                     $client->setName($togglClient->getName());
 
-                    $clients[$togglClient->getName()] = $this->invoiceNinjaClient->saveNewClient($client)->getId();
+                    $clients[ $togglClient->getName() ] = $this->invoiceNinjaClient->saveNewClient($client)->getId();
 
-                    $this->io->success('Client ('. $togglClient->getName() . ') created in InvoiceNinja');
+                    $this->io->success('Client (' . $togglClient->getName() . ') created in InvoiceNinja');
                 }
             }
         }
@@ -290,14 +290,14 @@ 
         $togglClients = $this->togglClient->getClientsForWorkspace($workspaceId);
         $invoiceNinjaProjects = $this->invoiceNinjaClient->getProjects();
 
-        $projects = [];
+        $projects = [ ];
 
         foreach ($togglProjects as $togglProject) {
-            if (!isset($workspaceProjects[$togglProject->getName()])) {
+            if (!isset($workspaceProjects[ $togglProject->getName() ])) {
                 $found = false;
                 foreach ($invoiceNinjaProjects as $invoiceNinjaProject) {
                     if ($invoiceNinjaProject->getIsDeleted() == false && strcasecmp($togglProject->getName(), $invoiceNinjaProject->getName()) == 0) {
-                        $projects[$invoiceNinjaProject->getName()] = $invoiceNinjaProject->getId();
+                        $projects[ $invoiceNinjaProject->getName() ] = $invoiceNinjaProject->getId();
                         $found = true;
                     }
                 }
@@ -311,7 +311,7 @@ 
                     if ($togglProject->getCid() !== NULL) {
                         foreach ($togglClients as $togglClient) {
                             if ($togglClient->getWid() == $workspaceId && $togglClient->getId() == $togglProject->getCid()) {
-                                $project->setClientId($workspaceClients[$togglClient->getName()]);
+                                $project->setClientId($workspaceClients[ $togglClient->getName() ]);
                             }
                         }
                     } else if ($this->useProjectsAsClients) {
@@ -319,20 +319,20 @@ 
 
                         $client->setName($togglProject->getName());
 
-                        $workspaceClients[$togglProject->getName()] = $this->invoiceNinjaClient->saveNewClient($client)->getId();
+                        $workspaceClients[ $togglProject->getName() ] = $this->invoiceNinjaClient->saveNewClient($client)->getId();
 
-                        $this->io->success('Project ('. $togglProject->getName() . ') created as Client in InvoiceNinja');
+                        $this->io->success('Project (' . $togglProject->getName() . ') created as Client in InvoiceNinja');
 
-                        $project->setClientId($workspaceClients[$togglProject->getName()]);
+                        $project->setClientId($workspaceClients[ $togglProject->getName() ]);
                     } else {
-                        $this->io->error('Client not provided for Project ('. $togglProject->getName() . ') in Toggl');
+                        $this->io->error('Client not provided for Project (' . $togglProject->getName() . ') in Toggl');
                         $this->io->warning("To allow using projects as clients enable 'use_projects_as_clients' in parameters config file");
                     }
 
                     if ($clientPresent) {
-                        $projects[$togglProject->getName()] = $this->invoiceNinjaClient->saveNewProject($project)->getId();
+                        $projects[ $togglProject->getName() ] = $this->invoiceNinjaClient->saveNewProject($project)->getId();
 
-                        $this->io->success('Project ('. $togglProject->getName() . ') created in InvoiceNinja');
+                        $this->io->success('Project (' . $togglProject->getName() . ') created in InvoiceNinja');
                     }
                 }
             }
@@ -348,7 +348,7 @@ 
      */
     private function getInvoiceNinjaClientIdForProject(string $projectName): ?int
     {
-        $invoiceNinjaProjects  = $this->invoiceNinjaClient->getProjects();
+        $invoiceNinjaProjects = $this->invoiceNinjaClient->getProjects();
         $invoiceNinjaClientId = NULL;
 
         foreach ($invoiceNinjaProjects as $invoiceNinjaProject) {
@@ -378,7 +378,7 @@ 
         $this->sentTimeEntries = unserialize(file_get_contents($this->storageDir . $this->storageFileName));
 
         if (!is_array($this->sentTimeEntries)) {
-            $this->sentTimeEntries = [];
+            $this->sentTimeEntries = [ ];
         }
 
         return $this->sentTimeEntries;

Braces +1 added lines, -3 removed lines

@@ -143,9 +143,7 @@
                 // a project is required, but a client is not.
                 if (!isset($timeEntryProject)) {
                     $this->io->warning('No project set for TimeEntry (' . $timeEntry->getDescription() . ')');
-                }
-
-                else if (!isset($timeEntryClient)) {
+                } else if (!isset($timeEntryClient)) {
                     if ($this->useProjectsAsClients) {
                         $timeEntryClient = $timeEntryProject;
                         $workspaceClients[$timeEntryProject] = $this->getInvoiceNinjaClientIdForProject($timeEntryProject);