Review Request: Fix special case arising from r954107

Aaron Seigo aseigo at kde.org
Mon Apr 20 17:53:36 CEST 2009


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

Ship it!


- Aaron


On 2009-04-19 23:54:07, Alan Alpert wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/614/
> -----------------------------------------------------------
> 
> (Updated 2009-04-19 23:54:07)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> r954107 seems to have caused the weather dataengine to do nothing if Solid cannot find out the network status. This patch lets the weather dataengine attempt to obtain the data in the event that solid cannot discover the network status.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/plasma/dataengines/weather/weatherengine.cpp 956394 
> 
> Diff: http://reviewboard.kde.org/r/614/diff
> 
> 
> Testing
> -------
> 
> It was confirmed that solid returns a network status of 'Unknown' on my machine. After applying this patch to r955998 of the weather dataengine the weather plasmoids were able to both retrieve weather data and location verification data (when configuring) - neither of which was working without this patch. 
> 
> 
> Thanks,
> 
> Alan
> 
>



More information about the Plasma-devel mailing list