From 6c251f797d448d9db4e2eed2e631e440f3f6e8a4 Mon Sep 17 00:00:00 2001
From: Julien Calixte <julienc@bam.tech>
Date: Sun, 2 Jul 2023 23:25:14 +0200
Subject: [PATCH] =?UTF-8?q?=E2=9C=A8=20(card)=20make=20the=20buttons=20wor?=
 =?UTF-8?q?k?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/data/data.ts                              |  25 +++-
 src/hooks/useMarkdown.hook.ts                 |   8 +-
 src/modules/card/components/FlipCard.vue      |  42 +++---
 .../card/hooks/useSpacedRepetitionCards.ts    | 123 +++++++++++-------
 src/views/SpacedRepetitionCard.vue            |  59 ++++-----
 5 files changed, 148 insertions(+), 109 deletions(-)

diff --git a/src/data/data.ts b/src/data/data.ts
index d354d1a..831ec69 100644
--- a/src/data/data.ts
+++ b/src/data/data.ts
@@ -1,8 +1,8 @@
+import { nanoid } from 'nanoid'
+import indexedDb from 'pouchdb-adapter-indexeddb'
+import PouchDb from 'pouchdb-browser'
 import { DataType } from './DataType.enum'
 import { Model } from './models/Model'
-import PouchDb from 'pouchdb-browser'
-import indexedDb from 'pouchdb-adapter-indexeddb'
-import { nanoid } from 'nanoid'
 
 PouchDb.plugin(indexedDb)
 
@@ -38,7 +38,9 @@ class Data {
     }
   }
 
