active_storage-send_zip 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b7486688b7fedf6040f5c36b5fa24547bb16d9426e0e61b5a00c88bb3bf87d17
-  data.tar.gz: 712e6d73af9d4515a367192f11bec18a0a0c7f5581a5ca0f8665503e9d95fbd6
+  metadata.gz: 8207888c37b524f02c443b39a3a886c570b4fe6d8a1fab0dc962ede883a24f3d
+  data.tar.gz: 4b960e75cdc3f948e6e77eddea418e8c6e8b07df5166d4739006569957f60d99
 SHA512:
-  metadata.gz: 60c8312e5c80fb0663cfd396dfcbb4a72ad3e2575184ce476c27108d28b1803b7a37308d8b50618bd32787a8c780aa8fd5a5dd56554ee5ff5703f9e7099cf992
-  data.tar.gz: 57326de525bd8cceab5005c975b89a78bc01939a8a2ee359b9591c20ff41baf9dd47e9a36ffdeadbbaa316461a366f7e7aaa7ec4b0c4a0b4ce98a07886bdd883
+  metadata.gz: 5558ce4b108d0a28a046f774adea7e73e15db5def44e31943ba13c28329ce2a5a35c16285e44b32beef7aa7089f0d3c518208e1cee927316195d84fb4f87f36f
+  data.tar.gz: 27511b5c8a954c4bf1176fc2433b005c42a0265549f649857c46de051e31a6723482e1f741f18f2b7662f0fee8ef8385ecab9f51cea360891bc66bb7d00c216a

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    active_storage-send_zip (0.1.0)
+    active_storage-send_zip (0.1.1)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -1,5 +1,7 @@
 # ActiveStorage::SendZip
 
+[![Gem Version](https://badge.fury.io/rb/active_storage-send_zip.svg)](https://badge.fury.io/rb/active_storage-send_zip)
+
 Create a zip from one or more Active Storage objects and return it in a rails controller
 
 ## Installation

data/lib/active_storage/send_zip.rb

@@ -1,6 +1,8 @@
 require 'active_storage/send_zip/version'
 require 'rails'
 require 'zip'
+require 'tempfile'
+require 'pathname'
 
 module ActiveStorage
   module SendZip
@@ -29,16 +31,32 @@ module ActiveStorage
     def save_files_on_server(files)
       require 'zip'
       # get a temporary folder and create it
-      temp_folder = File.join(Dir.tmpdir, 'user')
-      FileUtils.mkdir_p(temp_folder) unless Dir.exist?(temp_folder)
+      temp_folder = Dir.mktmpdir 'active_storage-send_zip'
+
+      # count each files to avoid duplicates
+      filepaths = []
 
       # download all ActiveStorage into
-      files.map do |picture|
+      files.each do |picture|
         filename = picture.filename.to_s
         filepath = File.join temp_folder, filename
+
+        # ensure that filename not exists
+        if filepaths.include? filepath
+          # create a new random filenames
+          basename = File.basename filename
+          extension = File.extname filename
+
+          filename = "#{basename}_#{SecureRandom.uuid}#{extension}"
+          filepath = File.join temp_folder, filename
+        end
+
         File.open(filepath, 'wb') { |f| f.write(picture.download) }
-        filepath
+
+        filepaths << filepath
       end
+
+      filepaths
     end
 
     # Create a temporary zip file & return the content as bytes

data/lib/active_storage/send_zip/version.rb

@@ -1,5 +1,5 @@
 module ActiveStorage
   module SendZip
-    VERSION = "0.1.0"
+    VERSION = '0.1.1'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_storage-send_zip
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Alexandre Rousseau
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-11-30 00:00:00.000000000 Z
+date: 2018-12-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler