Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 513583 - [Embedded Editor] Can not set a new Papyrus embedded editor for Port as the default editor
Summary: [Embedded Editor] Can not set a new Papyrus embedded editor for Port as the d...
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Others (show other bugs)
Version: 2.0.1   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: smaoui asma CLA
QA Contact:
URL:
Whiteboard: blockingpapyrusrt
Keywords:
Depends on:
Blocks: 513695
  Show dependency tree
 
Reported: 2017-03-13 12:33 EDT by smaoui asma CLA
Modified: 2017-06-19 06:54 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description smaoui asma CLA 2017-03-13 12:33:54 EDT
In Papyrus rt, we added a new default editor for transition and it works (because the xtext one have no priority) I plan to do the same for Port but it did not work because the xtext one have a priority set to medium and even with high and highest priority , the contributed configuration is not taken into account

I think that typo in org.eclipse.papyrus.uml.textedit.transition.xtext.ui:

 <extension
         point="org.eclipse.papyrus.extensionpoints.editors.DirectEditor">
      <DirectEditor
            language="Advanced transition textual editor"
            objectToEdit="org.eclipse.uml2.uml.Transition">
         <popupeditor
               editorConfiguration="org.eclipse.papyrus.uml.textedit.transition.xtext.ui.contributions.TransitionEditorConfigurationContribution">
         </popupeditor>
      </DirectEditor>
         <Priority
               name="Medium">
         </Priority>      
   </extension>

: the  <Priority
               name="Medium">
         </Priority>   
should be set to the direct editor and not outside the direct editoir balise.


should be corrected and see why we can not contribute a new configuration and set it as the default one when an xtext configuration is already in place.
Comment 1 Eclipse Genie CLA 2017-04-12 08:15:27 EDT
New Gerrit change created: https://git.eclipse.org/r/94914
Comment 2 Eclipse Genie CLA 2017-04-12 08:23:45 EDT
New Gerrit change created: https://git.eclipse.org/r/94919
Comment 3 Eclipse Genie CLA 2017-04-24 10:15:50 EDT
Gerrit change https://git.eclipse.org/r/94919 was merged to [streams/2.0-maintenance].
Commit: http://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=3a7f1d7e23febb8043083a9b59bad3765dd2e1f8
Comment 4 Eclipse Genie CLA 2017-05-04 07:57:14 EDT
New Gerrit change created: https://git.eclipse.org/r/96391