[perl-DateTime/f19] Restore object after failed truncate() call

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



commit 73fd00e0c9974a26b2ec4f924bcbebd6107ad609
Author: Petr Písař <ppisar@xxxxxxxxxx>
Date:   Wed May 7 09:31:14 2014 +0200

    Restore object after failed truncate() call

 ...-the-object-in-a-modified-state-after-a-f.patch |   96 ++++++++++++++++++++
 perl-DateTime.spec                                 |    9 ++-
 2 files changed, 104 insertions(+), 1 deletions(-)
---
diff --git a/DateTime-1.06-Don-t-leave-the-object-in-a-modified-state-after-a-f.patch b/DateTime-1.06-Don-t-leave-the-object-in-a-modified-state-after-a-f.patch
new file mode 100644
index 0000000..cd847db
--- /dev/null
+++ b/DateTime-1.06-Don-t-leave-the-object-in-a-modified-state-after-a-f.patch
@@ -0,0 +1,96 @@
+From d744bf17b7a0e0158eb813a8605cc0d8635f8959 Mon Sep 17 00:00:00 2001
+From: Dave Rolsky <autarch@xxxxxxxx>
+Date: Sat, 3 May 2014 11:39:47 +0800
+Subject: [PATCH] Don't leave the object in a modified state after a failed
+ truncate( to => 'week' )
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Petr Pisar: Ported to 1.06.
+
+Signed-off-by: Petr Písař <ppisar@xxxxxxxxxx>
+---
+ lib/DateTime.pm | 11 ++++++++++-
+ t/16truncate.t  | 38 ++++++++++++++++++++++++++++++++++++++
+ 2 files changed, 48 insertions(+), 1 deletion(-)
+
+diff --git a/lib/DateTime.pm b/lib/DateTime.pm
+index 1ff4c2e..a8663b2 100644
+--- a/lib/DateTime.pm
++++ b/lib/DateTime.pm
+@@ -1985,7 +1985,16 @@ sub set_formatter {
+                 $self->add( days => -1 * $day_diff );
+             }
+ 
+-            return $self->truncate( to => 'day' );
++            # This can fail if the truncate ends up giving us an invalid local
++            # date time. If that happens we need to reverse the addition we
++            # just did. See https://rt.cpan.org/Ticket/Display.html?id=93347.
++            try {
++                $self->truncate( to => 'day' );
++            }
++            catch {
++                $self->add( days => $day_diff );
++                die $_;
++            };
+         }
+         else {
+             my $truncate;
+diff --git a/t/16truncate.t b/t/16truncate.t
+index 0058f50..a478760 100644
+--- a/t/16truncate.t
++++ b/t/16truncate.t
+@@ -5,6 +5,7 @@ use Test::Fatal;
+ use Test::More 0.88;
+ 
+ use DateTime;
++use Try::Tiny;
+ 
+ my %vals = (
+     year       => 50,
+@@ -233,4 +234,41 @@ my %vals = (
+     }
+ }
+ 
++{
++    my $dt = DateTime->new(
++        year      => 2010,
++        month     => 3,
++        day       => 25,
++        hour      => 1,
++        minute    => 5,
++        time_zone => 'Asia/Tehran',
++    );
++
++    is(
++        $dt->day_of_week(),
++        4,
++        'day of week is Thursday'
++    );
++
++    my $error;
++    try {
++        $dt->truncate( to => 'week' );
++    }
++    catch {
++        $error = $_;
++    };
++
++    like(
++        $error,
++        qr/Invalid local time for date/,
++        'truncate operation threw an error because of an invalid local datetime'
++    );
++
++    is(
++        $dt->day_of_week(),
++        4,
++        'day of week does not change after failed truncate() call'
++    );
++}
++
+ done_testing();
+-- 
+1.9.0
+
diff --git a/perl-DateTime.spec b/perl-DateTime.spec
index 68f65f7..60e6806 100644
--- a/perl-DateTime.spec
+++ b/perl-DateTime.spec
@@ -1,12 +1,15 @@
 Name:           perl-DateTime
 Epoch:          2
 Version:        1.06
-Release:        1%{?dist}
+Release:        2%{?dist}
 Summary:        Date and time object
 License:        Artistic 2.0
 Group:          Development/Libraries
 URL:            http://search.cpan.org/dist/DateTime/
 Source0:        http://www.cpan.org/authors/id/D/DR/DROLSKY/DateTime-%{version}.tar.gz
+# Restore object after failed truncate() call, bug #1095104, CPAN RT#93347,
+# in 1.09
+Patch0:         DateTime-1.06-Don-t-leave-the-object-in-a-modified-state-after-a-f.patch
 BuildRequires:  perl
 BuildRequires:  perl(Module::Build)
 BuildRequires:  perl(strict)
@@ -55,6 +58,7 @@ believed to be the birth of Jesus Christ.
 
 %prep
 %setup -q -n DateTime-%{version}
+%patch0 -p1
 
 %build
 %{__perl} Build.PL installdirs=vendor optimize="%{optflags}"
@@ -75,6 +79,9 @@ find %{buildroot} -type f -name '*.bs' -size 0 -exec rm -f {} \;
 %{_mandir}/man3/*
 
 %changelog
+* Wed May 07 2014 Petr Pisar <ppisar@xxxxxxxxxx> - 2:1.06-2
+- Restore object after failed truncate() call (bug #1095104)
+
 * Fri Jan 03 2014 Petr Pisar <ppisar@xxxxxxxxxx> - 2:1.06-1
 - 1.06 bump
 
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/perl-devel





[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Legacy Announce]     [Fedora PHP Devel]     [Kernel Devel]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Big List of Linux Books]     [Gimp]     [Yosemite Information]
  Powered by Linux