search

opendena / jade.php

HAML-like template engine for PHP 5.3
MIT License
18 stars 0 forks source link

Added empty $html variable #1

Closed sameerkanda closed 11 years ago

sameerkanda commented 11 years ago

I'm using Jade with Laravel and it's going all crazy throwing an error! Plz fix.

SebastienElet commented 11 years ago

Thanks a lot, i miss this one. Could you send me an example of this case to add the test and avoid regression ?

sameerkanda commented 11 years ago

Basically the error popped up anytime I ran a PHP code in Jade, example:

!!! 5 html(ng-app="loadtest") head body

The "- foreach..." caused this error:

Undefined variable: $html in Dumper.php line 267: "$html .= '<?php ' . preg_replace('/^ +/', '', $node->code) . ' ?>';" ($html is not defined, so $html .= will screw up)

sameerkanda commented 11 years ago

tabs are not showing properly in my previous comment, but you get the idea

SebastienElet commented 11 years ago

Outch, i was focus on native jade test suite and i miss the main feature. Thanks a lot for your feedback.