|limit accepts strings

Author: dg

src/Latte/Essential/CoreExtension.php

@@ -152,7 +152,7 @@ public function getFilters(): array
 			'join' => $this->filters->implode(...),
 			'last' => $this->filters->last(...),
 			'length' => $this->filters->length(...),
-			'limit' => fn(iterable $value, int $length, int $offset = 0) => Filters::slice($value, $offset, $length, preserveKeys: true),
+			'limit' => fn(string|iterable $value, int $length) => Filters::slice($value, 0, $length, preserveKeys: true),
 			'localDate' => $this->filters->localDate(...),
 			'lower' => extension_loaded('mbstring')
 				? $this->filters->lower(...)

tests/filters/limit.phpt

@@ -0,0 +1,64 @@
+<?php declare(strict_types=1);
+
+/**
+ * Test: |limit filter
+ */
+
+use Tester\Assert;
+
+require __DIR__ . '/../bootstrap.php';
+
+
+function limit(string|iterable $value, int $length): string|array|\Generator
+{
+	return \Latte\Essential\Filters::slice($value, 0, $length, preserveKeys: true);
+}
+
+
+test('arrays', function () {
+	Assert::same([], limit([], 5));
+	Assert::same([0 => 'a', 1 => 'b'], limit(['a', 'b', 'c', 'd'], 2));
+	Assert::same([0 => 'a', 1 => 'b', 2 => 'c', 3 => 'd'], limit(['a', 'b', 'c', 'd'], 99));
+	Assert::same([], limit(['a', 'b', 'c'], 0));
+});
+
+
+test('arrays preserve keys', function () {
+	$arr = ['a', 'b', 10 => 'c', 'd'];
+	Assert::same([0 => 'a', 1 => 'b'], limit($arr, 2));
+	Assert::same([0 => 'a', 1 => 'b', 10 => 'c'], limit($arr, 3));
+});
+
+
+test('iterators', function () {
+	$gen = function () {
+		yield 'a' => 1;
+		yield 'b' => 2;
+		yield 'c' => 3;
+		yield 'd' => 4;
+	};
+
+	Assert::same(['a' => 1, 'b' => 2], iterator_to_array(limit($gen(), 2)));
+	Assert::same(['a' => 1, 'b' => 2, 'c' => 3, 'd' => 4], iterator_to_array(limit($gen(), 99)));
+	Assert::same([], iterator_to_array(limit($gen(), 0)));
+});
+
+
+test('strings (UTF-8)', function () {
+	Assert::same('Příl', limit('Příliš', 4));
+	Assert::same('Příliš', limit('Příliš', 99));
+	Assert::same('', limit('Příliš', 0));
+	Assert::same('', limit('', 5));
+	Assert::same('ř', limit('řeč', 1));
+});
+
+
+test('via Latte', function () {
+	$latte = new Latte\Engine;
+	$latte->setLoader(new Latte\Loaders\StringLoader);
+	$latte->setTempDirectory(sys_get_temp_dir());
+
+	Assert::same('ab', $latte->renderToString('{$s|limit:2}', ['s' => 'abcdef']));
+	Assert::same('Př', $latte->renderToString('{$s|limit:2}', ['s' => 'Příliš']));
+	Assert::same('a, b', $latte->renderToString('{$arr|limit:2|implode:", "}', ['arr' => ['a', 'b', 'c']]));
+});