Review Request 120829: Use full URL as filename for cached image files

Christoph Feck christoph at maxiom.de
Mon Oct 27 20:39:47 UTC 2014


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

Review request for Plasma and Anne-Marie Mahfouf.


Bugs: 340408
    http://bugs.kde.org/show_bug.cgi?id=340408


Repository: kdeplasma-addons


Description
-------

Otherwise, two different URLs with the same filename would show up as identical images. prettyUrl() makes sure the password is removed from the string.


Diffs
-----

  applets/frame/picture.cpp 635d845 

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


Testing
-------

Yes. Cache files look like plasma-frame/http:||www.yr.no|place|United_States|New_York|New_York|meteogram.png


Thanks,

Christoph Feck

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


More information about the Plasma-devel mailing list