dis 1.1.4 → 1.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0118e21d454d4ce58bcf4ad8d3abc5a5eece09f7aa425869e04853d6a6a3ea10'
-  data.tar.gz: 9cb58ea82f314f2c615a40773fa8e3e283171bbe6e0673c9f8938324c334d7c7
+  metadata.gz: e3b626d4874d661f0cfbb9fca2ae7b4c8d91f4a7d3b53d50a8eef7e7e26e001b
+  data.tar.gz: eb0b0009f3205724f8a23b505fb0ad0e2b91bf32c2a092cff75fbe668fd9f840
 SHA512:
-  metadata.gz: 35ad213ed5e7765c117fca8cc1b9496b348c93366996451bd44c93cadac33cd147580fd28f317ad4be83ee3db8304a4074bb53c692a96a230956aeb400ab8269
-  data.tar.gz: 54929beef973a5c1fbfa1e938c39e0bde0a2977d577325523dcf1d4eafe824b98e64ef2ac8d1d47cd2e4717341591fc659913b3a2dddd5fe2eae8195c603ae85
+  metadata.gz: dc99b33278935629fa9e8b498eb17c41d26e21cbcf97251e50feee83f8fc419de7a837c1246df563dfe34b9f5fc8c19a0056286b2dc0de9053a8a170750069d4
+  data.tar.gz: 067f17cb21587a0a33d8a03bf16bbba1e88e56a33110c316172e888720f235c8f215ae66abe789532dc6c18fd94cea76ead18f45942141d7598dcf9d9bc7e899

data/README.md

@@ -1,5 +1,5 @@
 [![Version](https://img.shields.io/gem/v/dis.svg?style=flat)](https://rubygems.org/gems/dis)
-[![Build Status](https://travis-ci.org/elektronaut/dis.svg?branch=master)](https://travis-ci.org/elektronaut/dis)
+![Build](https://github.com/elektronaut/dis/workflows/Build/badge.svg)
 [![Code Climate](https://codeclimate.com/github/elektronaut/dis/badges/gpa.svg)](https://codeclimate.com/github/elektronaut/dis)
 [![Code Climate](https://codeclimate.com/github/elektronaut/dis/badges/coverage.svg)](https://codeclimate.com/github/elektronaut/dis)
 [![Inline docs](http://inch-ci.org/github/elektronaut/dis.svg)](http://inch-ci.org/github/elektronaut/dis)
@@ -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,11 +1,15 @@
-# encoding: utf-8
+# frozen_string_literal: true
 
 module Dis
   module Errors
     class Error < StandardError; end
+
     class ReadOnlyError < Dis::Errors::Error; end
+
     class NoLayersError < Dis::Errors::Error; end
+
     class NotFoundError < Dis::Errors::Error; end
+
     class NoDataError < Dis::Errors::Error; end
   end
 end

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
@@ -12,6 +12,10 @@ module Dis
 
       def perform(type, key)
         Dis::Storage.delayed_store(type, key)
+      rescue Dis::Errors::NotFoundError
+        Rails.logger.warn(
+          "Delayed store failed, object not found: #{[type, key].inspect}"
+        )
       end
     end
   end

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,7 +102,13 @@ module Dis
     #
     #    layer.existing("documents", keys)
     def existing(type, keys)
-      list = directory(type, keys.first).files.map(&:key)
+      return [] if keys.empty?
+
+      list = []
+      directory(type, keys.first).files.each do |file|
+        list << file.key
+      end
+
       keys.select { |key| list.include?(key_component(type, key)) }
     end
 
@@ -109,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
@@ -123,6 +129,7 @@ module Dis
     def get(type, key)
       dir = directory(type, key)
       return unless dir
+
       dir.files.get(key_component(type, key))
     end
 
@@ -134,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
 
@@ -151,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
 
@@ -170,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
@@ -178,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
@@ -22,8 +22,8 @@ module Dis
     end
 
     # Iterates over the layers.
-    def each
-      @layers.each { |layer| yield layer }
+    def each(&block)
+      @layers.each { |layer| block.call(layer) }
     end
 
     # Returns a new instance containing only the delayed 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,14 +106,13 @@ module Dis
       end
 
       def stored?
-        @record.dis_stored? && !content_hash.blank?
+        content_hash.present? &&
+          (@record.dis_stored? ||
+           Dis::Storage.exists?(storage_type, content_hash))
       end
 
       def stored
-        Dis::Storage.get(
-          storage_type,
-          content_hash
-        )
+        Dis::Storage.get(storage_type, content_hash)
       end
 
       attr_reader :raw

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.4'.freeze
+  VERSION = "1.1.9"
 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
 
@@ -17,10 +17,7 @@ namespace :dis do
       puts "-- #{model.name} --"
 
       content_hash_attr = model.dis_attributes[:content_hash]
-      objects = model
-                .select(content_hash_attr)
-                .uniq
-                .map { |r| r.send(content_hash_attr) }
+      objects = model.pluck(content_hash_attr).uniq
       global_missing = objects.dup
 
       puts "Unique objects: #{objects.length}"
@@ -31,9 +28,14 @@ namespace :dis do
         existing = layer.existing(model.dis_type, objects)
         missing = objects - existing
         global_missing -= existing
-        puts "#{existing.length} existing, #{missing.length} missing"
+        puts "#{existing.length} existing, #{missing.length} missing" +
+             (layer.readonly? ? " (read-only)" : "")
 
-        next unless layer.delayed?
+        next unless layer.delayed? && !layer.readonly?
+
+        jobs += (missing - global_missing).pmap do |hash|
+          [model.dis_type, hash]
+        end.compact
       end
 
       if global_missing.any?
@@ -41,18 +43,14 @@ namespace :dis do
         pp global_missing
       end
 
-      jobs += (objects - global_missing).pmap do |hash|
-        [model.dis_type, hash]
-      end.compact
-
       puts
     end
 
     if jobs.any?
       print "#{jobs.length} objects can be transferred to delayed layers, " \
             "queue now? (y/n) "
-      response = STDIN.gets.chomp
-      if response =~ /^y/i
+      response = $stdin.gets.chomp
+      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.4
+  version: 1.1.9
 platform: ruby
 authors:
 - Inge Jørgensen
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-06-19 00:00:00.000000000 Z
+date: 2021-01-19 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,7 +53,21 @@ 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: '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: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,19 +81,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec-rails
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.5'
+        version: 0.17.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.5'
+        version: 0.17.1
+- !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:
@@ -127,7 +141,7 @@ homepage: https://github.com/elektronaut/dis
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -135,15 +149,15 @@ 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:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.1
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: A file store for your Rails app
 test_files: []