Discussion:
[kholidays] [Bug 383896] New: Israel's holidays are not displayed
Wolfgang Bauer
2017-08-23 09:43:17 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

Bug ID: 383896
Summary: Israel's holidays are not displayed
Product: kholidays
Version: git
Platform: openSUSE RPMs
URL: http://bugzilla.opensuse.org/show_bug.cgi?id=1053947
OS: Linux
Status: UNCONFIRMED
Severity: normal
Priority: NOR
Component: general
Assignee: kdepim-***@kde.org
Reporter: ***@tmo.at
Target Milestone: ---

If you choose "Israel (English)" or "Israel (Hebrew)" holidays in KOrganizer's
settings (and restart KOrganizer), no holidays are displayed at all.

I suppose that's because kholidays' QCalendarSystem class
(src/parsers/qcalendarsystem.cpp) seems to have no support for the Hebrew
calendar implemented.

That's with the latest version 5.6.0 (from KDE Applications 17.08.0).
--
You are receiving this mail because:
You are watching all bug changes.
b***@kde.org
2017-08-23 21:47:16 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

***@orbsky.homelinux.org changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@orbsky.homelinux.org
--
You are receiving this mail because:
You are watching all bug changes.
Luigi Toscano
2018-12-06 18:30:24 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

Luigi Toscano <***@tiscali.it> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@tiscali.it
Status|REPORTED |CONFIRMED
Ever confirmed|0 |1

--- Comment #1 from Luigi Toscano <***@tiscali.it> ---
This is still valid. Reported on IRC and confirmed by a quick check with
kholidays 5.52 (now in Frameworks).
--
You are receiving this mail because:
You are watching all bug changes.
b***@kde.org
2018-12-06 18:40:15 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

***@shimi.net changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@shimi.net

