dub-mirrors/poodinis
1
0
Fork 0
mirror of https://github.com/mbierlee/poodinis.git synced 2024-11-15 04:04:01 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix unnecessary re-registration of types when registerBeforeResolving is specified

Browse source
This commit is contained in:
Mike Bierlee 2022-10-27 01:42:09 +03:00
parent e1cb511178
commit f7578d79a3
2 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
CHANGES.md
View file

@ -1,5 +1,9 @@
Poodinis Changelog Poodinis Changelog
================== ==================
**Version NEXT**
* FIX unnecessary re-registration of types when registerBeforeResolving is specified.
**Version 8.1.2** **Version 8.1.2**
* FIX compilation error on importing template types that are not actually types. * FIX compilation error on importing template types that are not actually types.

9
source/poodinis/container.d
View file

@ -343,18 +343,19 @@ synchronized class DependencyContainer
writeln("DEBUG: Resolving type " ~ resolveType.toString() ~ " with qualifier " ~ qualifierType.toString()); writeln("DEBUG: Resolving type " ~ resolveType.toString() ~ " with qualifier " ~ qualifierType.toString());
} }
static if (__traits(compiles, new QualifierType())) auto candidates = resolveType in registrations;
if (!candidates)
{ {
if (hasOption(resolveOptions, persistentResolveOptions, if (hasOption(resolveOptions, persistentResolveOptions,
ResolveOption.registerBeforeResolving)) ResolveOption.registerBeforeResolving))
{
static if (__traits(compiles, new QualifierType()))
{ {
register!(RegistrationType, QualifierType)(); register!(RegistrationType, QualifierType)();
return resolve!(RegistrationType, QualifierType)(resolveOptions);
} }
} }
auto candidates = resolveType in registrations;
if (!candidates)
{
if (hasOption(resolveOptions, persistentResolveOptions, if (hasOption(resolveOptions, persistentResolveOptions,
ResolveOption.noResolveException)) ResolveOption.noResolveException))
{ {