Development

Changeset 32416

You must first sign up to be able to contribute.

Changeset 32416

Show
Ignore:
Timestamp:
03/30/11 19:15:05 (2 years ago)
Author:
fabien
Message:

Merge branch 'master' of git://github.com/symfony/symfony

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php

    r32345 r32416  
    144144                            return $v; 
    145145                        }) 
    146                     ->end()                     
     146                    ->end() 
    147147                    ->children() 
    148148                        ->booleanNode('auto_start')->end() 
    149149                        ->scalarNode('class')->end() 
    150                         ->scalarNode('default_locale')->end() 
     150                        ->scalarNode('default_locale')->defaultValue('en')->end() 
    151151                        ->scalarNode('storage_id')->defaultValue('native')->end() 
    152152                        // NativeSessionStorage options 
    …… 
    175175                    ->canBeUnset() 
    176176                    ->children() 
    177                         ->scalarNode('assets_version')->end() 
     177                        ->scalarNode('assets_version')->defaultValue(null)->end() 
    178178                    ->end() 
    179179                    ->fixXmlConfig('assets_base_url') 

  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php

    r32369 r32416  
    7575                $container->getDefinition('error_handler')->setMethodCalls(array()); 
    7676            } else { 
    77                 $container->getDefinition('error_handler')->addMethodCall('register', array()); 
    78                 $container->setParameter('error_handler.level', $config['error_handler']); 
     77                $container 
     78                    ->getDefinition('error_handler')->addMethodCall('register', array()) 
     79                    ->setParameter(0, $config['error_handler']) 
     80                ; 
    7981            } 
    8082        } 
    …… 
    294296        } 
    295297 
    296         if (isset($config['default_locale'])) { 
    297             $container->setParameter('session.default_locale', $config['default_locale']); 
    298         } 
     298        $container->getDefinition('session')->setArgument(1, $config['default_locale']); 
    299299 
    300300        $container->setAlias('session.storage', 'session.storage.'.$config['storage_id']); 
    …… 
    329329        if ($container->getParameter('kernel.debug')) { 
    330330            $loader->load('templating_debug.xml'); 
    331         } 
    332  
    333         if (isset($config['assets_version'])) { 
    334             $container->setParameter('templating.assets.version', $config['assets_version']); 
    335         } 
    336  
    337         if (isset($config['assets_base_urls'])) { 
    338             $container->setParameter('templating.assets.base_urls', $config['assets_base_urls']); 
    339331        } 
    340332 
    …… 
    346338            )); 
    347339        } 
    348         $container->getDefinition('templating.helper.assets')->setArgument(3, $packages); 
     340        $container 
     341            ->getDefinition('templating.helper.assets') 
     342            ->setArgument(1, isset($config['assets_base_urls']) ? $config['assets_base_urls'] : array()) 
     343            ->setArgument(2, $config['assets_version']) 
     344            ->setArgument(3, $packages) 
     345        ; 
    349346 
    350347        if (!empty($config['loaders'])) { 

  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/Resources/config/services.xml

    r32378 r32416  
    99        <parameter key="http_kernel.class">Symfony\Bundle\FrameworkBundle\HttpKernel</parameter> 
    1010        <parameter key="error_handler.class">Symfony\Component\HttpKernel\Debug\ErrorHandler</parameter> 
    11         <parameter key="error_handler.level">null</parameter> 
    1211        <parameter key="filesystem.class">Symfony\Component\HttpKernel\Util\Filesystem</parameter> 
    1312        <parameter key="cache_warmer.class">Symfony\Component\HttpKernel\CacheWarmer\CacheWarmerAggregate</parameter> 
    …… 
    2221 
    2322        <service id="error_handler" class="%error_handler.class%" public="false"> 
    24             <argument>%error_handler.level%</argument> 
     23            <argument /> <!-- level (null by default) --> 
    2524        </service> 
    2625 

  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/Resources/config/session.xml

    r32184 r32416  
    77    <parameters> 
    88        <parameter key="session.class">Symfony\Component\HttpFoundation\Session</parameter> 
    9         <parameter key="session.default_locale">en</parameter> 
    109        <parameter key="session.storage.native.class">Symfony\Component\HttpFoundation\SessionStorage\NativeSessionStorage</parameter> 
    1110        <parameter key="session.storage.native.options" type="collection" /> 
    …… 
    1918        <service id="session" class="%session.class%"> 
    2019            <argument type="service" id="session.storage" /> 
    21             <argument type="collection"> 
    22                 <argument key="default_locale">%session.default_locale%</argument> 
    23             </argument> 
     20            <argument /> <!-- default locale --> 
    2421        </service> 
    2522 

  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/Resources/config/templating.xml

    r32349 r32416  
    1414        <parameter key="templating.loader.cache.class">Symfony\Component\Templating\Loader\CacheLoader</parameter> 
    1515        <parameter key="templating.loader.chain.class">Symfony\Component\Templating\Loader\ChainLoader</parameter> 
    16         <parameter key="templating.assets.version">null</parameter> 
    17         <parameter key="templating.assets.base_urls" type="collection"></parameter> 
    1816    </parameters> 
    1917 

  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/Resources/config/templating_php.xml

    r32369 r32416  
    3636            <tag name="templating.helper" alias="assets" /> 
    3737            <argument type="service" id="request" strict="false" /> 
    38             <argument>%templating.assets.base_urls%</argument> 
    39             <argument>%templating.assets.version%</argument> 
    40             <argument type="collection" /> 
     38            <argument type="collection" /> <!-- assets base URLs --> 
     39            <argument /> <!-- assets version --> 
     40            <argument type="collection" /> <!-- packages --> 
    4141        </service> 
    4242 

  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTest.php

    r32398 r32416  
    7373 
    7474        $this->assertTrue($container->hasDefinition('session'), '->registerSessionConfiguration() loads session.xml'); 
    75         $this->assertEquals('fr', $container->getParameter('session.default_locale')); 
     75        $arguments = $container->getDefinition('session')->getArguments(); 
     76        $this->assertEquals('fr', $arguments[1]); 
    7677        $this->assertTrue($container->getDefinition('session')->hasMethodCall('start')); 
    7778        $this->assertEquals('Session', $container->getParameter('session.class')); 
    …… 
    104105 
    105106        $this->assertTrue($container->hasDefinition('templating.name_parser'), '->registerTemplatingConfiguration() loads templating.xml'); 
    106         $this->assertEquals('SomeVersionScheme', $container->getParameter('templating.assets.version')); 
    107         $this->assertEquals(array('http://cdn.example.com'), $container->getParameter('templating.assets.base_urls')); 
     107        $arguments = $container->getDefinition('templating.helper.assets')->getArguments(); 
     108        $this->assertEquals('SomeVersionScheme', $arguments[2]); 
     109        $this->assertEquals(array('http://cdn.example.com'), $arguments[1]); 
    108110 
    109111        $this->assertTrue($container->getDefinition('templating.cache_warmer.template_paths')->hasTag('kernel.cache_warmer'), '->registerTemplatingConfiguration() tags templating cache warmer if cache warming is set'); 

  • branches/2.0/src/Symfony/Component/HttpFoundation/Session.php

    r32233 r32416  
    2525    protected $oldFlashes; 
    2626    protected $started; 
    27     protected $options; 
     27    protected $defaultLocale; 
    2828 
    2929    /** 
    3030     * Constructor. 
    3131     * 
    32      * @param SessionStorageInterface $session A SessionStorageInterface instance 
    33      * @param array                   $options An array of options 
    34      */ 
    35     public function __construct(SessionStorageInterface $storage, array $options = array()) 
     32     * @param SessionStorageInterface $session       A SessionStorageInterface instance 
     33     * @param string                  $defaultLocale The default locale 
     34     */ 
     35    public function __construct(SessionStorageInterface $storage, $defaultLocale = 'en') 
    3636    { 
    3737        $this->storage = $storage; 
    38         $this->options = $options; 
     38        $this->defaultLocale = $defaultLocale; 
    3939        $this->attributes = array('_flash' => array(), '_locale' => $this->getDefaultLocale()); 
    4040        $this->started = false; 
    …… 
    289289    public function serialize() 
    290290    { 
    291         return serialize(array($this->storage, $this->options)); 
     291        return serialize(array($this->storage, $this->defaultLocale)); 
    292292    } 
    293293 
    294294    public function unserialize($serialized) 
    295295    { 
    296         list($this->storage, $this->options) = unserialize($serialized); 
     296        list($this->storage, $this->defaultLocale) = unserialize($serialized); 
    297297        $this->attributes = array(); 
    298298        $this->started = false; 
    299299    } 
    300300 
    301     protected function getDefaultLocale() 
    302     { 
    303         return isset($this->options['default_locale']) ? $this->options['default_locale'] : 'en'; 
     301    private function getDefaultLocale() 
     302    { 
     303        return $this->defaultLocale; 
    304304    } 
    305305} 

  • branches/2.0/tests/Symfony/Tests/Component/HttpFoundation/SessionTest.php

    r32233 r32416  
    124124    public function testSerialize() 
    125125    { 
    126         $options = array('foo' => 'bar'); 
    127         $this->session = new Session($this->storage, $options); 
     126        $defaultLocale = 'en'; 
     127        $this->session = new Session($this->storage, $defaultLocale); 
    128128 
    129         $compare = serialize(array($this->storage, $options)); 
     129        $compare = serialize(array($this->storage, $defaultLocale)); 
    130130 
    131131        $this->assertSame($compare, $this->session->serialize()); 
    …… 
    133133        $this->session->unserialize($compare); 
    134134 
    135         $_options = new \ReflectionProperty(get_class($this->session), 'options'); 
    136         $_options->setAccessible(true); 
     135        $_defaultLocale = new \ReflectionProperty(get_class($this->session), 'defaultLocale'); 
     136        $_defaultLocale->setAccessible(true); 
    137137 
    138138        $_storage = new \ReflectionProperty(get_class($this->session), 'storage'); 
    139139        $_storage->setAccessible(true); 
    140140 
    141         $this->assertEquals($_options->getValue($this->session), $options, 'options match'); 
     141        $this->assertEquals($_defaultLocale->getValue($this->session), $defaultLocale, 'options match'); 
    142142        $this->assertEquals($_storage->getValue($this->session), $this->storage, 'storage match'); 
    143143    } 
    …… 
    146146    { 
    147147        $this->storage = new ArraySessionStorage(); 
    148         $options = array('foo' => 'bar'); 
    149         $this->session = new Session($this->storage, $options); 
     148        $defaultLocale = 'fr'; 
     149        $this->session = new Session($this->storage, $defaultLocale); 
    150150        $this->session->set('foo', 'bar'); 
    151151 
    152152        $this->session->save(); 
    153         $compare = array('_symfony2' => array('_flash' => array() ,'_locale' => 'en', 'foo' => 'bar')); 
     153        $compare = array('_symfony2' => array('_flash' => array(), '_locale' => 'fr', 'foo' => 'bar')); 
    154154 
    155155        $r = new \ReflectionObject($this->storage); 
    …… 
    162162    public function testLocale() 
    163163    { 
    164         $this->assertSame('en', $this->session->getLocale(),'default locale is en'); 
     164        $this->assertSame('en', $this->session->getLocale(), 'default locale is en'); 
    165165 
    166166        $this->session->set('_locale','de'); 
    167167 
    168         $this->assertSame('de', $this->session->getLocale(),'locale is de'); 
     168        $this->assertSame('de', $this->session->getLocale(), 'locale is de'); 
    169169 
    170170        $this->session = $this->getSession(); 
    171171        $this->session->setLocale('fr'); 
    172         $this->assertSame('fr', $this->session->getLocale(),'locale is fr'); 
     172        $this->assertSame('fr', $this->session->getLocale(), 'locale is fr'); 
    173173    } 
    174174