D26167: Update holidays and add flagdays and namedays for Sweden
This revision was automatically updated to reflect the committed changes. Closed by commit R175:2d9ea97d1cdd: Update holidays and add flagdays and namedays for Sweden (authored by riiga, committed by aacid). REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26167?vs=72048=74529 REVISION DETAIL https://phabricator.kde.org/D26167 AFFECTED FILES holidays/holidays.qrc holidays/plan2/holiday_se_sv holidays/plan2/holiday_se_sv_nameday To: riiga, #kde_pim, winterz Cc: ltoscano, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D26167: Update holidays and add flagdays and namedays for Sweden
winterz accepted this revision. winterz added a comment. This revision is now accepted and ready to land. looks fine. I also tested and the changes don't break anything. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D26167 To: riiga, #kde_pim, winterz Cc: ltoscano, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D26167: Update holidays and add flagdays and namedays for Sweden
nicolasfella added reviewers: KDE PIM, winterz. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D26167 To: riiga, #kde_pim, winterz Cc: ltoscano, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D26167: Update holidays and add flagdays and namedays for Sweden
riiga updated this revision to Diff 72048. riiga edited the summary of this revision. riiga added a comment. Fixed Annandag pingst calculation. REPOSITORY R175 KHolidays CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26167?vs=72024=72048 REVISION DETAIL https://phabricator.kde.org/D26167 AFFECTED FILES holidays/holidays.qrc holidays/plan2/holiday_se_sv holidays/plan2/holiday_se_sv_nameday To: riiga Cc: ltoscano, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D26167: Update holidays and add flagdays and namedays for Sweden
ltoscano added a comment. Try to use [easter] + 50 days Example (from Italian holidays): "Ascensione (Chiesa Cattolica)" religious on ((year >= 1977) ? ([easter] + 42 days): noop) https://cgit.kde.org/kholidays.git/tree/holidays/plan2/holiday_it_it?#n47 REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D26167 To: riiga Cc: ltoscano, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
D26167: Update holidays and add flagdays and namedays for Sweden
riiga created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. riiga requested review of this revision. REVISION SUMMARY Holidays for Sweden have been corrected and updated. Flag days have been added along with a few other significant cultural days. A separate name day calendar has been added. Where known, support has been added for holidays in the recent past. I wasn't able to add support for both versions of "Annandag pingst". It changed in 2005 from a public religious holiday to just a religious holiday (the public holiday was moved to Nationaldagen instead), but something with the calculation of "easter plus 50 days" in brackets seem to result in a syntax error. If someone knows how to solve this, I'll fix it, otherwise the best is just to leave it as is (religious in entire calendar). TEST PLAN Tested locally with the testholidayregions tool for syntax errors. REPOSITORY R175 KHolidays REVISION DETAIL https://phabricator.kde.org/D26167 AFFECTED FILES holidays/holidays.qrc holidays/plan2/holiday_se_sv holidays/plan2/holiday_se_sv_nameday To: riiga Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns