mirror/fenom
1
0
Fork 0
mirror of https://github.com/fenom-template/fenom.git synced 2023-08-10 21:13:07 +03:00
Code Issues Projects Releases Wiki Activity

proper fix for array params (now works with both positional and named params), tests

Browse Source
This commit is contained in:
klkvsk 2013-07-11 16:13:11 +04:00
parent eb1fa83174
commit a2b5b43cb1
2 changed files with 14 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/Fenom/Template.php
View File

@ -644,7 +644,11 @@ class Template extends Render {
} else {
break;
}
} else {
} elseif($tokens->is('[')) {
$_exp .= $this->parseArray($tokens);
} else {
break;
}
}
@ -1126,11 +1130,7 @@ class Template extends Render {
}
if($tokens->is("=")) {
$tokens->next();
if ($tokens->is('[')) {
$params[ $key ] = $this->parseArray($tokens);
} else {
$params[ $key ] = $this->parseExp($tokens);
}
$params[ $key ] = $this->parseExp($tokens);
} else {
$params[ $key ] = 'true';
}

10
tests/cases/Fenom/FunctionsTest.php
View File

@ -29,7 +29,8 @@ class FunctionsTest extends TestCase {
$this->tpl('function_default_param_empty_array.tpl', '{sum}');
$this->tpl('function_default_param_const.tpl', '{inc a=1}');
$this->tpl('function_array_param.tpl', '{sum of=[1, 2, 3, 4, 5]}');
}
$this->tpl('function_array_param_pos.tpl', '{sum [1, 2, 3, 4, 5]}');
}
public function testFunctionWithParams() {
$output = $this->fenom->fetch('function_params_scalar.tpl');
@ -56,9 +57,14 @@ class FunctionsTest extends TestCase {
$this->assertEquals('2', $output);
}
public function testFunctionWithArrayParam() {
public function testFunctionWithArrayNamedParam() {
$output = $this->fenom->fetch('function_array_param.tpl');
$this->assertEquals('15', $output);
}
public function testFunctionWithArrayPositionalParam() {
$output = $this->fenom->fetch('function_array_param_pos.tpl');
$this->assertEquals('15', $output);
}
}