Add redirect for packages without any published versions

Signed-off-by: Tim Goudriaan <tim@codedmonkey.com>

Author: codedmonkey

src/Controller/Dashboard/DashboardPackagesInfoController.php

@@ -32,6 +32,10 @@ public function info(string $packageName): Response
         $package = $this->packageRepository->findOneBy(['name' => $packageName]);
         $version = $package->getLatestVersion();
 
+        if (!$version) {
+            return $this->redirectToRoute('dashboard_packages_versions', ['packageName' => $packageName]);
+        }
+
         return $this->versionInfo($packageName, $version->getNormalizedVersion());
     }
 

templates/dashboard/packages/_package_header.html.twig

@@ -4,8 +4,12 @@
 
 <ul class="nav nav-pills nav-justified d-grid d-md-flex gap-1 pb-3 mb-3 border-bottom">
   <li class="nav-item">
-    {% set packageInfoUrl = path('dashboard_packages_info', {packageName: package.name}) %}
-    <a {{ (currentPage == 'info' ? attrActive : attr)|raw }} href="{{ packageInfoUrl }}">{{ 'Info'|trans }}</a>
+    {% if package.versions|length > 0 %}
+      {% set packageInfoUrl = path('dashboard_packages_info', {packageName: package.name}) %}
+      <a {{ (currentPage == 'info' ? attrActive : attr)|raw }} href="{{ packageInfoUrl }}">{{ 'Info'|trans }}</a>
+    {% else %}
+      <span class="nav-link disabled">{{ 'Info'|trans }}</span>
+    {% endif %}
   </li>
   <li class="nav-item">
     {% set packageVersionsUrl = path('dashboard_packages_versions', {packageName: package.name}) %}

templates/dashboard/packages/package_versions.html.twig

@@ -5,10 +5,35 @@
 {% block page_content %}
   {{ include('dashboard/packages/_package_header.html.twig', {currentPage: 'versions'}) }}
 
-  {{ _self.versionList('Latest versions', package.activeVersions) }}
-  {{ _self.versionList('Development versions', package.devVersions) }}
-  {{ _self.versionList('Historical versions', package.historicalVersions) }}
-  {{ _self.versionList('Development branch versions', package.devBranchVersions) }}
+  {% if package.versions|length > 0 %}
+    {{ _self.versionList('Latest versions', package.activeVersions) }}
+    {{ _self.versionList('Development versions', package.devVersions) }}
+    {{ _self.versionList('Historical versions', package.historicalVersions) }}
+    {{ _self.versionList('Development branch versions', package.devBranchVersions) }}
+  {% else %}
+    <table class="table datagrid datagrid-empty">
+      <tbody>
+        {% for i in 1..14 %}
+          <tr class="empty-row">
+            <td><span></span></td>
+            <td><span></span></td>
+            <td><span></span></td>
+            <td><span></span></td>
+            <td><span></span></td>
+            <td><span></span></td>
+          </tr>
+
+          {% if 3 == loop.index %}
+            <tr class="no-results">
+              <td colspan="100">
+                {{ 'No published versions found'|trans }}
+              </td>
+            </tr>
+          {% endif %}
+        {% endfor %}
+      </tbody>
+    </table>
+  {% endif %}
 {% endblock %}
 
 {% macro versionList(title, versions) %}

translations/messages.en.yaml

@@ -52,6 +52,8 @@ Providers: Providers
 Statistics: Statistics
 Suggesters: Suggesters
 
+No published versions found: No published versions found.
+
 # Package labels (Statistics)
 Daily installations: Daily installations
 Daily installations per version: Daily installations per version