From: Jan Schneider Date: Thu, 3 Dec 2009 16:43:59 +0000 (+0100) Subject: E_STRICT fixes. X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=97d2e7e849a21f50578932050f7d390366d3005f;p=horde.git E_STRICT fixes. --- diff --git a/framework/Date/lib/Horde/Date.php b/framework/Date/lib/Horde/Date.php index 70cb97b7c..40f8eadfa 100644 --- a/framework/Date/lib/Horde/Date.php +++ b/framework/Date/lib/Horde/Date.php @@ -642,7 +642,7 @@ class Horde_Date */ public function compareDate($other) { - if (!is_a($other, 'Horde_Date')) { + if (!($other instanceof Horde_Date)) { $other = new Horde_Date($other); } @@ -705,7 +705,7 @@ class Horde_Date */ public function compareTime($other) { - if (!is_a($other, 'Horde_Date')) { + if (!($other instanceof Horde_Date)) { $other = new Horde_Date($other); } @@ -732,7 +732,7 @@ class Horde_Date */ public function compareDateTime($other) { - if (!is_a($other, 'Horde_Date')) { + if (!($other instanceof Horde_Date)) { $other = new Horde_Date($other); } diff --git a/framework/Date/lib/Horde/Date/Repeater.php b/framework/Date/lib/Horde/Date/Repeater.php index 5fdd9641b..7ee5b074c 100644 --- a/framework/Date/lib/Horde/Date/Repeater.php +++ b/framework/Date/lib/Horde/Date/Repeater.php @@ -27,7 +27,7 @@ abstract class Horde_Date_Repeater } } - public function this($pointer) + public function this($pointer = 'future') { if (is_null($this->now)) { throw new Horde_Date_Repeater_Exception('Start point must be set before calling this()'); diff --git a/framework/Date/test/Horde/Date/DateTest.php b/framework/Date/test/Horde/Date/DateTest.php index eb35f0c1c..626d6968c 100644 --- a/framework/Date/test/Horde/Date/DateTest.php +++ b/framework/Date/test/Horde/Date/DateTest.php @@ -8,6 +8,7 @@ require_once dirname(__FILE__) . '/../../../lib/Horde/Date.php'; require_once dirname(__FILE__) . '/../../../lib/Horde/Date/Utils.php'; require_once dirname(__FILE__) . '/../../../lib/Horde/Date/Span.php'; +date_default_timezone_set('Europe/Berlin'); /** * @category Horde @@ -18,9 +19,6 @@ class Horde_Date_DateTest extends PHPUnit_Framework_TestCase { public function testConstructor() { - $oldTimezone = date_default_timezone_get(); - date_default_timezone_set('Europe/Berlin'); - $date = new stdClass(); $date->year = 2001; $date->month = 2; @@ -36,8 +34,6 @@ class Horde_Date_DateTest extends PHPUnit_Framework_TestCase $this->assertEquals('2001-02-03 05:05:06', (string)new Horde_Date('20010203T040506Z')); $this->assertEquals('2001-02-03 04:05:06', (string)new Horde_Date('2001-02-03 04:05:06')); $this->assertEquals('2001-02-03 04:05:06', (string)new Horde_Date(981169506)); - - date_default_timezone_set($oldTimezone); } public function testDateCorrection() @@ -65,6 +61,7 @@ class Horde_Date_DateTest extends PHPUnit_Framework_TestCase public function testTimestamp() { + $oldTimezone = date_default_timezone_get(); date_default_timezone_set('America/New_York'); $date = new Horde_Date(array('mday' => 1, 'month' => 10, 'year' => 2004)); @@ -74,6 +71,8 @@ class Horde_Date_DateTest extends PHPUnit_Framework_TestCase $date = new Horde_Date(array('mday' => 1, 'month' => 5, 'year' => 1948)); $this->assertEquals('-683841600', $date->timestamp()); $this->assertEquals('-683841600', mktime(0, 0, 0, $date->month, $date->mday, $date->year)); + + date_default_timezone_set($oldTimezone); } public function testStrftime() @@ -121,6 +120,7 @@ class Horde_Date_DateTest extends PHPUnit_Framework_TestCase public function testSetTimezone() { + $oldTimezone = date_default_timezone_get(); date_default_timezone_set('America/New_York'); $date = new Horde_Date('20010203040506'); @@ -134,6 +134,8 @@ class Horde_Date_DateTest extends PHPUnit_Framework_TestCase $date->setTimezone('Europe/Berlin'); $this->assertEquals('2001-02-03 05:05:06', (string)$date); + + date_default_timezone_set($oldTimezone); } public function testDateMath() diff --git a/framework/Date/test/Horde/Date/RecurrenceTest.php b/framework/Date/test/Horde/Date/RecurrenceTest.php index 808735912..f3c82be79 100644 --- a/framework/Date/test/Horde/Date/RecurrenceTest.php +++ b/framework/Date/test/Horde/Date/RecurrenceTest.php @@ -11,6 +11,7 @@ require_once 'Horde/iCalendar.php'; require_once dirname(__FILE__) . '/../../../lib/Horde/Date.php'; require_once dirname(__FILE__) . '/../../../lib/Horde/Date/Recurrence.php'; require_once dirname(__FILE__) . '/../../../lib/Horde/Date/Utils.php'; +date_default_timezone_set('Europe/Berlin'); /** * @category Horde