Review Request: Allow to know if the machine have batteries during plasma init ( scripting )

Sebastian Kügler sebas at kde.org
Wed Jan 27 16:29:32 CET 2010


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



/trunk/KDE/kdebase/workspace/plasma/desktop/shell/scripting/scriptengine.cpp
<http://reviewboard.kde.org/r/2741/#comment3277>

    Should probably is "hasBattery()"


- Sebastian


On 2010-01-27 14:40:36, Nicolas Lécureuil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2741/
> -----------------------------------------------------------
> 
> (Updated 2010-01-27 14:40:36)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> When creating scripts for plasma first init there is for the moment no way to add the battery widget only if the machine have batteries. this patch try to work on this issue.
> 
> With this function , haveBattery() return true if the machine have some.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/desktop/shell/scripting/scriptengine.h 1081032 
>   /trunk/KDE/kdebase/workspace/plasma/desktop/shell/scripting/scriptengine.cpp 1081032 
> 
> Diff: http://reviewboard.kde.org/r/2741/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nicolas
> 
>



More information about the Plasma-devel mailing list