[ZendBundle] removed all non-class parameters

src/Symfony/Bundle/ZendBundle/DependencyInjection/Configuration.php

@@ -40,6 +40,7 @@ class Configuration
                     ->children()
                         ->scalarNode('priority')->defaultValue('INFO')->end()
                         ->scalarNode('path')->defaultValue('%kernel.logs_dir%/%kernel.environment%.log')->end()
+                        ->scalarNode('format')->defaultValue("%%timestamp%% %%priorityName%%: %%message%%\n")->end()
                         ->booleanNode('log_errors')->defaultFalse()->end()
                     ->end()
                 ->end()

src/Symfony/Bundle/ZendBundle/DependencyInjection/ZendExtension.php

@@ -49,18 +49,16 @@ class ZendExtension extends Extension
 
             $config = $config['logger'];
 
-            $container->setParameter('zend.logger.priority', is_int($config['priority']) ? $config['priority'] : constant('\\Zend\\Log\\Logger::'.strtoupper($config['priority'])));
-            $container->setParameter('zend.logger.path', $config['path']);
+            $container->getDefinition('zend.logger.filter')->setArgument(0, is_int($config['priority']) ? $config['priority'] : constant('\\Zend\\Log\\Logger::'.strtoupper($config['priority'])));
+            $container->getDefinition('zend.logger.writer.filesystem')->setArgument(0, $config['path']);
 
             $definition = $container->findDefinition('zend.logger');
             if ($config['log_errors']) {
                 $container->findDefinition('zend.logger')->addMethodCall('registerErrorHandler');
-            } else {
-                if ($definition->hasMethodCall('registerErrorHandler')) {
-                    $container->findDefinition('zend.logger')->removeMethodCall('registerErrorHandler');
-                }
             }
 
+            $container->getDefinition('zend.formatter.filesystem')->setArgument(0, $config['format']);
+
             $this->addClassesToCompile(array(
                 'Zend\\Log\\Factory',
                 'Zend\\Log\\Filter',

src/Symfony/Bundle/ZendBundle/Resources/config/logger.xml

@@ -6,12 +6,9 @@
 
     <parameters>
         <parameter key="zend.logger.class">Symfony\Bundle\ZendBundle\Logger\Logger</parameter>
-        <parameter key="zend.logger.log_errors">true</parameter>
         <parameter key="zend.logger.writer.debug.class">Symfony\Bundle\ZendBundle\Logger\DebugLogger</parameter>
         <parameter key="zend.logger.writer.filesystem.class">Zend\Log\Writer\Stream</parameter>
         <parameter key="zend.formatter.filesystem.class">Zend\Log\Formatter\Simple</parameter>
-        <parameter key="zend.formatter.filesystem.format">%%timestamp%% %%priorityName%%: %%message%%
-</parameter>
     </parameters>
 
     <services>
@@ -19,13 +16,13 @@
 
         <service id="zend.logger.writer.filesystem" class="%zend.logger.writer.filesystem.class%" public="false">
             <tag name="zend.logger.writer" />
-            <argument>%zend.logger.path%</argument>
+            <argument /> <!-- path -->
             <call method="addFilter"><argument type="service" id="zend.logger.filter" /></call>
             <call method="setFormatter"><argument type="service" id="zend.formatter.filesystem" /></call>
         </service>
 
         <service id="zend.formatter.filesystem" class="%zend.formatter.filesystem.class%" public="false">
-            <argument>%zend.formatter.filesystem.format%</argument>
+            <argument /> <!-- format -->
         </service>
 
         <service id="zend.logger.writer.debug" class="%zend.logger.writer.debug.class%" public="false">
@@ -34,7 +31,7 @@
         </service>
 
         <service id="zend.logger.filter" class="Zend\Log\Filter\Priority" public="false">
-            <argument>%zend.logger.priority%</argument>
+            <argument /> <!-- priority -->
         </service>
     </services>
 </container>

src/Symfony/Bundle/ZendBundle/Tests/DependencyInjection/ZendExtensionTest.php

@@ -1,32 +0,0 @@
-<?php
-
-/*
- * This file is part of the Symfony package.
- *
- * (c) Fabien Potencier <fabien@symfony.com>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Symfony\Bundle\ZendBundle\Tests\DependencyInjection;
-
-use Symfony\Bundle\ZendBundle\Tests\TestCase;
-use Symfony\Bundle\ZendBundle\DependencyInjection\ZendExtension;
-use Symfony\Component\DependencyInjection\ContainerBuilder;
-
-class ZendExtensionTest extends TestCase
-{
-    public function testLoad()
-    {
-        // logger
-        $container = new ContainerBuilder();
-        $loader = new ZendExtension();
-
-        $loader->load(array(array('logger' => array())), $container);
-        $this->assertEquals('Symfony\\Bundle\\ZendBundle\\Logger\\Logger', $container->getParameter('zend.logger.class'), '->loggerLoad() loads the logger.xml file if not already loaded');
-
-        $loader->load(array(array('logger' => array('priority' => 3))), $container);
-        $this->assertEquals(3, $container->getParameter('zend.logger.priority'), '->loggerLoad() overrides existing configuration options');
-    }
-}

src/Symfony/Bundle/ZendBundle/Tests/TestCase.php

@@ -1,22 +0,0 @@
-<?php
-
-/*
- * This file is part of the Symfony package.
- *
- * (c) Fabien Potencier <fabien@symfony.com>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Symfony\Bundle\ZendBundle\Tests;
-
-class TestCase extends \PHPUnit_Framework_TestCase
-{
-    protected function setUp()
-    {
-        if (!class_exists('Zend\\Log\\Logger')) {
-            $this->markTestSkipped('Zend Framework is not available.');
-        }
-    }
-}