NeoBlack / T3Bot

Classes/Commands/UtilCommand.php

Spacing +1 added lines, -1 removed lines

@@ -54,6 +54,6 @@
 
         $option = $options[random_int(0, count($options) - 1)];
 
-        return '*Botty says:* _' . $option . '_';
+        return '*Botty says:* _'.$option.'_';
     }
 }

config/config.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-(new Dotenv\Dotenv(__DIR__ . '/../'))->load();
+(new Dotenv\Dotenv(__DIR__.'/../'))->load();
 
 $GLOBALS['config'] = [
     // project phase

Classes/Traits/SlackTrait.php

Spacing +11 added lines, -11 removed lines

@@ -26,7 +26,7 @@ 
      */
     protected function bold($string) : string
     {
-        return '*' . $string . '*';
+        return '*'.$string.'*';
     }
 
     /**
@@ -38,7 +38,7 @@ 
      */
     protected function italic($string) : string
     {
-        return '_' . $string . '_';
+        return '_'.$string.'_';
     }
 
     /**
@@ -50,8 +50,8 @@ 
      */
     protected function buildReviewLine($item) : string
     {
-        return $this->bold($item->subject) . ' <https://review.typo3.org/' . $item->_number
-        . '|Review #' . $item->_number . ' now>';
+        return $this->bold($item->subject).' <https://review.typo3.org/'.$item->_number
+        . '|Review #'.$item->_number.' now>';
     }
 
     /**
@@ -63,15 +63,15 @@ 
     {
         $created = substr($item->created_on, 0, 19);
         $updated = substr($item->updated_on, 0, 19);
-        $text = $this->bold('[' . $item->tracker->name . '] ' . $item->subject)
-            . ' by ' . $this->italic($item->author->name) . chr(10);
-        $text .= 'Project: ' . $this->bold($item->project->name);
+        $text = $this->bold('['.$item->tracker->name.'] '.$item->subject)
+            . ' by '.$this->italic($item->author->name).chr(10);
+        $text .= 'Project: '.$this->bold($item->project->name);
         if (!empty($item->category->name)) {
-            $text .= ' | Category: ' . $this->bold($item->category->name);
+            $text .= ' | Category: '.$this->bold($item->category->name);
         }
-        $text .= ' | Status: ' . $this->bold($item->status->name) . chr(10);
-        $text .= ':calendar: Created: ' . $this->bold($created) . ' | Last update: ' . $this->bold($updated) . chr(10);
-        $text .= '<https://forge.typo3.org/issues/' . $item->id . '|:arrow_right: View on Forge>';
+        $text .= ' | Status: '.$this->bold($item->status->name).chr(10);
+        $text .= ':calendar: Created: '.$this->bold($created).' | Last update: '.$this->bold($updated).chr(10);
+        $text .= '<https://forge.typo3.org/issues/'.$item->id.'|:arrow_right: View on Forge>';
 
         return $text;
     }

Classes/Traits/GerritTrait.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
      */
     protected function queryGerrit($query)
     {
-        return $this->remoteCall('https://review.typo3.org/changes/?q=' . urlencode($query));
+        return $this->remoteCall('https://review.typo3.org/changes/?q='.urlencode($query));
     }
 
     /**
@@ -30,7 +30,7 @@ 
     protected function getFilesForPatch($changeId, $revision)
     {
         return $this->remoteCall(
-            'https://review.typo3.org/changes/' . $changeId . '/revisions/' . $revision . '/files'
+            'https://review.typo3.org/changes/'.$changeId.'/revisions/'.$revision.'/files'
         );
     }
 
@@ -51,7 +51,7 @@ 
         $result = false;
         if (!curl_errno($ch)) {
             curl_close($ch);
-            $result = json_decode(str_replace(")]}'" . chr(10), '', $data));
+            $result = json_decode(str_replace(")]}'".chr(10), '', $data));
         }
 
         return $result;

Classes/Slack/CommandResolver.php

Spacing +1 added lines, -1 removed lines

@@ -64,6 +64,6 @@
     {
         $delimiter = strpos($message, ':') !== false ? ':' : ' ';
         $parts = explode($delimiter, $message);
-        return '\\T3Bot\\Commands\\' . ucfirst(strtolower($parts[0])) . 'Command';
+        return '\\T3Bot\\Commands\\'.ucfirst(strtolower($parts[0])).'Command';
     }
 }

Classes/Commands/BeerCommand.php

Spacing +6 added lines, -6 removed lines

@@ -49,7 +49,7 @@ 
      */
     protected function processAll() : string
     {
-        return 'Yeah, ' . $this->getBeerCountAll() . ' :t3beer: spend to all people';
+        return 'Yeah, '.$this->getBeerCountAll().' :t3beer: spend to all people';
     }
 
     /**
@@ -64,7 +64,7 @@ 
         $rows = $this->getBeerTop10();
         $text = ['*Yeah, here are the TOP 10*'];
         foreach ($rows as $row) {
-            $text[] = '<@' . $row['username'] . '> has received ' . $row['cnt'] . ' :t3beer:';
+            $text[] = '<@'.$row['username'].'> has received '.$row['cnt'].' :t3beer:';
         }
 
         return implode(chr(10), $text);
@@ -85,7 +85,7 @@ 
         if (strpos($username, '<') === 0 && $username[1] === '@') {
             $username = str_replace(['<', '>', '@'], '', $username);
 
-            return '<@' . $username . '> has received ' . $this->getBeerCountByUsername($username) . ' :t3beer: so far';
+            return '<@'.$username.'> has received '.$this->getBeerCountByUsername($username).' :t3beer: so far';
         }
         return '*Sorry, a username must start with a @-sign:*';
     }
@@ -109,10 +109,10 @@ 
                     'from_user' => $from_user,
                     'tstamp' => time()
                 ]);
-                return 'Yeah, one more :t3beer: for <@' . $username . '>' . chr(10) . '<@' . $username . '> has received '
-                    . $this->getBeerCountByUsername($username) . ' :t3beer: so far';
+                return 'Yeah, one more :t3beer: for <@'.$username.'>'.chr(10).'<@'.$username.'> has received '
+                    . $this->getBeerCountByUsername($username).' :t3beer: so far';
             }
-            return 'You spend one :t3beer: to <@' . $username . '> within in last 24 hours. Too much beer is unhealthy ;)';
+            return 'You spend one :t3beer: to <@'.$username.'> within in last 24 hours. Too much beer is unhealthy ;)';
         }
         return '*Sorry, a username must start with a @-sign:*';
     }

Classes/Commands/TellCommand.php

Spacing +8 added lines, -8 removed lines

@@ -104,8 +104,8 @@ 
     {
         $parts = explode(':', $notification['message']);
         $refId = (int) trim($parts[1]);
-        $result = $this->queryGerrit('change:' . $refId);
-        $msg = '*Hi <@' . $user . '>, <@' . $notification['from_user'] . '>'
+        $result = $this->queryGerrit('change:'.$refId);
+        $msg = '*Hi <@'.$user.'>, <@'.$notification['from_user'].'>'
             . ' ask you to look at this patch:*';
 
         if (is_array($result)) {
@@ -127,11 +127,11 @@ 
     {
         $parts = explode(':', $notification['message']);
         $issueNumber = (int) trim($parts[1]);
-        $result = $this->queryForge('issues/' . $issueNumber);
+        $result = $this->queryForge('issues/'.$issueNumber);
         if ($result) {
-            $msg = '*Hi <@' . $user . '>, <@' . $notification['from_user'] . '>'
+            $msg = '*Hi <@'.$user.'>, <@'.$notification['from_user'].'>'
                 . ' ask you to look at this issue:*';
-            $this->sendResponse($msg . chr(10) . $this->buildIssueMessage($result->issue), $user);
+            $this->sendResponse($msg.chr(10).$this->buildIssueMessage($result->issue), $user);
         }
     }
 
@@ -141,8 +141,8 @@ 
      */
     protected function processTextMessage(array $notification, string $user)
     {
-        $msg = '*Hi <@' . $user . '>, here is a message from <@' . $notification['from_user'] . '>' . ' for you:*';
-        $this->sendResponse($msg . chr(10) . $notification['message'], $user);
+        $msg = '*Hi <@'.$user.'>, here is a message from <@'.$notification['from_user'].'>'.' for you:*';
+        $this->sendResponse($msg.chr(10).$notification['message'], $user);
     }
 
     /**
@@ -168,6 +168,6 @@ 
             'message' => $message,
         ]);
 
-        return 'OK, I will tell <@' . $toUser . '> about your message';
+        return 'OK, I will tell <@'.$toUser.'> about your message';
     }
 }

Classes/Commands/ForgeCommand.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
             $issueNumber = (int) $matches[1][0];
         }
         $issueNumber = $issueNumber ?? $value;
-        if ((int)$issueNumber === 0) {
+        if ((int) $issueNumber === 0) {
             return 'hey, I need an issue number!';
         }
         $result = $this->queryForge("issues/{$issueNumber}");

Classes/Controller/GerritHookController.php

Spacing +5 added lines, -5 removed lines

@@ -51,15 +51,15 @@ 
         $patchId = (int) str_replace('https://review.typo3.org/', '', $json->{'change-url'});
         $patchSet = property_exists($json, 'patchset') ? (int) $json->patchset : 0;
 
-        $item = $this->queryGerrit('change:' . $patchId)[0];
+        $item = $this->queryGerrit('change:'.$patchId)[0];
         $created = substr($item->created, 0, 19);
-        $text = "Branch: {$json->branch} | :calendar: {$created} | ID: {$item->_number}" . chr(10);
+        $text = "Branch: {$json->branch} | :calendar: {$created} | ID: {$item->_number}".chr(10);
         $text .= ":link: <https://review.typo3.org/{$item->_number}|Goto Review>";
         if ($hook === 'patchset-created' && $patchSet === 1 && $json->branch === 'master') {
-            $message = $this->buildMessage('[NEW] ' . $item->subject, $text);
+            $message = $this->buildMessage('[NEW] '.$item->subject, $text);
             $this->sendMessageToChannel($hook, $message);
         } elseif ($hook === 'change-merged') {
-            $message = $this->buildMessage(':white_check_mark: [MERGED] ' . $item->subject, $text, Message\Attachment::COLOR_GOOD);
+            $message = $this->buildMessage(':white_check_mark: [MERGED] '.$item->subject, $text, Message\Attachment::COLOR_GOOD);
             $this->sendMessageToChannel($hook, $message);
             $this->checkFiles($patchId, $json->commit);
         }
@@ -138,7 +138,7 @@ 
         $attachment->setColor($color[$status]);
         $attachment->setTitle($text[$status]);
 
-        $text = ':link: <https://git.typo3.org/Packages/TYPO3.CMS.git/blob/HEAD:/' . $fileName . '|' . $fileName . '>';
+        $text = ':link: <https://git.typo3.org/Packages/TYPO3.CMS.git/blob/HEAD:/'.$fileName.'|'.$fileName.'>';
         $attachment->setText($text);
         $attachment->setFallback($text);
         return $attachment;