twistofreality / dilmun

app/config/app_constants.php

Spacing +6 added lines, -6 removed lines

@@ -2,29 +2,29 @@
 /**
  * @const DILMUN_BASE_DIR The base directory at which Dilmun resides
  */
-define('DILMUN_BASE_DIR', dirname(dirname(dirname(__FILE__))) . "/");
+define('DILMUN_BASE_DIR', dirname(dirname(dirname(__FILE__)))."/");
 
 /**
  * @const DILMUN_SOURCE_DIR The source directory
  */
-define('DILMUN_SOURCE_DIR', DILMUN_BASE_DIR . "src/");
+define('DILMUN_SOURCE_DIR', DILMUN_BASE_DIR."src/");
 
 /**
  * @const DILMUN_VENDOR_DIR The vendor directory
  */
-define('DILMUN_VENDOR_DIR', DILMUN_BASE_DIR . "vendor/");
+define('DILMUN_VENDOR_DIR', DILMUN_BASE_DIR."vendor/");
 
 /**
  * @const DILMUN_LOGS_DIR The log directory
  */
-define('DILMUN_LOGS_DIR', DILMUN_BASE_DIR . "app/logs/");
+define('DILMUN_LOGS_DIR', DILMUN_BASE_DIR."app/logs/");
 
 /**
  * @const DILMUN_CONFIG_DIR The configurations directory
  */
-define('DILMUN_CONFIG_DIR', DILMUN_BASE_DIR . "app/config/");
+define('DILMUN_CONFIG_DIR', DILMUN_BASE_DIR."app/config/");
 
 /**
  * @const DILMUN_TEST_DIR The tests directory
  */
-define('DILMUN_TEST_DIR', DILMUN_BASE_DIR . "tests/");
+define('DILMUN_TEST_DIR', DILMUN_BASE_DIR."tests/");

src/Nabu/LoggerHandler/File.php

Spacing +2 added lines, -2 removed lines

@@ -176,7 +176,7 @@ 
         $full_file = "";
 
         if (isset($this->file_path)) {
-            $full_file = $this->file_path . DIRECTORY_SEPARATOR;
+            $full_file = $this->file_path.DIRECTORY_SEPARATOR;
         }
 
         $full_file .= $this->file_name;
