Completion (#165)

* Add support for method/property completion

* Move completion fixtures into directory

* Add support for variable suggestions

Refactor logic into CompletionProvider class

* Allow getTypeFromNode() to take Variable nodes

* Use property and constant values

* Fix using @var tag for variables

* Improve completion

* classes
* variables with prefix filtering

* Make FQNs more distinct

* use -> for instance methods/properties
* use ::$ for static properties

* Add tests for static access

* Properly filter completion on empty property

* Fix existing tests

* Add support for static access without prefix

* Fix testFullyQualifiedClass

* Add missing fixtures

* Correct file number in init test

* Only insert backslash if not typed yet

* Completion for keywords and bug fixes

* Correct variable insertion

* Support completion for namespaces

* Use CompletionList

* Always set isIncomplete to true

* Update PHPCodeSniffer

* Remove unused method

* And the call

* Handle case where FQN could not be resolved

.editorconfig

@@ -13,5 +13,5 @@ indent_size = 2
 [composer.json]
 indent_size = 4
 
-[*.md]
+[{*.md,fixtures/**}]
 trim_trailing_whitespace = false

composer.json

@@ -24,11 +24,11 @@
     "bin": ["bin/php-language-server.php"],
     "require": {
         "php": ">=7.0",
-        "nikic/php-parser": "^3.0.0beta2",
+        "nikic/php-parser": "dev-master#e52ffc4447e034514339a03b450aab9cd625e37c",
         "phpdocumentor/reflection-docblock": "^3.0",
         "sabre/event": "^5.0",
         "felixfbecker/advanced-json-rpc": "^2.0",
-        "squizlabs/php_codesniffer" : "3.0.0RC1",
+        "squizlabs/php_codesniffer" : "3.0.x-dev#e8acf8e029301b0e3ea7e7c9eef0aee914db78bf",
         "netresearch/jsonmapper": "^1.0",
         "webmozart/path-util": "^2.3",
         "webmozart/glob": "^4.1",

fixtures/completion/class_const_with_prefix.php

@@ -0,0 +1,3 @@
+<?php
+
+TestClass::TE

fixtures/completion/fully_qualified_class.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace Whatever;
+
+use TestNamespace\{TestClass, TestInterface};
+
+\TestC
+
+class OtherClass {}

fixtures/completion/html_with_prefix.php

@@ -0,0 +1 @@
+<

fixtures/completion/keywords.php

@@ -0,0 +1,3 @@
+<?php
+
+cl

fixtures/completion/namespace.php

@@ -0,0 +1,5 @@
+<?php
+
+namespace SomeNamespace {}
+
+SomeNa

fixtures/completion/property.php

@@ -0,0 +1,4 @@
+<?php
+
+$obj = new TestClass;
+$obj->

fixtures/completion/property_with_prefix.php

@@ -0,0 +1,4 @@
+<?php
+
+$obj = new TestClass;
+$obj->t

fixtures/completion/static.php

@@ -0,0 +1,3 @@
+<?php
+
+TestClass::

fixtures/completion/static_method_with_prefix.php

@@ -0,0 +1,3 @@
+<?php
+
+TestClass::st

fixtures/completion/static_property_with_prefix.php

@@ -0,0 +1,3 @@
+<?php
+
+TestClass::$st

fixtures/completion/used_class.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace Whatever;
+
+use TestNamespace\{TestClass, TestInterface};
+
+TestC
+
+class OtherClass {}

fixtures/completion/used_new.php

@@ -0,0 +1,7 @@
+<?php
+
+namespace Whatever;
+
+use TestNamespace\{TestClass, TestInterface};
+
+$obj = new

fixtures/completion/variable.php

@@ -0,0 +1,10 @@
+<?php
+
+/**
+ * @param string|null $param A parameter
+ */
+function test(string $param = null)
+{
+    $var = 123;
+    $
+}

fixtures/completion/variable_with_prefix.php

@@ -0,0 +1,10 @@
+<?php
+
+/**
+ * @param string|null $param A parameter
+ */
+function test(string $param = null)
+{
+    $var = 123;
+    $p
+}