Merge pull request #3 from Padam87/bundle3

Exchange, currency pair support
Padam87 · Feb 18, 2021 · 375c8ef · 375c8ef
2 parents b6a07ea + 5d76fd1
commit 375c8ef
Show file tree

Hide file tree

Showing 8 changed files with 159 additions and 5 deletions.
diff --git a/DependencyInjection/Padam87MoneyExtension.php b/DependencyInjection/Padam87MoneyExtension.php
@@ -55,11 +55,21 @@ public function process(ContainerBuilder $container)
     {
         $config = $container->getParameter('padam87_money.config');
 
-        $container->getDefinition('doctrine.orm.default_metadata_driver')->addMethodCall(
+        $driver = $container->getDefinition('doctrine.orm.default_metadata_driver');
+
+        $driver->addMethodCall(
             'addDriver',
             [
                 $container->getDefinition('Padam87\MoneyBundle\Doctrine\Mapping\Driver\MoneyEmbeddedDriver'),
-                'Money'
+                'Money\Money'
+            ]
+        );
+
+        $driver->addMethodCall(
+            'addDriver',
+            [
+                $container->getDefinition('Padam87\MoneyBundle\Doctrine\Mapping\Driver\CurrencyPairEmbeddedDriver'),
+                'Money\CurrencyPair'
             ]
         );
 

diff --git a/Doctrine/Mapping/Driver/CurrencyPairEmbeddedDriver.php b/Doctrine/Mapping/Driver/CurrencyPairEmbeddedDriver.php
@@ -0,0 +1,59 @@
+<?php
+
+namespace Padam87\MoneyBundle\Doctrine\Mapping\Driver;
+
+use Doctrine\Persistence\Mapping\ClassMetadata;
+use Doctrine\Persistence\Mapping\Driver\MappingDriver;
+use Money\CurrencyPair;
+
+class CurrencyPairEmbeddedDriver implements MappingDriver
+{
+    /**
+     * {@inheritdoc}
+     */
+    public function loadMetadataForClass($className, ClassMetadata $metadata)
+    {
+        /* @var \Doctrine\ORM\Mapping\ClassMetadataInfo $metadata */
+
+        $metadata->isEmbeddedClass = true;
+
+        $metadata->mapField(
+            [
+                'fieldName' => 'baseCurrency',
+                'type' => 'currency',
+            ]
+        );
+
+        $metadata->mapField(
+            [
+                'fieldName' => 'counterCurrency',
+                'type' => 'currency',
+            ]
+        );
+
+        $metadata->mapField(
+            [
+                'fieldName' => 'conversionRatio',
+                'type' => 'float',
+            ]
+        );
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getAllClassNames()
+    {
+        return [
+            CurrencyPair::class
+        ];
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function isTransient($className)
+    {
+        return false;
+    }
+}
diff --git a/Doctrine/Mapping/Driver/MoneyEmbeddedDriver.php b/Doctrine/Mapping/Driver/MoneyEmbeddedDriver.php
@@ -2,8 +2,8 @@
 
 namespace Padam87\MoneyBundle\Doctrine\Mapping\Driver;
 
-use Doctrine\Common\Persistence\Mapping\ClassMetadata;
-use Doctrine\Common\Persistence\Mapping\Driver\MappingDriver;
+use Doctrine\Persistence\Mapping\ClassMetadata;
+use Doctrine\Persistence\Mapping\Driver\MappingDriver;
 use Money\Money;
 
 class MoneyEmbeddedDriver implements MappingDriver

diff --git a/Entity/ExchangeRateInterface.php b/Entity/ExchangeRateInterface.php
@@ -0,0 +1,10 @@
+<?php
+
+namespace Padam87\MoneyBundle\Entity;
+
+use Money\CurrencyPair;
+
+interface ExchangeRateInterface
+{
+    public function getCurrencyPair(): CurrencyPair;
+}
diff --git a/Exchange/DatabaseExchange.php b/Exchange/DatabaseExchange.php
@@ -0,0 +1,48 @@
+<?php
+
+namespace Padam87\MoneyBundle\Exchange;
+
+use Decimal\Decimal;
+use Doctrine\Persistence\ManagerRegistry;
+use Money\Currency;
+use Money\CurrencyPair;
+use App\Entity\ExchangeRate;
+use Money\Exception\UnresolvableCurrencyPairException;
+use Padam87\MoneyBundle\Entity\ExchangeRateInterface;
+use Padam87\MoneyBundle\Repository\ExchangeRateRepositoryInterface;
+
+class DatabaseExchange implements \Money\Exchange
+{
+    private $doctrine;
+
+    public function __construct(ManagerRegistry $doctrine)
+    {
+        $this->doctrine = $doctrine;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function quote(Currency $baseCurrency, Currency $counterCurrency)
+    {
+        if ($baseCurrency->getCode() === $counterCurrency->getCode()) {
+            return new CurrencyPair($baseCurrency, $counterCurrency, 1);
+        }
+
+        $repo = $this->doctrine->getRepository(ExchangeRateInterface::class);
+
+        if (!$repo instanceof ExchangeRateRepositoryInterface) {
+            throw new \LogicException(
+                sprintf('"%s" must implement %s', $repo->getClassName(), ExchangeRateRepositoryInterface::class)
+            );
+        }
+
+        if (null === $exchangeRate = $repo->getExchangeRate($baseCurrency, $counterCurrency)) {
+            throw new UnresolvableCurrencyPairException(
+                sprintf('%s - >%s ratio not found in the database.', $baseCurrency->getCode(), $counterCurrency->getCode())
+            );
+        }
+
+        return $exchangeRate->getCurrencyPair();
+    }
+}
diff --git a/Repository/ExchangeRateRepositoryInterface.php b/Repository/ExchangeRateRepositoryInterface.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Padam87\MoneyBundle\Repository;
+
+use Money\Currency;
+use Padam87\MoneyBundle\Entity\ExchangeRateInterface;
+
+interface ExchangeRateRepositoryInterface
+{
+    public function getExchangeRate(Currency $baseCurrency, Currency $counterCurrency): ?ExchangeRateInterface;
+}
diff --git a/Resources/config/services.yaml b/Resources/config/services.yaml
@@ -8,6 +8,8 @@ services:
         arguments:
             $config: '%padam87_money.config%'
 
+    Padam87\MoneyBundle\Doctrine\Mapping\Driver\CurrencyPairEmbeddedDriver: ~
+
     Padam87\MoneyBundle\Money\Formatter\NumberFormatterFactory: ~
 
     padam87_money.number_formatter:
@@ -32,3 +34,17 @@ services:
     Padam87\MoneyBundle\Form\CurrencyType:
         arguments:
             $config: '%padam87_money.config%'
+
+    Padam87\MoneyBundle\Exchange\DatabaseExchange:
+        arguments:
+            - '@Doctrine\Persistence\ManagerRegistry'
+
+    Money\Exchange\ReversedCurrenciesExchange:
+        arguments:
+            - '@Padam87\MoneyBundle\Exchange\DatabaseExchange'
+
+    Money\Converter:
+        class: Money\Converter
+        arguments:
+            - '@Money\Currencies\CurrencyList'
+            - '@Money\Exchange\ReversedCurrenciesExchange'
diff --git a/composer.json b/composer.json
@@ -3,7 +3,7 @@
     "description": "Symfony bundle for https://github.com/moneyphp/money",
     "type": "symfony-bundle",
     "require": {
-        "php": "^7.0",
+        "php": "^7.4 || ^8.0",
         "ext-bcmath": "*",
         "symfony/framework-bundle": "^4.0 || ^5.0",
         "moneyphp/money": "^3.1",