@@ -230,7 +230,7 @@ 
 
             if ($locked) {
                 $now = date('Y-m-d H:i:s');
-                $this->file_pointer->fwrite("$now : $message" . "\r\n");
+                $this->file_pointer->fwrite("$now : $message"."\r\n");
                 $this->file_pointer->flock(LOCK_UN);
 
                 return true;

src/Nabu/StaticLogger.php

Spacing +1 added lines, -1 removed lines

@@ -218,7 +218,7 @@
         $replace = array();
 
         foreach ($context as $key => $value) {
-            $templated           = "{" . $key . "}";
+            $templated           = "{".$key."}";
             $replace[$templated] = $value;
         }
 

app/bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -4,8 +4,8 @@
  */
 
 require "config/app_constants.php";
-require DILMUN_VENDOR_DIR . "autoload.php";
-require DILMUN_SOURCE_DIR . "Nabu/Autoloader.php";
+require DILMUN_VENDOR_DIR."autoload.php";
+require DILMUN_SOURCE_DIR."Nabu/Autoloader.php";
 
 /**
  * @todo Pull this out somehow so that logging on bootstrap is more abstract

src/Nabu/Autoloader.php

Spacing +5 added lines, -5 removed lines

@@ -23,7 +23,7 @@ 
      */
     public function register()
     {
-        spl_autoload_register(array($this,"includeClass"));
+        spl_autoload_register(array($this, "includeClass"));
     }
 
     /**
@@ -41,10 +41,10 @@ 
         $this->updateLog("info", "Preparing package {$package} for registration with path {$path}.");
 
         $package = trim($package, "\\");
-        $package = $package . "\\";
+        $package = $package."\\";
 
         $path = rtrim($path, DIRECTORY_SEPARATOR);
-        $path = $path . "/";
+        $path = $path."/";
 
         $path_registered = $this->registrationExists($package, $path);
 
@@ -82,7 +82,7 @@ 
         $package_stack = array();
 
         foreach ($class_bits as $bit) {
-            $package = $package . $bit . "\\";
+            $package = $package.$bit."\\";
 
             $package_registered = $this->packageRegistered($package);
 
@@ -140,7 +140,7 @@ 
 
         foreach ($this->packages[$package_name] as $path) {
             $converted_class = str_replace('\\', DIRECTORY_SEPARATOR, $class_name);
-            $class_path      = $path . $converted_class . ".php";
+            $class_path      = $path.$converted_class.".php";
 
             $this->updateLog("debug", "Class file path is {$class_path}");
 

Upper-Lower-Casing +6 added lines, -6 removed lines

@@ -114,7 +114,7 @@ 
             $package_loaded = $this->loadPackage($package, $class_path);
 
             if ($package_loaded) {
-                $this->updateLog("info", "Loaded {$class_path} using package {$package}");
+                $this->updateLog("info", "loaded {$class_path} using package {$package}");
                 return true;
             }
         }
@@ -163,9 +163,9 @@ 
         $package_registered = array_key_exists($package, $this->packages);
 
         if ($package_registered) {
-            $this->updateLog("info", "Package {$package} is registered");
+            $this->updateLog("info", "package {$package} is registered");
         } else {
-            $this->updateLog("info", "Package {$package} is not registered");
+            $this->updateLog("info", "package {$package} is not registered");
         }
 
         return $package_registered;
@@ -190,7 +190,7 @@ 
             if ($path_registered !== false) {
                 $path_registered = true;
 
-                $this->updateLog("info", "Path {$path} already registered for package {$package}");
+                $this->updateLog("info", "path {$path} already registered for package {$package}");
             }
         } else {
             $path_registered = false;
@@ -210,7 +210,7 @@ 
     private function requireFile($path)
     {
         if (file_exists($path)) {
-            $this->updateLog("debug", "File {$path} exists; requiring file");
+            $this->updateLog("debug", "file {$path} exists; requiring file");
 
             /** @noinspection PhpIncludeInspection */
             require $path;
@@ -219,7 +219,7 @@ 
 
             return true;
         } else {
-            $this->updateLog("debug", "File {$path} does not exist");
+            $this->updateLog("debug", "file {$path} does not exist");
 
             return false;
         }

src/Kishar/Page.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
         } else {
             $this->contents = false;
             
-            $this->updateLog("alert", "File {$contents} not found");
+            $this->updateLog("alert", "file {$contents} not found");
         }
 
         return $contents_exist;

spec/Anshar/HttpReaderSpec.php

Unused Use Statements +1 added lines, -3 removed lines

@@ -1,10 +1,8 @@
 <?php
 namespace spec\Subreality\Dilmun\Anshar;
 
