From 074c596994786dba428c505e61bf55446e7d0b53 Mon Sep 17 00:00:00 2001
From: ornicar <thibault.duplessis@gmail.com>
Date: Tue, 7 Jun 2011 11:48:11 -0700
Subject: [PATCH] Apply @Stof comments

---
 DependencyInjection/FOQElasticaExtension.php | 1 -
 Persister/ObjectPersister.php                | 2 +-
 Resources/config/config.xml                  | 2 +-
 3 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/DependencyInjection/FOQElasticaExtension.php b/DependencyInjection/FOQElasticaExtension.php
index e6ed072..87a8050 100644
--- a/DependencyInjection/FOQElasticaExtension.php
+++ b/DependencyInjection/FOQElasticaExtension.php
@@ -224,7 +224,6 @@ class FOQElasticaExtension extends Extension
         $serviceDef->replaceArgument(1, new Reference($transformerId));
         $serviceDef->replaceArgument(2, $typeConfig['model']);
         $serviceDef->replaceArgument(3, new Reference('foq_elastica.type_inspector'));
-        $serviceDef->replaceArgument(4, new Reference('logger'));
         $container->setDefinition($serviceId, $serviceDef);
 
         return $serviceId;
diff --git a/Persister/ObjectPersister.php b/Persister/ObjectPersister.php
index 81b7f76..893c3da 100644
--- a/Persister/ObjectPersister.php
+++ b/Persister/ObjectPersister.php
@@ -141,7 +141,7 @@ class ObjectPersister implements ObjectPersisterInterface
 
         if ($this->logger) {
             $message = sprintf('Elastica object persister failure (%s: %s)', get_class($exception), $exception->getMessage());
-            $this->logger->addWarning($message);
+            $this->logger->warn($message);
         }
     }
 }
diff --git a/Resources/config/config.xml b/Resources/config/config.xml
index b42ec18..0aae9f5 100644
--- a/Resources/config/config.xml
+++ b/Resources/config/config.xml
@@ -37,7 +37,7 @@
             <argument /> <!-- model to elastica transformer -->
             <argument /> <!-- model -->
             <argument /> <!-- type inspector -->
-            <argument /> <!-- logger -->
+            <argument type="service" id="logger" on-invalid="null" />
             <argument>%kernel.debug%</argument>
         </service>