From 3d9946a6bdc57863ba6ff3728a67f24fcdb540ba Mon Sep 17 00:00:00 2001
From: Daniel Wagner-Hall <dawagner@gmail.com>
Date: Thu, 9 Jan 2025 21:14:53 +0000
Subject: [PATCH] Give friendly error on empty backlog repo

Currently you get a complicated error about a nil dereference. This
makes it clear what happened.
---
 common-theme/layouts/_default/backlog.html      |  2 +-
 common-theme/layouts/_default/success.html      | 17 +++++++++++++++--
 .../partials/block/issues-list-as-blocks.html   | 11 ++++++++---
 org-cyf-piscine/go.mod                          |  4 ++--
 4 files changed, 26 insertions(+), 8 deletions(-)

diff --git a/common-theme/layouts/_default/backlog.html b/common-theme/layouts/_default/backlog.html
index bccadea0e..f3d1ceb42 100644
--- a/common-theme/layouts/_default/backlog.html
+++ b/common-theme/layouts/_default/backlog.html
@@ -14,7 +14,7 @@
       for each repo, grab the issues that match the filter
     */}}
     {{ range $repos }}
-      {{ $issueBlocks := partial "block/issues-list-as-blocks.html" (dict "backlog" . "backlog_filter" $backlog_filter) }}
+      {{ $issueBlocks := partial "block/issues-list-as-blocks.html" (dict "backlog" . "backlog_filter" $backlog_filter "path" $currentPath) }}
       {{ $repo := . }}    
       {{ range $issueBlocks }}
         <div class="c-alert" role="alert" hidden>
diff --git a/common-theme/layouts/_default/success.html b/common-theme/layouts/_default/success.html
index 6360bb2df..1f0e03d46 100644
--- a/common-theme/layouts/_default/success.html
+++ b/common-theme/layouts/_default/success.html
@@ -28,13 +28,26 @@ <h2 class="e-heading__4 c-objectives__title is-invisible">
           {{ $prep := .Site.GetPage (printf "%sprep/index.md"
             .CurrentSection.RelPermalink)
           }}
-          {{ $backlog := .Site.GetPage (printf "%sbacklog/index.md"
+          {{ $backlogPage := .Site.GetPage (printf "%sbacklog/index.md"
             .CurrentSection.RelPermalink)
           }}
           {{ $dayplan := .Site.GetPage (printf "%sday-plan/index.md"
             .CurrentSection.RelPermalink)
           }}
-          {{ $issueBlocks := partial "block/issues-list-as-blocks.html" (dict "backlog" $backlog.Params.backlog "backlog_filter" $backlog.Params.backlog_filter) }}
+          {{ $issueBlocks := slice }}
+          {{ if $backlogPage }}
+            {{ $backlogs := $backlogPage.Params.backlog }}
+            {{ if $backlogs }}
+              {{ if reflect.IsSlice $backlogs }}
+                {{ range $backlogs }}
+                  {{ $issueBlocks = append (partial "block/issues-list-as-blocks.html" (dict "backlog" . "backlog_filter" $backlogPage.Params.backlog_filter "path" $pageContext.Path)) $issueBlocks }}
+                {{ end }}
+              {{ else }}
+                {{ $backlog := $backlogs }}
+                {{ $issueBlocks = append (partial "block/issues-list-as-blocks.html" (dict "backlog" $backlog "backlog_filter" $backlogPage.Params.backlog_filter "path" $pageContext.Path)) $issueBlocks }}
+              {{ end }}
+            {{ end }}
+          {{ end }}
           {{ $blocks := slice }}
           {{/* append non-nil blocks */}}
           {{ range $prep.Params.blocks }}
diff --git a/common-theme/layouts/partials/block/issues-list-as-blocks.html b/common-theme/layouts/partials/block/issues-list-as-blocks.html
index 628b32636..741654026 100644
--- a/common-theme/layouts/partials/block/issues-list-as-blocks.html
+++ b/common-theme/layouts/partials/block/issues-list-as-blocks.html
@@ -11,6 +11,7 @@
   "owner" site.Params.owner
   "repo" .backlog
 }}
+{{ $currentPath := .path }}
 {{ if .backlog_filter }}
   {{ $variables = merge $variables (dict "labels" .backlog_filter) }}
 {{ end }}
@@ -53,9 +54,13 @@
 {{ $issueBlocks := slice }}
 {{ with resources.GetRemote "https://api.github.com/graphql" (merge $request (partial "github-auth.html" .)) }}
   {{ with .Content | unmarshal }}
-    {{ with .data.repository.issues.nodes }}
-      {{ range sort . "title" "asc" }}
-        {{ $issueBlocks = $issueBlocks | append (dict "name" .title "src" .url "number" .number "labels" .labels) }}
+    {{ if not .data.repository }}
+      {{ errorf "GitHub repository %s/%s referenced on %s had no issues" site.Params.owner (index $variables "repo") $currentPath }}
+    {{ else }}
+      {{ with .data.repository.issues.nodes }}
+        {{ range sort . "title" "asc" }}
+          {{ $issueBlocks = $issueBlocks | append (dict "name" .title "src" .url "number" .number "labels" .labels) }}
+        {{ end }}
       {{ end }}
     {{ end }}
   {{ end }}
diff --git a/org-cyf-piscine/go.mod b/org-cyf-piscine/go.mod
index 5037e665f..e0a033ea6 100644
--- a/org-cyf-piscine/go.mod
+++ b/org-cyf-piscine/go.mod
@@ -3,8 +3,8 @@ module github.com/CodeYourFuture/curriculum/org-cyf-piscine
 go 1.22.6
 
 require (
-	github.com/CodeYourFuture/curriculum/common-content v0.0.0-20240121151641-e52b73ad527d // indirect
-	github.com/CodeYourFuture/curriculum/common-theme v0.0.0-20240121151641-e52b73ad527d // indirect
+	github.com/CodeYourFuture/curriculum/common-content v0.0.0-20241008095008-281a5517463f // indirect
+	github.com/CodeYourFuture/curriculum/common-theme v0.0.0-20241008095008-281a5517463f // indirect
 	github.com/CodeYourFuture/curriculum/org-cyf-theme v0.0.0-20240818070728-e8702788f7cb // indirect
 )