Adhere feedback

- rename hooks.config.php to strategies.config.php
- change all corresponding classes and tests

doc/StrategyHooks.md

@@ -43,7 +43,7 @@ public class ConcreteClassB implements ExampleInterface
 	}
 }
 
-/** @var \Friendica\Core\Hooks\Capabilities\ICanRegisterInstances $instanceRegister */
+/** @var \Friendica\Core\Hooks\Capabilities\ICanRegisterStrategies $instanceRegister */
 $instanceRegister->registerStrategy(ExampleInterface::class, ConcreteClassA::class, 'A');
 $instanceRegister->registerStrategy(ExampleInterface::class, ConcreteClassB::class, 'B');
 

src/Core/Hooks/Capabilities/ICanCreateInstances.php

@@ -32,10 +32,10 @@ interface ICanCreateInstances
 	 * The instance will be build based on the registered strategy and the (unique) name
 	 *
 	 * @param string $class     The fully-qualified name of the given class or interface which will get returned
-	 * @param string $name      An arbitrary identifier to find a concrete instance strategy.
+	 * @param string $strategy  An arbitrary identifier to find a concrete instance strategy.
 	 * @param array  $arguments Additional arguments, which can be passed to the constructor of "$class" at runtime
 	 *
 	 * @return object The concrete instance of the type "$class"
 	 */
-	public function create(string $class, string $name, array $arguments = []): object;
+	public function create(string $class, string $strategy, array $arguments = []): object;
 }

src/Core/Hooks/Capabilities/ICanRegisterStrategies.php

@@ -26,7 +26,7 @@ use Friendica\Core\Hooks\Exceptions\HookRegisterArgumentException;
 /**
  * Register strategies for given classes
  */
