dis 1.1.5 → 1.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 25df4cff8fae7044ae9409c77cde74fb8dcd48ce2c36dc07f87e662223752bd6
-  data.tar.gz: 1f7c68208264c977f5733c6a7553728d0dac92c6d47a0ea97aa756e5632e1621
+  metadata.gz: 75c6c1d614d13eaa8ae3b3fd30762e1a22069fb60629ebc2bb6b396caf72a25e
+  data.tar.gz: fa6552876ae2e87d50416d004b34ee9039ceabfe459a2dbff899028650346a43
 SHA512:
-  metadata.gz: 6178ecc70a1ba31c9dff52389ab42b9b2f5bebcd7a0d47a89f95292f0b87706110b1e08afb38431298cd038ee81b85f8ea20be6e32753804f4215eb4083d4cde
-  data.tar.gz: b128f8665c386e720a8d9413a56a6a8123454716da2e2efa54f0e14c9336aa3c7df5a18eef309af6a3658625b6f08477648617aa3856586259d6db3395fca839
+  metadata.gz: 3e299491b62a599f7e0cb16e2de1ad67152e96fd6496b58e01d407f728c3739530822626c551405fd06585f0d7590ea7ca438a2bf0774770776d4ce4dc8b288f
+  data.tar.gz: 78e7d8efea50cff8bc50d4a412166b2f814644781e2c86b99041a554eb29bab70ff81f61f0898039b9c108072669ecbe17247aed3706805fdada761e51345941

data/README.md

