more like builder pattern

na razie propozycja, po krótkiej medytacji :)

Zastanowiłem się jak faktycznie wygląda wzorzec budowniczego i poszedłem mocniej w tą stronę. Dzięki temu sam interfejs budowniczego jest jeszcze bardziej elastyczny. Nie proponowałem na razie directora - directorem jest teraz sam skrypt plugin-create. "Storage" na pluginy wyjąłem w buildera - to było bezpośrednie złamanie SRP i pewnie kilku innych zasad OOP, poza tym dzięki temu storage może być czymkolwiek - nawet zewnętrznym serwerem cache. Użycie fabryki na storage pozwoli wygodnie podmieniać storage gdyby była taka potrzeba w obrębie całej instancji WP.

src/Builder/AbstractBuilder.php

@@ -3,31 +3,62 @@
 namespace WPDesk\PluginBuilder\Builder;
 
 use WPDesk\PluginBuilder\Plugin\AbstractPlugin;
+use WPDesk\PluginBuilder\Storage\PluginStorage;
 
-class AbstractBuilder {
-	protected static $instances = [];
+abstract class AbstractBuilder {
+	/**
+	 * Create plugin class
+	 */
+	public function build_plugin() {
+	}
 
 	/**
-	 * @param string $class
-	 * @param AbstractPlugin $object
+	 * Store plugin class in some kind of storage
 	 */
-	protected function addToStorage( $class, $object ) {
-		if ( isset( self::$instances[ $class ] ) ) {
-			throw new Exception\ClassAlreadyExists( "Class {$class} already exists" );
+	public function store_plugin( PluginStorage $storage ) {
 	}
-		self::$instances[ $class ] = $object;
+
+	/**
+	 * Init plugin internal structure
+	 */
+	public function init_plugin() {
 	}
 
 	/**
-	 * @param string $class
-	 *
+	 * Return built plugin
 	 * @return AbstractPlugin
 	 */
-	protected function getFromStorage( $class ) {
-		if ( isset( self::$instances[ $class ] ) ) {
-			return self::$instances[ $class ];
-		} else {
-			throw new Exception\ClassNotExists( "Class {$class} not exists in storage" );
+	abstract function get_plugin();
+
+	/**
+	 * Set settings class in plugin
+	 *
+	 * @param $settings
+	 */
+	public function set_settings( $settings ) {
 	}
+
+	/**
+	 * Set view class in plugin
+	 *
+	 * @param $view
+	 */
+	public function set_view( $view ) {
+	}
+
+	/**
+	 * Set tracker class in plugin
+	 *
+	 * @param $tracker
+	 */
+	public function set_tracker( $tracker ) {
+	}
+
+	/**
+	 * Set helper class in plugin
+	 *
+	 * @param $helper
+	 */
+	public function set_helper( $helper ) {
 	}
 }
\ No newline at end of file