Merge pull request #37284 from jasonmalinowski/fix-missing-nullable-annotations

Fix missing nullable annotations

src/EditorFeatures/CSharp/NavigationBar/CSharpNavigationBarItemService.cs

@@ -36,7 +36,9 @@ internal class CSharpNavigationBarItemService : AbstractNavigationBarItemService
                                   SymbolDisplayParameterOptions.IncludeName |
                                   SymbolDisplayParameterOptions.IncludeDefaultValue |
                                   SymbolDisplayParameterOptions.IncludeParamsRefOut,
-                miscellaneousOptions: SymbolDisplayMiscellaneousOptions.UseSpecialTypes | SymbolDisplayMiscellaneousOptions.AllowDefaultLiteral);
+                miscellaneousOptions: SymbolDisplayMiscellaneousOptions.UseSpecialTypes |
+                                      SymbolDisplayMiscellaneousOptions.AllowDefaultLiteral |
+                                      SymbolDisplayMiscellaneousOptions.IncludeNullableReferenceTypeModifier);
 
         [ImportingConstructor]
         public CSharpNavigationBarItemService()

src/EditorFeatures/Test2/NavigationBar/CSharpNavigationBarTests.vb

@@ -240,5 +240,19 @@ Namespace Microsoft.CodeAnalysis.Editor.UnitTests.NavigationBar
                     Item("M()", Glyph.MethodPrivate),
                     Item("M()", Glyph.MethodPrivate, grayed:=True)}))
         End Function
+
+        <Fact, Trait(Traits.Feature, Traits.Features.NavigationBar), WorkItem(37183, "https://github.com/dotnet/roslyn/issues/37183")>
+        Public Async Function TestNullableReferenceTypesInParameters() As Task
+            Await AssertItemsAreAsync(
+                <Workspace>
+                    <Project Language="C#" CommonReferences="true">
+                        <Document>#nullable enable
+                        using System.Collections.Generic;
+                        class C { void M(string? s, IEnumerable&lt;string?&gt; e) { }</Document>
+                    </Project>
+                </Workspace>,
+                Item("C", Glyph.ClassInternal, children:={
+                    Item("M(string? s, IEnumerable<string?> e)", Glyph.MethodPrivate)}))
+        End Function
     End Class
 End Namespace

src/VisualStudio/Core/Def/Implementation/ExtractInterface/ExtractInterfaceDialog.xaml

@@ -136,17 +136,17 @@
                         <DataTemplate>
                             <StackPanel Orientation="Horizontal">
                                 <CheckBox x:Uid="SelectableMemberCheckBox"
-                                          AutomationProperties.Name="{Binding MemberAutomationText}"
+                                          AutomationProperties.Name="{Binding SymbolAutomationText}"
                                           IsChecked="{Binding IsChecked, Mode=TwoWay}" 
                                           Width="Auto"
                                           Focusable="False"
-                                          AutomationProperties.AutomationId="{Binding MemberName}">
+                                          AutomationProperties.AutomationId="{Binding SymbolName}">
                                 </CheckBox>
                                 <Image x:Uid="SelectableMemberGlyph" 
                                        Margin="8,0,0,0"
                                                Source="{Binding Glyph}"/>
                                 <TextBlock x:Uid="SelectableMemberName" 
-                                                   Text="{Binding MemberName}"/>
+                                                   Text="{Binding SymbolName}"/>
                             </StackPanel>
                         </DataTemplate>
                     </u:AutomationDelegatingListView.ItemTemplate>

src/VisualStudio/Core/Def/Implementation/ExtractInterface/ExtractInterfaceDialogViewModel.cs

@@ -4,15 +4,11 @@
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
-using System.Windows.Media;
 using Microsoft.CodeAnalysis;
-using Microsoft.CodeAnalysis.Editor.Shared.Extensions;
 using Microsoft.CodeAnalysis.LanguageServices;
 using Microsoft.CodeAnalysis.Notification;
-using Microsoft.CodeAnalysis.Shared.Extensions;
 using Microsoft.VisualStudio.Language.Intellisense;
 using Microsoft.VisualStudio.LanguageServices.Implementation.Utilities;
-using Microsoft.VisualStudio.LanguageServices.Utilities;
 using Roslyn.Utilities;
 
 namespace Microsoft.VisualStudio.LanguageServices.Implementation.ExtractInterface
@@ -55,7 +51,7 @@ internal class ExtractInterfaceDialogViewModel : AbstractNotifyPropertyChanged
             _generatedNameTypeParameterSuffix = generatedNameTypeParameterSuffix;
             _languageName = languageName;
 
-            MemberContainers = extractableMembers.Select(m => new MemberSymbolViewModel(m, glyphService)).OrderBy(s => s.MemberName).ToList();
+            MemberContainers = extractableMembers.Select(m => new MemberSymbolViewModel(m, glyphService)).OrderBy(s => s.SymbolName).ToList();
         }
 
         internal bool TrySubmit()