@@ -17,7 +17,7 @@ your own on. If you're looking to handle image uploads, check out
 [DynamicImage](https://github.com/elektronaut/dynamic_image). It's
 built on top of Dis and handles resizing, cropping and more on demand.
 
-Requires Rails 4.2+.
+Requires Rails 5+
 
 ## Layers
 

data/lib/dis.rb

@@ -1,18 +1,18 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
-require 'digest/sha1'
-require 'fog/core'
-require 'fog/local'
-require 'active_job'
-require 'pmap'
-require 'dis/engine'
-require 'dis/errors'
-require 'dis/jobs'
-require 'dis/layer'
-require 'dis/layers'
-require 'dis/model'
-require 'dis/storage'
-require 'dis/validations'
+require "digest/sha1"
+require "fog/core"
+require "fog/local"
+require "active_job"
+require "pmap"
+require "dis/engine"
+require "dis/errors"
+require "dis/jobs"
+require "dis/layer"
+require "dis/layers"
+require "dis/model"
+require "dis/storage"
+require "dis/validations"
 
 module Dis
 end

data/lib/dis/engine.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   class Engine < ::Rails::Engine

data/lib/dis/errors.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Errors

data/lib/dis/jobs.rb

@@ -1,5 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
-require 'dis/jobs/delete'
-require 'dis/jobs/store'
-require 'dis/jobs/change_type'
+require "dis/jobs/delete"
+require "dis/jobs/store"
+require "dis/jobs/change_type"

data/lib/dis/jobs/change_type.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Jobs

data/lib/dis/jobs/delete.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Jobs

data/lib/dis/jobs/store.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Jobs

data/lib/dis/layer.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   # = Dis Layer
@@ -94,6 +94,7 @@ module Dis
     # is readonly.
     def store(type, key, file)
       raise Dis::Errors::ReadOnlyError if readonly?
+
       store!(type, key, file)
     end
 
@@ -101,6 +102,8 @@ module Dis
     #
     #    layer.existing("documents", keys)
     def existing(type, keys)
+      return [] if keys.empty?
+
       list = []
       directory(type, keys.first).files.each do |file|
         list << file.key
@@ -113,8 +116,7 @@ module Dis
     #
     #    layer.exists?("documents", key)
     def exists?(type, key)
-      if directory(type, key) &&
-         directory(type, key).files.head(key_component(type, key))
+      if directory(type, key)&.files&.head(key_component(type, key))
         true
       else
         false
@@ -127,6 +129,7 @@ module Dis
     def get(type, key)
       dir = directory(type, key)
       return unless dir
+
       dir.files.get(key_component(type, key))
     end
 
@@ -138,6 +141,7 @@ module Dis
     # Raises an error if the layer is readonly.
     def delete(type, key)
       raise Dis::Errors::ReadOnlyError if readonly?
+
       delete!(type, key)
     end
 
@@ -155,15 +159,16 @@ module Dis
     end
 
     def directory_component(_type, _key)
-      path || ''
+      path || ""
     end
 
     def key_component(type, key)
-      [type, key[0...2], key[2..key.length]].compact.join('/')
+      [type, key[0...2], key[2..key.length]].compact.join("/")
     end
 
     def delete!(type, key)
       return false unless exists?(type, key)
+
       get(type, key).destroy
     end
 
@@ -174,7 +179,7 @@ module Dis
     def directory!(type, key)
       dir = directory(type, key)
       dir ||= connection.directories.create(
-        key:    directory_component(type, key),
+        key: directory_component(type, key),
         public: public?
       )
       dir
@@ -182,16 +187,17 @@ module Dis
 
     def store!(type, key, file)
       return get(type, key) if exists?(type, key)
+
       file.rewind if file.respond_to?(:rewind)
       directory!(type, key).files.create(
-        key:    key_component(type, key),
-        body:   (file.is_a?(Fog::Model) ? file.body : file),
+        key: key_component(type, key),
+        body: (file.is_a?(Fog::Model) ? file.body : file),
         public: public?
       )
     end
 
     def path
-      @path && !@path.empty? ? @path : nil
+      @path.presence
     end
   end
 end

data/lib/dis/layers.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   # = Dis Layers

data/lib/dis/model.rb

@@ -1,7 +1,7 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
-require 'dis/model/class_methods'
-require 'dis/model/data'
+require "dis/model/class_methods"
+require "dis/model/data"
 
 module Dis
   # = Dis Model
@@ -103,7 +103,7 @@ module Dis
     # <tt>content_hash</tt> to nil.
     def data=(raw_data)
       new_data = Dis::Model::Data.new(self, raw_data)
-      attribute_will_change!('data') unless new_data == dis_data
+      attribute_will_change!("data") unless new_data == dis_data
       @dis_data = new_data
       dis_set :content_hash, if raw_data.nil?
                                nil
@@ -115,7 +115,7 @@ module Dis
 
     # Returns true if the data has been changed since the object was last saved.
     def data_changed?
-      changes.include?('data')
+      changes.include?("data")
     end
 
     def dis_stored?
@@ -135,6 +135,7 @@ module Dis
 
     def cleanup_data
       return if @previous_content_hash.blank?
+
       dis_data.expire(@previous_content_hash)
       @previous_content_hash = nil
     end
@@ -167,7 +168,7 @@ module Dis
 
     # We don't want the data column when doing a partial write.
     def keys_for_partial_write
-      super.reject { |a| a == 'data' }
+      super.reject { |a| a == "data" }
     end
   end
 end

data/lib/dis/model/class_methods.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Model

data/lib/dis/model/data.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Model
@@ -26,7 +26,7 @@ module Dis
 
       # Returns the data as a binary string.
       def read
-        @cached ||= read_from(closest)
+        @read ||= read_from(closest)
       end
 
       # Will be true if data has been explicitely set.
@@ -60,6 +60,7 @@ module Dis
       # Stores the data. Returns a hash of the content for reference.
       def store!
         raise Dis::Errors::NoDataError unless raw?
+
         Dis::Storage.store(storage_type, raw)
       end
 
@@ -83,6 +84,7 @@ module Dis
 
       def read_from(object)
         return nil unless object
+
         if object.respond_to?(:body)
           object.body
         elsif object.respond_to?(:read)
@@ -104,7 +106,7 @@ module Dis
       end
 
       def stored?
-        @record.dis_stored? && !content_hash.blank?
+        @record.dis_stored? && content_hash.present?
       end
 
       def stored

data/lib/dis/storage.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   # = Dis Storage
@@ -107,9 +107,7 @@ module Dis
             fetch_count += 1
             layer.get(type, key)
           end.call
-        end
-
-        raise Dis::Errors::NotFoundError unless result
+        end || raise(Dis::Errors::NotFoundError)
 
         store_immediately!(type, result) if fetch_count > 1
         result

data/lib/dis/validations.rb

@@ -1,3 +1,3 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
-require 'dis/validations/data_presence'
+require "dis/validations/data_presence"

data/lib/dis/validations/data_presence.rb

@@ -1,4 +1,4 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Validations

data/lib/dis/version.rb

@@ -1,5 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
-  VERSION = '1.1.5'.freeze
+  VERSION = "1.1.6"
 end

data/lib/rails/generators/dis/install/install_generator.rb

@@ -1,16 +1,16 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
-require 'rails/generators'
-require 'rails/generators/rails/model/model_generator'
+require "rails/generators"
+require "rails/generators/rails/model/model_generator"
 
 module Dis
   module Generators
     class InstallGenerator < Rails::Generators::Base
-      desc 'Creates the Dis initializer'
-      source_root File.expand_path('../templates', __FILE__)
+      desc "Creates the Dis initializer"
+      source_root File.expand_path("templates", __dir__)
 
       def create_initializer
-        template 'initializer.rb', File.join('config', 'initializers', 'dis.rb')
+        template "initializer.rb", File.join("config", "initializers", "dis.rb")
       end
     end
   end

data/lib/rails/generators/dis/install/templates/initializer.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 # Be sure to restart your server when you modify this file.
 
 # Creates a local storage layer in db/dis:
 
 Dis::Storage.layers << Dis::Layer.new(
-  Fog::Storage.new(provider: 'Local', local_root: Rails.root.join('db', 'dis')),
+  Fog::Storage.new(provider: "Local", local_root: Rails.root.join("db", "dis")),
   path: Rails.env
 )
 

data/lib/rails/generators/dis/model/model_generator.rb

@@ -1,12 +1,12 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
-require 'rails/generators'
-require 'rails/generators/rails/model/model_generator'
+require "rails/generators"
+require "rails/generators/rails/model/model_generator"
 
 module Dis
   module Generators
     class ModelGenerator < Rails::Generators::ModelGenerator
-      desc 'Creates a Dis model'
+      desc "Creates a Dis model"
 
       def initialize(args, *options)
         super(inject_dis_attributes(args), *options)
@@ -14,7 +14,7 @@ module Dis
 
       def add_model_extension
         inject_into_file(
-          File.join('app/models', class_path, "#{file_name}.rb"),
+          File.join("app/models", class_path, "#{file_name}.rb"),
           after: "ActiveRecord::Base\n"
         ) do
           "  include Dis::Model\n"
@@ -32,12 +32,12 @@ module Dis
       end
 
       def dis_attributes
-        %w(
+        %w[
           content_hash:string
           content_type:string
           content_length:integer
           filename:string
-        )
+        ]
       end
     end
   end

data/lib/tasks/dis.rake

@@ -1,15 +1,15 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 namespace :dis do
-  desc 'Check stuff'
+  desc "Check stuff"
   task consistency_check: :environment do
-    unless ENV['MODELS']
+    unless ENV["MODELS"]
       puts "Usage: #{$PROGRAM_NAME} dis:consistency_check " \
-           'MODELS=Avatar,Document'
+           "MODELS=Avatar,Document"
       exit
     end
 
-    models = ENV['MODELS'].split(',').map(&:strip).map(&:constantize)
+    models = ENV["MODELS"].split(",").map(&:strip).map(&:constantize)
 
     jobs = Set.new
 
@@ -52,7 +52,7 @@ namespace :dis do
       print "#{jobs.length} objects can be transferred to delayed layers, " \
             "queue now? (y/n) "
       response = STDIN.gets.chomp
-      if response =~ /^y/i
+      if /^y/i.match?(response)
         puts "Queueing jobs..."
         jobs.each { |type, hash| Dis::Jobs::Store.perform_later(type, hash) }
       end

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: dis
 version: !ruby/object:Gem::Version
-  version: 1.1.5
+  version: 1.1.6
 platform: ruby
 authors:
 - Inge Jørgensen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-19 00:00:00.000000000 Z
+date: 2019-08-17 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: rails
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: fog-core
   requirement: !ruby/object:Gem::Requirement
@@ -67,19 +53,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.1.0
 - !ruby/object:Gem::Dependency
-  name: sqlite3
+  name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: '5.0'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.5'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Dis is a Rails plugin that stores your file uploads and other binary
   blobs.
 email:
@@ -135,7 +135,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.2
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="