From e5562cddb10acdf4ce47081f511b1c25dea80e54 Mon Sep 17 00:00:00 2001
From: krzysiej <krzysztofplaczek@techtube.pl>
Date: Thu, 2 Jun 2022 13:43:08 +0200
Subject: [PATCH] Vue event bus, uploading files handles from file service.

---
 ...ntroller.js => filedropzone_controller.js} | 13 ++---
 assets/js/event-bus.js                        |  3 ++
 assets/js/pages/book.vue                      |  1 -
 assets/js/pages/files.vue                     | 15 +++++-
 src/Controller/BookController.php             | 38 ++++-----------
 src/Form/BookType.php                         |  2 +-
 src/Form/FileType.php                         |  2 +-
 src/Service/FileService.php                   | 47 ++++++++++++++++---
 8 files changed, 69 insertions(+), 52 deletions(-)
 rename assets/controllers/{mydropzone_controller.js => filedropzone_controller.js} (83%)
 create mode 100644 assets/js/event-bus.js

diff --git a/assets/controllers/mydropzone_controller.js b/assets/controllers/filedropzone_controller.js
similarity index 83%
rename from assets/controllers/mydropzone_controller.js
rename to assets/controllers/filedropzone_controller.js
index 11dba72..4e28a5a 100644
--- a/assets/controllers/mydropzone_controller.js
+++ b/assets/controllers/filedropzone_controller.js
@@ -1,4 +1,4 @@
-// mydropzone_controller.js
+// filedropzone_controller.js
 
 import { Controller } from '@hotwired/stimulus';
 import axios from 'axios';
@@ -19,13 +19,7 @@ export default class extends Controller {
 
     _onConnect(event) {
         // The dropzone was just created
-        // console.info(event);
         console.info('onconnect');
-        //
-        // axios.get('/book/search/modyfikowany+węgiel').then(response => {
-        //     console.info(response.data);
-        // });
-
     }
 
     _onChange(event) {
@@ -36,14 +30,13 @@ export default class extends Controller {
                 'Content-Type': 'multipart/form-data'
             }
         }).then(() => {
-            event.target.querySelector('.dropzone-preview-button').click()
+            event.target.querySelector('.dropzone-preview-button').click();
+            window.EventBus.$emit('fileUploaded');
         })
-
     }
 
     _onClear(event) {
         // The dropzone has just been cleared
-        // console.info(event);
         console.info('onclear');
     }
 }
