Review Request: Fix Plasma clocks not being aware of timezone changes until next plasma-desktop restart, alt patch

[Jacopo De Simoi]

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


Much better than mine!
It would be excellent if we could trigger an update of all the containers attached but I just can't find the way to do it (is it possible?)
Right now it waits for next trigger to update all clocks, which can be in fact be quite confusing, as the user can have to wait 1 minute to have it set.

- Jacopo


On 2009-06-25 16:34:27, Aaron Seigo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/891/
> -----------------------------------------------------------
> 
> (Updated 2009-06-25 16:34:27)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Alternate patch to http://reviewboard.kde.org/r/889/ 
> 
> from 889: "The plasma timeengine reacts to timezone changes, but the timesource does not; this patch filles the gap."
> 
> the benefit of this patch is that the change is done in one place, no synchronization issues based on who gets a signal first. (i wonder if it needs to update all the timezones, even, or just the local one? not that updateAllSources is slow, so it shouldn't matter..)
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/plasma/dataengines/time/timeengine.h 980060 
>   trunk/KDE/kdebase/workspace/plasma/dataengines/time/timeengine.cpp 980060 
>   trunk/KDE/kdebase/workspace/plasma/dataengines/time/timesource.h 980060 
>   trunk/KDE/kdebase/workspace/plasma/dataengines/time/timesource.cpp 980060 
> 
> Diff: http://reviewboard.kde.org/r/891/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Aaron
> 
>