@@ -173,45 +169,10 @@ public InterfaceDestination Destination
 
         public bool FileNameEnabled => Destination == InterfaceDestination.NewFile;
 
-        internal class MemberSymbolViewModel : AbstractNotifyPropertyChanged
+        internal class MemberSymbolViewModel : SymbolViewModel<ISymbol>
         {
-            private readonly IGlyphService _glyphService;
-
-            public ISymbol MemberSymbol { get; }
-
-            private static readonly SymbolDisplayFormat s_memberDisplayFormat = new SymbolDisplayFormat(
-                genericsOptions: SymbolDisplayGenericsOptions.IncludeTypeParameters,
-                memberOptions: SymbolDisplayMemberOptions.IncludeParameters,
-                parameterOptions: SymbolDisplayParameterOptions.IncludeType | SymbolDisplayParameterOptions.IncludeParamsRefOut | SymbolDisplayParameterOptions.IncludeOptionalBrackets,
-                miscellaneousOptions: SymbolDisplayMiscellaneousOptions.EscapeKeywordIdentifiers | SymbolDisplayMiscellaneousOptions.UseSpecialTypes);
-
-            public MemberSymbolViewModel(ISymbol symbol, IGlyphService glyphService)
-            {
-                MemberSymbol = symbol;
-                _glyphService = glyphService;
-                _isChecked = true;
-            }
-
-            private bool _isChecked;
-            public bool IsChecked
-            {
-                get { return _isChecked; }
-                set { SetProperty(ref _isChecked, value); }
-            }
-
-            public string MemberName
-            {
-                get { return MemberSymbol.ToDisplayString(s_memberDisplayFormat); }
-            }
-
-            public ImageSource Glyph
-            {
-                get { return MemberSymbol.GetGlyph().GetImageSource(_glyphService); }
-            }
-
-            public string MemberAutomationText
+            public MemberSymbolViewModel(ISymbol symbol, IGlyphService glyphService) : base(symbol, glyphService)
             {
-                get { return MemberSymbol.Kind + " " + MemberName; }
             }
         }
     }

src/VisualStudio/Core/Def/Implementation/ExtractInterface/VisualStudioExtractInterfaceOptionsService.cs

