8000 gh-89039: call subclass constructors in datetime.*.replace · python/cpython@b5ebbc5 · GitHub
[go: up one dir, main page]
Skip to content

Commit b5ebbc5

Browse files
eltodereltoder
committed
gh-89039: call subclass constructors in datetime.*.replace
When replace() method is called on a subclass of datetime, date or time, properly call derived constructor. Previously, only the base class's constructor was called. Also, make sure to pass non-zero fold values when creating subclasses in various methods. Previously, fold was silently ignored.
1 parent 6de8aa3 commit b5ebbc5

File tree

3 files changed

+120
-20
lines changed

3 files changed

+120
-20
lines changed

‎Lib/test/datetimetester.py

Lines changed: 56 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1723,11 +1723,24 @@ def test_replace(self):
17231723

17241724
def test_subclass_replace(self):
17251725
class DateSubclass(self.theclass):
1726-
pass
1726+
def __new__(cls, *args, **kwargs):
1727+
result = self.theclass.__new__(cls, *args, **kwargs)
1728+
result.extra = 7
1729+
return result
17271730

17281731
dt = DateSubclass(2012, 1, 1)
1729-
self.assertIs(type(dt.replace(year=2013)), DateSubclass)
1730-
self.assertIs(type(copy.replace(dt, year=2013)), DateSubclass)
1732+
1733+
test_cases = [
1734+
('self.replace', dt.replace(year=2013)),
1735+
('copy.replace', copy.replace(dt, year=2013)),
1736+
]
1737+
1738+
for name, res in test_cases:
1739+
with self.subTest(name):
1740+
self.assertIs(type(res), DateSubclass)
1741+
self.assertEqual(res.year, 2013)
1742+
self.assertEqual(res.month, 1)
1743+
self.assertEqual(res.extra, 7)
17311744

17321745
def test_subclass_date(self):
17331746

@@ -3025,6 +3038,26 @@ def __new__(cls, *args, **kwargs):
30253038
self.assertIsInstance(dt, DateTimeSubclass)
30263039
self.assertEqual(dt.extra, 7)
30273040

