ID: 22418
Updated by: [EMAIL PROTECTED]
Reported By: pear at yogamala dot com
-Status: Assigned
+Status: Bogus
-Bug Type: PEAR related
+Bug Type: *General Issues
Operating System: Windows 2000
PHP Version: 4.3.1
Assigned To: pajoye
New Comment:
PEAR has it's own bug system, report this there..
Previous Comments:
------------------------------------------------------------------------
[2003-08-16 14:40:59] pear at yogamala dot com
> Your solution is not crossplatform.
Of course, it's windows specific. I don't understand your statement.
> on how many win32 OSes did you try your patch?
Windows 2000, Windows XP, others used it on Windows 98 with success.
> Do you really think that include a win32 only file is a good thing?
Yes. How else are you going to do it? I only include it if the system
is Windows.
------------------------------------------------------------------------
[2003-08-16 14:14:34] [EMAIL PROTECTED]
Hello,
Your solution is not crossplatform.
on how many win32 OSes did you try your patch?
Do you really think that include a win32 only file is a good thing?
pierre
------------------------------------------------------------------------
[2003-08-16 14:00:20] pear at yogamala dot com
> win32 platforms do not have any standart (sic) for TZ.
Yes it does. It's called the registry.
> This patch does not fix the problem.
It fixes the problem fine, and it is a "real" solution. And unless you
can come up with a better solution, why not include it? It's works far
better than TimeZone.php (see below) as Windows has standardized the
timezones in the registry. Vaguely critizing a solution without an
alternatively is not productive.
IMHO, the TimeZone class is flawed. For example, BST is not a unique
timezone identifier. In TimeZone.php, BST points to "Bangladesh Time",
where in practice, it more often refers to "British Summer Time".
Therefore, the whole class needs to be reworked. We spent several
months trying to use TimeZone.php in tikiwiki.sf.net but finally gave
up, as too many users reported problems. It's especially broken during
the summer as it doesn't handle summer identifiers very poorly.
I would strongly suggest you use date('Z') (timezone offset in seconds)
instead to determine the timezone. It is far more reliable than
date('T').
------------------------------------------------------------------------
[2003-08-16 12:19:59] [EMAIL PROTECTED]
No
See peardev archives for the status and "real" solutions.
In a few words:
win32 platforms do not have any standart for TZ. This patch does not
fix the problem.
The only good solution is to use the same TZ DB on all platforms (this
solution is used by near all date/time related system). That means to
use the public TZ DB.
pierre
------------------------------------------------------------------------
[2003-08-16 11:35:21] [EMAIL PROTECTED]
has this been fixed in cvs yet so it can be closed?
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/22418
--
Edit this bug report at http://bugs.php.net/?id=22418&edit=1