@@ -57,7 +57,7 @@ public VisualStudioExtractInterfaceOptionsService(IGlyphService glyphService, IT
 
             if (result.HasValue && result.Value)
             {
-                var includedMembers = viewModel.MemberContainers.Where(c => c.IsChecked).Select(c => c.MemberSymbol);
+                var includedMembers = viewModel.MemberContainers.Where(c => c.IsChecked).Select(c => c.Symbol);
 
                 return new ExtractInterfaceOptionsResult(
                     isCancelled: false,

src/VisualStudio/Core/Def/Implementation/PickMembers/PickMembersDialog.xaml

@@ -74,17 +74,17 @@
                         <DataTemplate>
                             <StackPanel Orientation="Horizontal">
                                 <CheckBox x:Uid="SelectableMemberCheckBox"
-                                          AutomationProperties.Name="{Binding MemberAutomationText}"
+                                          AutomationProperties.Name="{Binding SymbolAutomationText}"
                                           IsChecked="{Binding IsChecked, Mode=TwoWay}" 
                                           Width="Auto"
                                           Focusable="False"
-                                          AutomationProperties.AutomationId="{Binding MemberName}">
+                                          AutomationProperties.AutomationId="{Binding SymbolName}">
                                 </CheckBox>
                                 <Image x:Uid="SelectableMemberGlyph" 
                                        Margin="8,0,0,0"
                                                Source="{Binding Glyph}"/>
                                 <TextBlock x:Uid="SelectableMemberName" 
-                                                   Text="{Binding MemberName}"/>
+                                                   Text="{Binding SymbolName}"/>
                             </StackPanel>
                         </DataTemplate>
                     </u:AutomationDelegatingListView.ItemTemplate>

src/VisualStudio/Core/Def/Implementation/PickMembers/PickMembersDialogViewModel.cs

@@ -4,15 +4,10 @@
 using System.Collections.Immutable;
 using System.Diagnostics;
 using System.Linq;
-using System.Windows.Media;
 using Microsoft.CodeAnalysis;
-using Microsoft.CodeAnalysis.Editor.Shared.Extensions;
 using Microsoft.CodeAnalysis.PickMembers;
-using Microsoft.CodeAnalysis.Shared.Extensions;
 using Microsoft.VisualStudio.Language.Intellisense;
 using Microsoft.VisualStudio.LanguageServices.Implementation.Utilities;
-using Microsoft.VisualStudio.LanguageServices.Utilities;
-using Roslyn.Utilities;
 
 namespace Microsoft.VisualStudio.LanguageServices.Implementation.PickMembers
 {
@@ -80,7 +75,7 @@ public string MoveUpAutomationText
                     return string.Empty;
                 }
 
-                return string.Format(ServicesVSResources.Move_0_above_1, MemberContainers[SelectedIndex.Value].MemberAutomationText, MemberContainers[SelectedIndex.Value - 1].MemberAutomationText);
+                return string.Format(ServicesVSResources.Move_0_above_1, MemberContainers[SelectedIndex.Value].SymbolAutomationText, MemberContainers[SelectedIndex.Value - 1].SymbolAutomationText);
             }
         }
 
@@ -93,12 +88,10 @@ public string MoveDownAutomationText
                     return string.Empty;
                 }
 
-                return string.Format(ServicesVSResources.Move_0_below_1, MemberContainers[SelectedIndex.Value].MemberAutomationText, MemberContainers[SelectedIndex.Value + 1].MemberAutomationText);
+                return string.Format(ServicesVSResources.Move_0_below_1, MemberContainers[SelectedIndex.Value].SymbolAutomationText, MemberContainers[SelectedIndex.Value + 1].SymbolAutomationText);
             }
         }
 
-
-
         public bool CanMoveUp
         {
             get
@@ -152,37 +145,11 @@ private void Move(List<MemberSymbolViewModel> list, int index, int delta)
             SelectedIndex += delta;
         }
 
-        internal class MemberSymbolViewModel : AbstractNotifyPropertyChanged
+        internal class MemberSymbolViewModel : SymbolViewModel<ISymbol>
         {
-            private readonly IGlyphService _glyphService;
-
-            public ISymbol MemberSymbol { get; }
-
-            private static SymbolDisplayFormat s_memberDisplayFormat = new SymbolDisplayFormat(
-                genericsOptions: SymbolDisplayGenericsOptions.IncludeTypeParameters,
-                memberOptions: SymbolDisplayMemberOptions.IncludeParameters,
-                parameterOptions: SymbolDisplayParameterOptions.IncludeType | SymbolDisplayParameterOptions.IncludeParamsRefOut | SymbolDisplayParameterOptions.IncludeOptionalBrackets,
-                miscellaneousOptions: SymbolDisplayMiscellaneousOptions.EscapeKeywordIdentifiers | SymbolDisplayMiscellaneousOptions.UseSpecialTypes);
-
-            public MemberSymbolViewModel(ISymbol symbol, IGlyphService glyphService)
+            public MemberSymbolViewModel(ISymbol symbol, IGlyphService glyphService) : base(symbol, glyphService)
             {
-                MemberSymbol = symbol;
-                _glyphService = glyphService;
-                _isChecked = true;
             }
-
-            private bool _isChecked;
-            public bool IsChecked
-            {
-                get { return _isChecked; }
-                set { SetProperty(ref _isChecked, value); }
-            }
-
-            public string MemberName => MemberSymbol.ToDisplayString(s_memberDisplayFormat);
-
-            public ImageSource Glyph => MemberSymbol.GetGlyph().GetImageSource(_glyphService);
-
-            public string MemberAutomationText => MemberSymbol.Kind + " " + MemberName;
         }
 
         internal class OptionViewModel : AbstractNotifyPropertyChanged

