fixes #75 (#289)

* fixes #75

* remove loop | refactor writer

* cs fixes

* small refactor on dispatcher | added log for writer

* remove xmlable | added some functions

* add if check

* Scrutinizer Auto-Fixes (#290)

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com

* added a new class for handler

* fixed tests

* phpdoc fixes

* Added more tests

* remove on in writer

* Scrutinizer Auto-Fixes (#291)

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com

* added more tests

* added more tests

* phpdoc fix

* Scrutinizer Auto-Fixes (#292)

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com

Author: prisis

phpunit.xml.dist

@@ -70,13 +70,10 @@
         <testsuite name="Narrowspark Http Component Test Suite">
             <directory>./src/Viserio/Http/Tests</directory>
         </testsuite>
-<!--        <testsuite name="Narrowspark Log Component Test Suite">
+        <testsuite name="Narrowspark Log Component Test Suite">
             <directory>./src/Viserio/Log/Tests</directory>
         </testsuite>
-        <testsuite name="Narrowspark Loop Component Test Suite">
-            <directory>./src/Viserio/Loop/Tests</directory>
-        </testsuite>
-        <testsuite name="Narrowspark Mail Component Test Suite">
+<!--         <testsuite name="Narrowspark Mail Component Test Suite">
             <directory>./src/Viserio/Mail/Tests</directory>
         </testsuite> -->
         <testsuite name="Narrowspark Middleware Component Test Suite">

src/Viserio/Contracts/Log/Log.php

@@ -0,0 +1,43 @@
+<?php
+namespace Viserio\Contracts\Log;
+
+use Psr\Log\LoggerInterface as PsrLoggerInterface;
+
+interface Log extends PsrLoggerInterface
+{
+    /**
+     * Register a file log handler.
+     *
+     * @param string      $path
+     * @param string      $level
+     * @param object|null $processor
+     * @param object|null $formatter
+     *
+     * @return void
+     */
+    public function useFiles(
+        string $path,
+        string $level = 'debug',
+        $processor = null,
+        $formatter = null
+    );
+
+    /**
+     * Register a daily file log handler.
+     *
+     * @param string      $path
+     * @param int         $days
+     * @param string      $level
+     * @param object|null $processor
+     * @param object|null $formatter
+     *
+     * @return void
+     */
+    public function useDailyFiles(
+        string $path,
+        int $days = 0,
+        string $level = 'debug',
+        $processor = null,
+        $formatter = null
+    );
+}

src/Viserio/Contracts/Logging/Log.php

@@ -99,19 +99,7 @@ public function emit(string $eventName, array $arguments = [], $continue = null)
         $listeners = $this->getListeners($eventName);
 
         if ($continue === null) {
-            foreach ($listeners as $listener) {
-                $result = false;
-
-                if ($listener !== null) {
-                    $result = $this->invoker->call($listener, $arguments);
-                }
-
-                if ($result === false) {
-                    return false;
-                }
-            }
-
-            return true;
+            return $this->continueEmit($listeners, $arguments);
         }
 
         $counter = count($listeners);
@@ -308,4 +296,30 @@ protected function removeListenerPattern(string $eventPattern, $listener)
             }
         }
     }
+
+    /**
+     * If the continue is specified, this callback will be called every
+     * time before the next event handler is called.
+     *
+     * @param array $listeners
+     * @param array $arguments
+     *
+     * @return bool
+     */
+    protected function continueEmit(array $listeners, array $arguments): bool
+    {
+        foreach ($listeners as $listener) {
+            $result = false;
+
+            if ($listener !== null) {
+                $result = $this->invoker->call($listener, $arguments);
+            }
+
+            if ($result === false) {
+                return false;
+            }
+        }
+
+        return true;
+    }
 }

src/Viserio/Contracts/Loop/Loop.php

@@ -8,6 +8,8 @@ final class RequestFactory implements RequestFactoryContract
 {
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createRequest(
         string $method = 'GET',

src/Viserio/Contracts/Loop/LoopInitializedException.php

@@ -8,6 +8,8 @@ final class ResponseFactory implements ResponseFactoryContract
 {
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createResponse(
         int $code = 200

src/Viserio/Contracts/Loop/ResourceBusyException.php

@@ -33,6 +33,8 @@ public function createServerRequestFromGlobals(): ServerRequestInterface
 
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createServerRequest(string $method, $uri): ServerRequestInterface
     {

src/Viserio/Contracts/Loop/RunningException.php

@@ -8,6 +8,8 @@ final class StreamFactory implements StreamFactoryContract
 {
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createStream(): StreamInterface
     {
@@ -16,6 +18,8 @@ public function createStream(): StreamInterface
 
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createStreamFromCallback(callable $callback): StreamInterface
     {
@@ -24,6 +28,8 @@ public function createStreamFromCallback(callable $callback): StreamInterface
 
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createStreamFromResource($body): StreamInterface
     {
@@ -32,6 +38,8 @@ public function createStreamFromResource($body): StreamInterface
 
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createStreamFromString(string $body): StreamInterface
     {

src/Viserio/Contracts/Loop/Timer.php

@@ -8,6 +8,8 @@ final class UploadedFileFactory implements UploadedFileFactoryContract
 {
     /**
      * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
      */
     public function createUploadedFile(
         $file,