Merge branch 'main' of https://github.com/florinp/holidays

.github/workflows/fix-php-code-style-issues.yml

@@ -1,9 +1,6 @@
 name: Fix PHP code style issues
 
-on:
-  push:
-    paths:
-      - '**.php'
+on: [push]
 
 permissions:
   contents: write

.github/workflows/phpstan.yml

@@ -1,6 +1,6 @@
 name: PHPStan
 
-on: [push]
+on: [push, pull_request]
 
 jobs:
   phpstan:

.github/workflows/run-tests.yml

@@ -1,13 +1,6 @@
 name: Tests
 
-on:
-  push:
-    paths:
-      - '**.php'
-      - '.github/workflows/run-tests-pest.yml'
-      - 'phpunit.xml.dist'
-      - 'composer.json'
-      - 'composer.lock'
+on: [push, pull_request]
 
 jobs:
   test:

README.md

@@ -11,7 +11,7 @@ use Spatie\Holidays\Holidays;
 
 // returns an array of Belgian holidays
 // for the current year
-$holidays = Holidays::for('be')->get(); 
+$holidays = Holidays::for('be')->get();
 ```
 
 ## Support us
@@ -40,21 +40,21 @@ You can get all holidays for a country by using the `get` method.
 
 ```php
 use Spatie\Holidays\Holidays;
+use Spatie\Holidays\Countries\Belgium;
 
 // returns an array of Belgian holidays
 // for the current year
-$holidays = Holidays::for('be')->get(); 
+$holidays = Holidays::for(Belgium::make())->get(); 
 ```
 
-Alternatively, you could also pass an instance of `Country` to the `for` method.
+Alternatively, you could also pass an ISO code to the `for` method.
 
 ```php
 use Spatie\Holidays\Holidays;
-use Spatie\Holidays\Countries\Belgium;
 
 // returns an array of Belgian holidays
 // for the current year
-$holidays = Holidays::for(Belgium::make())->get(); 
+$holidays = Holidays::for('be')->get();
 ```
 
 ### Getting holidays for a specific year
@@ -87,6 +87,40 @@ use Spatie\Holidays\Holidays;
 Holidays::for('be')->getName('2024-01-01'); // Nieuwjaar
 ```
 