src/VisualStudio/Core/Def/Implementation/PickMembers/VisualStudioPickMembersService.cs

@@ -34,7 +34,7 @@ public VisualStudioPickMembersService(IGlyphService glyphService)
             {
                 return new PickMembersResult(
                     viewModel.MemberContainers.Where(c => c.IsChecked)
-                                              .Select(c => c.MemberSymbol)
+                                              .Select(c => c.Symbol)
                                               .ToImmutableArray(),
                     options);
             }

src/VisualStudio/Core/Def/Implementation/Utilities/SymbolViewModel.cs

@@ -17,14 +17,14 @@ internal class SymbolViewModel<T> : AbstractNotifyPropertyChanged where T : ISym
             genericsOptions: SymbolDisplayGenericsOptions.IncludeTypeParameters,
             memberOptions: SymbolDisplayMemberOptions.IncludeParameters,
             parameterOptions: SymbolDisplayParameterOptions.IncludeType | SymbolDisplayParameterOptions.IncludeParamsRefOut | SymbolDisplayParameterOptions.IncludeOptionalBrackets,
-            miscellaneousOptions: SymbolDisplayMiscellaneousOptions.EscapeKeywordIdentifiers | SymbolDisplayMiscellaneousOptions.UseSpecialTypes);
+            miscellaneousOptions: SymbolDisplayMiscellaneousOptions.EscapeKeywordIdentifiers | SymbolDisplayMiscellaneousOptions.UseSpecialTypes | SymbolDisplayMiscellaneousOptions.IncludeNullableReferenceTypeModifier);
 
         private static readonly SymbolDisplayFormat s_symbolAutomationFormat = new SymbolDisplayFormat(
             genericsOptions: SymbolDisplayGenericsOptions.IncludeTypeParameters,
             memberOptions: SymbolDisplayMemberOptions.IncludeParameters,
             parameterOptions: SymbolDisplayParameterOptions.IncludeType | SymbolDisplayParameterOptions.IncludeParamsRefOut | SymbolDisplayParameterOptions.IncludeOptionalBrackets,
             kindOptions: SymbolDisplayKindOptions.IncludeTypeKeyword,
-            miscellaneousOptions: SymbolDisplayMiscellaneousOptions.EscapeKeywordIdentifiers | SymbolDisplayMiscellaneousOptions.UseSpecialTypes);
+            miscellaneousOptions: SymbolDisplayMiscellaneousOptions.EscapeKeywordIdentifiers | SymbolDisplayMiscellaneousOptions.UseSpecialTypes | SymbolDisplayMiscellaneousOptions.IncludeNullableReferenceTypeModifier);
 
         public SymbolViewModel(T symbol, IGlyphService glyphService)
         {

src/VisualStudio/Core/Test/ExtractInterface/ExtractInterfaceViewModelTests.vb

@@ -394,7 +394,7 @@ class $$MyClass
 }"]]></Text>
 
             Dim viewModel = Await GetViewModelAsync(markup, LanguageNames.CSharp, "IMyClass")
-            Assert.Equal("Goo<T>(T, CorrelationManager, ref int, [int?], [string], params int[])", viewModel.MemberContainers.Single().MemberName)
+            Assert.Equal("Goo<T>(T, CorrelationManager, ref int, [int?], [string], params int[])", viewModel.MemberContainers.Single().SymbolName)
         End Function
 
         <Fact, Trait(Traits.Feature, Traits.Features.ExtractInterface)>
@@ -411,7 +411,7 @@ class $$MyClass
 }"]]></Text>
 
             Dim viewModel = Await GetViewModelAsync(markup, LanguageNames.CSharp, "IMyClass")
-            Assert.Equal("Goo", viewModel.MemberContainers.Where(Function(c) c.MemberSymbol.IsKind(SymbolKind.Property)).Single().MemberName)
+            Assert.Equal("Goo", viewModel.MemberContainers.Where(Function(c) c.Symbol.IsKind(SymbolKind.Property)).Single().SymbolName)
         End Function
 
         <Fact, Trait(Traits.Feature, Traits.Features.ExtractInterface)>
