Issue 6332 in pharo: Zero in DateAndTime offset

Status: Accepted
Owner: [hidden email]
Labels: Type-FailingTest Milestone-2.0

New issue 6332 by [hidden email]: Zero in DateAndTime offset
http://code.google.com/p/pharo/issues/detail?id=6332

this tests are failing because of wrong object in DateAndTime offset:

CompressionTests.Archive.ZipArchiveTest.testCreateWithRelativeNames
CompressionTests.Archive.ZipArchiveTest.testFilePermissions
CompressionTests.Archive.ZipArchiveTest.testParseSqueakZipTimestamp
CompressionTests.Archive.ZipArchiveTest.testZip




_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

SLICE-Issue-6332-Zero-in-DateAndTime-offset-PavelKrivanek.1


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #2 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

Issue 6331 has been merged into this issue.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #3 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

I think the assumptions made in this slice are broken :/


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: WorkNeeded

Comment #4 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Cc: [hidden email]

Comment #5 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

I think we should integrate the slice and keep this issue opened. Because  
it is safer than to raise DNUs.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #6 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

ok I agree


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #7 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

We discussed it with Camillo and our result are this proposed changes:
- DateAndTime internally uses only UTC times
- offset changes only the timezone
- asSeconds returns seconds in UTC
- fromSeconds takes seconds in UTC and creates a DateAndTime in local time


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Issue 6332: Zero in DateAndTime offset
http://code.google.com/p/pharo/issues/detail?id=6332

This issue is now blocking issue pharo:6327.
See http://code.google.com/p/pharo/issues/detail?id=6327

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Issue 6332: Zero in DateAndTime offset
http://code.google.com/p/pharo/issues/detail?id=6332

This issue is now blocking issue pharo:5919.
See http://code.google.com/p/pharo/issues/detail?id=5919

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #10 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

see http://code.google.com/p/pharo/issues/detail?id=5919 for the specific  
DateAndTime fixes


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #11 on issue 6332 by [hidden email]: Zero in DateAndTime  
offset
http://code.google.com/p/pharo/issues/detail?id=6332

in 2.0 204


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker