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;
         }

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';

classes/IssueLinksException.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         /** @var \helper_plugin_struct $plugin */
         $plugin = plugin_load('helper', 'issuelinks_util');
         $trans = $plugin->getLang($this->trans_prefix . $message);
-        if(!$trans) {
+        if (!$trans) {
             $trans = $message;
         }
 

classes/ServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     protected $serviceClasses = [];
 
     public static function getInstance($forcereload = false) {
-        if(null === self::$instance || $forcereload) {
+        if (null === self::$instance || $forcereload) {
             self::$instance = new self();
         }
         return self::$instance;
@@ -49,7 +49,7 @@ 
         $serviceDir = __DIR__ . '/../services';
         $filenames = scandir($serviceDir, SCANDIR_SORT_ASCENDING);
         foreach ($filenames as $filename) {
-            if ($filename === '.' || $filename === '..' ) {
+            if ($filename === '.' || $filename === '..') {
                 continue;
             }
             list($service, $servicePostfix) = explode('.', $filename, 2);