Fix to make cover image not required. and optimization of my personal…

… code following the comments made on the Pull request
Klaiment · Oct 12, 2023 · 3b8604b · 3b8604b
1 parent 375db33
commit 3b8604b
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 25 deletions.
diff --git a/api/src/controllers/torrent.js b/api/src/controllers/torrent.js
@@ -508,7 +508,6 @@ export const getTorrentsPage = async ({
  created: 1,
  freeleech: 1,
  tags: 1,
- poster: 1,
  confidenceScore: 1,
  },
  },

diff --git a/client/pages/torrent/[infoHash].js b/client/pages/torrent/[infoHash].js
@@ -523,10 +523,12 @@ const Torrent = ({ token, torrent = {}, userId, userRole, uid, userStats }) => {
  userStats.ratio < Number(SQ_MINIMUM_RATIO)) ||
  (Number(SQ_MAXIMUM_HIT_N_RUNS) !== -1 &&
  userStats.hitnruns > Number(SQ_MAXIMUM_HIT_N_RUNS));
+
  function isPngImage(data) {
  const pngHeader = "data:image/png;base64,";
  return data.startsWith(pngHeader);
  }
+
  return (
  <>
  <SEO title={torrent.name} />
@@ -664,19 +666,27 @@ const Torrent = ({ token, torrent = {}, userId, userRole, uid, userStats }) => {
  : [getLocaleString("torrNo")],
  }}
  />
- <Infobox mb={5}>
- <Text fontWeight={600} fontSize={1} textTransform="uppercase" mb={3}>
- {getLocaleString("PosterImage")}
- </Text>
- <img
- src={`data:image/${
- isPngImage(torrent.poster) ? "png" : "jpeg"
- };base64,${torrent.poster}`}
- alt="Poster"
- width="auto"
- height={200}
- />
- </Infobox>
+ {torrent.poster && (
+ <Infobox mb={5}>
+ <Text
+ fontWeight={600}
+ fontSize={1}
+ _css={{ textTransform: "uppercase" }}
+ mb={3}
+ >
+ {getLocaleString("posterImage")}
+ </Text>
+ <img
+ src={`data:image/${
+ isPngImage(torrent.poster) ? "png" : "jpeg"
+ };base64,${torrent.poster}`}
+ alt="Poster"
+ width="auto"
+ height={200}
+ />
+ </Infobox>
+ )}
+
 
  <Infobox mb={5}>
  <Text
@@ -708,7 +718,6 @@ const Torrent = ({ token, torrent = {}, userId, userRole, uid, userStats }) => {
  </Box>
  </Infobox>
  )}
- {console.log(torrent)}
  <Infobox mb={5}>
  <Text
  fontWeight={600}

diff --git a/client/pages/upload.js b/client/pages/upload.js
@@ -275,6 +275,7 @@ const Upload = ({ token, userId }) => {
  "image/png": [".png"],
  },
  maxFiles: 1,
+ maxSize: 5242880, //5Mo
  });
  function isPngImage(data) {
  const pngHeader = "data:image/png;base64,";
@@ -287,9 +288,7 @@ const Upload = ({ token, userId }) => {
  const form = new FormData(e.target);
 
  try {
- form.append("poster", posterFile.b64); // Ajoutez la valeur de l'image de l'affiche à la requête
  if (!torrentFile) throw new Error("No .torrent file added");
-
  const uploadRes = await fetch(`${SQ_API_URL}/torrent/upload`, {
  method: "POST",
  headers: {
@@ -306,7 +305,7 @@ const Upload = ({ token, userId }) => {
  tags: form.get("tags"),
  groupWith,
  mediaInfo: form.get("mediaInfo"),
- poster: posterFile.b64,
+ poster: posterFile ? posterFile.b64 : null,
  }),
  });
 
@@ -436,18 +435,21 @@ const Upload = ({ token, userId }) => {
  width={"auto"}
  height={200}
  />
- ) : isPosterDragActive ? (
- <Text color="grey">
- {getLocaleString("uploadDropImageHere")}
- </Text>
  ) : (
- <Text color="grey">
- {getLocaleString("uploadDragDropClickSelectPoster")}
- </Text>
+ isPosterDragActive ? (
+ <Text color="grey">
+ {getLocaleString("uploadDropImageHere")}
+ </Text>
+ ) : (
+ <Text color="grey">
+ {getLocaleString("uploadDragDropClickSelectPoster")}
+ </Text>
+ )
  )}
  </FileUpload>
  </WrapLabel>
  </Box>
+
  <TorrentFields
  categories={SQ_TORRENT_CATEGORIES}
  handleGroupSearch={handleGroupSearch}