From 5cc64a09bddc94a205b279523a9f1d61dfcda5d8 Mon Sep 17 00:00:00 2001
From: Zack Krida <zackkrida@pm.me>
Date: Fri, 20 Jan 2023 10:28:55 +0300
Subject: [PATCH] Image reporting view (#2090)

Co-authored-by: Olga Bulat <obulat@gmail.com>
---
 .../VContentReport/VContentReportForm.vue     |   9 +-
 .../VContentReport/VContentReportPopover.vue  |   1 +
 src/composables/use-match-routes.ts           |   6 +
 src/locales/scripts/en.json5                  |   3 +
 src/pages/audio/{_id.vue => _id/index.vue}    |   0
 src/pages/image/{_id.vue => _id/index.vue}    |   0
 src/pages/image/_id/report.vue                | 106 ++++++++++++++++++
 7 files changed, 123 insertions(+), 2 deletions(-)
 rename src/pages/audio/{_id.vue => _id/index.vue} (100%)
 rename src/pages/image/{_id.vue => _id/index.vue} (100%)
 create mode 100644 src/pages/image/_id/report.vue

diff --git a/src/components/VContentReport/VContentReportForm.vue b/src/components/VContentReport/VContentReportForm.vue
index e870944f04..a420d70cad 100644
--- a/src/components/VContentReport/VContentReportForm.vue
+++ b/src/components/VContentReport/VContentReportForm.vue
@@ -1,5 +1,5 @@
 <template>
-  <div id="content-report-form" class="w-80 p-6">
+  <div id="content-report-form">
     <div v-if="status === SENT">
       <p class="heading-6 mb-4">
         {{ $t("media-details.content-report.success.title") }}
@@ -76,7 +76,11 @@
         </div>
 
         <div class="flex flex-row items-center justify-end gap-4">
-          <VButton variant="secondary-bordered" @click="handleCancel">
+          <VButton
+            v-if="allowCancel"
+            variant="secondary-bordered"
+            @click="handleCancel"
+          >
             {{ $t("media-details.content-report.form.cancel") }}
           </VButton>
 
@@ -146,6 +150,7 @@ export default defineComponent({
     providerName: { required: true },
     reportService: { required: false },
     closeFn: { required: true },
+    allowCancel: { type: Boolean, default: true },
   },
   setup(props) {
     const service = props.reportService || ReportService
diff --git a/src/components/VContentReport/VContentReportPopover.vue b/src/components/VContentReport/VContentReportPopover.vue
index d0a878c3ec..8ea4c7dbb0 100644
--- a/src/components/VContentReport/VContentReportPopover.vue
+++ b/src/components/VContentReport/VContentReportPopover.vue
@@ -17,6 +17,7 @@
           @click="close"
         />
         <VContentReportForm
+          class="w-80 p-6"
           :close-fn="close"
           :media="media"
           :provider-name="media.providerName"
diff --git a/src/composables/use-match-routes.ts b/src/composables/use-match-routes.ts
index c2cdb79418..d66cfdbd4f 100644
--- a/src/composables/use-match-routes.ts
+++ b/src/composables/use-match-routes.ts
@@ -61,6 +61,12 @@ export const useMatchSingleResultRoutes = () => {
       .filter((name) => name !== ALL_MEDIA)
       .map((name) => `${name}-id`),
   ]
+  // @TODO Switch to more generic implementation once
+  // an Audio reporting page is designed.
+  //
+  // routes = routes.concat(routes.map((name) => `${name}-report`))
+  routes.push("image-id-report")
+
   return useMatchRoute(routes)
 }
 
diff --git a/src/locales/scripts/en.json5 b/src/locales/scripts/en.json5
index 43db5dad50..24c3fe40f7 100644
--- a/src/locales/scripts/en.json5
+++ b/src/locales/scripts/en.json5
@@ -713,4 +713,7 @@
     none: "No recent searches to show.",
     disclaimer: "Openverse does not store your recent searches, this information is kept locally in your browser.",
   },
+  report: {
+    "image-details": "See image details",
+  },
 }
