Skip to content

Commit 83fd38d

Browse files
committed
Apply suggestion from code review
1 parent 68d5793 commit 83fd38d

File tree

4 files changed

+18
-18
lines changed

4 files changed

+18
-18
lines changed

wcfsetup/install/files/lib/system/gridView/filter/DateFilter.class.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public function renderValue(string $value): string
5353
$locale = WCF::getLanguage()->getLocale();
5454
$fromString = $toString = '';
5555
if ($values[0] !== '') {
56-
$fromDateTime = \DateTime::createFromFormat(
56+
$fromDateTime = \DateTimeImmutable::createFromFormat(
5757
'Y-m-d',
5858
$values[0],
5959
WCF::getUser()->getTimeZone()
@@ -70,7 +70,7 @@ public function renderValue(string $value): string
7070
}
7171
}
7272
if ($values[1] !== '') {
73-
$toDateTime = \DateTime::createFromFormat(
73+
$toDateTime = \DateTimeImmutable::createFromFormat(
7474
'Y-m-d',
7575
$values[1],
7676
WCF::getUser()->getTimeZone()
@@ -108,21 +108,21 @@ private function getTimestamps(string $value): array
108108

109109
$values = explode(';', $value);
110110
if (\count($values) === 2) {
111-
$fromDateTime = \DateTime::createFromFormat(
111+
$fromDateTime = \DateTimeImmutable::createFromFormat(
112112
'Y-m-d',
113113
$values[0]
114114
);
115115
if ($fromDateTime !== false) {
116-
$fromDateTime->setTime(0, 0);
116+
$fromDateTime = $fromDateTime->setTime(0, 0);
117117
$from = $fromDateTime->getTimestamp();
118118
}
119119

120-
$toDateTime = \DateTime::createFromFormat(
120+
$toDateTime = \DateTimeImmutable::createFromFormat(
121121
'Y-m-d',
122122
$values[1]
123123
);
124124
if ($toDateTime !== false) {
125-
$toDateTime->setTime(23, 59, 59);
125+
$toDateTime = $toDateTime->setTime(23, 59, 59);
126126
$to = $toDateTime->getTimestamp();
127127
}
128128
}

wcfsetup/install/files/lib/system/gridView/filter/TimeFilter.class.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ public function renderValue(string $value): string
5555
$locale = WCF::getLanguage()->getLocale();
5656
$fromString = $toString = '';
5757
if ($values[0] !== '') {
58-
$fromDateTime = \DateTime::createFromFormat(
58+
$fromDateTime = \DateTimeImmutable::createFromFormat(
5959
'Y-m-d\TH:i:sP',
6060
$values[0],
6161
WCF::getUser()->getTimeZone()
@@ -72,7 +72,7 @@ public function renderValue(string $value): string
7272
}
7373
}
7474
if ($values[1] !== '') {
75-
$toDateTime = \DateTime::createFromFormat(
75+
$toDateTime = \DateTimeImmutable::createFromFormat(
7676
'Y-m-d\TH:i:sP',
7777
$values[1],
7878
WCF::getUser()->getTimeZone()
@@ -122,7 +122,7 @@ private function getTimestamps(string $value): array
122122

123123
private function getTimestamp(string $date): int
124124
{
125-
$dateTime = \DateTime::createFromFormat(
125+
$dateTime = \DateTimeImmutable::createFromFormat(
126126
'Y-m-d\TH:i:sP',
127127
$date
128128
);

wcfsetup/install/files/lib/system/listView/filter/DateFilter.class.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public function renderValue(string $value): string
5353
$locale = WCF::getLanguage()->getLocale();
5454
$fromString = $toString = '';
5555
if ($values[0] !== '') {
56-
$fromDateTime = \DateTime::createFromFormat(
56+
$fromDateTime = \DateTimeImmutable::createFromFormat(
5757
'Y-m-d',
5858
$values[0],
5959
WCF::getUser()->getTimeZone()
@@ -70,7 +70,7 @@ public function renderValue(string $value): string
7070
}
7171
}
7272
if ($values[1] !== '') {
73-
$toDateTime = \DateTime::createFromFormat(
73+
$toDateTime = \DateTimeImmutable::createFromFormat(
7474
'Y-m-d',
7575
$values[1],
7676
WCF::getUser()->getTimeZone()
@@ -108,21 +108,21 @@ private function getTimestamps(string $value): array
108108

109109
$values = explode(';', $value);
110110
if (\count($values) === 2) {
111-
$fromDateTime = \DateTime::createFromFormat(
111+
$fromDateTime = \DateTimeImmutable::createFromFormat(
112112
'Y-m-d',
113113
$values[0]
114114
);
115115
if ($fromDateTime !== false) {
116-
$fromDateTime->setTime(0, 0);
116+
$fromDateTime = $fromDateTime->setTime(0, 0);
117117
$from = $fromDateTime->getTimestamp();
118118
}
119119

120-
$toDateTime = \DateTime::createFromFormat(
120+
$toDateTime = \DateTimeImmutable::createFromFormat(
121121
'Y-m-d',
122122
$values[1]
123123
);
124124
if ($toDateTime !== false) {
125-
$toDateTime->setTime(23, 59, 59);
125+
$toDateTime = $toDateTime->setTime(23, 59, 59);
126126
$to = $toDateTime->getTimestamp();
127127
}
128128
}

wcfsetup/install/files/lib/system/listView/filter/TimeFilter.class.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ public function renderValue(string $value): string
5555
$locale = WCF::getLanguage()->getLocale();
5656
$fromString = $toString = '';
5757
if ($values[0] !== '') {
58-
$fromDateTime = \DateTime::createFromFormat(
58+
$fromDateTime = \DateTimeImmutable::createFromFormat(
5959
'Y-m-d\TH:i:sP',
6060
$values[0],
6161
WCF::getUser()->getTimeZone()
@@ -72,7 +72,7 @@ public function renderValue(string $value): string
7272
}
7373
}
7474
if ($values[1] !== '') {
75-
$toDateTime = \DateTime::createFromFormat(
75+
$toDateTime = \DateTimeImmutable::createFromFormat(
7676
'Y-m-d\TH:i:sP',
7777
$values[1],
7878
WCF::getUser()->getTimeZone()
@@ -122,7 +122,7 @@ private function getTimestamps(string $value): array
122122

123123
private function getTimestamp(string $date): int
124124
{
125-
$dateTime = \DateTime::createFromFormat(
125+
$dateTime = \DateTimeImmutable::createFromFormat(
126126
'Y-m-d\TH:i:sP',
127127
$date
128128
);

0 commit comments

Comments
 (0)