----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106665/ -----------------------------------------------------------
Review request for Plasma. Description ------- Until now plasmate was creating all the Package::files() on the load of the project, with this patch it will create only the Package::requiredFiles(). Also we **have** to s/Package::files()/Package::requiredFiles(), because right now 1. the Plasma themes are broken: plasmate creates all the listing files of the theme package with QFile,so QSvgRenderer can't open them, and if the user adds one more foo.svgz file, he will end up with 2 foo.svg files and even if he deletes the corrupted one it will be recreated on the next load 2. plasmate forces the user to have a main.xml and a default-configrc, also even if the user deletes those files, on the reload they will be recreated. I haven't s/Package::dirs()/Package::RequiredDirs() because we don't have any ui element in order to inform the user that the foo dir(for examples the images or the animations dir) exists, so he will have to figure it out and create it manually and even if he does the dirs won't appear in the plasmate's package model. The only drawback of this patch is that the user won't see anymore the description of the file but he will see their names. I believe that for the moment we should add this patch and to fix the PackageModel issues after the beta. P.S.: I replaced some deprecated methods. Diffs ----- packagemodel.cpp d9cc35e Diff: http://git.reviewboard.kde.org/r/106665/diff/ Testing ------- Thanks, Giorgos Tsiapaliwkas
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
