From 68f216a137743fb645b7e031c7928ace93c3188c Mon Sep 17 00:00:00 2001 From: Dmitriy Makarov Date: Mon, 19 Sep 2016 22:50:23 +0300 Subject: [PATCH 1/2] :book: Wrap "null" [skip ci] --- docs/guide-ru/helper-html.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/guide-ru/helper-html.md b/docs/guide-ru/helper-html.md index caf1b5c..e80afc9 100644 --- a/docs/guide-ru/helper-html.md +++ b/docs/guide-ru/helper-html.md @@ -43,7 +43,7 @@ Html-помощник Дополнительные атрибуты используются во многих методах Html-помощника и в различных виджетах. Во всех этих случаях в дело вступают механизмы дополнительной обработки данных, о которых следует знать: -- Если значение равно null, соответствующий атрибут не будет выведен. +- Если значение равно `null`, соответствующий атрибут не будет выведен. - Атрибуты, значения которых имеют тип boolean, будут интерпретированы как [логические атрибуты](http://www.w3.org/TR/html5/infrastructure.html#boolean-attributes). - Значения атрибутов будут экранированы с использованием метода [[yii\helpers\Html::encode()|Html::encode()]]. @@ -441,4 +441,4 @@ echo Html::getAttributeName('dates[0]'); }]) ?> ``` -Для формирования упорядоченных списков используйте `Html::ol()`. \ No newline at end of file +Для формирования упорядоченных списков используйте `Html::ol()`. From e327d018b436b823d2e430f83a80e1ab1b9e92e3 Mon Sep 17 00:00:00 2001 From: Carsten Brandt Date: Tue, 20 Sep 2016 09:48:44 +0200 Subject: [PATCH 2/2] added test for pagination params verify #12569 --- tests/framework/data/PaginationTest.php | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/tests/framework/data/PaginationTest.php b/tests/framework/data/PaginationTest.php index 5254e74..47412ff 100644 --- a/tests/framework/data/PaginationTest.php +++ b/tests/framework/data/PaginationTest.php @@ -5,6 +5,9 @@ namespace yiiunit\framework\data; use yii\data\Pagination; use yiiunit\TestCase; +/** + * @group data + */ class PaginationTest extends TestCase { protected function setUp() @@ -30,11 +33,25 @@ class PaginationTest extends TestCase 2, null, '/index.php?r=item%2Flist&page=3', + null, ], [ 2, 5, '/index.php?r=item%2Flist&page=3&per-page=5', + null, + ], + [ + 2, + null, + '/index.php?r=item%2Flist&q=test&page=3', + ['q' => 'test'], + ], + [ + 2, + 5, + '/index.php?r=item%2Flist&q=test&page=3&per-page=5', + ['q' => 'test'], ], ]; } @@ -46,10 +63,11 @@ class PaginationTest extends TestCase * @param integer $pageSize * @param string $expectedUrl */ - public function testCreateUrl($page, $pageSize, $expectedUrl) + public function testCreateUrl($page, $pageSize, $expectedUrl, $params) { $pagination = new Pagination(); $pagination->route = 'item/list'; + $pagination->params = $params; $this->assertEquals($expectedUrl, $pagination->createUrl($page, $pageSize)); }