Development

Changeset 32459

You must first sign up to be able to contribute.

Changeset 32459

Show
Ignore:
Timestamp:
04/08/11 17:30:04 (4 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/Templating/Loader/TemplateLocator.php

    r32457 r32459  
    6161            return $this->cache[$key] = $this->locator->locate($template->getPath(), $this->path); 
    6262        } catch (\InvalidArgumentException $e) { 
    63             throw new \InvalidArgumentException(sprintf('Unable to find template "%s" in "%s".', $template->getPath(), $this->path), 0, $e); 
     63            throw new \InvalidArgumentException(sprintf('Unable to find template "%s" in "%s".', $template->getLogicalName(), $this->path), 0, $e); 
    6464        } 
    6565    } 
  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/Templating/TemplateReference.php

    r32184 r32459  
    3636     *  - as a path when the template is not part of a bundle 
    3737     *  - as a resource when the template is part of a bundle 
    38      *  
     38     * 
    3939     * @return string A path to the template or a resource 
    4040     */ 
     
    4747    } 
    4848 
     49    /** 
     50     * {@inheritdoc} 
     51     */ 
     52    public function getLogicalName() 
     53    { 
     54        $parts = sprintf('%s:%s:', $this->get('bundle'), $this->get('controller')); 
     55        $elements = sprintf('%s.%s.%s', $this->get('name'), $this->get('format'), $this->get('engine')); 
     56 
     57        return $parts . $elements; 
     58    } 
     59 
    4960} 
  • branches/2.0/src/Symfony/Bundle/FrameworkBundle/Tests/Templating/TemplateNameParserTest.php

    r32439 r32459  
    4242 
    4343        $this->assertEquals($template->getSignature(), $ref->getSignature()); 
     44        $this->assertEquals($template->getLogicalName(), $ref->getLogicalName()); 
     45        $this->assertEquals($template->getLogicalName(), $name); 
    4446    } 
    4547 
     
    8385    { 
    8486        $template = $this->parser->parseFromFilename($file); 
    85          
     87 
    8688        if ($ref === false) { 
    8789            $this->assertFalse($template); 
  • branches/2.0/src/Symfony/Bundle/TwigBundle/Loader/FilesystemLoader.php

    r32445 r32459  
    9595 
    9696        if (false === $file || null === $file) { 
    97             throw new \Twig_Error_Loader(sprintf('Unable to find the template %s', json_encode($name)), -1, null, $previous); 
     97            throw new \Twig_Error_Loader(sprintf('Unable to find template "%s".', $tpl->getLogicalName()), -1, null, $previous); 
    9898        } 
    9999 
  • branches/2.0/src/Symfony/Component/Templating/DelegatingEngine.php

    r32184 r32459  
    107107        } 
    108108 
    109         throw new \RuntimeException(sprintf('No engine is able to work with the "%s" template.', $name)); 
     109        throw new \RuntimeException(sprintf('No engine is able to work with the %s template.', json_encode($name))); 
    110110    } 
    111111} 
  • branches/2.0/src/Symfony/Component/Templating/PhpEngine.php

    r32184 r32459  
    8383        // render 
    8484        if (false === $content = $this->evaluate($storage, $parameters)) { 
    85             throw new \RuntimeException(sprintf('The template "%s" cannot be rendered.', json_encode($name))); 
     85            throw new \RuntimeException(sprintf('The template "%s" cannot be rendered.', $this->parser->parse($name)->getLogicalName())); 
    8686        } 
    8787 
  • branches/2.0/src/Symfony/Component/Templating/TemplateReference.php

    r32184 r32459  
    3131    public function __toString() 
    3232    { 
    33         return json_encode($this->parameters); 
     33        return $this->getLogicalName(); 
    3434    } 
    3535 
    3636    /** 
    37      * {@inheritDoc} 
     37     * Returns the template signature 
     38     * 
     39     * @return string A UID for the template 
    3840     */ 
    3941    public function getSignature() 
     
    4345 
    4446    /** 
    45      * {@inheritDoc} 
     47     * Sets a template parameter. 
     48     * 
     49     * @param string $name   The parameter name 
     50     * @param string $value  The parameter value 
     51     * 
     52     * @return TemplateReferenceInterface The TemplateReferenceInterface instance 
     53     * 
     54     * @throws  \InvalidArgumentException if the parameter is not defined 
    4655     */ 
    4756    public function set($name, $value) 
     
    5766 
    5867    /** 
    59      * {@inheritDoc} 
     68     * Gets a template parameter. 
     69     * 
     70     * @param string $name The parameter name 
     71     * 
     72     * @return string The parameter value 
     73     * 
     74     * @throws  \InvalidArgumentException if the parameter is not defined 
    6075     */ 
    6176    public function get($name) 
     
    6984 
    7085    /** 
    71      * {@inheritDoc} 
     86     * Gets the template parameters. 
     87     * 
     88     * @return array An array of parameters 
    7289     */ 
    7390    public function all() 
     
    7794 
    7895    /** 
    79      * {@inheritDoc} 
     96     * Returns the path to the template. 
     97     * 
     98     * By default, it just returns the template name. 
     99     * 
     100     * @return string A path to the template or a resource 
    80101     */ 
    81102    public function getPath() 
     
    83104        return $this->parameters['name']; 
    84105    } 
     106 
     107    /** 
     108     * Returns the template name 
     109     * 
     110     * @return string The template name 
     111     */ 
     112    public function getLogicalName() 
     113    { 
     114        return $this->parameters['name']; 
     115    } 
    85116} 
  • branches/2.0/src/Symfony/Component/Templating/TemplateReferenceInterface.php

    r32184 r32459  
    6464     */ 
    6565    function getPath(); 
     66 
     67    /** 
     68     * Returns the template name 
     69     * 
     70     * @return string The template name 
     71     */ 
     72    function getLogicalName(); 
    6673}