-  public async update<DT extends DataType>(model: Model<DT>): Promise<boolean> {
+  public async update<DT extends DataType, T extends Model<DT>>(
+    model: T
+  ): Promise<boolean> {
     try {
       if (model._id) {
         const oldModel = await this.get(model._id)
@@ -82,6 +84,21 @@ class Data {
     }
   }
 
+  public async getOrCreate<DT extends DataType, T extends Model<DT>>(
+    id: string,
+    initialValue: T
+  ): Promise<T> {
+    const element = await this.get<DT, T>(id)
+
+    if (element) {
+      return element
+    }
+
+    await data.add<DT>({ ...initialValue, _id: id })
+
+    return this.getOrCreate(id, initialValue)
+  }
+
   public async getAll<DT extends DataType, T extends Model<DT>>({
     prefix,
     includeDocs = true,
diff --git a/src/hooks/useMarkdown.hook.ts b/src/hooks/useMarkdown.hook.ts
index c79bb26..a4cd7b7 100644
--- a/src/hooks/useMarkdown.hook.ts
+++ b/src/hooks/useMarkdown.hook.ts
@@ -1,11 +1,11 @@
-import MarkdownIt from 'markdown-it'
+import { decodeBase64ToUTF8 } from '@/utils/decodeBase64ToUTF8'
+import { html5Media } from '@/utils/markdown/markdown-html5-media'
 import markdownItClass from '@toycode/markdown-it-class'
+import MarkdownIt from 'markdown-it'
 import blockEmbedPlugin from 'markdown-it-block-embed'
 import markdownItCheckbox from 'markdown-it-checkbox'
-import markdownItSvgCodeCopy from 'markdown-it-svg-code-copy'
 import markdownItFootnote from 'markdown-it-footnote'
-import { html5Media } from '@/utils/markdown/markdown-html5-media'
-import { decodeBase64ToUTF8 } from '@/utils/decodeBase64ToUTF8'
+import markdownItSvgCodeCopy from 'markdown-it-svg-code-copy'
 
 const md = new MarkdownIt({
   typographer: true,
diff --git a/src/modules/card/components/FlipCard.vue b/src/modules/card/components/FlipCard.vue
index 2392b77..985671e 100644
--- a/src/modules/card/components/FlipCard.vue
+++ b/src/modules/card/components/FlipCard.vue
@@ -1,3 +1,18 @@
+<script lang="ts" setup>
+import { ref } from 'vue'
+import { Card } from '../models/Card'
+
+defineProps<{ card: Card }>()
+const emit = defineEmits(['success', 'fail'])
+
+const flipped = ref(false)
+const flip = () => (flipped.value = !flipped.value)
+
+const success = () => emit('success')
+
+const fail = () => emit('fail')
+</script>
+
 <template>
   <div class="flip-card" :class="{ flipped }" @click="flip">
     <div class="flip-card-inner">
@@ -8,10 +23,10 @@
         <div class="references" v-html="card.references"></div>
         <hr />
         <div class="buttons is-centered">
-          <div class="button is-warning is-light" @click.stop="success">
+          <div class="button is-warning is-light" @click.stop="fail">
             failed
           </div>
-          <div class="button is-success is-light" @click.stop="fail">
+          <div class="button is-success is-light" @click.stop="success">
             got it
           </div>
         </div>
@@ -20,29 +35,6 @@
   </div>
 </template>
 
-<script lang="ts">
-import { defineComponent, PropType, ref } from 'vue'
-import { Card } from '../models/Card'
-
-export default defineComponent({
-  name: 'FlipCard',
-  props: {
-    card: { type: Object as PropType<Card>, required: true }
-  },
-  emits: ['success', 'fail'],
-  setup(_, context) {
-    const flipped = ref(false)
-    const flip = () => (flipped.value = !flipped.value)
-
-    const success = () => context.emit('success')
-
-    const fail = () => context.emit('fail')
-
-    return { flip, flipped, success, fail }
-  }
-})
-</script>
-
 <style scoped lang="scss">
 $border-radius: 0.5rem;
 
diff --git a/src/modules/card/hooks/useSpacedRepetitionCards.ts b/src/modules/card/hooks/useSpacedRepetitionCards.ts
index d97d5e5..9b0680b 100644
--- a/src/modules/card/hooks/useSpacedRepetitionCards.ts
+++ b/src/modules/card/hooks/useSpacedRepetitionCards.ts
@@ -7,8 +7,8 @@ import { Card } from '@/modules/card/models/Card'
 import { RepetitionCard } from '@/modules/card/models/RepetitionCard'
 import { useUserRepoStore } from '@/modules/repo/store/userRepo.store'
 import { decodeBase64ToUTF8 } from '@/utils/decodeBase64ToUTF8'
-import { asyncComputed } from '@vueuse/core'
-import { isAfter } from 'date-fns'
+import { useAsyncState } from '@vueuse/core'
+import { addDays, isAfter } from 'date-fns'
 import { computed, nextTick, watch } from 'vue'
 
 interface Repetition {
@@ -23,62 +23,95 @@ export const useSpacedRepetitionCards = () => {
 
   const cardFiles = computed(() =>
     store.files.filter(
-      (file) => file.path?.startsWith('_cards') && file.path?.endsWith('.md')
+      (file) =>
+        file.path !== undefined &&
+        file.path.startsWith('_cards') &&
+        file.path.endsWith('.md')
     )
   )
 
-  const cards = asyncComputed(async () => {
-    const cards: Repetition[] = []
+  const {
+    state: cards,
+    isReady,
+    execute
+  } = useAsyncState(
+    async () => {
+      const cards: Repetition[] = []
 
-    for (const cardFile of cardFiles.value) {
-      if (!cardFile.sha) {
-        continue
-      }
+      for (const cardFile of cardFiles.value) {
+        if (!cardFile.sha) {
+          continue
+        }
 
-      const repetitionId = data.generateId(
-        DataType.RepetitionCard,
-        cardFile.sha
-      )
-
-      let repetition = await data.get<DataType.RepetitionCard, RepetitionCard>(
-        repetitionId
-      )
-
-      if (!repetition) {
-        const newRepetition: RepetitionCard = {
-          _id: repetitionId,
+        const repetition = await data.getOrCreate<
+          DataType.RepetitionCard,
+          RepetitionCard
+        >(data.generateId(DataType.RepetitionCard, cardFile.path), {
           $type: DataType.RepetitionCard,
           level: 1,
           repeatDate: new Date()
+        })
+
+        if (isAfter(new Date(repetition.repeatDate), new Date())) {
+          continue
         }
-        await data.add<DataType.RepetitionCard>(newRepetition)
-        repetition = (await data.get<DataType.RepetitionCard, RepetitionCard>(
-          repetitionId
-        )) as RepetitionCard
+
+        const { getRawContent } = useFile(cardFile.sha, false)
+        const content = (await getRawContent()) ?? ''
+
+        const [front, back, references] =
+          decodeBase64ToUTF8(content).split('___') ?? []
+
+        cards.push({
+          repetition,
+          card: {
+            front: toHTML(front),
+            back: toHTML(back),
+            references: toHTML(references)
+          }
+        })
       }
 
-      if (isAfter(new Date(repetition.repeatDate), new Date())) {
-        continue
-      }
+      console.log(cards)
 
-      const { getRawContent } = useFile(cardFile.sha, false)
-      const content = await getRawContent()
+      return cards
+    },
+    [],
+    { immediate: false }
+  )
 
-      const [front, back, references] =
-        decodeBase64ToUTF8(content ?? '').split('___') ?? []
-
-      cards.push({
-        repetition,
-        card: {
-          front: toHTML(front),
-          back: toHTML(back),
-          references: toHTML(references)
-        }
-      })
+  const failRepetition = async (cardId: string) => {
+    const repetition = await data.get<DataType.RepetitionCard, RepetitionCard>(
+      cardId
+    )
+    if (!repetition) {
+      return
     }
 
-    return cards
-  }, [])
+    await data.update<DataType.RepetitionCard, RepetitionCard>({
+      ...repetition,
+      repeatDate: addDays(new Date(), 1)
+    })
+
+    await execute()
+  }
+
+  const successRepetition = async (cardId: string) => {
+    const repetition = await data.get<DataType.RepetitionCard, RepetitionCard>(
+      cardId
+    )
+    if (!repetition) {
+      return
+    }
+
+    await data.update<DataType.RepetitionCard, RepetitionCard>({
+      ...repetition,
+      level: repetition.level,
+      repeatDate: addDays(new Date(), repetition.level)
+    })
+
+    await execute()
+  }
 
   watch(
     cards,
@@ -89,5 +122,7 @@ export const useSpacedRepetitionCards = () => {
     { immediate: true }
   )
 
-  return { cards }
+  watch(cardFiles, () => execute(), { immediate: true })
+
+  return { cards, failRepetition, successRepetition, isLoading: !isReady }
 }
diff --git a/src/views/SpacedRepetitionCard.vue b/src/views/SpacedRepetitionCard.vue
index 7284524..28afbf1 100644
--- a/src/views/SpacedRepetitionCard.vue
+++ b/src/views/SpacedRepetitionCard.vue
@@ -1,3 +1,23 @@
+<script lang="ts" setup>
+import FluxNote from '@/components/FluxNote.vue'
+import FlipCard from '@/modules/card/components/FlipCard.vue'
+import { useSpacedRepetitionCards } from '@/modules/card/hooks/useSpacedRepetitionCards'
+import { computed } from 'vue'
+
+defineProps<{
+  user: string
+  repo: string
+}>()
+
+const { cards, isLoading, successRepetition, failRepetition } =
+  useSpacedRepetitionCards()
+
+const firstCard = computed(() => {
+  const [repetitionCard] = cards.value
+  return repetitionCard
+})
+</script>
+
 <template>
   <div class="spaced-repetition-card repo-note">
     <flux-note
@@ -6,45 +26,20 @@
       :repo="repo"
       :with-content="false"
     >
-      <section v-if="firstCard">
+      <section v-if="isLoading">Loading...</section>
+      <section v-else-if="firstCard">
         <h3 class="subtitle is-3">Level: {{ firstCard.repetition.level }}</h3>
-        <flip-card :card="firstCard.card" />
+        <flip-card
+          :card="firstCard.card"
+          @success="() => successRepetition(firstCard.repetition._id ?? '')"
+          @fail="() => failRepetition(firstCard.repetition._id ?? '')"
+        />
       </section>
       <section v-else>No cards to review!</section>
     </flux-note>
   </div>
 </template>
 
-<script lang="ts">
-import { useSpacedRepetitionCards } from '@/modules/card/hooks/useSpacedRepetitionCards'
-import { computed, defineComponent } from 'vue'
-import FluxNote from '@/components/FluxNote.vue'
-import FlipCard from '@/modules/card/components/FlipCard.vue'
-
-export default defineComponent({
-  name: 'SpacedRepetitionCard',
-  components: {
-    FluxNote,
-    FlipCard
-  },
-  props: {
-    user: { type: String, required: true },
-    repo: { type: String, required: true }
-  },
-  setup() {
-    const { cards } = useSpacedRepetitionCards()
-    const firstCard = computed(() => {
-      const [repetitionCard] = cards.value
-      return repetitionCard
-    })
-
-    return {
-      firstCard
-    }
-  }
-})
-</script>
-
 <style scoped lang="scss">
 .spaced-repetition-card {
   display: flex;