sufia-models 5.0.0 → 6.0.0.beta1

data/lib/generators/sufia/models/proxies_generator.rb

@@ -1,12 +1,30 @@
-require_relative 'abstract_migration_generator'
+# -*- encoding : utf-8 -*-
+require 'rails/generators'
+require 'rails/generators/migration'
+
+class Sufia::Models::ProxiesGenerator < Rails::Generators::Base
+  include Rails::Generators::Migration
 
-class Sufia::Models::ProxiesGenerator < Sufia::Models::AbstractMigrationGenerator
   source_root File.expand_path('../templates', __FILE__)
 
   desc """
 This generator adds proxies and transfers to your application:
  1. Creates several database migrations if they do not exist in /db/migrate
        """
+  # Implement the required interface for Rails::Generators::Migration.
+  # taken from http://github.com/rails/rails/blob/master/activerecord/lib/generators/active_record.rb
+  def self.next_migration_number(path)
+    if @prev_migration_nr
+      @prev_migration_nr += 1
+    else
+      if last_migration = Dir[File.join(path, '*.rb')].sort.last
+        @prev_migration_nr = last_migration.sub(File.join(path, '/'), '').to_i + 1
+      else
+        @prev_migration_nr = Time.now.utc.strftime("%Y%m%d%H%M%S").to_i
+      end
+    end
+    @prev_migration_nr.to_s
+  end
 
   def banner
     say_status("warning", "ADDING PROXY/TRANSFER-RELATED SUFIA MODELS", :yellow)
@@ -14,6 +32,7 @@ This generator adds proxies and transfers to your application:
 
   # Setup the database migrations
   def copy_migrations
