IMAP list/rename commands
Needs ReviewPublic
Actions

Authored by mollekopf on Thu, Oct 10, 10:13 AM.

Tags

None

Referenced Files

	F16222962: D4989.diff
	Fri, Oct 18, 1:24 PM

	F16198227: D4989.id14286.diff
	Thu, Oct 17, 6:56 PM

	Unknown Object (File)
	Thu, Oct 17, 12:08 PM

	Unknown Object (File)
	Thu, Oct 17, 3:30 AM

	Unknown Object (File)
	Wed, Oct 16, 1:55 PM

	Unknown Object (File)
	Wed, Oct 16, 4:31 AM

	Unknown Object (File)
	Wed, Oct 16, 4:31 AM

	Unknown Object (File)
	Wed, Oct 16, 4:13 AM

View All 10 Files

Subscribers

machniak

Details

Reviewers

None

Group Reviewers

Restricted Project

Diff Detail

Repository

rK kolab

Lint

Lint Skipped

Unit

No Test Coverage

Build Status

Buildable 50745
Build 18468: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Thu, Oct 10, 10:13 AM

mollekopf created this revision.

Harbormaster completed remote builds in B50745: Diff 14286.Thu, Oct 10, 10:13 AM

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

Looks to me that the folders passed to the command need to be in Cyrus internal format user/{$localpart}/{$mailbox}@{$domain}. That's not very user-friendly. Also tests for the new IMAP methods would be welcome, less important for the command itself.