--- Comment #2 from ***@shimi.net ---
(In reply to Luigi Toscano from comment #1)
Post by Luigi Toscano
This is still valid. Reported on IRC and confirmed by a quick check with
kholidays 5.52 (now in Frameworks).
I'm the reporter on IRC.

I would like to emphasize that this is not sure Israel's holidays as can be
understood from the title of this bug. Right, they're "assigned" to "Israel",
but in fact all Israel's holidays are in fact the Jewish holidays, celebrated
by all Jews across the world. We're talking about Passover, Hannukah etc. So
whenever someone, Jewish or not, wants to know when is Hannukah this year
(which is a different day every year on the Gregorian calendar) - they
basically can't do that with Plasma 5.

I would ALSO like to emphasize, that this is A REGRESSION. This worked
perfectly in KDE 4, and was actually one of the "Linux" features I showed off
to friends as lacking in their corporate operating systems. Now it is gone :(
--
You are receiving this mail because:
You are watching all bug changes.
Shimi
2018-12-06 18:45:34 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #3 from Shimi <***@shimi.net> ---
(In reply to Shimi from comment #2)
Post by b***@kde.org
I would like to emphasize that this is not sure Israel's holidays as can be
Sorry: "this is not only about Israel's holidays" is what I meant to write.
Sorry about spam. Not sure how this got typed :)
--
You are receiving this mail because:
You are watching all bug changes.
JanKusanagi
2018-12-06 18:55:10 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

JanKusanagi <jan-***@gmx.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |jan-***@gmx.co.uk

--- Comment #4 from JanKusanagi <jan-***@gmx.co.uk> ---
(In reply to Shimi from comment #2)
Post by b***@kde.org
was actually one of the "Linux" features I showed
off to friends as lacking in their corporate operating systems. Now it is
gone :(
The feature is not gone, there's just a bug with this particular locale.
The feature works perfectly fine for most other countries/locales.

I'll add that "Victory Day", the only Gregorian-based event in the Israel
holiday file, is handled correctly, so it's not like parsing the file fails.
--
You are receiving this mail because:
You are watching all bug changes.
Allen Winter
2018-12-06 22:25:00 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

Allen Winter <***@kde.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@kde.org

--- Comment #5 from Allen Winter <***@kde.org> ---
This comment is about KOrganizer and not the digital clock plasmoid that can
also show holidays.

We had a plugin for KOrganizer in the KDE4 days for a Jewish holidays and what
made it special was that it knew the Hebrew calendar. Indeed we never ported
that to KDE5.

In KDE5 days, we have holidays for Israel in English and translated to Hebrew.
for the locales en-us and he. It should know about the Hebrew calendar too.

When testing in KOrganizer, however, I don't see the religious holidays for
Israeli calendar . I do see the one holiday "Victory day" on the Gregorian
calendar.

For people wanting to see the Jewish holidays and don't want to load the Israel
holidays, then it makes sense to try and restore the Jewish holidays KOrganizer
plugin. except that doesn't work for the digital clock plasmoid.

Summary:
- restore KOrganizer Jewish holidays plugin
- figure out why religious holidays from the Israel calendar don't show up in
KOrganizer (should they? or should that function belong to the Jewish holidays
plugin?)
- what to do in digital clock about all this

I am the kholidays maintainer and former KOrganizer maintainer but my memory of
all this is kinda fuzzy.
--
You are receiving this mail because:
You are watching all bug changes.
Allen Winter
2018-12-06 22:27:45 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #6 from Allen Winter <***@kde.org> ---
(In reply to JanKusanagi from comment #4)
Post by JanKusanagi
The feature is not gone, there's just a bug with this particular locale.
The feature works perfectly fine for most other countries/locales.
I'll add that "Victory Day", the only Gregorian-based event in the Israel
holiday file, is handled correctly, so it's not like parsing the file fails.
"Perfectly fine"? really? where? how?
I'm not seeing any Jewish holidays when I load the Israel holiday file.
--
You are receiving this mail because:
You are watching all bug changes.
b***@kde.org
2018-12-07 05:53:19 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #7 from ***@orbsky.homelinux.org ---
Yep.... the problem still presists.
--
You are receiving this mail because:
You are watching all bug changes.
Allen Winter
2018-12-09 20:44:46 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #10 from Allen Winter <***@kde.org> ---
I may be the kholidays maintainer now, but I really need John Layt (the
original author) to help figure out why holidays on the Hebrew calendar aren't
showing up.

I don't know how to rotate hebrew calendar dates onto the gregorian calendar.
There isn't any code in kholidays that I can see where this might happen.

for now the best I can do is restore the Jewish holidays plugin for KOrganizer,
which is done in the master branch
--
You are receiving this mail because:
You are watching all bug changes.
Shimi
2018-12-09 20:56:47 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #11 from Shimi <***@shimi.net> ---
(In reply to Allen Winter from comment #10)
Post by Allen Winter
I don't know how to rotate hebrew calendar dates onto the gregorian
calendar. There isn't any code in kholidays that I can see where this might
happen.
Maybe
https://api.kde.org/4.12-api/kdepim-apidocs/korganizer/html/classHebrewDate.html
and
https://api.kde.org/4.12-api/kdepim-apidocs/korganizer/html/holiday_8cpp_source.html
?
--
You are receiving this mail because:
You are watching all bug changes.
Allen Winter
2018-12-09 20:09:32 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #8 from Allen Winter <***@kde.org> ---
Git commit ac97505f1dc575116460fc10adaa470f70bd8c80 by Allen Winter.
Committed on 09/12/2018 at 20:05.
Pushed by winterz into branch 'master'.

restore Jewish holidays KOrganizer plugin

M +1 -2 korganizer/plugins/CMakeLists.txt
M +2 -0 korganizer/plugins/hebrew/CMakeLists.txt
A +1271 -0 korganizer/plugins/hebrew/qcalendarsystem.cpp [License: LGPL
(v2+)]
A +134 -0 korganizer/plugins/hebrew/qcalendarsystem_p.h [License: LGPL
(v2+)]

https://commits.kde.org/kdepim-addons/ac97505f1dc575116460fc10adaa470f70bd8c80
--
You are receiving this mail because:
You are watching all bug changes.
Allen Winter
2018-12-09 20:14:16 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #9 from Allen Winter <***@kde.org> ---
Git commit f4af74a7847a49e4acd506b2a685b666c00ea68a by Allen Winter.
Committed on 09/12/2018 at 20:12.
Pushed by winterz into branch 'master'.

plugins/hebrew - restore Jewish holidays KOrganizer plugin
CMakeList.txt - find KDELibs4Support, needed for Hebrew plugin

M +1 -0 CMakeLists.txt
M +1 -2 korganizer/plugins/CMakeLists.txt
M +2 -0 korganizer/plugins/hebrew/CMakeLists.txt
A +1271 -0 korganizer/plugins/hebrew/qcalendarsystem.cpp [License: LGPL
(v2+)]
A +134 -0 korganizer/plugins/hebrew/qcalendarsystem_p.h [License: LGPL
(v2+)]

https://commits.kde.org/kdepim-addons/f4af74a7847a49e4acd506b2a685b666c00ea68a
--
You are receiving this mail because:
You are watching all bug changes.
Allen Winter
2018-12-10 13:44:28 UTC
Permalink
https://bugs.kde.org/show_bug.cgi?id=383896

--- Comment #12 from Allen Winter <***@kde.org> ---
Git commit cae534ca44d2ed501c1fce8bdf3243ccf499fbdf by Allen Winter.
Committed on 10/12/2018 at 13:44.
Pushed by winterz into branch 'master'.

CMakeLists.txt - revert restoring the Hebrew plugin

not allowed in kdepim to use kde4support library

M +2 -1 korganizer/plugins/CMakeLists.txt

https://commits.kde.org/kdepim-addons/cae534ca44d2ed501c1fce8bdf3243ccf499fbdf
--
You are receiving this mail because:
You are watching all bug changes.
Loading...