+## Contributing a new country
+
+If you want to add a new country, you can create a pull request.
+
+1. Create a new class in the `Countries` directory. It should extend the `Country` class.
+2. Add a test for the new country in the `tests` directory.
+3. Run the tests so a snapshot gets created.
+4. Verify the result in the newly created snapshot is correct.
+
+In case your country has specific rules for calculating holidays,
+for example region specific holidays, you can pass this to the constructor of your country class.
+
+```php
+$holidays = Holidays::for(Austria::make(region: 'de-bw'))->get();
+```
+
+The value, `de-bw`, will be passed to the region parameter of the contructor of a country.
+
+```php
+class Austria extends Country
+{
+    protected function __construct(
+        protected ?string $region = null,
+    ) {
+    }
+
+    protected function allHolidays(int $year): array
+    {
+        // Here you can use $this->region (or other variables) to calculate holidays
+    }
+```
+
+Please see [CONTRIBUTING](https://github.com/spatie/.github/blob/main/CONTRIBUTING.md) for more details.
+
 ## Testing
 
 ```bash
@@ -97,10 +131,6 @@ composer test
 
 Please see [CHANGELOG](CHANGELOG.md) for more information on what has changed recently.
 
-## Contributing
-
-Please see [CONTRIBUTING](https://github.com/spatie/.github/blob/main/CONTRIBUTING.md) for details.
-
 ## Security Vulnerabilities
 
 Please review [our security policy](../../security/policy) on how to report security vulnerabilities.

phpstan-baseline.neon

@@ -1,17 +1,17 @@
 parameters:
 	ignoreErrors:
 		-
-			message: "#^Method Spatie\\\\Holidays\\\\Countries\\\\Belgium\\:\\:allHolidays\\(\\) should return array\\<string, Carbon\\\\CarbonImmutable\\> but returns array\\<string, Carbon\\\\CarbonImmutable\\|string\\>\\.$#"
+			message: "#^Unreachable statement \\- code above always terminates\\.$#"
 			count: 1
-			path: src/Countries/Belgium.php
+			path: src/Countries/Brazil.php
 
 		-
 			message: "#^Argument of an invalid type array\\<int, string\\>\\|false supplied for foreach, only iterables are supported\\.$#"
 			count: 1
 			path: src/Countries/Country.php
 
 		-
-			message: "#^Method Spatie\\\\Holidays\\\\Countries\\\\Country\\:\\:get\\(\\) should return array\\<string, Carbon\\\\CarbonImmutable\\> but returns array\\<string, Carbon\\\\CarbonImmutable\\|false\\>\\.$#"
+			message: "#^Method Spatie\\\\Holidays\\\\Countries\\\\Country\\:\\:get\\(\\) should return array\\<string, Carbon\\\\CarbonImmutable\\|string\\> but returns array\\<string, Carbon\\\\CarbonImmutable\\|false\\>\\.$#"
 			count: 1
 			path: src/Countries/Country.php
 
@@ -20,11 +20,6 @@ parameters:
 			count: 1
 			path: src/Countries/Country.php
 
-		-
-			message: "#^Method Spatie\\\\Holidays\\\\Countries\\\\Hungary\\:\\:allHolidays\\(\\) should return array\\<string, Carbon\\\\CarbonImmutable\\> but returns array\\<string, Carbon\\\\CarbonImmutable\\|string\\>\\.$#"
-			count: 1
-			path: src/Countries/Hungary.php
-
 		-
 			message: "#^Cannot call method isSunday\\(\\) on Carbon\\\\CarbonImmutable\\|false\\.$#"
 			count: 1
@@ -44,3 +39,8 @@ parameters:
 			message: "#^Parameter \\#1 \\$callback of function array_map expects \\(callable\\(Carbon\\\\CarbonImmutable\\)\\: mixed\\)\\|null, Closure\\(string\\)\\: non\\-falsy\\-string given\\.$#"
 			count: 1
 			path: src/Holidays.php
+
+		-
+			message: "#^Property Spatie\\\\Holidays\\\\Holidays\\:\\:\\$holidays \\(array\\<string, Carbon\\\\CarbonImmutable\\>\\) does not accept array\\<string, Carbon\\\\CarbonImmutable\\|string\\>\\.$#"
+			count: 1
+			path: src/Holidays.php

src/Countries/Andorra.php

@@ -0,0 +1,42 @@
+<?php
+
+namespace Spatie\Holidays\Countries;
+
+use Carbon\CarbonImmutable;
+
+class Andorra extends Country
+{
+    public function countryCode(): string
+    {
+        return 'ad';
+    }
+
+    protected function allHolidays(int $year): array
+    {
+        return array_merge([
+            'Any nou' => '01-01',
+            'Reis' => '01-06',
+            'Dia de la Constitució' => '03-14',
+            'Festa del Treball' => '05-01',
+            'Assumpció' => '08-15',
+            'Mare de Déu de Meritxell' => '09-08',
+            'Tots Sants' => '11-01',
+            'Immaculada Concepció' => '11-08',
+            'Nadal' => '12-25',
+            'Sant Esteve' => '12-26',
+        ], $this->variableHolidays($year));
+    }
+
+    /** @return array<string, CarbonImmutable> */
+    protected function variableHolidays(int $year): array
+    {
+        $easter = CarbonImmutable::createFromTimestamp(easter_date($year))
+            ->setTimezone('Europe/Brussels');
+
+        return [
+            'Divendres Sant' => $easter->subDays(2),
+            'Dilluns de Pasqua' => $easter->addDay(),
+            'Dilluns de Pentecosta' => $easter->addDays(50),
+        ];
+    }
+}

src/Countries/Austria.php

@@ -6,12 +6,16 @@
 
 class Austria extends Country
 {
+    protected function __construct(
+        public ?string $region = null
+    ) {
+    }
+
     public function countryCode(): string
     {
         return 'at';
     }
 
-    /** @return array<string, CarbonImmutable> */
     protected function allHolidays(int $year): array
     {
         return array_merge([
@@ -34,7 +38,7 @@ protected function variableHolidays(int $year): array
             ->setTimezone('Europe/Vienna');
 
         return [
-            'Ostermontag' => $easter->addDay(1),
+            'Ostermontag' => $easter->addDay(),
             'Christi Himmelfahrt' => $easter->addDays(39),
             'Pfingstmontag' => $easter->addDays(50),
             'Fronleichnam' => $easter->addDays(60),

src/Countries/Belgium.php

@@ -11,7 +11,6 @@ public function countryCode(): string
         return 'be';
     }
 
-    /** @return array<string, CarbonImmutable> */
     protected function allHolidays(int $year): array
     {
         return array_merge([

src/Countries/Brazil.php

@@ -11,9 +11,10 @@ public function countryCode(): string
         return 'br';
     }
 
-    /** @return array<string, CarbonImmutable> */
     protected function allHolidays(int $year): array
     {
+        throw new \Exception('Not implemented yet.');
+
         return array_merge([
             'Dia de Ano Novo' => '01-01',
             'Dia de Tiradentes' => '04-21',

src/Countries/Country.php

@@ -13,7 +13,7 @@ abstract public function countryCode(): string;
     /** @return array<string, string|CarbonImmutable> */
     abstract protected function allHolidays(int $year): array;
 
-    /** @return array<string, CarbonImmutable> */
+    /** @return array<string, CarbonImmutable|string> */
     public function get(int $year): array
     {
         $this->ensureYearCanBeCalculated($year);
@@ -37,7 +37,7 @@ public function get(int $year): array
 
     public static function make(): static
     {
-        return new static();
+        return new static(...func_get_args());
     }
 
     public static function find(string $countryCode): ?Country

src/Countries/Denmark.php

@@ -0,0 +1,46 @@
+<?php
+
+namespace Spatie\Holidays\Countries;
+
+use Carbon\CarbonImmutable;
+
+class Denmark extends Country
+{
+    public function countryCode(): string
+    {
+        return 'da';
+    }
+
+    protected function allHolidays(int $year): array
+    {
+        return array_merge([
+            'Nytår' => '01-01',
+            'Juleaften' => '12-24',
+            'Juledag' => '12-25',
+            'Anden Juledag' => '12-26',
+        ], $this->variableHolidays($year));
+    }
+
+    /** @return array<string, CarbonImmutable> */
+    protected function variableHolidays(int $year): array
+    {
+        $easter = CarbonImmutable::createFromTimestamp(easter_date($year))
+            ->setTimezone('Europe/Copenhagen');
+
+        $holidays = [
+            'Påskedag' => $easter->addDay(),
+            'Skærtorsdag' => $easter->subDays(3),
+            'Langfredag' => $easter->subDays(2),
+            'Anden Påskedag' => $easter->addDays(2),
+            'Kristi Himmelfartsdag' => $easter->addDays(39),
+            'Pinse' => $easter->addDays(49),
+            'Anden Pinsedag' => $easter->addDays(50),
+        ];
+
+        if ($year < 2024) {
+            $holidays['Store Bededag'] = $easter->addDays(26);
+        }
+
+        return $holidays;
+    }
+}

src/Countries/Hungary.php

@@ -11,7 +11,6 @@ public function countryCode(): string
         return 'hu';
     }
 
-    /** @return array<string, CarbonImmutable> */
     protected function allHolidays(int $year): array
     {
         return array_merge([

src/Countries/Netherlands.php

@@ -11,7 +11,6 @@ public function countryCode(): string
         return 'nl';
     }
 
-    /** @return array<string, string|CarbonImmutable> */
     protected function allHolidays(int $year): array
     {
         return array_merge([

tests/.pest/snapshots/Countries/AndorraTest/it_can_calculate_andorra_holidays.snap

@@ -0,0 +1,54 @@
+[
+    {
+        "name": "Any nou",
+        "date": "2024-01-01"
+    },
+    {
+        "name": "Reis",
+        "date": "2024-01-06"
+    },
+    {
+        "name": "Dia de la Constituci\u00f3",
+        "date": "2024-03-14"
+    },
+    {
+        "name": "Divendres Sant",
+        "date": "2024-03-29"
+    },
+    {
+        "name": "Dilluns de Pasqua",
+        "date": "2024-04-01"
+    },
+    {
+        "name": "Festa del Treball",
+        "date": "2024-05-01"
+    },
+    {
+        "name": "Dilluns de Pentecosta",
+        "date": "2024-05-20"
+    },
+    {
+        "name": "Assumpci\u00f3",
+        "date": "2024-08-15"
+    },
+    {
+        "name": "Mare de D\u00e9u de Meritxell",
+        "date": "2024-09-08"
+    },
+    {
+        "name": "Tots Sants",
+        "date": "2024-11-01"
+    },
+    {
+        "name": "Immaculada Concepci\u00f3",
+        "date": "2024-11-08"
+    },
+    {
+        "name": "Nadal",
+        "date": "2024-12-25"
+    },
+    {
+        "name": "Sant Esteve",
+        "date": "2024-12-26"
+    }
+]