After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 662598 - Store server folder cache in user's cache directory
Store server folder cache in user's cache directory
Status: RESOLVED FIXED
Product: evolution-ews
Classification: Other
Component: Mail
3.3.x
Other Linux
: Normal normal
: ---
Assigned To: Evolution EWS maintainer(s)
Evolution EWS maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2011-10-24 11:59 UTC by Milan Crha
Modified: 2011-10-25 08:04 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed ews patch (3.05 KB, patch)
2011-10-24 11:59 UTC, Milan Crha 		needs-work Details | Review
proposed ews patch ][ (4.29 KB, patch)
2011-10-24 12:41 UTC, Milan Crha 		committed Details | Review

Description Milan Crha 2011-10-24 11:59:07 UTC 
Created attachment 199811 [details] [review]
proposed ews patch

As was done within bug #656473 for other mail providers, the ews local folder cache should be stored in user's cache directory, rather than in their data dir, thus the cache will not be part of evolution backup file. This is a new material for 3.3.1+ only.

Patch implementing the functionality attached.
Comment 1 David Woodhouse 2011-10-24 12:05:26 UTC 
Review of attachment 199811 [details] [review]:

Looks sane; thanks. I would like to make one change though: I'd like to take the opportunity to rename the 'folder-tree-v2' file back to 'folder-tree'.
Comment 2 Milan Crha 2011-10-24 12:41:31 UTC 
Created attachment 199813 [details] [review]
proposed ews patch ][

for evolution-ews;

Say something like this?
Comment 3 David Woodhouse 2011-10-24 13:07:19 UTC 
Review of attachment 199813 [details] [review]:

Looks perfect; thanks. Haven't tested it, of course, but if you have then please go ahead and commit.

Thanks.
Comment 4 Milan Crha 2011-10-25 08:03:54 UTC 
Yup, I tested it.

Created commit ea37425 in ews master (3.3.2+)