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

Bug 328346

Summary:	[Model Browser] warning: recursive attempt to activate part PropertySheet
Product:	[Modeling] MoDisco	Reporter:	Nicolas Bros <nicolas.bros>
Component:	Infrastructure	Assignee:	Fabien Giquel <fabien.giquel>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P2	CC:	fabien.giquel, gdupe, hugo.bruneliere, modisco.web-inbox
Version:	0.9.0	Flags:	fabien.giquel: indigo+
Target Milestone:	0.9.0 M4
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Nicolas Bros

2010-10-21 09:27:45 EDT

- click on a model element in the browser
- open the properties view
- click an editable field so that the field goes into edit mode
- (don't leave editing mode in-between)
- click on a model element in the model browser

You get:
java.lang.RuntimeException: WARNING: Prevented recursive attempt to activate part org.eclipse.ui.views.PropertySheet while still in the middle of activating part org.eclipse.gmt.modisco.infra.browser.editorID

Comment 1 Fabien Giquel

2010-12-14 11:51:22 EST

Fix the EcoreBrowser.setFocus() method : behavior has been modified (Bug 325120) for initialization step, but old behavior (giving focus to the tree viewer) prevents from the WorkBench.setActivePart() loop, as indicated here : http://www.eclipse.org/forums/index.php?t=tree&th=162976&

commited into head for 0.9 (revision 3640).
No issue on 0.8.1

Comment 2 Hugo Bruneliere

2011-01-11 05:06:01 EST

Bug fixed.