Browse Source

Fix published() to work with FrozenTime

mscherer 6 years ago
parent
commit
0d54da15b6
2 changed files with 5 additions and 7 deletions
  1. 3 6
      src/View/Helper/TimeHelper.php
  2. 2 1
      tests/TestCase/View/Helper/TimeHelperTest.php

+ 3 - 6
src/View/Helper/TimeHelper.php

@@ -143,20 +143,17 @@ class TimeHelper extends CakeTimeHelper {
 	 * Returns red/specialGreen/green date depending on the current day
 	 * // TODO refactor! $userOffset is deprecated!
 	 *
-	 * @param \DateTime $date Date
+	 * @param \DateTimeInterface $date Date
 	 * @param array $options
 	 * @param array $attr HTML attributes
 	 * @return string Nicely formatted date
 	 */
-	public function published(DateTime $date, array $options = [], array $attr = []) {
-		$niceDate = '';
+	public function published($date, array $options = [], array $attr = []) {
 		$when = null;
-		$span = '';
-		$spanEnd = '';
 		$whenArray = ['-1' => 'already', '0' => 'today', '1' => 'notyet'];
 		$titles = ['-1' => __d('tools', 'publishedAlready'), '0' => __d('tools', 'publishedToday'), '1' => __d('tools', 'publishedNotYet')];
 
-		$y = $this->isThisYear($date) ? '' : ' Y';
+		//$y = $this->isThisYear($date) ? '' : ' Y';
 
 		$format = (!empty($options['format']) ? $options['format'] : FORMAT_NICE_YMD);
 

+ 2 - 1
tests/TestCase/View/Helper/TimeHelperTest.php

@@ -2,6 +2,7 @@
 
 namespace Tools\Test\TestCase\View\Helper;
 
+use Cake\I18n\FrozenTime;
 use Cake\View\View;
 use Tools\TestSuite\TestCase;
 use Tools\Utility\Time;
@@ -78,7 +79,7 @@ class TimeHelperTest extends TestCase {
 		$expected = 'class="published notyet';
 		$this->assertContains($expected, $result);
 
-		$result = $this->Time->published((new Time(date(FORMAT_DB_DATETIME)))->subDays(2));
+		$result = $this->Time->published((new FrozenTime(date(FORMAT_DB_DATETIME)))->subDays(2));
 		$expected = 'class="published already';
 		$this->assertContains($expected, $result);
 	}