Skip to content

Commit 8b2628b

Browse files
committed
fix: inconsistent new line view in date_helper
1 parent 09028f6 commit 8b2628b

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

system/Helpers/date_helper.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,13 @@ function timezone_select(string $class = '', string $default = '', int $what = D
5454
{
5555
$timezones = DateTimeZone::listIdentifiers($what, $country);
5656

57-
$buffer = "<select name='timezone' class='{$class}'>" . PHP_EOL;
57+
$buffer = "<select name='timezone' class='{$class}'>\n";
5858

5959
foreach ($timezones as $timezone) {
6060
$selected = ($timezone === $default) ? 'selected' : '';
61-
$buffer .= "<option value='{$timezone}' {$selected}>{$timezone}</option>" . PHP_EOL;
61+
$buffer .= "<option value='{$timezone}' {$selected}>{$timezone}</option>\n";
6262
}
6363

64-
return $buffer . ('</select>' . PHP_EOL);
64+
return $buffer . ("</select>\n");
6565
}
6666
}

tests/system/Helpers/DateHelperTest.php

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -40,14 +40,14 @@ public function testTimezoneSelectDefault()
4040
{
4141
$timezones = DateTimeZone::listIdentifiers(DateTimeZone::ALL, null);
4242

43-
$expected = "<select name='timezone' class='custom-select'>" . PHP_EOL;
43+
$expected = "<select name='timezone' class='custom-select'>\n";
4444

4545
foreach ($timezones as $timezone) {
4646
$selected = ($timezone === 'Asia/Jakarta') ? 'selected' : '';
47-
$expected .= "<option value='{$timezone}' {$selected}>{$timezone}</option>" . PHP_EOL;
47+
$expected .= "<option value='{$timezone}' {$selected}>{$timezone}</option>\n";
4848
}
4949

50-
$expected .= ('</select>' . PHP_EOL);
50+
$expected .= ("</select>\n");
5151

5252
$this->assertSame($expected, timezone_select('custom-select', 'Asia/Jakarta'));
5353
}
@@ -57,14 +57,14 @@ public function testTimezoneSelectSpecific()
5757
$spesificRegion = DateTimeZone::ASIA;
5858
$timezones = DateTimeZone::listIdentifiers($spesificRegion, null);
5959

60-
$expected = "<select name='timezone' class='custom-select'>" . PHP_EOL;
60+
$expected = "<select name='timezone' class='custom-select'>\n";
6161

6262
foreach ($timezones as $timezone) {
6363
$selected = ($timezone === 'Asia/Jakarta') ? 'selected' : '';
64-
$expected .= "<option value='{$timezone}' {$selected}>{$timezone}</option>" . PHP_EOL;
64+
$expected .= "<option value='{$timezone}' {$selected}>{$timezone}</option>\n";
6565
}
6666

67-
$expected .= ('</select>' . PHP_EOL);
67+
$expected .= ("</select>\n");
6868

6969
$this->assertSame($expected, timezone_select('custom-select', 'Asia/Jakarta', $spesificRegion));
7070
}
@@ -75,14 +75,14 @@ public function testTimezoneSelectSingle()
7575
$country = 'ID';
7676
$timezones = DateTimeZone::listIdentifiers($spesificRegion, $country);
7777

78-
$expected = "<select name='timezone' class='custom-select'>" . PHP_EOL;
78+
$expected = "<select name='timezone' class='custom-select'>\n";
7979

8080
foreach ($timezones as $timezone) {
8181
$selected = ($timezone === 'Asia/Jakarta') ? 'selected' : '';
82-
$expected .= "<option value='{$timezone}' {$selected}>{$timezone}</option>" . PHP_EOL;
82+
$expected .= "<option value='{$timezone}' {$selected}>{$timezone}</option>\n";
8383
}
8484

85-
$expected .= ('</select>' . PHP_EOL);
85+
$expected .= ("</select>\n");
8686

8787
$this->assertSame($expected, timezone_select('custom-select', 'Asia/Jakarta', $spesificRegion, $country));
8888
}

0 commit comments

Comments
 (0)