meadsteve / DiceApi

src/Renderer/Json.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
     private function diceAsAssocArrays(array $diceCollection)
     {
-        return array_map(function (Dice $dice) {
+        return array_map(function(Dice $dice) {
             return [
                 "value" => $dice->roll(),
                 "size" => "d" . $dice->size()

src/RequestHandler/DiceRequestHandler.php

Spacing +4 added lines, -4 removed lines

@@ -42,10 +42,10 @@ 
             $this->diceCounter->count($dice);
         } catch (UncreatableDiceException $creationError) {
             $diceResponse = $diceResponse->withStatus(400)
-                ->write("Unable to roll dice: " . $creationError->getMessage());
+                ->write("Unable to roll dice: ".$creationError->getMessage());
         } catch (UnrenderableDiceException $renderError) {
             $diceResponse = $diceResponse->withStatus(400)
-                ->write("Unable to render request: " . $renderError->getMessage());
+                ->write("Unable to render request: ".$renderError->getMessage());
         }
         return $diceResponse;
     }
@@ -60,7 +60,7 @@ 
                 ->withHeader("Content-Type", $renderer->contentType());
         } catch (UnknownRendererException $error) {
             $responseWithOutput = $response->withStatus(406);
-            $responseWithOutput->write("Not sure how to respond with: " . $requestedContentType);
+            $responseWithOutput->write("Not sure how to respond with: ".$requestedContentType);
         }
         return $responseWithOutput;
     }
@@ -69,7 +69,7 @@ 
     {
         $rolledValue = $request->getHeader('totally-legit')[0];
         return array_map(
-            function (Dice $dice) use ($rolledValue) {
+            function(Dice $dice) use ($rolledValue) {
                 return new TotallyLegit($dice, (int) $rolledValue);
             },
             $dice

src/Renderer/Html.php

Spacing +1 added lines, -1 removed lines

@@ -35,6 +35,6 @@
         }
         $roll = $dice->roll();
         $url = "{$this->urlRoot}/images/poorly-drawn/d{$size}/{$roll}.png";
-        return '<img src="' . $url . '" />';
+        return '<img src="'.$url.'" />';
     }
 }

src/UrlDiceGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         $data = [];
         $valid = preg_match("/(?P<count>[0-9]+)?d(?P<type>[^\/]+)/i", $part, $data);
         if (!$valid) {
-            throw new UncreatableDiceException("Problem creating dice from incorrectly formated data: " . $part);
+            throw new UncreatableDiceException("Problem creating dice from incorrectly formated data: ".$part);
         }
         if (!$data["count"]) {
             $data["count"] = 1;

src/Dice/Factories/SpecialDiceFactory.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public function __construct()
     {
         $this->diceTypeCallbacks = [
-            'steve' => function ($_type, $diceCount) {
+            'steve' => function($_type, $diceCount) {
                 $newDice = [];
                 for ($i = 0; $i < $diceCount; $i++) {
                     $newDice[] = new SteveDice();

Unused Use Statements -3 removed lines

@@ -2,11 +2,8 @@
 
 namespace MeadSteve\DiceApi\Dice\Factories;
 
-use MeadSteve\DiceApi\Dice\BasicDice;
 use MeadSteve\DiceApi\Dice;
 use MeadSteve\DiceApi\Dice\SteveDice;
-use MeadSteve\DiceApi\Dice\UncreatableDiceException;
-use MeadSteve\DiceApi\Dice\ZeropointDice;
 
 class SpecialDiceFactory implements DiceFactory
 {

src/Dice/BasicDice.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 
     public function roll()
     {
-        if (! function_exists('random_int')) {
+        if (!function_exists('random_int')) {
             return mt_rand(1, $this->size);
         }
 

www/index.php

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
 use MeadSteve\DiceApi\RequestHandler\DiceRequestHandler;
 use Predis\Client;
 
-require __DIR__ . "/../vendor/autoload.php";
+require __DIR__."/../vendor/autoload.php";
 
 $diceGenerator = new UrlDiceGenerator(
     new DiceFactoryCollection([
@@ -24,7 +24,7 @@ 
 
 $rendererCollection = new RendererCollection([
     new Json(),
-    new Html('http://' . $_SERVER['HTTP_HOST'])
+    new Html('http://'.$_SERVER['HTTP_HOST'])
 ]);
 
 if (isset($_ENV['REDIS_URL'])) {

src/DiceApp.php

Spacing +5 added lines, -5 removed lines

@@ -27,11 +27,11 @@ 
 
     public function index(Request $request, Response $response)
     {
-        $indexFilePath = __DIR__ . "/generated-index.html";
+        $indexFilePath = __DIR__."/generated-index.html";
         if (!file_exists($indexFilePath)) {
             $converter = new CommonMarkConverter();
-            $indexBody = $converter->convertToHtml(file_get_contents(__DIR__ . "/../README.md"));
-            $indexContent = file_get_contents(__DIR__ . "/../www/templates/index.html");
+            $indexBody = $converter->convertToHtml(file_get_contents(__DIR__."/../README.md"));
+            $indexContent = file_get_contents(__DIR__."/../www/templates/index.html");
             $indexContent = str_replace("{{body}}", $indexBody, $indexContent);
             file_put_contents($indexFilePath, $indexContent);
             $response->write($indexContent);
@@ -59,8 +59,8 @@ 
 
         foreach ($this->diceRequestHandler->contentTypesForPaths() as $path => $contentType) {
             $this->get(
-                "/{$path}" . self::DICE_PATH_REGEX,
-                function (Request $request, $response, $args) use ($diceRequestHandler, $contentType) {
+                "/{$path}".self::DICE_PATH_REGEX,
+                function(Request $request, $response, $args) use ($diceRequestHandler, $contentType) {
                     return $diceRequestHandler->getDice(
                         $request->withHeader('accept', $contentType),
                         $response,