@@ -424,7 +424,22 @@ class $$MyClass
 }"]]></Text>
 
             Dim viewModel = Await GetViewModelAsync(markup, LanguageNames.CSharp, "IMyClass")
-            Assert.Equal("this[int?, [string]]", viewModel.MemberContainers.Where(Function(c) c.MemberSymbol.IsKind(SymbolKind.Property)).Single().MemberName)
+            Assert.Equal("this[int?, [string]]", viewModel.MemberContainers.Where(Function(c) c.Symbol.IsKind(SymbolKind.Property)).Single().SymbolName)
+        End Function
+
+        <Fact, Trait(Traits.Feature, Traits.Features.ExtractInterface)>
+        <WorkItem(37176, "https://github.com/dotnet/roslyn/issues/37176")>
+        Public Async Function TestExtractInterface_MemberDisplay_NullableReferenceType() As Task
+            Dim markup = <Text><![CDATA[
+#nullable enable
+using System.Collections.Generic;
+class $$MyClass
+{
+    public void M(string? s, IEnumerable<string?> e) { }
+}"]]></Text>
+
+            Dim viewModel = Await GetViewModelAsync(markup, LanguageNames.CSharp, "IMyClass")
+            Assert.Equal("M(string?, IEnumerable<string?>)", viewModel.MemberContainers.Single(Function(c) c.Symbol.IsKind(SymbolKind.Method)).SymbolName)
         End Function
 
         <Fact, Trait(Traits.Feature, Traits.Features.ExtractInterface)>
@@ -441,11 +456,11 @@ public class $$MyClass
 
             Dim viewModel = Await GetViewModelAsync(markup, LanguageNames.CSharp, "IMyClass")
             Assert.Equal(5, viewModel.MemberContainers.Count)
-            Assert.Equal("Goo()", viewModel.MemberContainers.ElementAt(0).MemberName)
-            Assert.Equal("Goo(int)", viewModel.MemberContainers.ElementAt(1).MemberName)
-            Assert.Equal("Goo(int, int)", viewModel.MemberContainers.ElementAt(2).MemberName)
-            Assert.Equal("Goo(int, string)", viewModel.MemberContainers.ElementAt(3).MemberName)
-            Assert.Equal("Goo(string)", viewModel.MemberContainers.ElementAt(4).MemberName)
+            Assert.Equal("Goo()", viewModel.MemberContainers.ElementAt(0).SymbolName)
+            Assert.Equal("Goo(int)", viewModel.MemberContainers.ElementAt(1).SymbolName)
+            Assert.Equal("Goo(int, int)", viewModel.MemberContainers.ElementAt(2).SymbolName)
+            Assert.Equal("Goo(int, string)", viewModel.MemberContainers.ElementAt(3).SymbolName)
+            Assert.Equal("Goo(string)", viewModel.MemberContainers.ElementAt(4).SymbolName)
         End Function
 
         Private Async Function GetViewModelAsync(markup As XElement,

src/VisualStudio/IntegrationTest/TestUtilities/InProcess/ExtractInterfaceDialog_InProc.cs

@@ -124,7 +124,7 @@ public string[] GetSelectedItems()
 
                     return listItems.Cast<ExtractInterfaceDialogViewModel.MemberSymbolViewModel>()
                         .Where(viewModel => viewModel.IsChecked)
-                        .Select(viewModel => viewModel.MemberName)
+                        .Select(viewModel => viewModel.SymbolName)
                         .ToArray();
                 });
             }
@@ -142,7 +142,7 @@ public void ToggleItem(string item)
 
                     var memberSelectionList = dialog.GetTestAccessor().Members;
                     var items = memberSelectionList.Items.Cast<ExtractInterfaceDialogViewModel.MemberSymbolViewModel>().ToArray();
-                    var itemViewModel = items.Single(x => x.MemberName == item);
+                    var itemViewModel = items.Single(x => x.SymbolName == item);
                     itemViewModel.IsChecked = !itemViewModel.IsChecked;
 
                     // Wait for changes to propagate