Summary: | Non-processing days no longer work for certain regions | ||
---|---|---|---|
Product: | [Applications] kmymoney | Reporter: | Ian Neal <iann_bugzilla> |
Component: | general | Assignee: | Ian Neal <iann_bugzilla> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | agander93, djhoulden |
Priority: | NOR | ||
Version: | git (master) | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: | Fix holiday region for KDE 4.5.0+ |
Description
Ian Neal
2013-04-01 13:41:07 UTC
Created attachment 78543 [details]
Fix holiday region for KDE 4.5.0+
This patch:
* Makes use of new HolidayRegion API to generate list of regions with languages for holiday region selector in schedule settings pane.
What this patch doesn't do:
* Check that current setting is a valid region.
Just to mention that I'm on KDE 4.6.5 with KMyMoney holiday calendar set to United Kingdom and my schedules for 1st April have moved to 2nd April as I expect. I'm not sure why but I don't see the problem described by Ian. (In reply to comment #2) > Just to mention that I'm on KDE 4.6.5 with KMyMoney holiday calendar set to > United Kingdom and my schedules for 1st April have moved to 2nd April as I > expect. I'm not sure why but I don't see the problem described by Ian. I'm on KDE 4.10.1, presumably at some point between 4.5 and 4.10 backwards compatibility was removed. (In reply to comment #1) > Created attachment 78543 [details] > Fix holiday region for KDE 4.5.0+ > > This patch: > * Makes use of new HolidayRegion API to generate list of regions with > languages for holiday region selector in schedule settings pane. > > What this patch doesn't do: > * Check that current setting is a valid region. Another option would be to use the new HolidayRegionSelector widget. (In reply to comment #2) > Just to mention that I'm on KDE 4.6.5 with KMyMoney holiday calendar set to > United Kingdom and my schedules for 1st April have moved to 2nd April as I > expect. I'm not sure why but I don't see the problem described by Ian. I'm on KDE 4.9.5 and, like Dave, get the correct results, without the patch. I've not taken particular note in the past of this feature's working. (In reply to comment #5) > (In reply to comment #2) > > Just to mention that I'm on KDE 4.6.5 with KMyMoney holiday calendar set to > > United Kingdom and my schedules for 1st April have moved to 2nd April as I > > expect. I'm not sure why but I don't see the problem described by Ian. > > I'm on KDE 4.9.5 and, like Dave, get the correct results, without the patch. > I've not taken particular note in the past of this feature's working. Well for KDE 4.10.1 upwards running on Fedora 17/18 x64, it definitely is required. (In reply to comment #6) > (In reply to comment #5) > > (In reply to comment #2) > > > Just to mention that I'm on KDE 4.6.5 with KMyMoney holiday calendar set to > > > United Kingdom and my schedules for 1st April have moved to 2nd April as I > > > expect. I'm not sure why but I don't see the problem described by Ian. > > > > I'm on KDE 4.9.5 and, like Dave, get the correct results, without the patch. > > I've not taken particular note in the past of this feature's working. > > Well for KDE 4.10.1 upwards running on Fedora 17/18 x64, it definitely is > required. Not doubting you, Ian, just responding to your comment on Reviewboard about checking on other revs. Compiled ok and tested ok on both KDE 4.6.5 and 4.8.5 which are the two versions I have access to. I don't have an id for reviewboard so haven't comments updated there. commit 795dfb4a9e158ea1ed2fef59d48c25f341c65a90 |