Fixes felixfbecker#27: Shift end position in SymbolInformation ranges

kaloyan-raev · Sep 13, 2016 · f28690c · f28690c
1 parent 4d5052b
commit f28690c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/SymbolFinder.php b/src/SymbolFinder.php
@@ -69,7 +69,7 @@ public function enterNode(Node $node)
             $this->uri,
             new Range(
                 new Position($node->getAttribute('startLine') - 1, $node->getAttribute('startColumn') - 1),
-                new Position($node->getAttribute('endLine') - 1, $node->getAttribute('endColumn') - 1)
+                new Position($node->getAttribute('endLine') - 1, $node->getAttribute('endColumn'))
             )
         );
         $symbol->containerName = $this->containerName;

diff --git a/tests/Server/TextDocumentTest.php b/tests/Server/TextDocumentTest.php
@@ -36,7 +36,7 @@ public function testDocumentSymbol()
                         ],
                         'end' => [
                             'line' => 2,
-                            'character' => 23
+                            'character' => 24
                         ]
                     ]
                 ],
@@ -54,7 +54,7 @@ public function testDocumentSymbol()
                         ],
                         'end' => [
                             'line' => 12,
-                            'character' => 0
+                            'character' => 1
                         ]
                     ]
                 ],
@@ -72,7 +72,7 @@ public function testDocumentSymbol()
                         ],
                         'end' => [
                             'line' => 6,
-                            'character' => 23
+                            'character' => 24
                         ]
                     ]
                 ],
@@ -90,7 +90,7 @@ public function testDocumentSymbol()
                         ],
                         'end' => [
                             'line' => 11,
-                            'character' => 4
+                            'character' => 5
                         ]
                     ]
                 ],
@@ -108,7 +108,7 @@ public function testDocumentSymbol()
                         ],
                         'end' => [
                             'line' => 17,
-                            'character' => 0
+                            'character' => 1
                         ]
                     ]
                 ],
@@ -126,7 +126,7 @@ public function testDocumentSymbol()
                         ],
                         'end' => [
                             'line' => 22,
-                            'character' => 0
+                            'character' => 1
                         ]
                     ]
                 ],