rails-uploader 0.4.1 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 5b4b00fbcd6be9841837569fcbd89b22d291b035
-  data.tar.gz: 528c4ad6195d462a333439e477456562a1fd4c65
+SHA256:
+  metadata.gz: b602b1677568ea5d878174c76224c6a61d431efdb93046f8bd3b897dbed63f55
+  data.tar.gz: e2464535aa4a6f329bee939b98a32fba4193f472111bda3c669c5f8465d1ac62
 SHA512:
-  metadata.gz: 88594863634e997ce0d3b0997b8345ff75a0c7495a0dc6ac1bd08995267122207bda33ba3099b7b7201c92ce486f18a7010e02dc54607eaaf78c63efe3789f1b
-  data.tar.gz: 1f6d1b8e0304f6dac05886493ae5c2aad2d1a8baec701664d3aa18ad8afac2ddff01dd3aa7b24230f35f015418b7421aa2791a263a75c92257dfaad52b4baf54
+  metadata.gz: 194158c8d9f7540ed210e11b3cb1b98ff346c35562b960365dba01113caea71b808363832b613a804bcd45b2df2f815506d8c74c25532011384dd027f38f9660
+  data.tar.gz: d66527e6277747de70d148af352391998c83a75e08418e0a665c31130c1ea5117e06c2c039ce4b8fad73930e75fe5cfe45c77da72e509ee4da6fdb34ed72c5ee

data/README.md

