From ef6f6dc04e35ab006747083bab50ead0353b0746 Mon Sep 17 00:00:00 2001
From: zverok <zverok.offline@gmail.com>
Date: Mon, 12 Dec 2022 22:48:54 +0200
Subject: [PATCH] fixup! Implement Date#deconstruct_keys and
 DateTime#deconstruct_keys

---
 ext/date/date_core.c   |  6 +-----
 test/date/test_date.rb | 12 +++++++++++-
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/ext/date/date_core.c b/ext/date/date_core.c
index dd06774..d1f2963 100644
--- a/ext/date/date_core.c
+++ b/ext/date/date_core.c
@@ -7438,14 +7438,12 @@ d_lite_jisx0301(VALUE self)
 
 static VALUE
 deconstruct_keys(VALUE self, VALUE keys, int is_datetime) {
-    VALUE h;
+    VALUE h = rb_hash_new();
     long i;
 
     get_d1(self);
 
     if (NIL_P(keys)) {
-        h = rb_hash_new();
-
         rb_hash_aset(h, sym_year, m_real_year(dat));
         rb_hash_aset(h, sym_month, INT2FIX(m_mon(dat)));
         rb_hash_aset(h, sym_day, INT2FIX(m_mday(dat)));
@@ -7468,8 +7466,6 @@ deconstruct_keys(VALUE self, VALUE keys, int is_datetime) {
 
     }
 
-    h = rb_hash_new();
-
     for (i=0; i<RARRAY_LEN(keys); i++) {
         VALUE key = RARRAY_AREF(keys, i);
 
diff --git a/test/date/test_date.rb b/test/date/test_date.rb
index e5d24b7..3f9c893 100644
--- a/test/date/test_date.rb
+++ b/test/date/test_date.rb
@@ -180,17 +180,27 @@ def test_deconstruct_keys
     d = Date.new(1999,5,23)
     assert_equal({year: 1999, month: 5, day: 23, wday: 0, yday: 143}, d.deconstruct_keys(nil))
     assert_equal({year: 1999}, d.deconstruct_keys([:year, :century]))
+    assert_equal(
+      {year: 1999, month: 5, day: 23, wday: 0, yday: 143},
+      d.deconstruct_keys([:year, :month, :day, :wday, :yday])
+    )
 
     dt = DateTime.new(1999, 5, 23, 4, 20, Rational(1, 10000))
 
     assert_equal(
       {year: 1999, month: 5, day: 23, wday: 0, yday: 143,
-       hour: 4, min: 20, sec: 13, sec: 0, sec_fraction: Rational(1, 10000), zone: "+00:00"},
+       hour: 4, min: 20, sec: 0, sec_fraction: Rational(1, 10000), zone: "+00:00"},
       dt.deconstruct_keys(nil)
     )
 
     assert_equal({year: 1999}, dt.deconstruct_keys([:year, :century]))
 
+    assert_equal(
+      {year: 1999, month: 5, day: 23, wday: 0, yday: 143,
+       hour: 4, min: 20, sec: 0, sec_fraction: Rational(1, 10000), zone: "+00:00"},
+      dt.deconstruct_keys([:year, :month, :day, :wday, :yday, :hour, :min, :sec, :sec_fraction, :zone])
+    )
+
     dtz = DateTime.parse('3rd Feb 2001 04:05:06+03:30')
     assert_equal({zone: '+03:30'}, dtz.deconstruct_keys([:zone]))
   end