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

[Wayne Speir]

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



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

    Per Sebastian:
    
    Shouldn't be necessary:
    
    "On failure, the fallback NullEngine (which does nothing and !isValid()) is
    returned."


- Wayne


On 2010-01-27 15:44:17, Nicolas Lécureuil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2741/
> -----------------------------------------------------------
> 
> (Updated 2010-01-27 15:44:17)
> 
> 
> 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
> 
>