@@ -1,3 +1,5 @@
+[![Build Status](https://travis-ci.org/superp/rails-uploader.svg?branch=master)](https://travis-ci.org/superp/rails-uploader)
+
 # HTML5 File uploader for rails
 
 This gem use https://github.com/blueimp/jQuery-File-Upload for upload files.
@@ -19,7 +21,7 @@ mount Uploader::Engine => '/uploader'
 Migration for ActiveRecord:
 
 ```bash
-$> rake uploader:install:migrations
+  $ bundle exec rails g uploader:install
 ```
 
 ## Usage

data/Rakefile

@@ -1,18 +1,7 @@
 # encoding: utf-8
-require 'rake'
-require 'rake/testtask'
-require 'rake/rdoctask'
-
-desc 'Default: run unit tests.'
-task :default => :test
-
-desc 'Test the rails-uploader plugin.'
-Rake::TestTask.new(:test) do |t|
-  t.libs << 'lib'
-  t.libs << 'test'
-  t.pattern = 'test/**/*_test.rb'
-  t.verbose = true
-end
+require 'rdoc/task'
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 desc 'Generate documentation for the rails-uploader plugin.'
 Rake::RDocTask.new(:rdoc) do |rdoc|
@@ -22,3 +11,7 @@ Rake::RDocTask.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('README.rdoc')
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
+
+RSpec::Core::RakeTask.new(:spec)
+
+task default: :spec

data/app/assets/javascripts/uploader/jquery.uploader.js.coffee

@@ -34,6 +34,7 @@ class UploaderWidget
     @options['filesContainer'] = @container
     @options['uploadTemplateId'] += @template
     @options['downloadTemplateId'] += @template
+    @options.singular = !@input.prop('multiple')
 
     this._initFileupload()
 

data/app/controllers/uploader/attachments_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Uploader
   class AttachmentsController < ActionController::Metal
     include AbstractController::Callbacks
@@ -5,8 +7,8 @@ module Uploader
     include Uploader::ChunkedUploads
 
     before_action :find_klass
-    before_action :build_asset, only: [:create]
-    before_action :find_asset, only: [:destroy]
+    before_action :build_asset, only: :create
+    before_action :find_asset, only: :destroy
 
     def index
       authorize!(:index, @klass)
@@ -46,7 +48,7 @@ module Uploader
 
     def find_klass
       @klass = Uploader.constantize(params[:klass])
-      raise ActionController::RoutingError, "Class not found #{params[:klass]}" if @klass.nil?
+      raise ActionController::ParameterMissing.new("klass parameter isn't set but #{params[:klass].inspect}") if @klass.nil?
     end
 
     def build_asset

data/config/locales/ru.yml

@@ -1,6 +1,6 @@
 ru:
   uploader:
-    button: "Выберите файл на компьютере"
+    button: "Выберите файл"
     drop: "Перетащите файл сюда"
     or: "или"
     confirm_delete: "Are you sure?"

data/config/locales/uk.yml

@@ -1,6 +1,6 @@
 uk:
   uploader:
-    button: "Виберіть файл на комп'ютері"
+    button: "Оберіть файл"
     drop: "Перетягніть файл сюди"
     or: "або"
     confirm_delete: "Are you sure?"

data/config/routes.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Uploader::Engine.routes.draw do
   resources :attachments, only: [:index, :create, :update, :destroy]
 end

data/lib/generators/uploader/install/install_generator.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require 'rails/generators/migration'
+
+module Uploader
+  module Generators
+    class InstallGenerator < ::Rails::Generators::Base
+      include Rails::Generators::Migration
+
+      source_root File.expand_path('templates', __dir__)
+
+      desc 'add the migrations'
+      def self.next_migration_number(_path)
+        if @prev_migration_nr
+          @prev_migration_nr += 1
+        else
+          @prev_migration_nr = Time.now.utc.strftime('%Y%m%d%H%M%S').to_i
+        end
+
+        @prev_migration_nr.to_s
+      end
+
+      def copy_migrations
+        migration_template '20130905144515_create_assets.rb', 'db/migrate/create_assets.rb'
+      end
+    end
+  end
+end

data/lib/generators/uploader/install/templates/20130905144515_create_assets.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+class CreateAssets < ActiveRecord::Migration
+  def self.up
+    create_table :assets do |t|
+      t.string  :data_file_name, null: false
+      t.string  :data_content_type
+      t.integer :data_file_size
+
+      t.integer :assetable_id, null: false
+      t.string  :assetable_type, limit: 30, null: false
+      t.string  :type, limit: 30
+
+      t.string  :guid, limit: 30
+      t.string  :public_token, limit: 30
+
+      t.integer :user_id
+      t.integer :sort_order, default: 0
+      t.integer :width
+      t.integer :height
+
+      t.timestamps
+    end
+
+    add_index :assets, [:assetable_type, :type, :assetable_id]
+    add_index :assets, :user_id
+    add_index :assets, :guid
+    add_index :assets, :public_token
+  end
+
+  def self.down
+    drop_table :assets
+  end
+end

data/lib/rails-uploader.rb

@@ -1,2 +1,3 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 require 'uploader'

data/lib/uploader.rb

@@ -1,4 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 require 'securerandom'
 require 'uploader/version'
 
@@ -64,6 +65,7 @@ module Uploader
 
   def self.constantize(klass)
     return if klass.blank?
+
     klass.safe_constantize
   end
 

data/lib/uploader/asset.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/concern'
 
 module Uploader
@@ -49,7 +51,22 @@ module Uploader
     #
     def fileupload_create(params, _request = nil)
       self[Uploader.guid_column] = params[:guid]
-      update_attributes(self.class.fileupload_assetable_options(params))
+      return false unless update_attributes(self.class.fileupload_assetable_options(params))
+
+      if fileupload_destroy_other_on_singular?(params)
+        self.class.fileupload_find_assets(params).where.not(id: id).destroy_all
+      end
+      true
+    end
+
+    def fileupload_destroy_other_on_singular?(params)
+      return unless params[:singular].to_s.downcase == 'true'
+      return true if params[:guid].present?
+
+      [
+        params["#{Uploader.assetable_column}_id"],
+        params["#{Uploader.assetable_column}_type"]
+      ].all?(&:present?)
     end
 
     # Destroy asset
@@ -80,7 +97,7 @@ module Uploader
         name: filename,
         content_type: content_type,
         size: size,
-        url:  url,
+        url: url,
         thumb_url: thumb_url
       }
     end

data/lib/uploader/authorization.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/concern'
 
 module Uploader
@@ -21,6 +23,7 @@ module Uploader
     # If the action is not allowd, it raises an Uploader::AccessDenied exception.
     def authorize!(action, subject = nil)
       return if authorized?(action, subject)
+
       raise Uploader::AccessDenied.new(current_uploader_user, action, subject)
     end
 
@@ -46,6 +49,7 @@ module Uploader
 
     def current_uploader_user
       return if Uploader.current_user_proc.nil?
+
       @current_uploader_user ||= Uploader.current_user_proc.call(request)
     end
   end

data/lib/uploader/authorization_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Uploader
   class AuthorizationAdapter
     attr_reader :user

data/lib/uploader/chunked_uploads.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Uploader
   module ChunkedUploads
     extend ActiveSupport::Concern
@@ -5,7 +7,7 @@ module Uploader
     protected
 
     def with_chunked_upload(file_or_part)
-      uploader = UploadRequest.new(env, file_or_part)
+      uploader = UploadRequest.new(request.env, file_or_part)
       return unless uploader.completed?
 
       yield uploader.file

data/lib/uploader/engine.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails'
 
 module Uploader

data/lib/uploader/file_part.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Uploader
   class FilePart < File
     def initialize(path, filename)

data/lib/uploader/fileupload_glue.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Uploader
   class FileuploadGlue
     attr_reader :record, :record_klass
@@ -5,8 +7,8 @@ module Uploader
     delegate :reflect_on_association, to: :record_klass
     delegate :fileupload_guid, :new_record?, to: :record
 
-    TARGET_TYPE = '_type'.freeze
-    TARGET_ID = '_id'.freeze
+    TARGET_TYPE = '_type'
+    TARGET_ID = '_id'
 
     def initialize(record)
       @record = record
@@ -36,6 +38,7 @@ module Uploader
 
     def asset(method_name)
       return unless available_fileuploads.include?(method_name.to_sym)
+
       find_asset_by_fileupload_guid(method_name, fileupload_guid) || build_asset(method_name)
     end
 
@@ -47,6 +50,7 @@ module Uploader
     # Find class by reflection
     def klass(method_name)
       return if association(method_name).nil?
+
       association(method_name).klass
     end
 
@@ -62,6 +66,7 @@ module Uploader
 
     def available_fileuploads
       return [] if @record_klass.fileupload_options.nil?
+
       @available_fileuploads ||= @record_klass.fileupload_options.keys
     end
 
@@ -97,7 +102,9 @@ module Uploader
     end
 
     def record_klass_type
-      if @record_klass.respond_to?(:base_class)
+      if @record_klass.respond_to?(:polymorphic_name)
+        @record_klass.polymorphic_name
+      elsif @record_klass.respond_to?(:base_class)
         @record_klass.base_class.name
       else
         @record_klass.name

data/lib/uploader/fileuploads.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/concern'
 
 module Uploader
@@ -40,7 +42,7 @@ module Uploader
     end
 
     def fileupload_guid=(value)
-      @fileupload_changed = (@fileupload_guid != value)
+      @fileupload_changed = (@fileupload_guid != value) if @fileupload_changed.nil?
       @fileupload_guid = value.blank? ? nil : value
     end
 

data/lib/uploader/helpers/field_tag.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module Uploader
   module Helpers
     class FieldTag
-      RESERVED_OPTIONS_KEYS = %(method_name object_name theme value object sortable).freeze
+      RESERVED_OPTIONS_KEYS = %w[method_name object_name theme value object sortable].freeze
 
       attr_reader :template, :object, :theme
 
@@ -73,12 +75,12 @@ module Uploader
       def input_html
         @input_html ||= { multiple: multiple?, class: 'uploader' }.merge(input_html_options)
         @input_html[:data] ||= {}
-        @input_html[:data][:url] ||= attachments_path
+        @input_html[:data][:url] ||= attachments_path(singular: !multiple?)
         @input_html
       end
 
       def input_html_options
-        @options.select { |key, _value| !RESERVED_OPTIONS_KEYS.include?(key.to_s) }
+        @options.reject { |key, _value| RESERVED_OPTIONS_KEYS.include?(key.to_s) }
       end
     end
   end

data/lib/uploader/helpers/form_builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Uploader
   module Helpers
     module FormBuilder

data/lib/uploader/helpers/form_tag_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Uploader
   module Helpers
     module FormTagHelper

data/lib/uploader/hooks/active_record.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_record'
 
 ActiveSupport.on_load :active_record do

data/lib/uploader/hooks/formtastic.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'formtastic'
 
 module Formtastic

data/lib/uploader/hooks/simple_form.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'simple_form'
 
 module SimpleForm

data/lib/uploader/upload_request.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rack/request'
 require 'fileutils'
 require 'digest/sha1'
@@ -5,7 +7,7 @@ require 'uri'
 
 module Uploader
   class UploadRequest < Rack::Request
-    SPLITTER = '/'.freeze
+    SPLITTER = '/'
 
     attr_reader :file
 
@@ -30,6 +32,7 @@ module Uploader
 
     def total_file_length
       return content_length.to_i unless chunked?
+
       @env['HTTP_CONTENT_RANGE'].split(SPLITTER).last.to_i
     end
 
@@ -73,7 +76,7 @@ module Uploader
 
     def extract_filename(value)
       value = value.match(/filename\s?=\s?\"?([^;"]+)\"?/i)[1]
-      URI.decode(value.force_encoding('binary'))
+      URI.decode_www_form(value.force_encoding('binary'))
     end
   end
 end