Merge pull request OmniSharp#123 from mholo65/feature/OmniSharpGH-122

GH122: DocumentSymbol returns DocumentSymbol[] or SymbolInformation[]

Author: david-driscoll

src/Protocol/Document/Client/DocumentSymbolExtensions.cs

@@ -9,9 +9,9 @@ namespace OmniSharp.Extensions.LanguageServer.Protocol.Client
 {
     public static class DocumentSymbolExtensions
     {
-        public static Task<DocumentSymbolInformationContainer> DocumentSymbol(this ILanguageClientDocument mediator, DocumentSymbolParams @params)
+        public static Task<DocumentSymbolInformationOrDocumentSymbolContainer> DocumentSymbol(this ILanguageClientDocument mediator, DocumentSymbolParams @params)
         {
-            return mediator.SendRequest<DocumentSymbolParams, DocumentSymbolInformationContainer>(DocumentNames.DocumentSymbol, @params);
+            return mediator.SendRequest<DocumentSymbolParams, DocumentSymbolInformationOrDocumentSymbolContainer>(DocumentNames.DocumentSymbol, @params);
         }
     }
 }

src/Protocol/Document/Server/IDocumentSymbolHandler.cs

@@ -8,5 +8,5 @@ namespace OmniSharp.Extensions.LanguageServer.Protocol.Server
 {
     using static DocumentNames;
     [Parallel, Method(DocumentSymbol)]
-    public interface IDocumentSymbolHandler : IJsonRpcRequestHandler<DocumentSymbolParams, DocumentSymbolInformationContainer>, IRegistration<TextDocumentRegistrationOptions>, ICapability<DocumentSymbolCapability> { }
+    public interface IDocumentSymbolHandler : IJsonRpcRequestHandler<DocumentSymbolParams, DocumentSymbolInformationOrDocumentSymbolContainer>, IRegistration<TextDocumentRegistrationOptions>, ICapability<DocumentSymbolCapability> { }
 }

src/Protocol/Models/DocumentSymbolInformationOrDocumentSymbolContainer.cs

@@ -0,0 +1,36 @@
+using System.Collections.Generic;
+using System.Collections.ObjectModel;
+using System.Linq;
+
+namespace OmniSharp.Extensions.LanguageServer.Protocol.Models
+{
+    public class DocumentSymbolInformationOrDocumentSymbolContainer : Container<DocumentSymbolInformationOrDocumentSymbol>
+    {
+        public DocumentSymbolInformationOrDocumentSymbolContainer() : this(Enumerable.Empty<DocumentSymbolInformationOrDocumentSymbol>())
+        {
+        }
+
+        public DocumentSymbolInformationOrDocumentSymbolContainer(IEnumerable<DocumentSymbolInformationOrDocumentSymbol> items) : base(items)
+        {
+        }
+
+        public DocumentSymbolInformationOrDocumentSymbolContainer(params DocumentSymbolInformationOrDocumentSymbol[] items) : base(items)
+        {
+        }
+
+        public static implicit operator DocumentSymbolInformationOrDocumentSymbolContainer(DocumentSymbolInformationOrDocumentSymbol[] items)
+        {
+            return new DocumentSymbolInformationOrDocumentSymbolContainer(items);
+        }
+
+        public static implicit operator DocumentSymbolInformationOrDocumentSymbolContainer(Collection<DocumentSymbolInformationOrDocumentSymbol> items)
+        {
+            return new DocumentSymbolInformationOrDocumentSymbolContainer(items);
+        }
+
+        public static implicit operator DocumentSymbolInformationOrDocumentSymbolContainer(List<DocumentSymbolInformationOrDocumentSymbol> items)
+        {
+            return new DocumentSymbolInformationOrDocumentSymbolContainer(items);
+        }
+    }
+}

src/Protocol/Models/DocumentSymbolParams.cs

@@ -4,7 +4,7 @@
 
 namespace OmniSharp.Extensions.LanguageServer.Protocol.Models
 {
-    public class DocumentSymbolParams : ITextDocumentIdentifierParams, IRequest<DocumentSymbolInformationContainer>
+    public class DocumentSymbolParams : ITextDocumentIdentifierParams, IRequest<DocumentSymbolInformationOrDocumentSymbolContainer>
     {
         /// <summary>
         /// The text document.