initialize db_nav context menu in trigger-hooks

to ensure exension is fully loaded when context menu is created

Author: PhilippSalvisberg

sqldev/extension.xml

@@ -14,6 +14,13 @@
 	</feature>
 
 	<trigger-hooks xmlns="http://xmlns.oracle.com/ide/extension">
+		<rules>
+			<rule-type class="org.utplsql.sqldev.menu.UtplsqlNavigatorRule"
+				id="utplsql-navigator-rule">
+				<supported-parameters />
+			</rule-type>
+			<rule id="dbnode-is-utplsql-ready" type="utplsql-navigator-rule" />
+		</rules>
 		<triggers>
 			<actions xmlns="http://xmlns.oracle.com/jdeveloper/1013/extension">
 				<action id="utplsql.test">
@@ -28,7 +35,7 @@
 				<controller class="org.utplsql.sqldev.menu.UtplsqlController">
 					<update-rules>
 						<update-rule rule="always-enabled">
-							<action id="utplsql.test"/>
+							<action id="utplsql.test" />
 						</update-rule>
 					</update-rules>
 				</controller>
@@ -38,7 +45,16 @@
 				<menu>
 					<section xmlns="http://jcp.org/jsr/198/extension-manifest"
 						id="UTPLSQL_MENU" weight="2.0">
-						<item action-ref="utplsql.test" weight="1.0"/>
+						<item action-ref="utplsql.test" weight="1.0" />
+					</section>
+				</menu>
+			</context-menu-hook>
+			<context-menu-hook rule="dbnode-is-utplsql-ready">
+				<site idref="db_nav" />
+				<menu>
+					<section xmlns="http://jcp.org/jsr/198/extension-manifest"
+						id="UTPLSQL_MENU" weight="2.0">
+						<item action-ref="utplsql.test" weight="12.1" />
 					</section>
 				</menu>
 			</context-menu-hook>
@@ -54,13 +70,4 @@
 			</settings-ui-hook>
 		</triggers>
 	</trigger-hooks>
-	<hooks xmlns:c="http://xmlns.oracle.com/ide/customization">
-		<jdeveloper-hook xmlns="http://xmlns.oracle.com/jdeveloper/1013/extension">
-			<context-menu-listeners>
-				<site idref="db_nav">
-					<listener-class>org.utplsql.sqldev.menu.UtplsqlContextMenuListener</listener-class>
-				</site>
-			</context-menu-listeners>
-		</jdeveloper-hook>	
- 	</hooks>
 </extension>

sqldev/src/main/java/org/utplsql/sqldev/menu/UtplsqlContextMenuListener.xtend renamed to sqldev/src/main/java/org/utplsql/sqldev/menu/UtplsqlNavigatorRule.xtend

@@ -14,51 +14,42 @@
  */
 package org.utplsql.sqldev.menu
 
+import java.util.Map
 import java.util.logging.Logger
 import oracle.dbtools.raptor.navigator.db.DatabaseConnection
 import oracle.dbtools.raptor.navigator.impl.ChildObjectElement
 import oracle.dbtools.raptor.navigator.impl.ObjectFolder
 import oracle.dbtools.raptor.navigator.plsql.PlSqlNode
-import oracle.ide.Context
-import oracle.ide.controller.ContextMenu
-import oracle.ide.controller.ContextMenuListener
+import oracle.ide.^extension.rules.RuleEvaluationContext
+import oracle.ide.^extension.rules.RuleEvaluationException
+import oracle.ide.^extension.rules.RuleFunction
+import oracle.ide.^extension.rules.RuleFunctionParameter
 import org.utplsql.sqldev.model.URLTools
 
-class UtplsqlContextMenuListener implements ContextMenuListener {
-	private static final Logger logger = Logger.getLogger(UtplsqlContextMenuListener.name);
+class UtplsqlNavigatorRule extends RuleFunction {
+	private static final Logger logger = Logger.getLogger(UtplsqlNavigatorRule.name);
 	private val extension URLTools urlTools = new URLTools
 
-
-	override handleDefaultAction(Context context) {
-		return false
-	}
-	
-	override menuWillHide(ContextMenu contextMenu) {
-	}
-	
-	override menuWillShow(ContextMenu contextMenu) {
-		val element = contextMenu.context.selection.get(0)
-		var boolean showMenu = false
+	override evaluate(RuleEvaluationContext ruleContext,
+		Map<String, RuleFunctionParameter> parameters) throws RuleEvaluationException {
+		val element = ruleContext.ideContext.selection.get(0)
+		var boolean enable = false
 		logger.fine('''selected object is of type «element.class.name»''')
 		if (element instanceof DatabaseConnection) {
-			showMenu = true
+			enable = true
 		} else if (element instanceof ObjectFolder) {
 			if (element.objectType == "PACKAGE") {
-				showMenu = true
+				enable = true
 			}
 		} else if (element instanceof PlSqlNode) {
 			if (element.objectType == "PACKAGE" || element.objectType == "PACKAGE BODY") {
-				showMenu = true
+				enable = true
 			}
 		} else if (element instanceof ChildObjectElement) {
 			if (element.URL.objectType == "PACKAGE") {
-				showMenu = true
+				enable = true
 			}
 		}
-		if (showMenu) {
-			val menuItem = contextMenu.createMenuItem(UtplsqlController.UTLPLSQL_TEST_ACTION, 1.0f)
-			contextMenu.add(menuItem, 12.1f)
-			logger.finer("context menu created.")
-		}
+		return enable
 	}
-}
+}