Skip to content

Commit 07d4ebb

Browse files
committed
TemplateFactory: filters |normalize, |toAscii, |reverse, |url, |null and |modifyDate are deprecated
1 parent 1a26321 commit 07d4ebb

File tree

2 files changed

+23
-13
lines changed

2 files changed

+23
-13
lines changed

src/Bridges/ApplicationLatte/TemplateFactory.php

+14-4
Original file line numberDiff line numberDiff line change
@@ -81,12 +81,22 @@ public function createTemplate(UI\Control $control = NULL)
8181
}
8282
});
8383

84-
$latte->addFilter('url', 'rawurlencode'); // back compatiblity
85-
foreach (['normalize', 'toAscii', 'webalize', 'reverse'] as $name) {
86-
$latte->addFilter($name, 'Nette\Utils\Strings::' . $name);
84+
$latte->addFilter('webalize', 'Nette\Utils\Strings::webalize');
85+
$latte->addFilter('url', function ($s) {
86+
trigger_error('Filter |url is deprecated, use |escapeUrl.', E_USER_DEPRECATED);
87+
return rawurlencode($s);
88+
});
89+
foreach (['normalize', 'toAscii', 'reverse'] as $name) {
90+
$latte->addFilter($name, function ($s) use ($name) {
91+
trigger_error("Filter |$name is deprecated.", E_USER_DEPRECATED);
92+
return [Nette\Utils\Strings::class, $name]($s);
93+
});
8794
}
88-
$latte->addFilter('null', function () {});
95+
$latte->addFilter('null', function () {
96+
trigger_error('Filter |null is deprecated.', E_USER_DEPRECATED);
97+
});
8998
$latte->addFilter('modifyDate', function ($time, $delta, $unit = NULL) {
99+
trigger_error('Filter |modifyDate is deprecated.', E_USER_DEPRECATED);
90100
return $time == NULL ? NULL : Nette\Utils\DateTime::from($time)->modify($delta . $unit); // intentionally ==
91101
});
92102

tests/Bridges.Latte/TemplateFactory.filters.phpt

+9-9
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,15 @@ $latte = $factory->createTemplate()->getLatte();
3535
setlocale(LC_TIME, 'C');
3636
date_default_timezone_set('Europe/Prague');
3737

38-
Assert::null($latte->invokeFilter('modifyDate', [NULL, NULL]));
39-
Assert::same('1978-01-24 11:40:00', (string) $latte->invokeFilter('modifyDate', [254400000, '+1 day']));
40-
Assert::same('1978-05-06 00:00:00', (string) $latte->invokeFilter('modifyDate', ['1978-05-05', '+1 day']));
41-
Assert::same('1978-05-06 00:00:00', (string) $latte->invokeFilter('modifyDate', [new DateTime('1978-05-05'), '1day']));
42-
Assert::same('1978-01-22 11:40:00', (string) $latte->invokeFilter('modifyDate', [254400000, -1, 'day']));
38+
Assert::null(@$latte->invokeFilter('modifyDate', [NULL, NULL])); // @ is deprecated
39+
Assert::same('1978-01-24 11:40:00', (string) @$latte->invokeFilter('modifyDate', [254400000, '+1 day']));
40+
Assert::same('1978-05-06 00:00:00', (string) @$latte->invokeFilter('modifyDate', ['1978-05-05', '+1 day']));
41+
Assert::same('1978-05-06 00:00:00', (string) @$latte->invokeFilter('modifyDate', [new DateTime('1978-05-05'), '1day']));
42+
Assert::same('1978-01-22 11:40:00', (string) @$latte->invokeFilter('modifyDate', [254400000, -1, 'day']));
4343

4444

45-
Assert::same('%25', $latte->invokeFilter('url', ['%']));
46-
Assert::null($latte->invokeFilter('null', ['x']));
47-
Assert::same('', $latte->invokeFilter('normalize', [' ']));
45+
Assert::same('%25', @$latte->invokeFilter('url', ['%'])); // @ is deprecated
46+
Assert::null(@$latte->invokeFilter('null', ['x'])); // @ is deprecated
47+
Assert::same('', @$latte->invokeFilter('normalize', [' '])); // @ is deprecated
4848
Assert::same('a-b', $latte->invokeFilter('webalize', ['a b']));
49-
Assert::same('cba', $latte->invokeFilter('reverse', ['abc']));
49+
Assert::same('cba', @$latte->invokeFilter('reverse', ['abc'])); // @ is deprecated

0 commit comments

Comments
 (0)