\ No newline at end of file
diff --git a/assets/js/event-bus.js b/assets/js/event-bus.js
new file mode 100644
index 0000000..0b39f8e
--- /dev/null
+++ b/assets/js/event-bus.js
@@ -0,0 +1,3 @@
+import Vue from 'vue';
+window.EventBus = new Vue();
+export const EventBus = window.EventBus;
\ No newline at end of file
diff --git a/assets/js/pages/book.vue b/assets/js/pages/book.vue
index 2083623..dc4e58c 100644
--- a/assets/js/pages/book.vue
+++ b/assets/js/pages/book.vue
@@ -47,7 +47,6 @@ export default {
   },
   methods: {
     search: function () {
-      console.info(this.searchTerm);
       axios.get('/book/search/' + this.searchTerm).then(response => {
         this.books = response.data;
       });
diff --git a/assets/js/pages/files.vue b/assets/js/pages/files.vue
index 52bd1fb..279849a 100644
--- a/assets/js/pages/files.vue
+++ b/assets/js/pages/files.vue
@@ -21,7 +21,8 @@
           <td class="text-end">{{ formatSize(file.fileSize) }}</td>
           <td>{{ file.extension }}</td>
           <td><a :href="'/file/'+ file.id" class="link-secondary">download</a></td>
-          <td><a :href="'/file/delete/'+file.id" class="link-danger">remove</a></td>
+          <td><a :href="'/file/delete/'+file.id" @click.prevent="deleteFile(file.id)" class="link-danger">remove</a>
+          </td>
         </tr>
         </tbody>
       </table>
@@ -31,9 +32,13 @@
 
 <script>
 import axios from 'axios';
+import {EventBus} from "../event-bus";
 
 export default {
   name: 'Files',
+  components: {
+    EventBus
+  },
   data() {
     return {
       files: []
@@ -42,9 +47,15 @@ export default {
   mounted() {
     this.getFiles();
   },
+  created() {
+    window.EventBus.$on('fileUploaded', this.getFiles);
+  },
   methods: {
+    deleteFile: function (fileId) {
+      axios.get(window.location.origin + '/file/delete/' + fileId).then(() => this.getFiles())
+    },
     formatSize: function (bytes) {
-      if (bytes == 0) {
+      if (bytes === 0) {
         return "0.00 B";
       }
       var e = Math.floor(Math.log(bytes) / Math.log(1024));
diff --git a/src/Controller/BookController.php b/src/Controller/BookController.php
index d8e1014..5e74c69 100644
--- a/src/Controller/BookController.php
+++ b/src/Controller/BookController.php
@@ -10,6 +10,7 @@ use App\Repository\BookRepository;
 use App\Repository\FileRepository;
 use App\Service\FileService;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\File\UploadedFile;
 use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
@@ -86,27 +87,15 @@ class BookController extends AbstractController
     }
 
     #[Route('/{id}', name: 'app_book_show', methods: ['GET', 'POST'])]
-    public function show(Book $book, Request $request, FileRepository $fileRepository): Response
+    public function show(Book $book, Request $request, FileRepository $fileRepository, FileService $fileService): Response
     {
         $fileForm = $this->createForm(FileType::class);
         $fileForm->handleRequest($request);
 
         if ($fileForm->isSubmitted() && $fileForm->isValid()) {
-            /** @var \Symfony\Component\HttpFoundation\File\UploadedFile[] $ebooks */
+            /** @var UploadedFile[] $ebooks */
             $ebook = $request->files->get('file')['file'];
-            $file = new File();
-            $file->setFileName(trim($ebook->getClientOriginalName()));
-            $file->setFileSize($ebook->getSize());
-            $file->setExtension($ebook->guessClientExtension());
-            $file->setBook($book);
-            $fileRepository->add($file, true);
-
-            $ebook->move(
-                $this->getParameter('book_files'),
-                'ebook_' . $book->getId() . '_' .
-                md5($file->getFileName()) . '.' .
-                $file->getExtension()
-            );
+            $fileService->saveFile($ebook, $book);
         }
 
         return $this->renderForm('book/show.html.twig', [
@@ -127,7 +116,8 @@ class BookController extends AbstractController
         Request $request,
         Book $book,
         BookRepository $bookRepository,
-        FileRepository $fileRepository
+        FileRepository $fileRepository,
+        FileService $fileService
     ): Response {
         $form = $this->createForm(BookType::class, $book);
         $form->handleRequest($request);
@@ -141,23 +131,11 @@ class BookController extends AbstractController
                 );
             }
 
-            /** @var \Symfony\Component\HttpFoundation\File\UploadedFile[] $ebooks */
+            /** @var UploadedFile[] $ebooks */
             $ebooks = $request->files->get('book')['ebook'];
             if (count($ebooks) > 0) {
                 foreach ($ebooks as $ebook) {
-                    $file = new File();
-                    $file->setFileName(trim($ebook->getClientOriginalName()));
-                    $file->setFileSize($ebook->getSize());
-                    $file->setExtension($ebook->guessClientExtension());
-                    $file->setBook($book);
-                    $fileRepository->add($file, true);
-
-                    $ebook->move(
-                        $this->getParameter('book_files'),
-                        'ebook_' . $book->getId() . '_' .
-                        md5($file->getFileName()) . '.' .
-                        $file->getExtension()
-                    );
+                    $fileService->saveFile($ebook, $book);
                 }
             }
 
diff --git a/src/Form/BookType.php b/src/Form/BookType.php
index 0ab09ad..94a7ec0 100644
--- a/src/Form/BookType.php
+++ b/src/Form/BookType.php
@@ -55,7 +55,7 @@ class BookType extends AbstractType
             ])
 //            ->add('cover', FileType::class, ['mapped' => false, 'data_class' => null, 'required' => false])
             ->add('cover_url', TextType::class, ['mapped' => false, 'help' => 'Fill in the field with a link to a cover image to use it as a cover for the book.', 'label'=> 'Cover url', 'required' => false])
-            ->add('cover', DropzoneType::class,  ['mapped' => false, 'data_class' => null, 'required' => false, 'attr' => ['data-controller' => 'mydropzone', 'accept' => "image/*", 'placeholder' => 'Drag and drop or browse']])
+            ->add('cover', DropzoneType::class,  ['mapped' => false, 'data_class' => null, 'required' => false, 'attr' => ['accept' => "image/*", 'placeholder' => 'Drag and drop or browse']])
         ;
     }
 
diff --git a/src/Form/FileType.php b/src/Form/FileType.php
index c8400ae..2418207 100644
--- a/src/Form/FileType.php
+++ b/src/Form/FileType.php
@@ -22,7 +22,7 @@ class FileType extends AbstractType
                     'data_class' => null,
                     'required' => true,
                     'attr' => [
-                        'data-controller' => 'mydropzone',
+                        'data-controller' => 'filedropzone',
                         'accept' => ".pdf, .epub, .mobi",
                         'placeholder' => 'Drag and drop or browse'
                     ]
diff --git a/src/Service/FileService.php b/src/Service/FileService.php
index ac042ba..da13b7a 100644
--- a/src/Service/FileService.php
+++ b/src/Service/FileService.php
@@ -2,35 +2,68 @@
 
 namespace App\Service;
 
+use App\Entity\Book;
 use App\Entity\File;
 use App\Repository\FileRepository;
 use Doctrine\Common\Collections\Collection;
 use Symfony\Component\Filesystem\Filesystem;
+use Symfony\Component\HttpFoundation\File\UploadedFile;
 
 class FileService
 {
     private Filesystem $fileSystem;
     private string $bookFiles;
+    private FileRepository $fileRepository;
 
-    public function __construct(string $bookFiles, FileSystem $filesystem)
+    public function __construct(string $bookFiles, FileSystem $filesystem, FileRepository $fileRepository,)
     {
         $this->bookFiles = $bookFiles;
         $this->fileSystem = $filesystem;
+        $this->fileRepository = $fileRepository;
     }
 
     /**
-     * @param File[] $files
+     * @param Collection $files
      * @return void
      */
     public function removeFiles(Collection $files)
     {
         foreach ($files as $file) {
-            $filePath = $this->bookFiles . '/' . 'ebook_' . $file->getBook()->getId() . '_' .
-                md5($file->getFileName()) . '.' .
-                $file->getExtension();
-
-            $this->fileSystem->remove($filePath);
+            $this->fileSystem->remove($this->getFilePath($file));
         }
     }
 
+    public function getFilePath(File $file): string
+    {
+        return $this->bookFiles . DIRECTORY_SEPARATOR . $this->getFileName($file);
+    }
+
+    public function getFileName(File $file): string
+    {
+        return 'ebook_' . $file->getBook()->getId() . '_' .
+            md5($file->getFileName()) . '.' .
+            $file->getExtension();
+    }
+
+    /**
+     * @param UploadedFile $ebook
+     * @param Book $book
+     * @return File
+     */
+    public function saveFile(UploadedFile $ebook, Book $book): File
+    {
+        $file = new File();
+        $file->setFileName(trim($ebook->getClientOriginalName()));
+        $file->setFileSize($ebook->getSize());
+        $file->setExtension($ebook->guessClientExtension());
+        $file->setBook($book);
+        $this->fileRepository->add($file, true);
+
+        $ebook->move(
+            $this->bookFiles,
+            $this->getFileName($file)
+        );
+        return $file;
+    }
+
 }
\ No newline at end of file