dis 1.1.21 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9926ad8c82fb6ed96ea76e41e3f93f7c4dc2c5e8acec7917e6131cb474e0404
-  data.tar.gz: 27521093c0575935ce489991f8250dac55a2dc86384ac207639fe03fe25ad2e0
+  metadata.gz: cf00e607df1b1aa62ffa3e025c22ac42e3363d14fb0a06cb6b8f41d3b7757db5
+  data.tar.gz: 2cfe7f5a47d87bdd5e6c59e7f98446bec2918a27c8e3476c123ef1164842d437
 SHA512:
-  metadata.gz: fcd0fa193bf99030dee1a1254f91c7bdad6237a6b899120a3a14569aa6d766a5629dc46bfb4c73ba8f29e8f67c1244bfdaf17f4d2d1438d9b5c96660d3dcd5c4
-  data.tar.gz: 2cb2dd1c9fca6bd489248980ab2d251d0d9fab85cbe5b98334dd52ccc6042a2faf6a5dccc1584a339b5899f4fb5157cc873ac235cebd16f26f2a8d45a5a728ab
+  metadata.gz: e2428a8aa2299b730e1db9702fbec9e725cae6a38bfd3e04bce0ac1ba41484f45840f52cdfe0f998dfe5ffa3c5c5a5fa31bc01c380a1aa8a307392e4a4c9a6a4
+  data.tar.gz: d1e9bfbc53cdd753fa0b52730b4c6ac88ec00ca1ba3dd519086fdea58f117947f30d0eb5bef297d952a8fb8deab6d642ce939d5a045d19f70f0586c578b3111e

data/lib/dis/layer.rb

@@ -136,6 +136,21 @@ module Dis
       end
     end
 
+    # Returns the absolute file path for a locally stored file, or nil
+    # if the provider is not local or the file does not exist.
+    #
+    #    layer.file_path("documents", key)
+    def file_path(type, key)
+      return unless connection.respond_to?(:local_root)
+      return unless exists?(type, key)
+
+      File.join(
+        connection.local_root,
+        directory_component(type, key),
+        key_component(type, key)
+      )
+    end
+
     # Deletes a file from the store.
     #
     #   layer.delete("documents", key)

data/lib/dis/model/data.rb

@@ -14,11 +14,13 @@ module Dis
 
       # Returns true if two Data objects represent the same data.
       def ==(other)
-        return false unless other.respond_to?(:read)
-
-        # TODO: This can be made faster by
-        # comparing hashes for stored objects.
-        other.read == read
+        if !raw? && other.is_a?(self.class) && !other.changed?
+          content_hash == other.content_hash
+        elsif other.respond_to?(:read)
+          other.read == read
+        else
+          false
+        end
       end
 
       # Returns true if data exists either in memory or in storage.
@@ -68,6 +70,22 @@ module Dis
         Dis::Storage.store(storage_type, raw)
       end
 
+      # Clears cached data and tempfiles, allowing them to be garbage
+      # collected. Subsequent calls to read or tempfile will re-fetch.
+      def reset_read_cache!
+        @read = nil
+        return unless @tempfile
+
+        @tempfile.close!
+        @tempfile = nil
+      end
+
+      # Returns the file path to the data. Prefers a local storage path
+      # to avoid unnecessary copies, falls back to a tempfile.
+      def file_path
+        local_path || tempfile.path
+      end
+
       # Writes the data to a temporary file.
       def tempfile
         unless @tempfile
@@ -78,6 +96,12 @@ module Dis
         @tempfile
       end
 
+      protected
+
+      def content_hash
+        @record[@record.class.dis_attributes[:content_hash]]
+      end
+
       private
 
       def closest
@@ -88,10 +112,6 @@ module Dis
         end
       end
 
-      def content_hash
-        @record[@record.class.dis_attributes[:content_hash]]
-      end
-
       def raw?
         raw ? true : false
       end
@@ -129,6 +149,12 @@ module Dis
         Dis::Storage.get(storage_type, content_hash)
       end
 
+      def local_path
+        return if raw?
+
+        Dis::Storage.file_path(storage_type, content_hash)
+      end
+
       attr_reader :raw
     end
   end

data/lib/dis/model.rb

@@ -131,6 +131,11 @@ module Dis
       dis_set :filename, file.original_filename
     end
 
+    # Returns a file path to the data, preferring local storage paths.
+    def data_file_path
+      dis_data.file_path
+    end
+
     # Returns the data as a temporary file.
     delegate :tempfile, to: :dis_data
 

data/lib/dis/storage.rb

@@ -113,6 +113,19 @@ module Dis
         result
       end
 
+      # Returns the absolute file path from the first layer that has a
+      # local copy, or nil if no layer stores files locally.
+      #
+      #   Dis::Storage.file_path("things", key)
+      def file_path(type, key)
+        require_layers!
+        layers.each do |layer|
+          path = layer.file_path(type, key)
+          return path if path
+        end
+        nil
+      end
+
       # Deletes a file from all layers. Kicks off a
       # <tt>Dis::Jobs::Delete</tt> job if any delayed layers are defined.
       # Returns true if the file existed in any immediate layers,

data/lib/dis/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Dis
-  VERSION = "1.1.21"
+  VERSION = "1.2.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dis
 version: !ruby/object:Gem::Version
-  version: 1.1.21
+  version: 1.2.0
 platform: ruby
 authors:
 - Inge Jørgensen