Browse Source

Fix ordering. I got desc/asc backwards like I often do.

Mark Story 12 years ago
parent
commit
e36e5de821
2 changed files with 11 additions and 11 deletions
  1. 1 1
      src/View/Widget/DateTime.php
  2. 10 10
      tests/TestCase/View/Widget/DateTimeTest.php

+ 1 - 1
src/View/Widget/DateTime.php

@@ -277,7 +277,7 @@ class DateTime implements WidgetInterface {
 		if (empty($options['options'])) {
 			$options['options'] = $this->_generateNumbers($options['start'], $options['end']);
 		}
-		if ($options['order'] === 'asc') {
+		if ($options['order'] === 'desc') {
 			$options['options'] = array_reverse($options['options'], true);
 		}
 		unset($options['start'], $options['end'], $options['order']);

+ 10 - 10
tests/TestCase/View/Widget/DateTimeTest.php

@@ -221,7 +221,7 @@ class DateTimeTest extends TestCase {
 				'start' => 2013,
 				'end' => 2015,
 				'data-foo' => 'test',
-				'order' => 'desc',
+				'order' => 'asc',
 			],
 			'month' => false,
 			'day' => false,
@@ -245,7 +245,7 @@ class DateTimeTest extends TestCase {
 			'year' => [
 				'start' => 2013,
 				'end' => 2015,
-				'order' => 'asc'
+				'order' => 'desc'
 			],
 			'month' => false,
 			'day' => false,
@@ -287,12 +287,12 @@ class DateTimeTest extends TestCase {
 		]);
 		$expected = [
 			'select' => ['name' => 'date[year]'],
-			['option' => ['value' => '2010', 'selected' => 'selected']], '2010', '/option',
-			['option' => ['value' => '2011']], '2011', '/option',
-			['option' => ['value' => '2012']], '2012', '/option',
-			['option' => ['value' => '2013']], '2013', '/option',
-			['option' => ['value' => '2014']], '2014', '/option',
 			['option' => ['value' => '2015']], '2015', '/option',
+			['option' => ['value' => '2014']], '2014', '/option',
+			['option' => ['value' => '2013']], '2013', '/option',
+			['option' => ['value' => '2012']], '2012', '/option',
+			['option' => ['value' => '2011']], '2011', '/option',
+			['option' => ['value' => '2010', 'selected' => 'selected']], '2010', '/option',
 			'/select',
 		];
 		$this->assertTags($result, $expected);
@@ -313,10 +313,10 @@ class DateTimeTest extends TestCase {
 		]);
 		$expected = [
 			'select' => ['name' => 'date[year]'],
-			['option' => ['value' => '2010']], '2010', '/option',
-			['option' => ['value' => '2011']], '2011', '/option',
-			['option' => ['value' => '2012']], '2012', '/option',
 			['option' => ['value' => '2013', 'selected' => 'selected']], '2013', '/option',
+			['option' => ['value' => '2012']], '2012', '/option',
+			['option' => ['value' => '2011']], '2011', '/option',
+			['option' => ['value' => '2010']], '2010', '/option',
 			'/select',
 		];
 		$this->assertTags($result, $expected);