Review Request 120284: sanity checking in desktops/activities scripting

Aaron J. Seigo aseigo at kde.org
Fri Sep 19 12:37:38 UTC 2014


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

Review request for Plasma.


Repository: plasma-workspace


Description
-------

This patch set

* checks that there are desktops for a given activity; happens when activities exist and a new shell config is being made
* checks that an activity actually exists before handing out containments for it
* allow fetching the current activity


Diffs
-----

  shell/scripting/scriptengine.h 2ea0e3170e2388780ea36eefdf58caf64afbb0cb 
  shell/scripting/scriptengine.cpp f458becb5df33116a6251ba3876aa6e06d16e2bf 
  shell/shellcorona.h 84919ff4ca22b9e481aa5719e28a664da0f2f9d1 

Diff: https://git.reviewboard.kde.org/r/120284/diff/


Testing
-------

Ran plasmashell and a layout.js that was failing due to the above issues.


Thanks,

Aaron J. Seigo

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140919/2bcdefba/attachment.html>


More information about the Plasma-devel mailing list