[Differential] [Commented On] D2283: patch from https://bugs.kde.org/show_bug.cgi?id=365989

[ivan (Ivan Čukić)]

ivan added a comment.


  @abondrov
  
  Have you applied any patch other than this one to plasma? I'm asking since neither Marco nor I can replicate this on our systems (with empty-config users)
  
  Another question, do you have a repository of what changes your distro is doing during packaging - maybe we are missing something?
  
  What is really confusing here is where is the containment assigned to a non-existent activity coming from - 
  it seems to always have the "c51bdf4c-5df8-4b04-9cf8-88a446ad2445" id which, is a UUID which should be random. Since it is not, it is getting it from somewhere else. Some config file or something.
  
  Is there a systemwide plasma*rc?

INLINE COMMENTS

> shellcorona.cpp:346
>      if (containments().isEmpty()) {
> +        currentActivityChanged(m_activityController->currentActivity());
>          loadDefaultLayout();

One thing I realized is that this one is useless - plasma will probably never get into this branch since loadLayout cals (virtually) ShellCorona::loadDefaultLayout().

REVISION DETAIL
  https://phabricator.kde.org/D2283

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, #plasma
Cc: ivan, abondrov, davidedmundson, plasma-devel, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160725/00e3b8a0/attachment.html>