+    # Can't get this any more DRY, because we need this order.
     [
       'create_proxy_deposit_rights.rb',
       'create_proxy_deposit_requests.rb'
@@ -21,4 +40,14 @@ This generator adds proxies and transfers to your application:
       better_migration_template file
     end
   end
+
+  private
+
+  def better_migration_template(file)
+    begin
+      migration_template "migrations/#{file}", "db/migrate/#{file}"
+    rescue Rails::Generators::Error => e
+      say_status("warning", e.message, :yellow)
+    end
+  end
 end

data/lib/generators/sufia/models/templates/config/sufia.rb

@@ -111,6 +111,16 @@ Sufia.config do |config|
   # Specify how many seconds back from the current time that we should show by default of the user's activity on the user's dashboard
   # config.activity_to_show_default_seconds_since_now = 24*60*60
 
+  # Specify a date you wish to start collecting Google Analytic statistics for.
+  # Leaving it blank will set the start date to when ever the file was uploaded by
+  # NOTE: if you have always sent analytics to GA for downloads and page views leave this commented out
+  # config.analytic_start_date = DateTime.new(2014,9,10)
+  #
+  # Method of converting pids into URIs for storage in Fedora
+  # config.translate_uri_to_id = lambda { |uri| uri.to_s.split('/')[-1] }
+  # config.translate_id_to_uri = lambda { |id|
+  #      "#{ActiveFedora.fedora.host}#{ActiveFedora.fedora.base_path}/#{Sufia::Noid.treeify(id)}" }
+
   # If browse-everything has been configured, load the configs.  Otherwise, set to nil.
   begin
     if defined? BrowseEverything

data/lib/generators/sufia/models/upgrade400_generator.rb

@@ -1,8 +1,13 @@
-require_relative 'abstract_migration_generator'
+# -*- encoding : utf-8 -*-
+require 'rails/generators'
+require 'rails/generators/migration'
+
+class Sufia::Models::Upgrade400Generator < Rails::Generators::Base
+  include Rails::Generators::Migration
 
-class Sufia::Models::Upgrade400Generator < Sufia::Models::AbstractMigrationGenerator
   source_root File.expand_path('../templates', __FILE__)
 
+  argument     :model_name, type: :string , default: "user"
   desc """
 This generator for upgrading sufia-models from 3.7.2 to 4.0 makes the following changes to your application:
  1. Creates several database migrations if they do not exist in /db/migrate
@@ -11,12 +16,28 @@ This generator for upgrading sufia-models from 3.7.2 to 4.0 makes the following
  4. Runs full-text generator
        """
 
+  # Implement the required interface for Rails::Generators::Migration.
+  # taken from http://github.com/rails/rails/blob/master/activerecord/lib/generators/active_record.rb
+  def self.next_migration_number(path)
+    if @prev_migration_nr
+      @prev_migration_nr += 1
+    else
+      if last_migration = Dir[File.join(path, '*.rb')].sort.last
+        @prev_migration_nr = last_migration.sub(File.join(path, '/'), '').to_i + 1
+      else
+        @prev_migration_nr = Time.now.utc.strftime("%Y%m%d%H%M%S").to_i
+      end
+    end
+    @prev_migration_nr.to_s
+  end
+
   def banner
     say_status("warning", "UPGRADING SUFIA MODELS", :yellow)
   end
 
   # Setup the database migrations
   def copy_migrations
+    # Can't get this any more DRY, because we need this order.
     [
       'create_tinymce_assets.rb',
       'create_content_blocks.rb',
@@ -51,4 +72,14 @@ This generator for upgrading sufia-models from 3.7.2 to 4.0 makes the following
   def full_text_indexing
     generate "sufia:models:fulltext"
   end
+
+  private
+
+  def better_migration_template(file)
+    begin
+      migration_template "migrations/#{file}", "db/migrate/#{file}"
+    rescue Rails::Generators::Error => e
+      say_status("warning", e.message, :yellow)
+    end
+  end
 end

data/lib/sufia/models/engine.rb

@@ -31,6 +31,10 @@ module Sufia
       # Defaulting analytic start date to when ever the file was uploaded by leaving it blank
       config.analytic_start_date = nil
 
+      config.translate_uri_to_id = lambda { |uri| uri.to_s.split('/')[-1] }
+      config.translate_id_to_uri = lambda { |id|
+        "#{ActiveFedora.fedora.host}#{ActiveFedora.fedora.base_path}/#{Sufia::Noid.treeify(id)}" }
+
       config.autoload_paths += %W(
         #{config.root}/app/models/datastreams
       )
@@ -58,10 +62,15 @@ module Sufia
       end
 
       initializer 'configure' do
-        Hydra::Derivatives.ffmpeg_path    = Sufia.config.ffmpeg_path
-        Hydra::Derivatives.temp_file_base = Sufia.config.temp_file_base
-        Hydra::Derivatives.fits_path      = Sufia.config.fits_path
-        Hydra::Derivatives.enable_ffmpeg  = Sufia.config.enable_ffmpeg
+        Sufia.config.tap do |c|
+          Hydra::Derivatives.ffmpeg_path    = c.ffmpeg_path
+          Hydra::Derivatives.temp_file_base = c.temp_file_base
+          Hydra::Derivatives.fits_path      = c.fits_path
+          Hydra::Derivatives.enable_ffmpeg  = c.enable_ffmpeg
+
+          ActiveFedora::Base.translate_uri_to_id = c.translate_uri_to_id
+          ActiveFedora::Base.translate_id_to_uri = c.translate_id_to_uri
+        end
       end
     end
   end

data/lib/sufia/models/file_content/versions.rb

@@ -1,23 +1,21 @@
 module Sufia
   module FileContent
     module Versions
-      def get_version(version_id)
-        self.versions.select { |v| v.versionID == version_id}.first
+      extend ActiveSupport::Concern
+
+      included do
+        has_many_versions
       end
 
       def latest_version
-        self.versions.first
+        versions.last unless versions.empty?
       end
 
-      def version_committer(version)
-        vc = VersionCommitter.where(obj_id: version.pid,
-                                    datastream_id: version.dsid,
-                                    version_id: version.versionID)
-        return vc.empty? ? nil : vc.first.committer_login
+      def save
+        super.tap do |passing|
+          create_version if passing
+        end
       end
-
-
     end
   end
 end
-

data/lib/sufia/models/stats/user_stat_importer.rb

@@ -12,11 +12,11 @@ module Sufia
         start_date = date_since_last_cache(user)
 
         stats = {}
-        file_ids_for_user(user).each do |file_id|
-          view_stats = FileViewStat.statistics(file_id, start_date, user.id)
+        files_for_user(user).each do |file|
+          view_stats = FileViewStat.statistics(file.id, start_date, user.id)
           stats = tally_results(view_stats, :views, stats)
 
-          dl_stats = FileDownloadStat.statistics(file_id, start_date, user.id)
+          dl_stats = FileDownloadStat.statistics(file.id, start_date, user.id)
           stats = tally_results(dl_stats, :downloads, stats)
         end
 
@@ -38,12 +38,8 @@ private
       end
     end
 
-    def file_ids_for_user(user)
-      ids = []
-      ::GenericFile.find_in_batches("#{Solrizer.solr_name('depositor', :symbol)}:\"#{user.user_key}\"", fl:"id") do |group|
-        ids.concat group.map { |doc| doc["id"] }
-      end
-      ids
+    def files_for_user(user)
+      ::GenericFile.where(Solrizer.solr_name('depositor', :symbol) => user.user_key)
     end
 
     # For each date, add the view and download counts for this

data/lib/sufia/models/version.rb

@@ -1,5 +1,5 @@
 module Sufia
   module Models
-    VERSION = "5.0.0"
+    VERSION = "6.0.0.beta1"
   end
 end

data/lib/sufia/permissions/writable.rb

@@ -8,32 +8,50 @@ module Sufia
       include Hydra::AccessControls::Visibility
 
       included do
-        has_metadata "rightsMetadata", type: ParanoidRightsDatastream
         validate :paranoid_permissions
       end
 
       def paranoid_permissions
-        # let the rightsMetadata ds make this determination
-        # - the object instance is passed in for easier access to the props ds
-        rightsMetadata.validate(self)
+        valid = true
+        VALIDATIONS.each do |validation|
+          if validation[:condition].call(self)
+            errors[validation[:key]] ||= []
+            errors[validation[:key]] << validation[:message]
+            valid = false
+          end
+        end
+        return valid
       end
 
-      ## Updates those permissions that are provided to it. Does not replace any permissions unless they are provided
-      def permissions=(params)
-        perm_hash = permission_hash
-        params[:new_user_name].each { |name, access| perm_hash['person'][name] = access } if params[:new_user_name].present?
-        params[:new_group_name].each { |name, access| perm_hash['group'][name] = access } if params[:new_group_name].present?
+      VALIDATIONS = [
+        {key: :edit_users, message: 'Depositor must have edit access', condition: lambda { |obj| !obj.edit_users.include?(obj.depositor) }},
+        {key: :edit_groups, message: 'Public cannot have edit access', condition: lambda { |obj| obj.edit_groups.include?('public') }},
+        {key: :edit_groups, message: 'Registered cannot have edit access', condition: lambda { |obj| obj.edit_groups.include?('registered') }}
+      ]
 
-        params[:user].each { |name, access| perm_hash['person'][name] = access} if params[:user]
-        params[:group].each { |name, access| perm_hash['group'][name] = access if ['read', 'edit'].include?(access)} if params[:group]
 
-        rightsMetadata.update_permissions(perm_hash)
+      def clear_permissions!
+        self.permissions = []
       end
 
-      def permissions
-        perms = super
-        perms.map {|p| { name: p.name, access: p.access, type:p.type } }
-      end
+      ## Updates those permissions that are provided to it. Does not replace any permissions unless they are provided
+      # def permissions=(params)
+      #   raise "Fixme #{params}"
+      #   perm_hash = permission_hash
+      #   params[:new_user_name].each { |name, access| perm_hash['person'][name] = access } if params[:new_user_name].present?
+      #   params[:new_group_name].each { |name, access| perm_hash['group'][name] = access } if params[:new_group_name].present?
+
+      #   params[:user].each { |name, access| perm_hash['person'][name] = access} if params[:user]
+      #   params[:group].each { |name, access| perm_hash['group'][name] = access if ['read', 'edit'].include?(access)} if params[:group]
+
+      #   # rightsMetadata.update_permissions(perm_hash)
+      # end
+
+      # def permissions
+      #   raise "Fixme "
+      #   perms = super
+      #   perms.map {|p| { name: p.name, access: p.access, type:p.type } }
+      # end
 
       private
 

data/sufia-models.gemspec

@@ -29,7 +29,10 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'rails', '~> 4.0', '< 5.0.0'
   spec.add_dependency 'activeresource', "~> 4.0" # No longer a dependency of rails 4.0
 
-  spec.add_dependency "hydra-head", "~> 7.0"
+  spec.add_dependency "hydra-head", "~> 9.0.0.beta1"
+  spec.add_dependency "active-fedora", "~> 9.0.0.beta5"
+  spec.add_dependency "hydra-collections", "~> 4.0.0.beta1"
+  spec.add_dependency 'hydra-derivatives', '~> 1.0.0.beta1'
   spec.add_dependency 'nest', '~> 1.1'
   spec.add_dependency 'resque', '~> 1.23'
   spec.add_dependency 'resque-pool', '~> 0.3'
@@ -37,7 +40,6 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'mailboxer', '~> 0.12'
   spec.add_dependency 'acts_as_follower', '>= 0.1.1', '< 0.3'
   spec.add_dependency 'carrierwave', '~> 0.9'
-  spec.add_dependency 'hydra-derivatives', '~> 0.1'
   spec.add_dependency 'oauth2', '~> 0.9'
   spec.add_dependency 'google-api-client', '~> 0.7'
   spec.add_dependency 'legato', '~> 0.3'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sufia-models
 version: !ruby/object:Gem::Version
-  version: 5.0.0
+  version: 6.0.0.beta1
 platform: ruby
 authors:
 - Jeremy Friesen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-06 00:00:00.000000000 Z
+date: 2014-12-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -78,14 +78,56 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.0'
+        version: 9.0.0.beta1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.0'
+        version: 9.0.0.beta1
+- !ruby/object:Gem::Dependency
+  name: active-fedora
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 9.0.0.beta5
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 9.0.0.beta5
+- !ruby/object:Gem::Dependency
+  name: hydra-collections
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.0.0.beta1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.0.0.beta1
+- !ruby/object:Gem::Dependency
+  name: hydra-derivatives
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.0.beta1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.0.beta1
 - !ruby/object:Gem::Dependency
   name: nest
   requirement: !ruby/object:Gem::Requirement
@@ -190,20 +232,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
-- !ruby/object:Gem::Dependency
-  name: hydra-derivatives
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.1'
 - !ruby/object:Gem::Dependency
   name: oauth2
   requirement: !ruby/object:Gem::Requirement
@@ -302,31 +330,27 @@ files:
 - app/models/concerns/sufia/file_stat_utils.rb
 - app/models/concerns/sufia/generic_file.rb
 - app/models/concerns/sufia/generic_file/accessible_attributes.rb
-- app/models/concerns/sufia/generic_file/audit.rb
+- app/models/concerns/sufia/generic_file/batches.rb
 - app/models/concerns/sufia/generic_file/characterization.rb
+- app/models/concerns/sufia/generic_file/content.rb
 - app/models/concerns/sufia/generic_file/derivatives.rb
 - app/models/concerns/sufia/generic_file/export.rb
 - app/models/concerns/sufia/generic_file/featured.rb
 - app/models/concerns/sufia/generic_file/full_text_indexing.rb
+- app/models/concerns/sufia/generic_file/indexing.rb
 - app/models/concerns/sufia/generic_file/metadata.rb
 - app/models/concerns/sufia/generic_file/mime_types.rb
 - app/models/concerns/sufia/generic_file/permissions.rb
 - app/models/concerns/sufia/generic_file/proxy_deposit.rb
-- app/models/concerns/sufia/generic_file/reload_on_save.rb
 - app/models/concerns/sufia/generic_file/trophies.rb
 - app/models/concerns/sufia/generic_file/versions.rb
 - app/models/concerns/sufia/generic_file/virus_check.rb
 - app/models/concerns/sufia/generic_file/web_form.rb
 - app/models/concerns/sufia/model_methods.rb
-- app/models/concerns/sufia/properties_datastream_behavior.rb
 - app/models/concerns/sufia/user.rb
 - app/models/concerns/sufia/user_usage_stats.rb
-- app/models/datastreams/batch_rdf_datastream.rb
 - app/models/datastreams/file_content_datastream.rb
 - app/models/datastreams/fits_datastream.rb
-- app/models/datastreams/generic_file_rdf_datastream.rb
-- app/models/datastreams/paranoid_rights_datastream.rb
-- app/models/datastreams/properties_datastream.rb
 - app/models/domain_term.rb
 - app/models/featured_work.rb
 - app/models/file_download_stat.rb
@@ -345,20 +369,19 @@ files:
 - app/models/sufia/avatar_uploader.rb
 - app/models/sufia/avatar_validator.rb
 - app/models/sufia/download.rb
-- app/models/sufia/orcid_validator.rb
 - app/models/sufia/pageview.rb
 - app/models/trophy.rb
 - app/models/user_stat.rb
 - app/models/version_committer.rb
 - app/services/sufia/analytics.rb
+- app/services/sufia/generic_file_audit_service.rb
 - app/services/sufia/id_service.rb
 - app/services/sufia/noid.rb
+- app/services/sufia/repository_audit_service.rb
 - config/locales/sufia.en.yml
-- lib/generators/sufia/models/abstract_migration_generator.rb
 - lib/generators/sufia/models/cached_stats_generator.rb
 - lib/generators/sufia/models/fulltext_generator.rb
 - lib/generators/sufia/models/install_generator.rb
-- lib/generators/sufia/models/orcid_field_generator.rb
 - lib/generators/sufia/models/proxies_generator.rb
 - lib/generators/sufia/models/templates/config/analytics.yml
 - lib/generators/sufia/models/templates/config/clamav.rb
@@ -377,7 +400,6 @@ files:
 - lib/generators/sufia/models/templates/migrations/add_groups_to_users.rb
 - lib/generators/sufia/models/templates/migrations/add_ldap_attrs_to_user.rb
 - lib/generators/sufia/models/templates/migrations/add_linkedin_to_users.rb
-- lib/generators/sufia/models/templates/migrations/add_orcid_to_users.rb
 - lib/generators/sufia/models/templates/migrations/add_social_to_users.rb
 - lib/generators/sufia/models/templates/migrations/create_checksum_audit_logs.rb
 - lib/generators/sufia/models/templates/migrations/create_content_blocks.rb
@@ -394,7 +416,6 @@ files:
 - lib/generators/sufia/models/templates/migrations/create_version_committers.rb
 - lib/generators/sufia/models/upgrade400_generator.rb
 - lib/generators/sufia/models/usagestats_generator.rb
-- lib/generators/sufia/models/user_stats_generator.rb
 - lib/sufia/messages.rb
 - lib/sufia/models.rb
 - lib/sufia/models/active_fedora/redis.rb
@@ -430,12 +451,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.6
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Models and services for sufia

data/app/models/concerns/sufia/generic_file/audit.rb

@@ -1,116 +0,0 @@
-module Sufia
-  module GenericFile
-    module Audit
-      extend ActiveSupport::Concern
-
-      NO_RUNS = 999
-
-      def audit(force = false)
-        logs = []
-        self.per_version do |ver|
-          logs << audit_each(ver, force)
-        end
-        logs
-      end
-
-      def per_version(&block)
-        self.datastreams.each do |dsid, ds|
-          next if ds == full_text
-          ds.versions.each do |ver|
-            block.call(ver)
-          end
-        end
-      end
-
-      def logs(dsid)
-        ChecksumAuditLog.where(dsid: dsid, pid: self.pid).order('created_at desc, id desc')
-      end
-
-      def audit!
-        audit(true)
-      end
-
-      def audit_stat!
-        audit_stat(true)
-      end
-
-      def audit_stat(force = false)
-        logs = audit(force)
-        audit_results = logs.collect { |result| result["pass"] }
-
-        # check how many non runs we had
-        non_runs = audit_results.reduce(0) { |sum, value| value == NO_RUNS ? sum += 1 : sum }
-        if non_runs == 0
-          result = audit_results.reduce(true) { |sum, value| sum && value }
-          return result
-        elsif non_runs < audit_results.length
-          result = audit_results.reduce(true) { |sum, value| value == NO_RUNS ? sum : sum && value }
-          return 'Some audits have not been run, but the ones run were '+ ((result)? 'passing' : 'failing') + '.'
-        else
-          return 'Audits have not yet been run on this file.'
-        end
-      end
-
-      def audit_each(version, force = false)
-        latest_audit = logs(version.dsid).first
-        return latest_audit unless force || ::GenericFile.needs_audit?(version, latest_audit)
-
-        #  Resque.enqueue(AuditJob, version.pid, version.dsid, version.versionID)
-        Sufia.queue.push(AuditJob.new(version.pid, version.dsid, version.versionID))
-
-        # run the find just incase the job has finished already
-        latest_audit = logs(version.dsid).first
-        latest_audit = ChecksumAuditLog.new(pass: NO_RUNS, pid: version.pid, dsid: version.dsid, version: version.versionID) unless latest_audit
-        latest_audit
-      end
-
-
-      module ClassMethods
-        def audit!(version)
-          ::GenericFile.audit(version, true)
-        end
-
-        def audit(version, force = false)
-          latest_audit = self.find(version.pid).audit_each( version, force)
-        end
-
-        def needs_audit?(version, latest_audit)
-          if latest_audit and latest_audit.updated_at
-            days_since_last_audit = (DateTime.now - latest_audit.updated_at.to_date).to_i
-            if days_since_last_audit < Sufia.config.max_days_between_audits
-              return false
-            end
-          else
-            logger.warn "***AUDIT*** problem with audit log!  Latest Audit is not nil, but updated_at is not set #{latest_audit}"  unless latest_audit.nil?
-          end
-          true
-        end
-
-        def audit_everything(force = false)
-          ::GenericFile.find_each do |gf|
-            gf.per_version do |ver|
-              ::GenericFile.audit(ver, force)
-            end
-          end
-        end
-
-        def audit_everything!
-          ::GenericFile.audit_everything(true)
-        end
-
-        def run_audit(version)
-          if version.dsChecksumValid
-            passing = 1
-            ChecksumAuditLog.prune_history(version)
-          else
-            logger.warn "***AUDIT*** Audit failed for #{version.pid} #{version.versionID}"
-            passing = 0
-          end
-          check = ChecksumAuditLog.create!(pass: passing, pid: version.pid,
-                                           dsid: version.dsid, version: version.versionID)
-          check
-        end
-      end
-    end
-  end
-end

data/app/models/concerns/sufia/generic_file/reload_on_save.rb

@@ -1,18 +0,0 @@
-# add this in here until we can use a version of Active Fedora that contains this ability
-module Sufia
-  module GenericFile
-    module ReloadOnSave
-
-      attr_writer :reload_on_save
-
-      def reload_on_save?
-        !!@reload_on_save
-      end
-
-      def refresh
-        self.reload if reload_on_save?
-      end
-    end
-  end
-end
-