Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328346 - [Model Browser] warning: recursive attempt to activate part PropertySheet
Summary: [Model Browser] warning: recursive attempt to activate part PropertySheet
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Infrastructure (show other bugs)
Version: 0.9.0   Edit
Hardware: PC Windows 7
: P2 normal (vote)
Target Milestone: 0.9.0 M4   Edit
Assignee: Fabien Giquel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-21 09:27 EDT by Nicolas Bros CLA
Modified: 2011-01-11 05:06 EST (History)
4 users (show)

See Also:
fabien.giquel: indigo+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Bros CLA 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 CLA 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 CLA 2011-01-11 05:06:01 EST 
Bug fixed.