WIP: Fix Delete Slide

[arifpriadi]

Issue #989

[pandigresik]

Untuk nama file hasil upload, usahakan tidak menggunakan nama file yang diupload $fileName = $file->getClientOriginalName()
referensi : https://laravel.com/docs/10.x/filesystem#other-uploaded-file-information

bisa juga menggunakan uuid untuk nama file gambar yang disimpan

[vickyrolanda]

bung @arifpriadi mohon untuk pembaharuannya agar tim OpenSID bisa masukkan PR ini ke Rilis.

[arifpriadi]

Untuk nama file hasil upload, usahakan tidak menggunakan nama file yang diupload $fileName = $file->getClientOriginalName() referensi : https://laravel.com/docs/10.x/filesystem#other-uploaded-file-information

bisa juga menggunakan uuid untuk nama file gambar yang disimpan

Pak @pandigresik, pada PR ini hanya memperbaiki notifikasi yang tidak sesuai disebabkan karena file yang akan dihapus tidak ditemukan. Tidak ada kaitannya dengan unggah. Jika memasang dengan data demo, maka pada terdapat beberapa slider. Bisa dicoba pada data tsb.

[pandigresik]

Untuk nama file hasil upload, usahakan tidak menggunakan nama file yang diupload $fileName = $file->getClientOriginalName() referensi : https://laravel.com/docs/10.x/filesystem#other-uploaded-file-information
bisa juga menggunakan uuid untuk nama file gambar yang disimpan

Pak @pandigresik, pada PR ini hanya memperbaiki notifikasi yang tidak sesuai disebabkan karena file yang akan dihapus tidak ditemukan. Tidak ada kaitannya dengan unggah. Jika memasang dengan data demo, maka pada terdapat beberapa slider. Bisa dicoba pada data tsb.

Ketika lakukan perbaikan lebih baik sekalian, pada fitur delete digunakan penghapusan melalui model. Maka alangkah baiknya ubah juga fitur create/update menggunakan metode yang sama agar seragam yaitu penghapusan melalui model. Pada fitur slide ketika unggah file masih menggunakan nama file asli dari unggahan, akan lebih baik jika gunakan nama file random yang sulit ditebak dengan alasan keamanan. Sekalian diperbaiki karena ubahan di model juga

[arifpriadi]

Untuk nama file hasil upload, usahakan tidak menggunakan nama file yang diupload $fileName = $file->getClientOriginalName() referensi : https://laravel.com/docs/10.x/filesystem#other-uploaded-file-information
bisa juga menggunakan uuid untuk nama file gambar yang disimpan

Pak @pandigresik, pada PR ini hanya memperbaiki notifikasi yang tidak sesuai disebabkan karena file yang akan dihapus tidak ditemukan. Tidak ada kaitannya dengan unggah. Jika memasang dengan data demo, maka pada terdapat beberapa slider. Bisa dicoba pada data tsb.

Ketika lakukan perbaikan lebih baik sekalian, pada fitur delete digunakan penghapusan melalui model. Maka alangkah baiknya ubah juga fitur create/update menggunakan metode yang sama agar seragam yaitu penghapusan melalui model. Pada fitur slide ketika unggah file masih menggunakan nama file asli dari unggahan, akan lebih baik jika gunakan nama file random yang sulit ditebak dengan alasan keamanan. Sekalian diperbaiki karena ubahan di model juga

Disesuaikan dengan saran yang diberikan. Mencontoh dari https://github.com/OpenSID/premium/blob/rilis-dev/app/Models/SinergiProgram.php#L112-L134