Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 497224

Summary:	IDomain.execute(ITransactionalOperation) should accept IProgressMonitor and pass it on.
Product:	[Tools] GEF	Reporter:	Alexander Nyßen <nyssen>
Component:	GEF MVC	Assignee:	Alexander Nyßen <nyssen>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	1.0.0
Target Milestone:	4.1.0 (Neon.1) M1
Hardware:	All
OS:	All
Whiteboard:

Description Alexander Nyßen

2016-07-04 05:55:12 EDT

Currently, AbstractDomain.execute(ITransactionalOperation) creates a NullProgressMonitor, which it passes on to the execution of the operations. We should add a second parameter to the execute method within IDomain and allow to pass in an IProgressMonitor. The current signature can be preserved, delegating to the new method with a NullProgressMonitor.

Comment 1 Alexander Nyßen

2016-07-12 12:50:23 EDT

Pushed the following changes to origin/master and origin/R4_0_maintenance:

- Added IDomain.execute(ITransactionalOperation, IProgressMonitor) to complement IDomain.execute(ITransactionalOperation).
- Moved implementation of AbstractDomain.execute(ITransactionalOperation) over to new execute method, and have it delegate to new method instead, passing over a NullProgressMonitor.
- Deprecated execute(ITransactionalOperation), which should no longer be used.
- Increased version of MVC and related doc, tests, example, and sdk bundles and features to 1.1.0.

Resolving as fixed in 4.1.0 (Neon.1) M1.