-use Subreality\Dilmun\Anshar\HttpReader;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
-use Subreality\Dilmun\Nabu\Logger;
+use Subreality\Dilmun\Anshar\HttpReader;
 
 class HttpReaderSpec extends ObjectBehavior
 {

spec/Anshar/Http/RequestSpec.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,10 +2,10 @@
 
 namespace spec\Subreality\Dilmun\Anshar\Http;
 
+use PhpSpec\ObjectBehavior;
 use Psr\Http\Message\StreamInterface;
 use Psr\Http\Message\UriInterface;
 use Subreality\Dilmun\Anshar\Http\Request;
-use PhpSpec\ObjectBehavior;
 
 /**
  * Class RequestSpec

Indentation +1 added lines, -1 removed lines

@@ -455,7 +455,7 @@
      * @param \PhpSpec\Wrapper\Collaborator|StreamInterface $new_body
      */
     function it_retains_its_body_when_returning_a_new_request_with_a_body(StreamInterface $old_body,
-                                                                          StreamInterface $new_body)
+                                                                            StreamInterface $new_body)
     {
         $old_body->getContents()->willReturn("I'm the original stream!");
         $new_body->getContents()->willReturn("I'm the new stream!");

Spacing +2 added lines, -2 removed lines

@@ -702,7 +702,7 @@ 
 
         $uri->getHost()->willReturn($host);
 
-        $empty_host = $this->withHeader("host","");
+        $empty_host = $this->withHeader("host", "");
 
         $request = $empty_host->withUri($uri, true);
 
@@ -723,7 +723,7 @@ 
     {
         $uri->getHost()->willReturn(null);
 
-        $empty_host = $this->withHeader("host","");
+        $empty_host = $this->withHeader("host", "");
 
         $request = $empty_host->withUri($uri);
 

Doc Comments +10 added lines, -10 removed lines

@@ -435,7 +435,7 @@ 
     /**
      * Return an instance with the specified message body.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|StreamInterface $body
+     * @param StreamInterface $body
      */
     function it_returns_a_new_request_with_a_body(StreamInterface $body)
     {
@@ -451,8 +451,8 @@ 
      * immutability of the message, and MUST return a new instance that has the
      * new body stream.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|StreamInterface $old_body
-     * @param \PhpSpec\Wrapper\Collaborator|StreamInterface $new_body
+     * @param StreamInterface $old_body
+     * @param StreamInterface $new_body
      */
     function it_retains_its_body_when_returning_a_new_request_with_a_body(StreamInterface $old_body,
                                                                           StreamInterface $new_body)
@@ -624,7 +624,7 @@ 
      * This method MUST update the Host header of the returned request by
      * default if the URI contains a host component.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|UriInterface $uri
+     * @param UriInterface $uri
      */
     function it_returns_a_new_request_with_a_uri_with_host_replacement(UriInterface $uri)
     {
@@ -648,7 +648,7 @@ 
      * - If a Host header is present and non-empty, this method MUST NOT update
      *   the Host header in the returned request.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|UriInterface $uri
+     * @param UriInterface $uri
      */
     function it_returns_a_new_request_with_a_uri_without_host_replacement(UriInterface $uri)
     {
@@ -671,7 +671,7 @@ 
      *   a host component, this method MUST update the Host header in the returned
      *   request.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|UriInterface $uri
+     * @param UriInterface $uri
      */
     function it_returns_a_new_request_with_a_host_when_no_host_set(UriInterface $uri)
     {
@@ -694,7 +694,7 @@ 
      *   a host component, this method MUST update the Host header in the returned
      *   request.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|UriInterface $uri
+     * @param UriInterface $uri
      */
     function it_returns_a_new_request_with_a_host_when_empty_host_set(UriInterface $uri)
     {
@@ -717,7 +717,7 @@ 
      *   host component, this method MUST NOT update the Host header in the returned
      *   request.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|UriInterface $uri
+     * @param UriInterface $uri
      */
     function it_preserves_its_original_empty_host_with_a_hostless_uri(UriInterface $uri)
     {
@@ -735,8 +735,8 @@ 
      * immutability of the message, and MUST return an instance that has the
      * new UriInterface instance.
      *
-     * @param \PhpSpec\Wrapper\Collaborator|UriInterface $old_uri
-     * @param \PhpSpec\Wrapper\Collaborator|UriInterface $new_uri
+     * @param UriInterface $old_uri
+     * @param UriInterface $new_uri
      */
     function it_retains_its_uri_when_returning_a_new_request_with_a_uri(UriInterface $old_uri,
                                                                         UriInterface $new_uri)

spec/Anshar/Http/UriSpec.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -1,9 +1,8 @@
 <?php
 namespace spec\Subreality\Dilmun\Anshar\Http;
 
-use Subreality\Dilmun\Anshar\Http\Uri;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
+use Subreality\Dilmun\Anshar\Http\Uri;
 
 class UriSpec extends ObjectBehavior
 {