From 84bd50994a8d462c94d1db4852865665adbd4482 Mon Sep 17 00:00:00 2001
From: anna <Anna.Kozlova@jetbrains.com>
Date: Wed, 27 Feb 2013 18:16:55 +0100
Subject: [PATCH] javafx: accept no params handlers (IDEA-102017)

---
 .../plugins/javaFX/fxml/JavaFXHighlightingTest.java          | 5 +++++
 .../javaFX/fxml/refs/JavaFxEventHandlerReference.java        | 1 +
 plugins/javaFX/testData/highlighting/NoParamsHandler.java    | 4 ++++
 plugins/javaFX/testData/highlighting/noParamsHandler.fxml    | 5 +++++
 4 files changed, 15 insertions(+)
 create mode 100644 plugins/javaFX/testData/highlighting/NoParamsHandler.java
 create mode 100644 plugins/javaFX/testData/highlighting/noParamsHandler.fxml

diff --git a/plugins/javaFX/javaFX-CE/testSrc/org/jetbrains/plugins/javaFX/fxml/JavaFXHighlightingTest.java b/plugins/javaFX/javaFX-CE/testSrc/org/jetbrains/plugins/javaFX/fxml/JavaFXHighlightingTest.java
index f66c68429f01..f6a1a408a01c 100644
--- a/plugins/javaFX/javaFX-CE/testSrc/org/jetbrains/plugins/javaFX/fxml/JavaFXHighlightingTest.java
+++ b/plugins/javaFX/javaFX-CE/testSrc/org/jetbrains/plugins/javaFX/fxml/JavaFXHighlightingTest.java
@@ -93,6 +93,11 @@ public class JavaFXHighlightingTest extends DaemonAnalyzerTestCase {
     doDoTest(false, false);
   }
 
+  public void testNoParamsHandler() throws Exception {
+    configureByFiles(null, getTestName(true) + ".fxml", getTestName(false) + ".java");
+    doDoTest(false, false);
+  }
+
   private void doTestIdController() throws Exception {
     final String controllerClassName = getTestName(false) + "Controller";
     configureByFiles(null, getTestName(true) + ".fxml", controllerClassName + ".java");
diff --git a/plugins/javaFX/src/org/jetbrains/plugins/javaFX/fxml/refs/JavaFxEventHandlerReference.java b/plugins/javaFX/src/org/jetbrains/plugins/javaFX/fxml/refs/JavaFxEventHandlerReference.java
index ed34c51a9cb5..432e3eeabe25 100644
--- a/plugins/javaFX/src/org/jetbrains/plugins/javaFX/fxml/refs/JavaFxEventHandlerReference.java
+++ b/plugins/javaFX/src/org/jetbrains/plugins/javaFX/fxml/refs/JavaFxEventHandlerReference.java
@@ -79,6 +79,7 @@ public class JavaFxEventHandlerReference extends PsiReferenceBase<XmlAttributeVa
           return true;
         }
       }
+      return parameters.length == 0;
     }
     return false;
   }
diff --git a/plugins/javaFX/testData/highlighting/NoParamsHandler.java b/plugins/javaFX/testData/highlighting/NoParamsHandler.java
new file mode 100644
index 000000000000..c96768e76861
--- /dev/null
+++ b/plugins/javaFX/testData/highlighting/NoParamsHandler.java
@@ -0,0 +1,4 @@
+public class Controller {
+  public void sayHelloWorld() {
+  }
+}
\ No newline at end of file
diff --git a/plugins/javaFX/testData/highlighting/noParamsHandler.fxml b/plugins/javaFX/testData/highlighting/noParamsHandler.fxml
new file mode 100644
index 000000000000..a4fc85b8606b
--- /dev/null
+++ b/plugins/javaFX/testData/highlighting/noParamsHandler.fxml
@@ -0,0 +1,5 @@
+<?import javafx.scene.control.*?>
+<?import javafx.scene.layout.GridPane?>
+<GridPane xmlns:fx="http://javafx.com/fxml" fx:controller="Controller">
+    <Button onAction="#sayHelloWorld"/>
+</GridPane>
\ No newline at end of file