Merge pull request #501 from mbabker/ip-provider

Use the new IP address provider interface

Author: stof

src/EventListener/IpTraceListener.php

@@ -12,11 +12,9 @@
             <call method="setAnnotationReader">
                 <argument type="service" id=".stof_doctrine_extensions.reader" on-invalid="ignore" />
             </call>
-        </service>
-
-        <service id="stof_doctrine_extensions.event_listener.ip_trace" class="Stof\DoctrineExtensionsBundle\EventListener\IpTraceListener" public="false">
-            <argument type="service" id="stof_doctrine_extensions.listener.ip_traceable" />
-            <tag name="kernel.event_subscriber" />
+            <call method="setIpAddressProvider">
+                <argument type="service" id="stof_doctrine_extensions.tool.ip_address_provider" />
+            </call>
         </service>
     </services>
 </container>

src/Resources/config/ip_traceable.xml

@@ -9,5 +9,9 @@
             <argument type="service" id="security.token_storage" on-invalid="null" />
             <argument type="service" id="security.authorization_checker" on-invalid="null" />
         </service>
+
+        <service id="stof_doctrine_extensions.tool.ip_address_provider" class="Stof\DoctrineExtensionsBundle\Tool\RequestStackIpAddressProvider" public="false">
+            <argument type="service" id="request_stack" on-invalid="null" />
+        </service>
     </services>
 </container>

src/Resources/config/tool.xml

@@ -0,0 +1,36 @@
+<?php
+
+namespace Stof\DoctrineExtensionsBundle\Tool;
+
+use Gedmo\Tool\IpAddressProviderInterface;
+use Symfony\Component\HttpFoundation\RequestStack;
+
+/**
+ * Provides an IP address for the extensions using an IP address reference.
+ *
+ * @internal
+ */
+final class RequestStackIpAddressProvider implements IpAddressProviderInterface
+{
+    private ?RequestStack $requestStack;
+
+    public function __construct(?RequestStack $requestStack)
+    {
+        $this->requestStack = $requestStack;
+    }
+
+    public function getAddress(): ?string
+    {
+        if (null === $this->requestStack) {
+            return null;
+        }
+
+        $request = $this->requestStack->getCurrentRequest();
+
+        if (null === $request) {
+            return null;
+        }
+
+        return $request->getClientIp();
+    }
+}