diff --git a/src/pages/audio/_id.vue b/src/pages/audio/_id/index.vue
similarity index 100%
rename from src/pages/audio/_id.vue
rename to src/pages/audio/_id/index.vue
diff --git a/src/pages/image/_id.vue b/src/pages/image/_id/index.vue
similarity index 100%
rename from src/pages/image/_id.vue
rename to src/pages/image/_id/index.vue
diff --git a/src/pages/image/_id/report.vue b/src/pages/image/_id/report.vue
new file mode 100644
index 0000000000..0326955da2
--- /dev/null
+++ b/src/pages/image/_id/report.vue
@@ -0,0 +1,106 @@
+<template>
+  <div
+    class="mx-auto mt-8 mb-6 max-w-none gap-x-10 px-4 md:grid md:max-w-4xl md:grid-cols-2 md:px-6 lg:mb-30 lg:px-0 xl:max-w-4xl"
+  >
+    <figure class="mb-6 flex flex-col items-start gap-y-4">
+      <img
+        id="main-image"
+        :src="imageSrc"
+        :alt="image.title"
+        class="mx-auto h-auto w-full rounded-sm"
+        :width="imageWidth"
+        :height="imageHeight"
+      />
+      <!-- Disable reason: We control the attribution HTML generation so this is safe and will not lead to XSS attacks -->
+      <!-- eslint-disable vue/no-v-html -->
+      <caption
+        class="block w-full text-left text-sr"
+        v-html="getAttributionMarkup({ includeIcons: false })"
+      />
+      <!-- eslint-enable vue/no-v-html -->
+      <VButton
+        variant="secondary-bordered"
+        :href="`/image/${image.id}`"
+        as="VLink"
+        size="disabled"
+        class="p-2 text-sr text-dark-charcoal"
+      >
+        {{ $t("report.image-details") }}
+      </VButton>
+    </figure>
+
+    <VContentReportForm
+      :close-fn="() => {}"
+      :media="image"
+      :allow-cancel="false"
+      :provider-name="image.providerName"
+    />
+  </div>
+</template>
+
+<script lang="ts">
+import { defineComponent, ref, computed } from "@nuxtjs/composition-api"
+
+import { useI18n } from "~/composables/use-i18n"
+
+import { IMAGE } from "~/constants/media"
+
+import { useSingleResultStore } from "~/stores/media/single-result"
+import type { ImageDetail } from "~/types/media"
+import { AttributionOptions, getAttribution } from "~/utils/attribution-html"
+
+import VButton from "~/components/VButton.vue"
+
+export default defineComponent({
+  name: "ReportImage",
+  components: {
+    VButton,
+  },
+  setup() {
+    const i18n = useI18n()
+    const singleResultStore = useSingleResultStore()
+    const image = computed(() =>
+      singleResultStore.mediaType === IMAGE
+        ? (singleResultStore.mediaItem as ImageDetail)
+        : null
+    )
+    const imageWidth = ref(0)
+    const imageHeight = ref(0)
+    const imageType = ref("Unknown")
+    /**
+     * To make sure that image is loaded fast, we `src` to `image.thumbnail`,
+     * and then replace it with the provider image once it is loaded.
+     */
+    const imageSrc = ref(image.value.thumbnail)
+
+    const getAttributionMarkup = (options?: AttributionOptions) =>
+      getAttribution(image.value, i18n, options)
+
+    return {
+      image,
+      imageWidth,
+      imageHeight,
+      imageType,
+      imageSrc,
+      getAttributionMarkup,
+    }
+  },
+  async asyncData({ app, error, route, $pinia }) {
+    const imageId = route.params.id
+    const singleResultStore = useSingleResultStore($pinia)
+    try {
+      await singleResultStore.fetch(IMAGE, imageId)
+    } catch (err) {
+      const errorMessage = app.i18n
+        .t("error.image-not-found", {
+          id: imageId,
+        })
+        .toString()
+      return error({
+        statusCode: 404,
+        message: errorMessage,
+      })
+    }
+  },
+})
+</script>