Fix crash when opening the PDF preview for an empty song

Desbeers · Oct 31, 2024 · 220c2b7 · 220c2b7
1 parent 78da2c1
commit 220c2b7
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 26 deletions.
diff --git a/Chord Provider/PDFBuild/SongExport/SongExport.swift b/Chord Provider/PDFBuild/SongExport/SongExport.swift
@@ -91,7 +91,9 @@ extension SongExport {
         items.append(PDFBuild.Spacer(10))
         items.append(PDFBuild.SongDetails(song: song))
         items.append(PDFBuild.Spacer(10))
-        items.append(PDFBuild.Chords(chords: song.chords, options: song.settings.diagram))
+        if !song.chords.isEmpty {
+            items.append(PDFBuild.Chords(chords: song.chords, options: song.settings.diagram))
+        }
         items.append(PDFBuild.Spacer(10))
         /// Add all the sections
         for section in song.sections {

diff --git a/Chord Provider/Views/SongView.swift b/Chord Provider/Views/SongView.swift
@@ -13,34 +13,39 @@ import SwiftUI
     @Environment(SceneStateModel.self) private var sceneState
     /// The body of the `View`
     var body: some View {
-        VStack {
-            switch sceneState.settings.song.paging {
-            case .asList:
-                ScrollView {
-                    ViewThatFits {
-                        RenderView(
-                            song: sceneState.song,
-                            paging: .asList,
-                            labelStyle: .grid
-                        )
-                        RenderView(
-                            song: sceneState.song,
-                            paging: .asList,
-                            labelStyle: .inline
-                        )
+        if sceneState.song.sections.isEmpty {
+            ContentUnavailableView("The Song is Empty", systemImage: "music.quarternote.3")
+                .frame(maxWidth: .infinity, maxHeight: .infinity)
+        } else {
+            VStack {
+                switch sceneState.settings.song.paging {
+                case .asList:
+                    ScrollView {
+                        ViewThatFits {
+                            RenderView(
+                                song: sceneState.song,
+                                paging: .asList,
+                                labelStyle: .grid
+                            )
+                            RenderView(
+                                song: sceneState.song,
+                                paging: .asList,
+                                labelStyle: .inline
+                            )
+                        }
+                        .frame(maxWidth: .infinity, maxHeight: .infinity)
                     }
-                    .frame(maxWidth: .infinity, maxHeight: .infinity)
+                case .asColumns:
+                    RenderView(
+                        song: sceneState.song,
+                        paging: .asColumns,
+                        labelStyle: .inline
+                    )
                 }
-            case .asColumns:
-                RenderView(
-                    song: sceneState.song,
-                    paging: .asColumns,
-                    labelStyle: .inline
-                )
             }
+            .contentMargins(.all, sceneState.settings.song.scale * 20, for: .scrollContent)
+            .contentShape(Rectangle())
+            .scaleModifier
         }
-        .contentMargins(.all, sceneState.settings.song.scale * 20, for: .scrollContent)
-        .contentShape(Rectangle())
-        .scaleModifier
     }
 }