clones/friendica
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Make InstanceManager strategy naming convention case insensitive.

This commit is contained in:
Philipp 2023-07-29 13:20:02 +02:00
parent 06df62a2b5
commit 6bcf5cb823
No known key found for this signature in database
GPG Key ID: 24A7501396EB5432
2 changed files with 23 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/Core/Hooks/Model/DiceInstanceManager.php
View File

@ -49,11 +49,11 @@ class DiceInstanceManager implements ICanCreateInstances, ICanRegisterStrategies
/** {@inheritDoc} */ /** {@inheritDoc} */
public function registerStrategy(string $interface, string $class, ?string $name = null): ICanRegisterStrategies public function registerStrategy(string $interface, string $class, ?string $name = null): ICanRegisterStrategies
{ {
if (!empty($this->instance[$interface][$name])) { if (!empty($this->instance[$interface][strtolower($name)])) {
throw new HookRegisterArgumentException(sprintf('A class with the name %s is already set for the interface %s', $name, $interface)); throw new HookRegisterArgumentException(sprintf('A class with the name %s is already set for the interface %s', $name, $interface));
} }
$this->instance[$interface][$name] = $class; $this->instance[$interface][strtolower($name)] = $class;
return $this; return $this;
} }
@ -61,10 +61,10 @@ class DiceInstanceManager implements ICanCreateInstances, ICanRegisterStrategies
/** {@inheritDoc} */ /** {@inheritDoc} */
public function create(string $class, string $strategy, array $arguments = []): object public function create(string $class, string $strategy, array $arguments = []): object
{ {
if (empty($this->instance[$class][$strategy])) { if (empty($this->instance[$class][strtolower($strategy)])) {
throw new HookInstanceException(sprintf('The class with the name %s isn\'t registered for the class or interface %s', $strategy, $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][$strategy], $arguments); return $this->dice->create($this->instance[$class][strtolower($strategy)], $arguments);
} }
} }

19
tests/src/Core/Hooks/Model/InstanceManagerTest.php
View File

@ -235,4 +235,23 @@ class InstanceManagerTest extends MockedTest
self::assertEquals($cBool, $getInstanceA->getCBool()); self::assertEquals($cBool, $getInstanceA->getCBool());
self::assertEquals($cBool, $getInstanceB->getCBool()); self::assertEquals($cBool, $getInstanceB->getCBool());
} }
/**
* @see https://github.com/friendica/friendica/issues/13318
*/
public function testCaseInsensitiveNames()
{
$instance = new DiceInstanceManager(new Dice(), $this->hookFileManager);
$instance->registerStrategy(IAmADecoratedInterface::class, FakeInstance::class, 'fake');
// CamelCase
self::assertInstanceOf(FakeInstance::class, $instance->create(IAmADecoratedInterface::class, 'Fake'));
// UPPER CASE
self::assertInstanceOf(FakeInstance::class, $instance->create(IAmADecoratedInterface::class, 'FAKE'));
// lower case
self::assertInstanceOf(FakeInstance::class, $instance->create(IAmADecoratedInterface::class, 'fake'));
// UnKnOwN
self::assertInstanceOf(FakeInstance::class, $instance->create(IAmADecoratedInterface::class, 'fAkE'));
}
} }