Skip to content

Commit

Permalink
Remove singleton scope from certain classes
Browse files Browse the repository at this point in the history
The previous implementation made use of the singleton scope for the RowInterface, RowListInterface, and InvokeInterface. This change removes this scope, allowing new instances of these classes to be created each time they are needed. This may improve flexibility in certain use cases and can prevent potential issues related to state persistence across multiple uses.
  • Loading branch information
koriym committed Jun 11, 2024
1 parent 103efbc commit 13f078d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/SqlQueryProviderModule.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ protected function configure()
{
$this->bind(SqlFinder::class)->in(Scope::SINGLETON);
$this->bind(ParamReaderInterface::class)->to(ParamReader::class)->in(Scope::SINGLETON);
$this->bind(RowInterface::class)->toProvider(RowInterfaceProvider::class)->in(Scope::SINGLETON);
$this->bind(RowListInterface::class)->toProvider(RowListInterfaceProvider::class)->in(Scope::SINGLETON);
$this->bind(InvokeInterface::class)->toProvider(RowListInterfaceProvider::class)->in(Scope::SINGLETON);
$this->bind(RowInterface::class)->toProvider(RowInterfaceProvider::class);
$this->bind(RowListInterface::class)->toProvider(RowListInterfaceProvider::class);
$this->bind(InvokeInterface::class)->toProvider(RowListInterfaceProvider::class);
}
}

0 comments on commit 13f078d

Please sign in to comment.