refactor: split binder interface

Possibility to check, if binder is available is not necessary for
outside usage besides `CompositeBinder` class, so this interface can be
split and implemented only by specific classes.

Signed-off-by: Bart Jaskulski <bjaskulski@protonmail.com>

src/Binding/Binder.php

@@ -6,7 +6,5 @@ namespace WPDesk\Init\Binding;
 
 interface Binder {
 
-	public function can_bind( Definition $def ): bool;
-
 	public function bind( Definition $def ): void;
 }

src/Binding/Binder/CallableBinder.php

@@ -5,11 +5,11 @@ declare(strict_types=1);
 namespace WPDesk\Init\Binding\Binder;
 
 use Psr\Container\ContainerInterface;
-use WPDesk\Init\Binding\Binder;
+use WPDesk\Init\Binding\ComposableBinder;
 use WPDesk\Init\Binding\Definition;
 use WPDesk\Init\Binding\Definition\CallableDefinition;
 
-class CallableBinder implements Binder {
+class CallableBinder implements ComposableBinder {
 
 	/** @var ContainerInterface */
 	private $container;
@@ -23,7 +23,6 @@ class CallableBinder implements Binder {
 	}
 
 	public function bind( Definition $def ): void {
-		if ( $def instanceof CallableDefinition ) {
 		$ref        = new \ReflectionFunction( $def->value() );
 		$parameters = [];
 		foreach ( $ref->getParameters() as $ref_param ) {
@@ -32,4 +31,3 @@ class CallableBinder implements Binder {
 		$ref->invokeArgs( $parameters );
 	}
 }
-}

src/Binding/Binder/CompositeBinder.php

@@ -21,16 +21,6 @@ final class CompositeBinder implements Binder {
 		$this->binders[] = $binder;
 	}
 
-	public function can_bind( Definition $def ): bool {
-		foreach ( $this->binders as $binder ) {
-			if ( $binder->can_bind( $def ) ) {
-				return true;
-			}
-		}
-
-		return false;
-	}
-
 	public function bind( Definition $def ): void {
 		foreach ( $this->binders as $binder ) {
 			if ( $binder->can_bind( $def ) ) {

src/Binding/Binder/HookableBinder.php

@@ -5,11 +5,11 @@ declare(strict_types=1);
 namespace WPDesk\Init\Binding\Binder;
 
 use Psr\Container\ContainerInterface;
-use WPDesk\Init\Binding\Binder;
+use WPDesk\Init\Binding\ComposableBinder;
 use WPDesk\Init\Binding\Definition;
 use WPDesk\Init\Binding\Definition\HookableDefinition;
 
-class HookableBinder implements Binder {
+class HookableBinder implements ComposableBinder {
 
 	/** @var ContainerInterface */
 	private $container;
@@ -23,8 +23,6 @@ class HookableBinder implements Binder {
 	}
 
 	public function bind( Definition $def ): void {
-		if ( $def instanceof HookableDefinition ) {
 		$this->container->get( $def->value() )->hooks();
 	}
 }
-}

src/Binding/Binder/StoppableBinder.php

@@ -5,12 +5,12 @@ declare(strict_types=1);
 namespace WPDesk\Init\Binding\Binder;
 
 use Psr\Container\ContainerInterface;
-use WPDesk\Init\Binding\Binder;
+use WPDesk\Init\Binding\ComposableBinder;
 use WPDesk\Init\Binding\StoppableBinder as Stop;
 use WPDesk\Init\Binding\Definition;
 use WPDesk\Init\Binding\Definition\HookableDefinition;
 
-class StoppableBinder implements Binder {
+class StoppableBinder implements ComposableBinder {
 
 	/** @var ContainerInterface */
 	private $container;

src/Binding/ComposableBinder.php

+<?php
+
+declare(strict_types=1);
+
+namespace WPDesk\Init\Binding;
+
+/**
+ * Can be composed with other binders within {@see CompositeBinder} class.
+ */
+interface ComposableBinder extends Binder {
+
+	public function can_bind( Definition $def ): bool;
+}