it was decided to remove the $mappers argument because mappers are mo…

…re elegantly supported by the Mappers trait and MapperProvider interface; closes #20

src/framework/Connection.php

@@ -16,20 +16,16 @@ interface Connection
     /**
      * Fetch the `Result` of executing an SQL "SELECT" statement.
      *
-     * If the given `Executable` implements `MapperProvider`, any Mappers provided by it will
-     * be prepended to the given list of Mappers.
-     *
      * Note that you can directly iterate over the `Result` instance.
      *
+     * @see MapperProvider optional interface enabling an Executable to provide Mappers
+     *
      * @param Statement $statement
      * @param int       $batch_size batch-size (when fetching large result sets)
-     * @param Mapper[]  $mappers    list of additional Mappers to apply while fetching results
      *
      * @return Result
-     *
-     * @see MapperProvider optional interface enabling an Executable to provide Mappers
      */
-    public function fetch(Statement $statement, $batch_size = 1000, array $mappers = []);
+    public function fetch(Statement $statement, $batch_size = 1000);
 
     /**
      * Execute an SQL statement, which does not produce a result, e.g. an "INSERT", "UPDATE" or "DELETE" statement.

src/framework/pdo/PDOConnection.php

@@ -94,12 +94,11 @@ public function prepare(Statement $statement)
     /**
      * @inheritdoc
      */
-    public function fetch(Statement $statement, $batch_size = 1000, array $mappers = [])
+    public function fetch(Statement $statement, $batch_size = 1000)
     {
-        if ($statement instanceof MapperProvider) {
-            // prepend Mappers provided by the Executable:
-            $mappers = array_merge($statement->getMappers(), $mappers);
-        }
+        $mappers = $statement instanceof MapperProvider
+            ? $statement->getMappers()
+            : [];
 
         return new Result(
             $this->prepare($statement),