Display 'No dependencies' if group has no dependencies (loic-sharma#686)

Author: loic-sharma

src/BaGet.Core/Entities/PackageDependency.cs

@@ -5,8 +5,16 @@ public class PackageDependency
     {
         public int Key { get; set; }
 
+        /// <summary>
+        /// The dependency's package ID. Null if this is a dependency group without any dependencies.
+        /// </summary>
         public string Id { get; set; }
+
+        /// <summary>
+        /// The dependency's package version. Null if this is a dependency group without any dependencies.
+        /// </summary>
         public string VersionRange { get; set; }
+
         public string TargetFramework { get; set; }
 
         public Package Package { get; set; }

src/BaGet.Web/Pages/Package.cshtml

@@ -139,17 +139,24 @@ else
                             </h4>
 
                             <ul class="list-unstyled dependency-group">
-                                @foreach (var dependency in group.Dependencies)
+                                @if (!group.Dependencies.Any())
                                 {
-                                    <li>
-                                        <a asp-page="Package"
-                                           asp-route-id="@dependency.PackageId"
-                                           asp-route-version="@null">
-                                            @dependency.PackageId
-                                        </a>
-
-                                        <span> @dependency.VersionSpec</span>
-                                    </li>
+                                    <li>No dependencies.</li>
+                                }
+                                else
+                                {
+                                    @foreach (var dependency in group.Dependencies)
+                                    {
+                                        <li>
+                                            <a asp-page="Package"
+                                               asp-route-id="@dependency.PackageId"
+                                               asp-route-version="@null">
+                                                @dependency.PackageId
+                                            </a>
+
+                                            <span> @dependency.VersionSpec</span>
+                                        </li>
+                                    }
                                 }
                             </ul>
                         }

src/BaGet.Web/Pages/Package.cshtml.cs

@@ -121,6 +121,7 @@ private IReadOnlyList<DependencyGroupModel> ToDependencyGroups(Package package)
                     {
                         Name = PrettifyTargetFramework(group.Key),
                         Dependencies = group
+                            .Where(d => d.Id != null)
                             .Select(d => new DependencyModel
                             {
                                 PackageId = d.Id,