[jbosstools-issues] [JBoss JIRA] (JBIDE-17650) Find better way to enable Tern cordova module for cordova projects

[Victor Rubezhny (JIRA)]

    [ https://issues.jboss.org/browse/JBIDE-17650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12980499#comment-12980499 ] 

Victor Rubezhny edited comment on JBIDE-17650 at 7/1/14 9:29 PM:
-----------------------------------------------------------------

The project nature-based configuration of Tern project is proposed: https://github.com/angelozerr/tern.java/pull/87 

If accepted, there will be no need to make additional calls to org.jboss.tools.jst.jsdt.utils.ConfigUtils.enableCordovaJSPlugin(project) from o.j.t.aerogear.hybrid.core plug-in. And even to depend on o.j.t.jst.jsdt plug-in directly somehow.
In order to see content assist on 'cordova' object, we'll need just to configure specify 'cordovajs' module to be loaded by default for HMT Nature by specifying it in extension point like:
{code}
<extension point="tern.eclipse.ide.core.ternNatureAdapters">
      <ternAdaptToNature
            id = "org.jboss.tools.aerogear.hybrid.core.HybridAppNature"
            name = "HybridAppNature">
      	 <defaultModules>
      	 	<module name = "cordovajs" />
      	 </defaultModules>
      </ternAdaptToNature>
</extension>
{code}

The change is made to jbosstools-aerogear project: https://github.com/vrubezhny/jbosstools-aerogear/tree/jbide17650 - No PR is made 'cause there are no according fix applied into Tern.java project and we've no included according Tern.java build into our TP.

PS: This comment and the branch are updated in order to match the latest changes in the PR for Tern.java project.


was (Author: vrubezhny):
The project nature-based configuration of Tern project is proposed: https://github.com/angelozerr/tern.java/pull/87 

If accepted, there will be no need to make additional calls to org.jboss.tools.jst.jsdt.utils.ConfigUtils.enableCordovaJSPlugin(project) from o.j.t.aerogear.hybrid.core plug-in. And even to depend on o.j.t.jst.jsdt plug-in directly somehow.
In order to see content assist on 'cordova' object, we'll need just to configure specify 'cordovajs' module to be loaded by default for HMT Nature by specifying it in extension point like:
{code}
<extension point="tern.eclipse.ide.core.ternNatureAdapters">
	<ternAdaptToNature
		id = "org.jboss.tools.aerogear.hybrid.core.HybridAppNature"
		name = "HybridAppNature"
		defaultModules = "cordovajs" />
</extension>
{code}

The change is made to jbosstools-aerogear project: https://github.com/vrubezhny/jbosstools-aerogear/tree/jbide17650 - No PR is made 'cause there are no according fix applied into Tern.java project and we've no included according Tern.java build into our TP.

> Find better way to enable Tern cordova module for cordova projects
> ------------------------------------------------------------------
>
>                 Key: JBIDE-17650
>                 URL: https://issues.jboss.org/browse/JBIDE-17650
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: aerogear-hybrid
>    Affects Versions: 4.2.0.Beta2
>            Reporter: Denis Golovin
>            Assignee: Victor Rubezhny
>             Fix For: 4.2.0.Beta3
>
>




--
This message was sent by Atlassian JIRA
(v6.2.6#6264)