Review Request: create only the required files during the load of the project

Giorgos Tsiapaliwkas terietor at gmail.com
Mon Oct 1 18:44:07 UTC 2012



> On Oct. 1, 2012, 10:34 a.m., Aaron J. Seigo wrote:
> > i think this change is necessary and can go in. however, the model MUST then be fixed to at least show all non-created files described in the package as well otherwise things like creating new images or populating the default settings will be hard/impossible for the user and making that easy is plasmate's #1 purpose in life. :)

> the model MUST then be fixed to at least show all non-created files described in the package

It will be fixed by then :)


- Giorgos


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106665/#review19676
-----------------------------------------------------------


On Oct. 1, 2012, 8:46 a.m., Giorgos Tsiapaliwkas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106665/
> -----------------------------------------------------------
> 
> (Updated Oct. 1, 2012, 8:46 a.m.)
> 
> 
> 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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121001/1e9e5f96/attachment.html>


More information about the Plasma-devel mailing list