Review Request 125648: Add method to run a script from DBus directly

[David Edmundson]

> On Oct. 19, 2015, 10:43 a.m., Marco Martin wrote:
> > is true that for a shell script would already be possible to poke with the config file..
> > but wouldn't this be a bit too much of a security hole/open invitation for anybody, script or app to poke in the desktop configuration without the user being aware of it?

How is this any different to the look and feel scripting you've literally just pushed?


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125648/#review87061
-----------------------------------------------------------


On Oct. 15, 2015, 5:22 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125648/
> -----------------------------------------------------------
> 
> (Updated Oct. 15, 2015, 5:22 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> loadScriptInInteractiveConsole requires user interaction which blocks
> some purposes, like setting the desktop background from an app.
> 
> This runs a given script directly from a passed string without any prompts.
> 
> 
> CCBUG: 217950
> 
> ---
> 
> Questions:
> 
> I also toyed with returning the value of a script as a QDBusVariant, thoughts?
> 
> Should I not run if widgets are locked?
> 
> 
> Diffs
> -----
> 
>   shell/dbus/org.kde.PlasmaShell.xml cf2d0bbfcdb5720bf19d42403ee8c7fd542adecd 
>   shell/shellcorona.h 0e6fe041787b2f75d791b38661b63614e27d33bf 
>   shell/shellcorona.cpp 37d05347d67b7178db4d29a1ac368aacaacf0c0a 
> 
> Diff: https://git.reviewboard.kde.org/r/125648/diff/
> 
> 
> Testing
> -------
> 
> Ran this:
> var allDesktops = desktops();
> print (allDesktops);
> 
> for (i=0;i<allDesktops.length;i++) {
>     d = allDesktops[i];
>     d.wallpaperPlugin = "org.kde.image";
>     d.currentConfigGroup = Array("Wallpaper", "org.kde.image", "General");
>     d.writeConfig("Image", "file:///home/david/pictures/Wallpapers/northern_lights-wide.jpg")
> }
> 
> 
> also ran some with errors, that behaved as expected too.
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151019/65a4b1af/attachment.html>