-interface ICanRegisterInstances
+interface ICanRegisterStrategies
 {
 	/**
 	 * Register a class(strategy) for a given interface with a unique name.
@@ -36,7 +36,7 @@ interface ICanRegisterInstances
 	 * @param string  $interface The interface, which the given class implements
 	 * @param string  $class     The fully-qualified given class name
 	 *                           A placeholder for dependencies is possible as well
-	 * @param ?string $name      An arbitrary identifier for the given class, which will be used for factories, dependency injections etc.
+	 * @param ?string $name      An arbitrary identifier for the given strategy, which will be used for factories, dependency injections etc.
 	 *
 	 * @return $this This interface for chain-calls
 	 *

src/Core/Hooks/Model/DiceInstanceManager.php

@@ -23,31 +23,31 @@ namespace Friendica\Core\Hooks\Model;
 
 use Dice\Dice;
 use Friendica\Core\Hooks\Capabilities\ICanCreateInstances;
-use Friendica\Core\Hooks\Capabilities\ICanRegisterInstances;
+use Friendica\Core\Hooks\Capabilities\ICanRegisterStrategies;
 use Friendica\Core\Hooks\Exceptions\HookInstanceException;
 use Friendica\Core\Hooks\Exceptions\HookRegisterArgumentException;
-use Friendica\Core\Hooks\Util\HookFileManager;
+use Friendica\Core\Hooks\Util\StrategiesFileManager;
 
 /**
  * This class represents an instance register, which uses Dice for creation
  *
  * @see Dice
  */
-class DiceInstanceManager implements ICanCreateInstances, ICanRegisterInstances
+class DiceInstanceManager implements ICanCreateInstances, ICanRegisterStrategies
 {
 	protected $instance = [];
 
 	/** @var Dice */
 	protected $dice;
 
-	public function __construct(Dice $dice, HookFileManager $hookFileManager)
+	public function __construct(Dice $dice, StrategiesFileManager $strategiesFileManager)
 	{
 		$this->dice = $dice;
-		$hookFileManager->setupHooks($this);
+		$strategiesFileManager->setupStrategies($this);
 	}
 
 	/** {@inheritDoc} */
-	public function registerStrategy(string $interface, string $class, ?string $name = null): ICanRegisterInstances
+	public function registerStrategy(string $interface, string $class, ?string $name = null): ICanRegisterStrategies
 	{
 		if (!empty($this->instance[$interface][$name])) {
 			throw new HookRegisterArgumentException(sprintf('A class with the name %s is already set for the interface %s', $name, $interface));
@@ -59,12 +59,12 @@ class DiceInstanceManager implements ICanCreateInstances, ICanRegisterInstances
 	}
 
 	/** {@inheritDoc} */
-	public function create(string $class, string $name, array $arguments = []): object
+	public function create(string $class, string $strategy, array $arguments = []): object
 	{
-		if (empty($this->instance[$class][$name])) {
+		if (empty($this->instance[$class][$strategy])) {
-			throw new HookInstanceException(sprintf('The class with the name %s isn\'t registered for the class or interface %s', $name, $class));
+			throw new HookInstanceException(sprintf('The class with the name %s isn\'t registered for the class or interface %s', $strategy, $class));
 		}
 
-		return $this->dice->create($this->instance[$class][$name], $arguments);
+		return $this->dice->create($this->instance[$class][$strategy], $arguments);
 	}
 }

src/Core/Hooks/Util/StrategiesFileManager.php

@@ -22,22 +22,21 @@
 namespace Friendica\Core\Hooks\Util;
 
 use Friendica\Core\Addon\Capabilities\ICanLoadAddons;
-use Friendica\Core\Hooks\Capabilities\BehavioralHookType;
+use Friendica\Core\Hooks\Capabilities\ICanRegisterStrategies;
-use Friendica\Core\Hooks\Capabilities\ICanRegisterInstances;
 use Friendica\Core\Hooks\Exceptions\HookConfigException;
 
 /**
- * Manage all hooks.config.php files
+ * Manage all strategies.config.php files
  */
-class HookFileManager
+class StrategiesFileManager
 {
 	const STATIC_DIR  = 'static';
-	const CONFIG_NAME = 'hooks';
+	const CONFIG_NAME = 'strategies';
 
 	/** @var ICanLoadAddons */
 	protected $addonLoader;
 	/** @var array */
-	protected $hookConfig = [];
+	protected $config = [];
 	/** @var string */
 	protected $basePath;
 
@@ -50,21 +49,13 @@ class HookFileManager
 	/**
 	 * Loads all kinds of hooks and registers the corresponding instances
 	 *
-	 * @param ICanRegisterInstances $instanceRegister The instance register
+	 * @param ICanRegisterStrategies $instanceRegister The instance register
 	 *
 	 * @return void
 	 */
-	public function setupHooks(ICanRegisterInstances $instanceRegister)
+	public function setupStrategies(ICanRegisterStrategies $instanceRegister)
 	{
-		// In case it wasn't used before, reload the whole hook config
+		foreach ($this->config as $interface => $strategy) {
-		if (empty($this->hookConfig)) {
-			$this->reloadHookConfig();
-		}
-
-		foreach ($this->hookConfig as $hookType => $classList) {
-			switch ($hookType) {
-				case BehavioralHookType::STRATEGY:
-					foreach ($classList as $interface => $strategy) {
 			foreach ($strategy as $dependencyName => $names) {
 				if (is_array($names)) {
 					foreach ($names as $name) {
@@ -75,9 +66,6 @@ class HookFileManager
 				}
 			}
 		}
-					break;
-			}
-		}
 	}
 
 	/**
@@ -87,7 +75,7 @@ class HookFileManager
 	 *
 	 * @return void
 	 */
-	protected function reloadHookConfig()
+	public function loadConfig()
 	{
 		// load core hook config
 		$configFile = $this->basePath . '/' . static::STATIC_DIR . '/' . static::CONFIG_NAME . '.config.php';
@@ -102,6 +90,6 @@ class HookFileManager
 			throw new HookConfigException(sprintf('Error loading config file %s.', $configFile));
 		}
 
-		$this->hookConfig = array_merge_recursive($config, $this->addonLoader->getActiveAddonConfig(static::CONFIG_NAME));
+		$this->config = array_merge_recursive($config, $this->addonLoader->getActiveAddonConfig(static::CONFIG_NAME));
 	}
 }

src/Core/Logger/Factory/Logger.php

@@ -43,7 +43,7 @@ class Logger
 		$this->channel = $channel;
 	}
 
-	public function create(ICanCreateInstances $createInstances, IManageConfigValues $config, Profiler $profiler): LoggerInterface
+	public function create(ICanCreateInstances $instanceCreator, IManageConfigValues $config, Profiler $profiler): LoggerInterface
 	{
 		if (empty($config->get('system', 'debugging') ?? false)) {
 			return new NullLogger();
@@ -53,7 +53,7 @@ class Logger
 
 		try {
 			/** @var LoggerInterface $logger */
-			$logger = $createInstances->create(LoggerInterface::class, $name, [$this->channel]);
+			$logger = $instanceCreator->create(LoggerInterface::class, $name, [$this->channel]);
 			if ($config->get('system', 'profiling') ?? false) {
 				return new ProfilerLoggerClass($logger, $profiler);
 			} else {

src/DI.php

@@ -297,7 +297,7 @@ abstract class DI
 		static::init($flushDice);
 	}
 
-	public static function loggCheck(): ICheckLoggerSettings
+	public static function logCheck(): ICheckLoggerSettings
 	{
 		return self::$dice->create(LoggerSettingsCheck::class);
 	}

src/Module/Admin/Summary.php

@@ -126,10 +126,10 @@ class Summary extends BaseAdmin
 		}
 
 		// Check logfile permission
-		if (($return = DI::loggCheck()->checkLogfile()) !== null) {
+		if (($return = DI::logCheck()->checkLogfile()) !== null) {
 			$warningtext[] = $return;
 		}
-		if (($return = DI::loggCheck()->checkDebugLogfile()) !== null) {
+		if (($return = DI::logCheck()->checkDebugLogfile()) !== null) {
 			$warningtext[] = $return;
 		}
 

static/dependencies.config.php

@@ -38,7 +38,7 @@ use Friendica\App;
 use Friendica\Core\Cache;
 use Friendica\Core\Config;
 use Friendica\Core\Hooks\Capabilities\ICanCreateInstances;
-use Friendica\Core\Hooks\Capabilities\ICanRegisterInstances;
+use Friendica\Core\Hooks\Capabilities\ICanRegisterStrategies;
 use Friendica\Core\Hooks\Model\DiceInstanceManager;
 use Friendica\Core\PConfig;
 use Friendica\Core\L10n;
@@ -91,12 +91,15 @@ return [
 			[Dice::INSTANCE => Dice::SELF],
 		]
 	],
-	\Friendica\Core\Hooks\Util\HookFileManager::class => [
+	\Friendica\Core\Hooks\Util\StrategiesFileManager::class => [
 		'constructParams' => [
 			[Dice::INSTANCE => '$basepath'],
 		],
+		'call' => [
+			['loadConfig'],
 		],
-	ICanRegisterInstances::class => [
+	],
+	ICanRegisterStrategies::class => [
 		'instanceOf' => DiceInstanceManager::class,
 		'constructParams' => [
 			[Dice::INSTANCE => Dice::SELF],

static/strategies.config.php

@@ -24,11 +24,9 @@ use Friendica\Core\Logger\Type;
 use Psr\Log;
 
 return [
-	H::STRATEGY  => [
 	Log\LoggerInterface::class => [
 		Log\NullLogger::class    => [''],
 		Type\SyslogLogger::class => ['syslog'],
 		Type\StreamLogger::class => ['stream'],
 	],
-	],
 ];

tests/src/Core/Hooks/Model/InstanceManagerTest.php

@@ -25,7 +25,7 @@ use Dice\Dice;
 use Friendica\Core\Hooks\Exceptions\HookInstanceException;
 use Friendica\Core\Hooks\Exceptions\HookRegisterArgumentException;
 use Friendica\Core\Hooks\Model\DiceInstanceManager;
-use Friendica\Core\Hooks\Util\HookFileManager;
+use Friendica\Core\Hooks\Util\StrategiesFileManager;
 use Friendica\Test\MockedTest;
 use Friendica\Test\Util\Hooks\InstanceMocks\FakeInstance;
 use Friendica\Test\Util\Hooks\InstanceMocks\FakeInstanceDecorator;
@@ -34,15 +34,15 @@ use Mockery\MockInterface;
 
 class InstanceManagerTest extends MockedTest
 {
-	/** @var HookFileManager|MockInterface */
+	/** @var StrategiesFileManager|MockInterface */
 	protected $hookFileManager;
 
 	protected function setUp(): void
 	{
 		parent::setUp();
 
-		$this->hookFileManager = \Mockery::mock(HookFileManager::class);
+		$this->hookFileManager = \Mockery::mock(StrategiesFileManager::class);
-		$this->hookFileManager->shouldReceive('setupHooks')->withAnyArgs();
+		$this->hookFileManager->shouldReceive('setupStrategies')->withAnyArgs();
 	}
 
 	protected function tearDown(): void

tests/src/Core/Hooks/Util/StrategiesFileManagerTest.php

@@ -22,16 +22,16 @@
 namespace Friendica\Test\src\Core\Hooks\Util;
 
 use Friendica\Core\Addon\Capabilities\ICanLoadAddons;
-use Friendica\Core\Hooks\Capabilities\ICanRegisterInstances;
+use Friendica\Core\Hooks\Capabilities\ICanRegisterStrategies;
 use Friendica\Core\Hooks\Exceptions\HookConfigException;
-use Friendica\Core\Hooks\Util\HookFileManager;
+use Friendica\Core\Hooks\Util\StrategiesFileManager;
 use Friendica\Test\MockedTest;
 use Friendica\Test\Util\VFSTrait;
 use org\bovigo\vfs\vfsStream;
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
 
-class HookFileManagerTest extends MockedTest
+class StrategiesFileManagerTest extends MockedTest
 {
 	use VFSTrait;
 
@@ -50,11 +50,9 @@ class HookFileManagerTest extends MockedTest
 <?php
 
 return [
-	\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 		\Psr\Log\LoggerInterface::class => [
 			\Psr\Log\NullLogger::class => [''],
 		],
-	],
 ];
 EOF,
 				'addonsArray'      => [],
@@ -67,11 +65,9 @@ EOF,
 <?php
 
 return [
-	\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 		\Psr\Log\LoggerInterface::class => [
 			\Psr\Log\NullLogger::class => '',
 		],
-	],
 ];
 EOF,
 				'addonsArray'      => [],
@@ -84,20 +80,16 @@ EOF,
 <?php
 
 return [
-	\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 		\Psr\Log\LoggerInterface::class => [
 			\Psr\Log\NullLogger::class => [''],
 		],
-	],
 ];
 EOF,
 				'addonsArray' => [
-					\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 						\Psr\Log\LoggerInterface::class => [
 							\Psr\Log\NullLogger::class => ['null'],
 						],
 				],
-				],
 				'assertStrategies' => [
 					[LoggerInterface::class, NullLogger::class, ''],
 					[LoggerInterface::class, NullLogger::class, 'null'],
@@ -108,20 +100,16 @@ EOF,
 <?php
 
 return [
-	\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 		\Psr\Log\LoggerInterface::class => [
 			\Psr\Log\NullLogger::class => [''],
 		],
-	],
 ];
 EOF,
 				'addonsArray' => [
-					\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 						\Psr\Log\LoggerInterface::class => [
 							\Psr\Log\NullLogger::class => 'null',
 						],
 				],
-				],
 				'assertStrategies' => [
 					[LoggerInterface::class, NullLogger::class, ''],
 					[LoggerInterface::class, NullLogger::class, 'null'],
@@ -133,48 +121,21 @@ EOF,
 <?php
 
 return [
-	\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 		\Psr\Log\LoggerInterface::class => [
 			\Psr\Log\NullLogger::class => [''],
 		],
-	],
 ];
 EOF,
 				'addonsArray' => [
-					\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
 						\Psr\Log\LoggerInterface::class => [
 							\Psr\Log\NullLogger::class => [''],
 						],
 				],
-				],
 				'assertStrategies' => [
 					[LoggerInterface::class, NullLogger::class, ''],
 					[LoggerInterface::class, NullLogger::class, ''],
 				],
 			],
-			'withWrongContentButAddons' => [
-				'content' => <<<EOF
-<?php
-
-return [
-	'REALLY_WRONG' => [
-		\Psr\Log\LoggerInterface::class => [
-			\Psr\Log\NullLogger::class => [''],
-		],
-	],
-];
-EOF,
-				'addonsArray' => [
-					\Friendica\Core\Hooks\Capabilities\BehavioralHookType::STRATEGY => [
-						\Psr\Log\LoggerInterface::class => [
-							\Psr\Log\NullLogger::class => [''],
-						],
-					],
-				],
-				'assertStrategies' => [
-					[LoggerInterface::class, NullLogger::class, ''],
-				],
-			],
 		];
 	}
 
@@ -183,58 +144,59 @@ EOF,
 	 */
 	public function testSetupHooks(string $content, array $addonsArray, array $assertStrategies)
 	{
-		vfsStream::newFile('static/hooks.config.php')
+		vfsStream::newFile(StrategiesFileManager::STATIC_DIR . '/' . StrategiesFileManager::CONFIG_NAME . '.config.php')
 			->withContent($content)
 			->at($this->root);
 
 		$addonLoader = \Mockery::mock(ICanLoadAddons::class);
 		$addonLoader->shouldReceive('getActiveAddonConfig')->andReturn($addonsArray)->once();
 
-		$hookFileManager = new HookFileManager($this->root->url(), $addonLoader);
+		$hookFileManager = new StrategiesFileManager($this->root->url(), $addonLoader);
 
-		$instanceManager = \Mockery::mock(ICanRegisterInstances::class);
+		$instanceManager = \Mockery::mock(ICanRegisterStrategies::class);
 		foreach ($assertStrategies as $assertStrategy) {
 			$instanceManager->shouldReceive('registerStrategy')->withArgs($assertStrategy)->once();
 		}
 
-		$hookFileManager->setupHooks($instanceManager);
+		$hookFileManager->loadConfig();
+		$hookFileManager->setupStrategies($instanceManager);
 
 		self::expectNotToPerformAssertions();
 	}
 
 	/**
-	 * Test the exception in case the hooks.config.php file is missing
+	 * Test the exception in case the strategies.config.php file is missing
 	 */
-	public function testMissingHooksFile()
+	public function testMissingStrategiesFile()
 	{
 		$addonLoader     = \Mockery::mock(ICanLoadAddons::class);
-		$instanceManager = \Mockery::mock(ICanRegisterInstances::class);
+		$instanceManager = \Mockery::mock(ICanRegisterStrategies::class);
-		$hookFileManager = new HookFileManager($this->root->url(), $addonLoader);
+		$hookFileManager = new StrategiesFileManager($this->root->url(), $addonLoader);
 
 		self::expectException(HookConfigException::class);
 		self::expectExceptionMessage(sprintf('config file %s does not exist.',
-				$this->root->url() . '/' . HookFileManager::STATIC_DIR . '/' . HookFileManager::CONFIG_NAME . '.config.php'));
+				$this->root->url() . '/' . StrategiesFileManager::STATIC_DIR . '/' . StrategiesFileManager::CONFIG_NAME . '.config.php'));
 
-		$hookFileManager->setupHooks($instanceManager);
+		$hookFileManager->loadConfig();
 	}
 
 	/**
-	 * Test the exception in case the hooks.config.php file is wrong
+	 * Test the exception in case the strategies.config.php file is wrong
 	 */
-	public function testWrongHooksFile()
+	public function testWrongStrategiesFile()
 	{
 		$addonLoader     = \Mockery::mock(ICanLoadAddons::class);
-		$instanceManager = \Mockery::mock(ICanRegisterInstances::class);
+		$instanceManager = \Mockery::mock(ICanRegisterStrategies::class);
-		$hookFileManager = new HookFileManager($this->root->url(), $addonLoader);
+		$hookFileManager = new StrategiesFileManager($this->root->url(), $addonLoader);
 
-		vfsStream::newFile('static/hooks.config.php')
+		vfsStream::newFile(StrategiesFileManager::STATIC_DIR . '/' . StrategiesFileManager::CONFIG_NAME . '.config.php')
 				 ->withContent("<php return 'WRONG_CONTENT';")
 				 ->at($this->root);
 
 		self::expectException(HookConfigException::class);
 		self::expectExceptionMessage(sprintf('Error loading config file %s.',
-			$this->root->url() . '/' . HookFileManager::STATIC_DIR . '/' . HookFileManager::CONFIG_NAME . '.config.php'));
+			$this->root->url() . '/' . StrategiesFileManager::STATIC_DIR . '/' . StrategiesFileManager::CONFIG_NAME . '.config.php'));
 
-		$hookFileManager->setupHooks($instanceManager);
+		$hookFileManager->loadConfig();
 	}
 }