Use separate model binding classes (#6993)

Author: Scott Bommarito

src/NuGetGallery/Controllers/PackagesController.cs

@@ -1387,7 +1387,7 @@ public virtual ActionResult ContactOwners(string id, string version)
                 PackageId = id,
                 PackageVersion = package.Version,
                 ProjectUrl = package.ProjectUrl,
-                Owners = package.PackageRegistration.Owners.Where(u => u.EmailAllowed),
+                Owners = package.PackageRegistration.Owners.Where(u => u.EmailAllowed).Select(u => u.Username),
                 CopySender = true,
                 HasOwners = hasOwners
             };
@@ -2141,7 +2141,7 @@ public virtual async Task<JsonResult> VerifyPackage(VerifyPackageRequest formDat
             }
         }
 
-        public virtual async Task<JsonResult> VerifySymbolsPackageInternal(
+        protected virtual async Task<JsonResult> VerifySymbolsPackageInternal(
             VerifyPackageRequest formData,
             Stream uploadFile,
             PackageArchiveReader packageArchiveReader,
@@ -2241,7 +2241,7 @@ await _auditingService.SaveAuditRecordAsync(
             }
         }
 
-        public virtual async Task<JsonResult> VerifyPackageInternal(
+        protected virtual async Task<JsonResult> VerifyPackageInternal(
             VerifyPackageRequest formData,
             Stream uploadFile,
             PackageArchiveReader packageArchiveReader,

src/NuGetGallery/ViewModels/ContactOwnersViewModel.cs

@@ -17,7 +17,7 @@ public class ContactOwnersViewModel
 
         public string ProjectUrl { get; set; }
 
-        public IEnumerable<User> Owners { get; set; }
+        public IEnumerable<string> Owners { get; set; }
 
         [Display(Name = "Send me a copy")]
         public bool CopySender { get; set; }

src/NuGetGallery/Views/Packages/ContactOwners.cshtml

@@ -32,7 +32,7 @@
                         <p>
                             @foreach (var owner in Model.Owners)
                             {
-                                <a href="@Url.User(owner)" title="View @owner.Username's profile" class="ms-font-xl">@owner.Username</a>
+                                <a href="@Url.User(owner)" title="View @owner's profile" class="ms-font-xl">@owner</a>
                             }
                         </p>
                     </div>

tests/NuGetGallery.Facts/Controllers/PackagesControllerFacts.cs

@@ -2161,7 +2161,7 @@ public void OnlyShowsOwnersWhoAllowReceivingEmails()
 
                 // assert
                 Assert.Equal(2, model.Owners.Count());
-                Assert.Empty(model.Owners.Where(u => u.Username == notAllowedUser));
+                Assert.Empty(model.Owners.Where(u => u == notAllowedUser));
             }
 
             [Fact]