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

[Marta Rybczyńska]

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

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/20110813/f3640d8f/attachment.html>