azuyalabs / yasumi

The easy PHP Library for calculating holidays
https://www.yasumi.dev
Other
1.04k stars 152 forks source link

Spain: Easter sunday not an official holiday? #83

Closed bkonetzny closed 6 years ago

bkonetzny commented 6 years ago

Yasumi lists 01.04.2018 (Easter Sunday) for Spain as an official holiday. I checked several sources and can't find any reference to this.

Sources I checked:

Good friday is a national holiday and easter monday is listed as a regional holiday, which is identical to listing in Yasumi dataset. I think (Easter Sunday) should be removed from Spain holidays?

lukosius commented 6 years ago

Good catch. I think the holiday should be changed to the observance type.

https://es.wikipedia.org/wiki/Calendario_laboral

stelgenhof commented 6 years ago

My assumption here was that if Easter Monday is celebrated, then Easter Sunday must be celebrated as well (as it happens as so in other countries). Likely my assumption was wrong then :)

I checked also this resource: https://www.boe.es/boe/dias/2017/10/11/pdfs/BOE-A-2017-11639.pdf and Easter Sunday isn't mentioned anywhere. So it does seem to be that it isn't a holiday at all.

lukosius commented 6 years ago

I thought we can leave it as an observance type because it ends the holy week in Spain, but I'm not sure.

https://en.wikipedia.org/wiki/Holy_Week_procession

stelgenhof commented 6 years ago

According to that source, it should be an observance in selected regions only.

bkonetzny commented 6 years ago

@stelgenhof Which source of the links above are you referring to? If thats an spanish source, can you point me to a english variant so I can fix the PR?

stelgenhof commented 6 years ago

@bkonetzny I was referring to the PDF in my above comment. However, after reading it a bit more thoroughly, my assumption that it was only observed in selected regions was wrong. Easter Sunday is not an 'official' holiday as such in all of Spain and setting this a 'observance' makes sense (also it falls on a Sunday anyway).

Don't need to fix your PR. I'll have another look and merge it (if it is fine).