3041+
def test_subclass_replace_fold(self):
3042+
class DateTimeSubclass(self.theclass):
3043+
pass
3044+
3045+
dt = DateTimeSubclass(2012, 1, 1)
3046+
dt2 = DateTimeSubclass(2012, 1, 1, fold=1)
3047+
3048+
test_cases = [
3049+
('self.replace', dt.replace(year=2013), 0),
3050+
('self.replace', dt2.replace(year=2013), 1),
3051+
('copy.replace', copy.replace(dt, year=2013), 0),
3052+
('copy.replace', copy.replace(dt2, year=2013), 1),
3053+
]
3054+
3055+
for name, res, fold in test_cases:
3056+
with self.subTest(name, fold=fold):
3057+
self.assertIs(type(res), DateTimeSubclass)
3058+
self.assertEqual(res.year, 2013)
3059+
self.assertEqual(res.fold, fold)
3060+
30283061
def test_fromisoformat_datetime(self):
30293062
# Test that isoformat() is reversible
30303063
base_dates = [
@@ -3705,11 +3738,28 @@ def test_replace(self):
37053738

37063739
def test_subclass_replace(self):
37073740
class TimeSubclass(self.theclass):
3708-
pass
3741+
def __new__(cls, *args, **kwargs):
3742+
result = self.theclass.__new__(cls, *args, **kwargs)
3743+
result.extra = 7
3744+
return result
37093745

37103746
ctime = TimeSubclass(12, 30)
3711-
self.assertIs(type(ctime.replace(hour=10)), TimeSubclass)
3712-
self.assertIs(type(copy.replace(ctime, hour=10)), TimeSubclass)
3747+
ctime2 = TimeSubclass(12, 30, fold=1)
3748+
3749+
test_cases = [
3750+
('self.replace', ctime.replace(hour=10), 0),
3751+
('self.replace', ctime2.replace(hour=10), 1),
3752+
('copy.replace', copy.replace(ctime, hour=10), 0),
3753+
('copy.replace', copy.replace(ctime2, hour=10), 1),
3754+
]
3755+
3756+
for name, res, fold in test_cases:
3757+
with self.subTest(name, fold=fold):
3758+
self.assertIs(type(res), TimeSubclass)
3759+
self.assertEqual(res.hour, 10)
3760+
self.assertEqual(res.minute, 30)
3761+
self.assertEqual(res.extra, 7)
3762+
self.assertEqual(res.fold, fold)
37133763

37143764
def test_subclass_time(self):
37153765

‎Misc/NEWS.d/next/Library/2023-12-18-20-10-50.gh-issue-89039.gqFdtU.rst

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
When replace() method is called o 292D n a subclass of datetime, date or time,
2+
properly call derived constructor. Previously, only the base class's
3+
constructor was called.
4+
5+
Also, make sure to pass non-zero fold values when creating subclasses in
6+
various methods. Previously, fold was silently ignored.

‎Modules/_datetimemodule.c

Lines changed: 58 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1045,6 +1045,39 @@ new_datetime_ex(int year, int month, int day, int hour, int minute,
10451045
new_datetime_ex2(y, m, d, hh, mm, ss, us, tzinfo, fold, \
10461046
&PyDateTime_DateTimeType)
10471047

1048+
static PyObject *
1049+
call_subclass_fold(PyObject *cls, int fold, const char *format, ...) {
1050+
PyObject *kwargs = NULL, *res = NULL;
1051+
va_list va;
1052+
1053+
va_start(va, format);
1054+
PyObject *args = Py_VaBuildValue(format, va);
1055+
va_end(va);
1056+
if (args == NULL) {
1057+
return NULL;
1058+
}
1059+
if (fold) {
1060+
kwargs = PyDict_New();
1061+
if (kwargs == NULL) {
1062+
goto Done;
1063+
}
1064+
PyObject *obj = PyLong_FromLong(fold);
1065+
if (obj == NULL) {
1066+
goto Done;
1067+
}
1068+
int res = PyDict_SetItemString(kwargs, "fold", obj);
1069+
Py_DECREF(obj);
1070+
if (res < 0) {
1071+
goto Done;
1072+
}
1073+
}
1074+
res = PyObject_Call(cls, args, kwargs);
1075+
Done:
1076+
Py_DECREF(args);
1077+
Py_XDECREF(kwargs);
1078+
return res;
1079+
}
1080+
10481081
static PyObject *
10491082
new_datetime_subclass_fold_ex(int year, int month, int day, int hour, int minute,
10501083
int second, int usecond, PyObject *tzinfo,
@@ -1056,15 +1089,8 @@ new_datetime_subclass_fold_ex(int year, int month, int day, int hour, int minute
10561089
tzinfo, fold);
10571090
} else {
10581091
// Subclass
1059-
dt = PyObject_CallFunction(cls, "iiiiiiiO",
1060-
year,
1061-
month,
1062-
day,
1063-
hour,
1064-
minute,
1065-
second,
1066-
usecond,
1067-
tzinfo);
1092+
dt = call_subclass_fold(cls, fold, "iiiiiiiO", year, month, day,
1093+
hour, minute, second, usecond, tzinfo);
10681094
}
10691095

10701096
return dt;
@@ -1120,6 +1146,23 @@ new_time_ex(int hour, int minute, int second, int usecond,
11201146
#define new_time(hh, mm, ss, us, tzinfo, fold) \
11211147
new_time_ex2(hh, mm, ss, us, tzinfo, fold, &PyDateTime_TimeType)
11221148

1149+
static PyObject *
1150+
new_time_subclass_fold_ex(int hour, int minute, int second, int usecond,
1151+
PyObject *tzinfo, int fold, PyObject *cls)
1152+
{
1153+
PyObject *t;
1154+
if ((PyTypeObject*)cls == &PyDateTime_TimeType) {
1155+
// Use the fast path constructor
1156+
t = new_time(hour, minute, second, usecond, tzinfo, fold);
1157+
} else {
1158+
// Subclass
1159+
t = call_subclass_fold(cls, fold, "iiiiO", hour, minute, second,
1160+
usecond, tzinfo);
1161+
}
1162+
1163+
return t;
1164+
}
1165+
11231166
/* Create a timedelta instance. Normalize the members iff normalize is
11241167
* true. Passing false is a speed optimization, if you know for sure
11251168
* that seconds and microseconds are already in their proper ranges. In any
@@ -3482,7 +3525,7 @@ datetime_date_replace_impl(PyDateTime_Date *self, int year, int month,
34823525
int day)
34833526
/*[clinic end generated code: output=2a9430d1e6318aeb input=0d1f02685b3e90f6]*/
34843527
{
3485-
return new_date_ex(year, month, day, Py_TYPE(self));
3528+
return new_date_subclass_ex(year, month, day, (PyObject *)Py_TYPE(self));
34863529
}
34873530

34883531
static Py_hash_t
@@ -4591,8 +4634,8 @@ datetime_time_replace_impl(PyDateTime_Time *self, int hour, int minute,
45914634
int fold)
45924635
/*[clinic end generated code: output=0b89a44c299e4f80 input=9b6a35b1e704b0ca]*/
45934636
{
4594-
return new_time_ex2(hour, minute, second, microsecond, tzinfo, fold,
4595-
Py_TYPE(self));
4637+
return new_time_subclass_fold_ex(hour, minute, second, microsecond, tzinfo,
4638+
fold, (PyObject *)Py_TYPE(self));
45964639
}
45974640

45984641
static PyObject *
@@ -6055,8 +6098,9 @@ datetime_datetime_replace_impl(PyDateTime_DateTime *self, int year,
60556098
int fold)
60566099
/*[clinic end generated code: output=00bc96536833fddb input=9b38253d56d9bcad]*/
60576100
{
6058-
return new_datetime_ex2(year, month, day, hour, minute, second,
6059-
microsecond, tzinfo, fold, Py_TYPE(self));
6101+
return new_datetime_subclass_fold_ex(year, month, day, hour, minute,
6102+
second, microsecond, tzinfo, fold,
6103+
(PyObject *)Py_TYPE(self));
60606104
}
60616105

60626106
static PyObject *

0 commit comments

Comments
 (0)
0