[Kstars-devel] Review Request: Daylight Savings Time saving correction

[Akarsh Simha]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102316/#review6013
-----------------------------------------------------------

Ship it!


Sorry for not taking a look at this earlier, and thanks for poking us. Looks good. Ship it. (Do you have commit access? Else, I can sign it for you)

- Akarsh


On Aug. 13, 2011, 6:53 p.m., Marta Rybczyńska wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102316/
> -----------------------------------------------------------
> 
> (Updated Aug. 13, 2011, 6:53 p.m.)
> 
> 
> Review request for KStars.
> 
> 
> Summary
> -------
> 
> This patch fixes an old bug. It turns out that DST (Daylight Savings Time) setting was not saved and that resulted with no DST correction (fallback "--" DST rule when all other location information is correct) when the program was run again. The workaround was to enter Settings->Geographic and click OK.
> 
> The patch adds change of the DST field in Option when the location is changed, that leads to the new value to be saved.
> 
> In the case of a configuration without DST saved, the configuration will be fixed next time the location is changed, also when the user follows the existing workaround.
> 
> Implementation explanation:
> The patch uses a loop, because at this point we do not have the rule name, which is to be saved. Further work will be to keep the rule name in the TimeZoneRule class and refactor this code and other similar places accordingly.
> 
> 
> This addresses bug 209647.
>     http://bugs.kde.org/show_bug.cgi?id=209647
> 
> 
> Diffs
> -----
> 
>   kstars/kstarsdata.cpp 2303c5f 
> 
> Diff: http://git.reviewboard.kde.org/r/102316/diff
> 
> 
> Testing
> -------
> 
> Tested with a config file with and without DST settings, multiple location changes etc. Works fine.
> 
> 
> Thanks,
> 
> Marta
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kstars-devel/attachments/20110825/e441c632/attachment.html>