fix build

johnpc · Jan 2, 2025 · 366cddf · 366cddf
1 parent c40297e
commit 366cddf
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 16 deletions.
diff --git a/src/components/pages/GenreReviewPage.tsx b/src/components/pages/GenreReviewPage.tsx
@@ -26,12 +26,14 @@ export default function GenreReviewPage() {
       setIsLoading(true);
       try {
         const movies = await listMovies();
-        setMovies(movies.filter((movie) => !hiddenIds.includes(movie.id)));
+        setMovies(
+          movies.filter((movie) => !hiddenIds.includes(movie.id ?? "")),
+        );
         const shows = await listShows();
         setShows(
           shows
             .map((show) => show.item)
-            .filter((show) => !hiddenIds.includes(show.id)),
+            .filter((show) => !hiddenIds.includes(show.id ?? "")),
         );
       } catch (error) {
         showBoundary(error);
@@ -60,6 +62,7 @@ export default function GenreReviewPage() {
   const allGenres = movies.flatMap((movie) => movie.genres).filter((g) => g);
   const genreCounts = allGenres.reduce(
     (counts, genre) => {
+      if (!genre) return counts;
       counts[genre] = (counts[genre] || 0) + 1;
       return counts;
     },
@@ -104,8 +107,8 @@ export default function GenreReviewPage() {
                     key={generateGuid()}
                     item={movie}
                     onHide={() => {
-                      setCachedHiddenId(movie.id);
-                      setHiddenIds([...hiddenIds, movie.id]);
+                      setCachedHiddenId(movie.id ?? "");
+                      setHiddenIds([...hiddenIds, movie.id ?? ""]);
                     }}
                   />
                 ))}

diff --git a/src/components/pages/MoviesReviewPage.tsx b/src/components/pages/MoviesReviewPage.tsx
@@ -21,7 +21,9 @@ export default function MoviesReviewPage() {
       setIsLoading(true);
       try {
         const movies = await listMovies();
-        setMovies(movies.filter((movie) => !hiddenIds.includes(movie.id)));
+        setMovies(
+          movies.filter((movie) => !hiddenIds.includes(movie.id ?? "")),
+        );
       } catch (error) {
         showBoundary(error);
       } finally {
@@ -65,8 +67,8 @@ export default function MoviesReviewPage() {
                 key={generateGuid()}
                 item={movie}
                 onHide={() => {
-                  setCachedHiddenId(movie.id);
-                  setHiddenIds([...hiddenIds, movie.id]);
+                  setCachedHiddenId(movie.id ?? "");
+                  setHiddenIds([...hiddenIds, movie.id ?? ""]);
                 }}
               />
             ))}

diff --git a/src/components/pages/MoviesReviewPage/ActorCard.tsx b/src/components/pages/MoviesReviewPage/ActorCard.tsx
@@ -25,7 +25,7 @@ export function ActorCard({
   useEffect(() => {
     const fetchImageUrl = async () => {
       try {
-        const url = await getImageUrlById(details.Id);
+        const url = await getImageUrlById(details.Id ?? "");
         setImageUrl(url);
       } catch (error) {
         console.error("Failed to fetch image URL:", error);

diff --git a/src/components/pages/MoviesReviewPage/MovieCard.tsx b/src/components/pages/MoviesReviewPage/MovieCard.tsx
@@ -24,7 +24,7 @@ export function MovieCard({
   useEffect(() => {
     const fetchImageUrl = async () => {
       try {
-        const url = await getImageUrlById(item.id);
+        const url = await getImageUrlById(item.id ?? "");
         setImageUrl(url);
       } catch (error) {
         console.error("Failed to fetch image URL:", error);

diff --git a/src/components/pages/OldestMoviePage.tsx b/src/components/pages/OldestMoviePage.tsx
@@ -21,8 +21,8 @@ export default function OldestMoviePage() {
       try {
         const movies = await listMovies();
         movies.sort((a, b) => {
-          const aDate = new Date(a.date);
-          const bDate = new Date(b.date);
+          const aDate = new Date(a.date ?? new Date());
+          const bDate = new Date(b.date ?? new Date());
           return aDate.getTime() - bDate.getTime();
         });
         const m = movies.find((s) => s);

diff --git a/src/components/pages/ShowReviewPage.tsx b/src/components/pages/ShowReviewPage.tsx
@@ -28,7 +28,7 @@ export default function ShowReviewPage() {
       try {
         const shows = await listShows();
         const filteredShows = shows.filter(
-          (show) => !hiddenIds.includes(show.item.id),
+          (show) => !hiddenIds.includes(show.item.id ?? ""),
         );
         setShows(filteredShows);
       } catch (e) {
@@ -97,8 +97,8 @@ export default function ShowReviewPage() {
                   episodeCount={show.episodeCount}
                   playbackTime={show.playbackTime}
                   onHide={() => {
-                    setCachedHiddenId(show.item.id);
-                    setHiddenIds([...hiddenIds, show.item.id]);
+                    setCachedHiddenId(show.item.id ?? "");
+                    setHiddenIds([...hiddenIds, show.item.id ?? ""]);
                   }}
                 />
               </>

diff --git a/src/lib/jellyfin-api.ts b/src/lib/jellyfin-api.ts
@@ -81,7 +81,7 @@ export const authenticateByUserName = async (
 
   try {
     // Authentication state is persisted on the api object
-    const authenticated = await api.authenticateUserByName(username, password);
+    await api.authenticateUserByName(username, password);
   } catch (error) {
     console.error("Connection failed:", error);
     alert(error);

diff --git a/src/main.tsx b/src/main.tsx
@@ -2,8 +2,10 @@ import { StrictMode } from "react";
 import { createRoot } from "react-dom/client";
 import App from "./App.tsx";
 import "./index.css";
+const rootElement = document.getElementById("root");
+if (!rootElement) throw new Error("Failed to find the root element");
 
-createRoot(document.getElementById("root")).render(
+createRoot(rootElement).render(
   <StrictMode>
     <App />
   </StrictMode>,