feat: remove compilation command

After thinking for some time, I've decided to get rid of compilation command. This would be useful to compile container and plugin data at build-time, but for the container the task is not trivial.

Compilation would require WordPress runtime, so the only viable solution would be to boot whole WordPress with database. Removing the use of WordPress functions wouldn't be possible or at least required some common WP functions to be rewritten in our codebase.

For current approach, I bias towards compilation in customer's environment at first runtime, with graceful handling of any IO operation issues.

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

src/Extension/ConditionalExtension.php

@@ -5,37 +5,38 @@ declare(strict_types=1);
 namespace WPDesk\Init\Extension;
 
 use DI\Definition\Helper\AutowireDefinitionHelper;
-use Monolog\Formatter\LineFormatter;
-use Monolog\Logger;
-use Monolog\Processor\PsrLogMessageProcessor;
-use Monolog\Processor\UidProcessor;
 use Psr\Container\ContainerInterface;
 use Psr\Log\LoggerInterface;
 use WPDesk\Init\Binding\Loader\ArrayDefinitions;
 use WPDesk\Init\Binding\Loader\BindingDefinitions;
+use WPDesk\Init\Configuration\Configuration;
 use WPDesk\Init\Configuration\ReadableConfig;
 use WPDesk\Init\DependencyInjection\ContainerBuilder;
 use WPDesk\Init\Extension\CommonBinding\RequirementsCheck;
 use WPDesk\Init\Extension\CommonBinding\WPDeskLicenseBridge;
 use WPDesk\Init\Extension\CommonBinding\WPDeskTrackerBridge;
 use WPDesk\Init\Plugin\Plugin;
-use WPDesk\Logger\WC\WooCommerceHandler;
+use WPDesk\Logger\SimpleLoggerFactory;
 
 class ConditionalExtension implements Extension {
 
 	public function bindings( ContainerInterface $c ): BindingDefinitions {
+		$config   = $c->get( Configuration::class );
 		$bindings = [];
 
 		if ( class_exists( \WPDesk_Basic_Requirement_Checker::class ) ) {
-			$bindings[] = RequirementsCheck::class;
+			$bindings[] = [
+				'priority' => -10,
+				'handler'  => RequirementsCheck::class,
+			];
 		}
 
-		if ( class_exists( \WPDesk\License\PluginRegistrator::class ) ) {
+		if ( class_exists( \WPDesk\License\LicenseServer\PluginRegistrator::class ) ) {
 			$bindings[] = WPDeskLicenseBridge::class;
 		}
 
 		if ( class_exists( \WPDesk_Tracker::class ) ) {
-			$bindings[] = WPDeskTrackerBridge::class;
+			$bindings['plugins_loaded'] = WPDeskTrackerBridge::class;
 		}
 
 		return new ArrayDefinitions( $bindings );
@@ -48,37 +49,27 @@ class ConditionalExtension implements Extension {
 			$definitions[ RequirementsCheck::class ] = new AutowireDefinitionHelper();
 		}
 
-		if ( class_exists( \WPDesk\License\PluginRegistrator::class ) ) {
-			$definitions[ WPDeskLicenseBridge::class ] = new AutowireDefinitionHelper();
+		if ( class_exists( \WPDesk\License\LicenseServer\PluginRegistrator::class ) ) {
+			$definitions[ WPDeskLicenseBridge::class ] = ( new AutowireDefinitionHelper() )
+				->constructorParameter( 'product_id', $config->get( 'product_id' ) )
+				->constructorParameter( 'shops', (array) $config->get( 'shops', [] ) );
 		}
 
 		if ( class_exists( \WPDesk_Tracker::class ) ) {
 			$definitions[ WPDeskTrackerBridge::class ] = new AutowireDefinitionHelper();
 		}
 
-		if ( class_exists( \WPDesk\Logger\WC\WooCommerceHandler::class ) ) {
-			$definitions[ LoggerInterface::class ] = static function () use ( $plugin ) {
-				$logger = new Logger(
-					$plugin->get_slug(),
-					[],
-					[ new PsrLogMessageProcessor( null, true ), new UidProcessor() ]
-				);
-
-				$attach_handler = function () use ( $logger, $plugin ) {
-					$handler = new WooCommerceHandler( wc_get_logger(), $plugin->get_slug() );
-					$handler->setFormatter(
-						new LineFormatter( '%channel%.%level_name% [%extra.uid%]: %message% %context% %extra%' )
-					);
-					$logger->pushHandler( $handler );
-				};
-
-				if ( \function_exists( 'wc_get_logger' ) ) {
-					$attach_handler();
-				} else {
-					\add_action( 'woocommerce_init', $attach_handler );
-				}
-
-				return $logger;
+		if ( class_exists( \WPDesk\Logger\SimpleLoggerFactory::class ) ) {
+			$definitions[ LoggerInterface::class ] = static function ( ContainerInterface $c ) {
+				$p = $c->get( Plugin::class );
+
+				return ( new SimpleLoggerFactory(
+					$p->get_slug(),
+					[
+						'level'        => $c->has( 'logger.level' ) ? $c->get( 'logger.level' ) : 'debug',
+						'action_level' => $c->has( 'logger.action_level' ) ? $c->get( 'logger.action_level' ) : null,
+					]
+				) )->getLogger();
 			};
 		}