Page MenuHomePhorge

Required changes for EWS import, including email
AcceptedPublic

Authored by mollekopf on Thu, Oct 10, 10:17 AM.
Tags
None
Referenced Files
F16200930: D4995.id14292.diff
Thu, Oct 17, 8:48 PM
Unknown Object (File)
Thu, Oct 17, 3:29 AM
Unknown Object (File)
Wed, Oct 16, 4:29 AM
Unknown Object (File)
Wed, Oct 16, 4:29 AM
Unknown Object (File)
Wed, Oct 16, 4:29 AM
Unknown Object (File)
Wed, Oct 16, 4:09 AM
Unknown Object (File)
Wed, Oct 16, 12:22 AM
Unknown Object (File)
Tue, Oct 15, 5:19 AM
Subscribers

Details

Reviewers
machniak
Group Reviewers
Restricted Project
Summary

EWS: Skip "Aufgabensuche"

--folder-mapping support for migrate command

migrator Further folders to skip / search depth=infinity for iRony compat

Email migration from EWS to IMAP

Subscribe to new folders

Add a filename to calendar attachments

After creating a folder there may be a delay until the folder becomes available.

EWS: Skip "Aufgabensuche"

--folder-mapping support for migrate command

migrator Further folders to skip / search depth=infinity for iRony compat

Email migration from EWS to IMAP

Subscribe to new folders

Add a filename to calendar attachments

Handle an odd case where we get ATTACH parts in ORGANIZER

Diff Detail

Repository
rK kolab
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 50751
Build 18474: arc lint + arc unit

Event Timeline

mollekopf created this revision.
mollekopf edited the summary of this revision. (Show Details)

Cleanup

mollekopf added a reviewer: Restricted Project.Thu, Oct 10, 10:29 AM

This diff consists of multiple commits that I've used for the migration. Since we're already using it like this I'd like to mostly just get it merged, but I'm open to small adjustments (the rest we can improve later on IMO).

src/app/DataMigrator/DAV.php
349–350

For new folders it can take a moment for them to appear.

mollekopf retitled this revision from After creating a folder there may be a delay until the folder becomes available. to Required changes for EWS import, including email.Thu, Oct 10, 10:30 AM
machniak subscribed.

It's good enough, but I added some comments.

src/app/DataMigrator/DAV.php
187

Will that not fetch the items from subfolders?

src/app/DataMigrator/EWS/Appointment.php
64

Maybe the preg_replace() above is wrong and is causing this missing end-line issue.

src/app/DataMigrator/Engine.php
119

This is fragile and can catch folders that should not be mapped. We should do proper equality and prefix matching.

This revision is now accepted and ready to land.Thu, Oct 10, 10:55 AM