cosmocode / dokuwiki-plugin-issuelinks

syntax.php

Spacing +3 added lines, -3 removed lines

@@ -62,14 +62,14 @@ 
      * @throws Exception
      */
     public function handle($match, $state, $pos, Doku_Handler $handler) {
-        list($pmServiceKey,$issueSyntax) = explode('>', trim($match,'[]'));
+        list($pmServiceKey, $issueSyntax) = explode('>', trim($match, '[]'));
 
         /** @var ServiceInterface $serviceClass */
         $serviceClass = $this->syntaxPatterns[$pmServiceKey]::getInstance();
 
         $issue = $serviceClass->parseIssueSyntax($issueSyntax);
 
-        if(null === $issue) {
+        if (null === $issue) {
             return [$pmServiceKey, $issueSyntax];
         }
 
@@ -96,7 +96,7 @@ 
      * @return bool If rendering was successful.
      */
     public function render($mode, Doku_Renderer $renderer, $data) {
-        if($mode !== 'xhtml' || count($data) === 2) {
+        if ($mode !== 'xhtml' || count($data) === 2) {
             $renderer->interwikilink(null, null, 'google.com', implode(' ', $data));
             return true;
         }

action/ajax.php

Braces +6 added lines, -2 removed lines

@@ -6,7 +6,9 @@ 
  * @author  Michael Große <dokuwiki@cosmocode.de>
  */
 
-if (!defined('DOKU_INC')) die();
+if (!defined('DOKU_INC')) {
+    die();
+}
 
 use dokuwiki\plugin\issuelinks\classes\Issue;
 use dokuwiki\plugin\issuelinks\classes\ServiceProvider;
@@ -200,7 +202,9 @@ 
      * @return void
      */
     public function handle_ajax(Doku_Event $event, $param) {
-        if ($event->data !== 'plugin_issuelinks') return;
+        if ($event->data !== 'plugin_issuelinks') {
+            return;
+        }
         $event->preventDefault();
         $event->stopPropagation();
 

Spacing +4 added lines, -4 removed lines

@@ -131,7 +131,7 @@ 
         $html = '<div class="org_repos">';
         $html .= '<p>Below are the repositories of the organisation to which the authorized user has access to. Click on the icon to create/delete the webhook.</p>';
         $html .= '<div><ul>';
-        usort($repos, function ($repo1, $repo2) {return $repo1->displayName < $repo2->displayName ? -1 : 1;});
+        usort($repos, function($repo1, $repo2) {return $repo1->displayName < $repo2->displayName ? -1 : 1; });
         foreach ($repos as $repo) {
             $stateIssue = empty($repo->hookID) ? 'inactive' : 'active';
             if ($repo->error === 403) {
@@ -175,11 +175,11 @@ 
 
 
         ignore_user_abort(true);
-        set_time_limit(60*20);
+        set_time_limit(60 * 20);
         ob_start();
         $this->util->sendResponse(202, 'Importing  issues...');
         header('Connection: close');
-        header('Content-Length: '.ob_get_length());
+        header('Content-Length: ' . ob_get_length());
         ob_end_flush();
         ob_end_flush();
         flush();
@@ -243,7 +243,7 @@ 
         $lockID = $data->getImportLockID($serviceName, $projectKey);
         $lockData = $data->getLockContent($lockID);
         if ($lockData === false) {
-            msg('Import not locked ' . $lockID,2);
+            msg('Import not locked ' . $lockID, 2);
             return [200, ['status' => 'done']];
         }
         if (!empty($lockData['status']) && $lockData['status'] === 'done') {

helper/db.php

Spacing +25 added lines, -25 removed lines

@@ -9,7 +9,7 @@ 
 // must be run within Dokuwiki
 use dokuwiki\plugin\issuelinks\classes\Issue;
 
-if(!defined('DOKU_INC')) die();
+if (!defined('DOKU_INC')) die();
 
 class helper_plugin_issuelinks_db extends DokuWiki_Plugin
 {
@@ -24,22 +24,22 @@ 
      * @throws Exception Only thrown in unittests
      */
     public function getDB() {
-        if(null === $this->db) {
+        if (null === $this->db) {
             /** @var helper_plugin_sqlite $sqlite */
             $sqlite = plugin_load('helper', 'sqlite');
-            if(!$sqlite) {
+            if (!$sqlite) {
                 msg('This plugin requires the sqlite plugin. Please install it', -1);
                 return null;
             }
 
-            if($sqlite->getAdapter()->getName() !== DOKU_EXT_PDO) {
-                if(defined('DOKU_UNITTEST')) throw new \Exception('Couldn\'t load PDO sqlite.');
+            if ($sqlite->getAdapter()->getName() !== DOKU_EXT_PDO) {
+                if (defined('DOKU_UNITTEST')) throw new \Exception('Couldn\'t load PDO sqlite.');
                 return null;
             }
             $sqlite->getAdapter()->setUseNativeAlter(true);
 
             // initialize the database connection
-            if(!$sqlite->init('issuelinks', DOKU_PLUGIN . 'issuelinks/db/')) {
+            if (!$sqlite->init('issuelinks', DOKU_PLUGIN . 'issuelinks/db/')) {
                 return null;
             }
 
@@ -59,7 +59,7 @@ 
      */
     public function saveKeyValuePair($key, $value) {
         $db = $this->getDB();
-        if(!$db) return false;
+        if (!$db) return false;
         $sql = 'REPLACE INTO opts VALUES (?, ?)';
         $db->query($sql, array($key, $value));
     }
@@ -72,7 +72,7 @@ 
      */
     public function getKeyValue($key) {
         $db = $this->getDB();
-        if(!$db) return false;
+        if (!$db) return false;
         $sql = 'SELECT val FROM opts WHERE opt = ?';
         $res = $db->query($sql, array($key));
         $value = $db->res2single($res);
@@ -132,7 +132,7 @@ 
      * @return bool
      */
     public function saveIssue(Issue $issue) {
-        $ok = $this->saveEntity('issues', array (
+        $ok = $this->saveEntity('issues', array(
             'service' => $issue->getServiceName(),
             'project' => $issue->getProject(),
             'id' => $issue->getKey(),
@@ -142,14 +142,14 @@ 
             'type' => $issue->getType(),
             'status' => $issue->getStatus(),
             'parent' => $issue->getParent(),
-            'components' => implode(',',$issue->getComponents()),
+            'components' => implode(',', $issue->getComponents()),
             'labels' => implode(',', $issue->getLabels()),
             'priority' => $issue->getPriority(),
             'duedate' => $issue->getDuedate(),
             'versions' => implode(',', $issue->getVersions()),
             'updated' => $issue->getUpdated()
         ));
-        return (bool)$ok;
+        return (bool) $ok;
     }
 
 
@@ -236,10 +236,10 @@ 
     public function removeOldLinks($serviceName, $projectKey, $issue_id, $isMergeRequest, $pages) {
         $activeLinks = array();
 
-        foreach($pages as $linkingPage) {
+        foreach ($pages as $linkingPage) {
             $changelog = new PageChangelog($linkingPage['page']);
             $currentRev = $changelog->getRelativeRevision(time(), -1);
-            if($linkingPage['rev'] < $currentRev) {
+            if ($linkingPage['rev'] < $currentRev) {
                 $entity = array(
                     'page'     => $linkingPage['page'],
                     'issue_id' => $issue_id,
@@ -276,7 +276,7 @@ 
         if (!$util->isValidTimeStamp($rev)) {
             throw new InvalidArgumentException("Second parameter must be a valid timestamp!");
         }
-        if ((int)$rev === 0) {
+        if ((int) $rev === 0) {
             $rev = filemtime(wikiFN($page));
             $changelog = new PageChangelog($page);
             $rev_info = $changelog->getRevisionInfo($rev);
@@ -284,10 +284,10 @@ 
             $this->savePageRev($page, $rev, $rev_info['sum'], $user);
         }
         /** @noinspection TypeUnsafeComparisonInspection this is done to ensure $issue_id is a natural number */
-        if (!is_numeric($issue_id) || (int)$issue_id != $issue_id) {
+        if (!is_numeric($issue_id) || (int) $issue_id != $issue_id) {
             throw new InvalidArgumentException("IssueId must be an integer!");
         }
-        $ok = $this->saveEntity('pagerev_issues', array (
+        $ok = $this->saveEntity('pagerev_issues', array(
             'page' => $page,
             'rev' => $rev,
             'service' => $serviceName,
@@ -297,7 +297,7 @@ 
             'type' => $type
         ));
 
-        return (bool)$ok;
+        return (bool) $ok;
     }
 
     /**
@@ -318,13 +318,13 @@ 
         if (!$util->isValidTimeStamp($rev)) {
             throw new InvalidArgumentException("Second parameter must be a valid timestamp!");
         }
-        $ok = $this->saveEntity('pagerevs', array (
+        $ok = $this->saveEntity('pagerevs', array(
             'page' => $page,
             'rev' => $rev,
             'summary' => $summary,
             'user' => $user,
         ));
-        return (bool)$ok;
+        return (bool) $ok;
     }
 
     /**
@@ -357,7 +357,7 @@ 
      */
     private function deleteEntity($table, $entity) {
         $db = $this->getDB();
-        if(!$db) return;
+        if (!$db) return;
 
         $where = implode(' = ? AND ', array_keys($entity)) . ' = ?';
         $vals = array_values($entity);
@@ -375,7 +375,7 @@ 
      */
     private function saveEntity($table, $entity) {
         $db = $this->getDB();
-        if(!$db) return false;
+        if (!$db) return false;
 
         $keys = implode(', ', array_keys($entity));
         $vals = array_values($entity);
@@ -403,16 +403,16 @@ 
      * @return array|bool
      */
     private function sqlArrayQuery($sql, $conditional) {
-        if (substr(trim($sql),0,strlen('SELECT')) !== 'SELECT') {
+        if (substr(trim($sql), 0, strlen('SELECT')) !== 'SELECT') {
             throw new InvalidArgumentException("SQL-Statement must be a SELECT statement! \n" . $sql);
         }
-        if (strpos(trim($sql,';'), ';') !== false) {
+        if (strpos(trim($sql, ';'), ';') !== false) {
             throw new InvalidArgumentException("SQL-Statement must be one single statement! \n" . $sql);
         }
         $db = $this->getDB();
-        if(!$db) return false;
+        if (!$db) return false;
 
-        $res = $db->query($sql,$conditional);
+        $res = $db->query($sql, $conditional);
         $result = $db->res2arr($res);
         $db->res_close($res);
         return $result;

Braces +21 added lines, -7 removed lines

@@ -9,7 +9,9 @@ 
 // must be run within Dokuwiki
 use dokuwiki\plugin\issuelinks\classes\Issue;
 
-if(!defined('DOKU_INC')) die();
+if(!defined('DOKU_INC')) {
+    die();
+}
 
 class helper_plugin_issuelinks_db extends DokuWiki_Plugin
 {
@@ -33,7 +35,9 @@ 
             }
 
             if($sqlite->getAdapter()->getName() !== DOKU_EXT_PDO) {
-                if(defined('DOKU_UNITTEST')) throw new \Exception('Couldn\'t load PDO sqlite.');
+                if(defined('DOKU_UNITTEST')) {
+                    throw new \Exception('Couldn\'t load PDO sqlite.');
+                }
                 return null;
             }
             $sqlite->getAdapter()->setUseNativeAlter(true);
@@ -59,7 +63,9 @@ 
      */
     public function saveKeyValuePair($key, $value) {
         $db = $this->getDB();
-        if(!$db) return false;
+        if(!$db) {
+            return false;
+        }
         $sql = 'REPLACE INTO opts VALUES (?, ?)';
         $db->query($sql, array($key, $value));
     }
@@ -72,7 +78,9 @@ 
      */
     public function getKeyValue($key) {
         $db = $this->getDB();
-        if(!$db) return false;
+        if(!$db) {
+            return false;
+        }
         $sql = 'SELECT val FROM opts WHERE opt = ?';
         $res = $db->query($sql, array($key));
         $value = $db->res2single($res);
@@ -357,7 +365,9 @@ 
      */
     private function deleteEntity($table, $entity) {
         $db = $this->getDB();
-        if(!$db) return;
+        if(!$db) {
+            return;
+        }
 
         $where = implode(' = ? AND ', array_keys($entity)) . ' = ?';
         $vals = array_values($entity);
@@ -375,7 +385,9 @@ 
      */
     private function saveEntity($table, $entity) {
         $db = $this->getDB();
-        if(!$db) return false;
+        if(!$db) {
+            return false;
+        }
 
         $keys = implode(', ', array_keys($entity));
         $vals = array_values($entity);
@@ -410,7 +422,9 @@ 
             throw new InvalidArgumentException("SQL-Statement must be one single statement! \n" . $sql);
         }
         $db = $this->getDB();
-        if(!$db) return false;
+        if(!$db) {
+            return false;
+        }
 
         $res = $db->query($sql,$conditional);
         $result = $db->res2arr($res);

helper/data.php

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
 use dokuwiki\plugin\issuelinks\classes\ServiceProvider;
 
 // must be run within Dokuwiki
-if(!defined('DOKU_INC')) die();
+if(!defined('DOKU_INC')) {
+    die();
+}
 
 class helper_plugin_issuelinks_data extends DokuWiki_Plugin {
 

Spacing +16 added lines, -16 removed lines

@@ -10,7 +10,7 @@ 
 use dokuwiki\plugin\issuelinks\classes\ServiceProvider;
 
 // must be run within Dokuwiki
-if(!defined('DOKU_INC')) die();
+if (!defined('DOKU_INC')) die();
 
 class helper_plugin_issuelinks_data extends DokuWiki_Plugin {
 
@@ -103,7 +103,7 @@ 
     public function getLockContent($id)
     {
         global $conf;
-        $lockFN = $conf['lockdir'].'/'.md5('_' . $id).'.lock';
+        $lockFN = $conf['lockdir'] . '/' . md5('_' . $id) . '.lock';
         if (!file_exists($lockFN)) {
             return false;
         }
@@ -123,7 +123,7 @@ 
     public function lockImport($id, $jsonData) {
         global $conf;
 
-        $lock = $conf['lockdir'].'/'.md5('_' . $id).'.lock';
+        $lock = $conf['lockdir'] . '/' . md5('_' . $id) . '.lock';
         dbglog('lock import: ' . $jsonData, __FILE__ . ': ' . __LINE__);
         io_saveFile($lock, $jsonData);
     }
@@ -140,13 +140,13 @@ 
      */
     public function isImportLocked($id) {
         global $conf;
-        $lockFN = $conf['lockdir'].'/'.md5('_' . $id).'.lock';
-        if(!file_exists($lockFN)) {
+        $lockFN = $conf['lockdir'] . '/' . md5('_' . $id) . '.lock';
+        if (!file_exists($lockFN)) {
             return false;
         }
 
         clearstatcache($lockFN);
-        if((time() - filemtime($lockFN)) > 120) { // todo: decide if we want this to be configurable?
+        if ((time() - filemtime($lockFN)) > 120) { // todo: decide if we want this to be configurable?
             @unlink($lockFN);
             dbglog('issuelinks: stale lock timeout');
             return false;
@@ -166,9 +166,9 @@ 
         }
 
         global $conf, $INPUT;
-        $lockFN = $conf['lockdir'].'/'.md5('_' . $id).'.lock';
+        $lockFN = $conf['lockdir'] . '/' . md5('_' . $id) . '.lock';
         $lockData = json_decode(io_readFile($lockFN), true);
-        if($lockData['user'] !== $INPUT->server->str('REMOTE_USER')) {
+        if ($lockData['user'] !== $INPUT->server->str('REMOTE_USER')) {
             return false;
         }
 
@@ -184,7 +184,7 @@ 
      */
     public function unlockImport($id) {
         global $conf;
-        $lockFN = $conf['lockdir'].'/'.md5('_' . $id).'.lock';
+        $lockFN = $conf['lockdir'] . '/' . md5('_' . $id) . '.lock';
         $lockData = json_decode(io_readFile($lockFN), true);
         $lockData['status'] = 'done';
         $lockData['total'] = $lockData['count'];
@@ -194,7 +194,7 @@ 
     public function removeLock($lockID)
     {
         global $conf;
-        $lockFN = $conf['lockdir'].'/'.md5('_' . $lockID).'.lock';
+        $lockFN = $conf['lockdir'] . '/' . md5('_' . $lockID) . '.lock';
         unlink($lockFN);
     }
 
@@ -239,7 +239,7 @@ 
      */
     public function getIssue($pmServiceName, $project, $issueid, $isMergeRequest) {
         $issue = Issue::getInstance($pmServiceName, $project, $issueid, $isMergeRequest);
-        if(!$issue->isValid()) {
+        if (!$issue->isValid()) {
             try {
                 $issue->getFromService();
                 $issue->saveToDB();
@@ -274,13 +274,13 @@ 
 
         $repoMatches = array();
         $repoPattern = '/(\w+\/)?([\w\.\-_]+)?([#!])(\d+)(?:[\.,\s]|$)/';
-        preg_match_all($repoPattern, $message, $repoMatches,PREG_SET_ORDER);
+        preg_match_all($repoPattern, $message, $repoMatches, PREG_SET_ORDER);
         list($currentNamespace, $currentRepo) = explode('/', $repoId);
         foreach ($repoMatches as $match) {
             if ($rmServiceName !== 'gitlab' && $match[3] === '!') {
                 continue; // only gitlab has `!` has separator
             }
-            $namespace = empty($match[1]) ? $currentNamespace : trim($match[1],'/');
+            $namespace = empty($match[1]) ? $currentNamespace : trim($match[1], '/');
             $repo = empty($match[2]) ? $currentRepo : $match[2];
             $issues[] = array('pmService' => $rmServiceName,
                 'project' => "$namespace/$repo",
@@ -319,7 +319,7 @@ 
     public function keepNewest($pages) {
         $uniquePages = array();
         foreach ($pages as $page) {
-            if (!array_key_exists($page['page'],$uniquePages) || $uniquePages[$page['page']]['rev'] < $page['rev'] ) {
+            if (!array_key_exists($page['page'], $uniquePages) || $uniquePages[$page['page']]['rev'] < $page['rev']) {
                 $uniquePages[$page['page']] = $page;
             }
         }
@@ -336,7 +336,7 @@ 
     private function filterPagesForACL($pages) {
         $allowedPagegs = array();
         foreach ($pages as $page) {
-            if(auth_quickaclcheck($page['page']) >= AUTH_READ) {
+            if (auth_quickaclcheck($page['page']) >= AUTH_READ) {
                 $allowedPagegs[] = $page;
             }
         }
@@ -435,7 +435,7 @@ 
         $config = new dokuwiki\plugin\struct\meta\ConfigParser($configLines);
         $search = new dokuwiki\plugin\struct\meta\SearchConfig($config->getConfig());
         $results = $search->execute();
-        $pages = array_map(function($result) { return array('page' => $result[0]->getRawValue(), 'rev' => 0);}, $results);
+        $pages = array_map(function($result) { return array('page' => $result[0]->getRawValue(), 'rev' => 0); }, $results);
         return $pages;
     }
 

helper/util.php

Spacing +5 added lines, -5 removed lines

@@ -7,7 +7,7 @@ 
  */
 
 // must be run within Dokuwiki
-if(!defined('DOKU_INC')) die();
+if (!defined('DOKU_INC')) die();
 
 class helper_plugin_issuelinks_util extends DokuWiki_Plugin {
 
@@ -17,7 +17,7 @@ 
      * @param string $linkHeader
      * @return array
      */
-    public function parseHTTPLinkHeaders ($linkHeader) {
+    public function parseHTTPLinkHeaders($linkHeader) {
         $links = explode(',', $linkHeader);
         $linkarray = array();
         foreach ($links as $linkstring) {
@@ -66,7 +66,7 @@ 
     public function reportException(Exception $e) {
         msg(hsc($e->getMessage()), -1, $e->getLine(), $e->getFile());
         global $conf;
-        if($conf['allowdebug']) {
+        if ($conf['allowdebug']) {
             msg('<pre>' . hsc($e->getTraceAsString()) . '</pre>', -1);
         }
     }
@@ -77,7 +77,7 @@ 
      */
     public function sendResponse($code, $msg) {
         header('Content-Type: application/json');
-        if ((int)$code === 204) {
+        if ((int) $code === 204) {
             http_status(200);
         } else {
             http_status($code);
@@ -96,7 +96,7 @@ 
      * @return bool
      */
     public function isValidTimeStamp($timestamp) {
-        return ((string) (int) $timestamp === (string)$timestamp);
+        return ((string) (int) $timestamp === (string) $timestamp);
     }
 
 }

Braces +3 added lines, -1 removed lines

@@ -7,7 +7,9 @@
  */
 
 // must be run within Dokuwiki
-if(!defined('DOKU_INC')) die();
+if(!defined('DOKU_INC')) {
+    die();
+}
 
 class helper_plugin_issuelinks_util extends DokuWiki_Plugin {
 

services/AbstractService.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
     public static function getInstance($forcereload = false) {
         $class = static::class;
-        if(empty(self::$instance[$class]) || $forcereload) {
+        if (empty(self::$instance[$class]) || $forcereload) {
             self::$instance[$class] = new $class();
         }
         return self::$instance[$class];

services/GitLab.service.php

Spacing +3 added lines, -3 removed lines

@@ -116,7 +116,7 @@ 
     {
         $groups = $this->makeSingleGitLabGetRequest('/groups');
 
-        return array_map(function ($group) {
+        return array_map(function($group) {
             return $group['full_path'];
         }, $groups);
 
@@ -139,7 +139,7 @@ 
             try {
                 $repoHooks = $this->makeSingleGitLabGetRequest("/projects/$organisation%2F{$project['path']}/hooks?per_page=100");
             } catch (HTTPRequestException $e) {
-                $repo->error = (int)$e->getCode();
+                $repo->error = (int) $e->getCode();
             }
 
             $repoHooks = array_filter($repoHooks, array($this, 'isOurIssueHook'));
@@ -315,7 +315,7 @@ 
     public function retrieveAllIssues($projectKey, &$startat = 0)
     {
         $perPage = 100;
-        $page = ceil(($startat+1)/$perPage);
+        $page = ceil(($startat + 1) / $perPage);
         $endpoint = '/projects/' . urlencode($projectKey) . "/issues?page=$page&per_page=$perPage";
         $issues = $this->makeSingleGitLabGetRequest($endpoint);
         $this->total = $this->estimateTotal($perPage, count($issues));

services/GitHub.service.php

Spacing +1 added lines, -1 removed lines

@@ -334,7 +334,7 @@
             }
         }
         // fixme: add 'user repos'!
-        return array_map(function ($org) {
+        return array_map(function($org) {
             return $org['login'];
         }, $this->orgs);
     }

classes/Issue.php

Spacing +14 added lines, -14 removed lines

@@ -68,8 +68,8 @@ 
      * @return Issue
      */
     public static function getInstance($serviceName, $projectKey, $issueId, $isMergeRequest = false, $forcereload = false) {
-        $issueHash = $serviceName.$projectKey.$issueId.'!'.$isMergeRequest;
-        if(empty(self::$instances[$issueHash]) || $forcereload) {
+        $issueHash = $serviceName . $projectKey . $issueId . '!' . $isMergeRequest;
+        if (empty(self::$instances[$issueHash]) || $forcereload) {
             self::$instances[$issueHash] = new Issue($serviceName, $projectKey, $issueId, $isMergeRequest);
         }
         return self::$instances[$issueHash];
@@ -130,7 +130,7 @@ 
 
     public function __toString() {
         $sep = $this->pmService->getProjectIssueSeparator($this->isMergeRequest);
-        return $this->projectId . $sep .$this->issueId;
+        return $this->projectId . $sep . $this->issueId;
     }
 
     /**
@@ -284,10 +284,10 @@ 
      */
     public function setComponents($components) {
         if (!is_array($components)) {
-            $components = array_filter(array_map('trim', explode(',',$components)));
+            $components = array_filter(array_map('trim', explode(',', $components)));
         }
         if (!empty($components[0]['name'])) {
-            $components = array_map(function($component){return $component['name'];},$components);
+            $components = array_map(function($component) {return $component['name']; },$components);
         }
         $this->components = $components;
         return $this;
@@ -299,7 +299,7 @@ 
      */
     public function setLabels($labels) {
         if (!is_array($labels)) {
-            $labels = array_filter(array_map('trim', explode(',',$labels)));
+            $labels = array_filter(array_map('trim', explode(',', $labels)));
         }
         $this->labels = $labels;
         return $this;
@@ -311,10 +311,10 @@ 
      */
     public function setVersions($versions) {
         if (!is_array($versions)) {
-            $versions = array_map('trim', explode(',',$versions));
+            $versions = array_map('trim', explode(',', $versions));
         }
         if (!empty($versions[0]['name'])) {
-            $versions = array_map(function($version){return $version['name'];},$versions);
+            $versions = array_map(function($version) {return $version['name']; },$versions);
         }
         $this->versions = $versions;
         return $this;
@@ -330,7 +330,7 @@ 
         if (!$util->isValidTimeStamp($updated)) {
             $updated = strtotime($updated);
         }
-        $this->updated = (int)$updated;
+        $this->updated = (int) $updated;
         return $this;
     }
 
@@ -639,16 +639,16 @@ 
      */
     private function calculateColor($color) {
         /** @noinspection PrintfScanfArgumentsInspection */
-        list($r, $g, $b) = array_map(function($color8bit){
-            $c = $color8bit/255;
+        list($r, $g, $b) = array_map(function($color8bit) {
+            $c = $color8bit / 255;
             if ($c <= 0.03928) {
-                $cl = $c/12.92;
+                $cl = $c / 12.92;
             } else {
-                $cl = pow(($c+0.055)/1.055,2.4);
+                $cl = pow(($c + 0.055) / 1.055, 2.4);
             }
             return $cl;
         }, sscanf($color, "%02x%02x%02x"));
-        if ($r*0.2126 + $g*0.7152 + $b*0.0722 > 0.179) {
+        if ($r * 0.2126 + $g * 0.7152 + $b * 0.0722 > 0.179) {
             return '#000000';
         }
         return '#FFFFFF';