This is an automatically generated e-mail. To reply, visit:

Review request for Plasma and Aleix Pol Gonzalez.

Repository: plasma-framework


When appending a list to an existing list using append or operator<< each item 
in the list is added individually whereas with initializer list it just adds 
the entire list as one entry.

This caused the applet args to never be empty usually and resulted in 
externalData being called in init() with a QVariantList as argument.

While this didn't do much for most applets it did break FolderView (and 
potentially others) which updates its URL in response to this signal and stored 
the "data" it got in plasmoid.configuration.url. FolderView's model did not 
accept this broken data [1] so the folder would still work initially. On next 
startup, however, it loaded said value from config and just went blank.

 Unable to assign QJSValue to QString"


  src/plasma/pluginloader.cpp 15e0ead 

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


I created a FolderView applet, restarted plasmashell twice, FolderView still 
there and also in subsequent startups both on desktop and in panel.


Kai Uwe Broulik

Reply via email to