Unable to specify a layout with relative path

Bug #885411 reported by Nicolas Hatier
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
AuthPuppy
New
Undecided
Unassigned

Bug Description

(possibly a symfony bug, but there's almost no hope getting that fixed there due to symfony2 being out)

I wanted to specify a different layout in frontend/config/view.yml, but store it in another directory. The layout parameter doesn't accept a relative path (../../plugins/...) due to a small bug in /lib/symfony/lib/view/sfView.class.php

in the function setDecoratorTemplate (line 413), the call to strpos should be compared to FALSE instead of just being treated as a boolean.

Please replace
    if (!strpos($template, '.'))
    {
      $template .= $this->getExtension();
    }
by
    if (strpos($template, '.') === FALSE)
    {
      $template .= $this->getExtension();
    }

This is the recommended PHP syntax for strpos - see the warning under Return Values at http://ca.php.net/manual/en/function.strpos.php

Revision history for this message
Robin Jones (robin-networkfusion) wrote :

Has this bug been recorded on the Symfony Trac?

Also I think we should update symfony to 1.4.16, as they have fixed quite a few bugs since we started... http://www.symfony-project.org/installation/1_4/changelog

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.