hoardable 0.11.0 → 0.12.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2d9f9ea3a4559d3a4515357eabc0c7ee05526ee1308a4e6eef111f2d6207aef6
-  data.tar.gz: be0011cb1ea9d3fcdcd5d1a4f4951d7185371cc1d05c91dfd27ea163b59620b7
+  metadata.gz: eec6cda69bb2bbb1aeec6a0c460881c33abbe1de123ef9178d2c16378374b2bd
+  data.tar.gz: f3f57007e08f958017483dfffd0275b5b70c0ef1522984ea78da4c595a9e5506
 SHA512:
-  metadata.gz: 20ebc8d7ec2924e43e6bbd806e7aaf08350653b695dc4aef16de28359ceb95d8de16da4c19790e8b1212bc8d8d202caeb3614e05b326977bd470defe48e4b731
-  data.tar.gz: fc3fa92ae0ab25bee4122456b889f86eec75b0eee4efba742dced8737d7bc9ab3a99e63e63839f49089183900df8ed54a315a4561aad634c0e2ea3fa56aa8732
+  metadata.gz: 04a13fec8198d4c4fda05b5364a7925755e7f2708004e15a564f243b64836577f4393cb502e7dd7d88ea123df29909ccb36a1d603cae8b2e711958250dc45763
+  data.tar.gz: 34a59cb91ed643784e4aa6ab88abac2d3e7a59a59945bcdce6e849aa441bc1490a6870c2661f65c31d246e4cb87bb39c4502e6cb8ce05cb3c42a6ad0e56f1140

data/README.md

@@ -84,9 +84,9 @@ of that model. As we continue our example from above:
 ```
 $ irb
 >> Post
-=> Post(id: integer, body: text, user_id: integer, created_at: datetime)
+=> Post(id: integer, body: text, user_id: integer, created_at: datetime, hoardable_id: integer)
 >> PostVersion
-=> PostVersion(id: integer, body: text, user_id: integer, created_at: datetime, _data: jsonb, _during: tsrange, hoardable_source_id: integer)
+=> PostVersion(id: integer, body: text, user_id: integer, created_at: datetime, hoardable_id: integer, _data: jsonb, _during: tsrange)
 ```
 
 A `Post` now `has_many :versions`. With the default configuration, whenever an update and deletion
@@ -144,7 +144,7 @@ If you want to look-up the version of a record at a specific time, you can use t
 ```ruby
 post.at(1.day.ago) # => #<PostVersion>
 # or you can use the scope on the version model class
-PostVersion.at(1.day.ago).find_by(hoardable_source_id: post.id) # => #<PostVersion>
+PostVersion.at(1.day.ago).find_by(hoardable_id: post.id) # => #<PostVersion>
 ```
 
 The source model class also has an `.at` method:
@@ -357,7 +357,7 @@ Hoardable.at(datetime) do
   post.comments.size # => 2
   post.id # => 2
   post.version? # => true
-  post.hoardable_source_id # => 1
+  post.hoardable_id # => 1
 end
 ```
 
@@ -370,7 +370,7 @@ version, even though it is masquerading as a `Post`.
 
 If you are ever unsure if a Hoardable record is a "source" or a "version", you can be sure by
 calling `version?` on it. If you want to get the true original source record ID, you can call
-`hoardable_source_id`. 
+`hoardable_id`. 
 
 Sometimes you’ll trash something that `has_many :children, dependent: :destroy` and want
 to untrash everything in a similar dependent manner. Whenever a hoardable version is created in a
@@ -393,7 +393,7 @@ class Post < ActiveRecord::Base
 end
 ```
 
-## ActionText
+## Action Text
 
 Hoardable provides support for ActiveRecord models with `has_rich_text`. First, you must create a
 temporal table for `ActionText::RichText`:
@@ -420,9 +420,9 @@ post = Post.create!(content: '<div>Hello World</div>')
 datetime = DateTime.current
 post.update!(content: '<div>Goodbye Cruel World</div>')
 post.content.versions.size # => 1
-assert_equal post.content.to_plain_text, 'Goodbye Cruel World'
+post.content.to_plain_text # => 'Goodbye Cruel World'
 Hoardable.at(datetime) do
-  assert_equal post.content.to_plain_text, 'Hello World'
+  post.content.to_plain_text # => 'Hello World'
 end
 ```
 

data/lib/generators/hoardable/install_generator.rb

@@ -23,7 +23,7 @@ module Hoardable
     end
 
     def create_migration_file
-      migration_template 'functions.rb.erb', 'db/migrate/install_hoardable.rb'
+      migration_template 'install.rb.erb', 'db/migrate/install_hoardable.rb'
     end
 
     def self.next_migration_number(dir)

data/lib/generators/hoardable/migration_generator.rb

@@ -12,7 +12,7 @@ module Hoardable
     class_option :foreign_key_type, type: :string
 
     def create_versions_table
-      migration_template migration_template_name, "db/migrate/create_#{singularized_table_name}_versions.rb"
+      migration_template 'migration.rb.erb', "db/migrate/create_#{singularized_table_name}_versions.rb"
     end
 
     no_tasks do
@@ -23,12 +23,10 @@ module Hoardable
         'bigint'
       end
 
-      def migration_template_name
-        if Gem::Version.new(ActiveRecord::Migration.current_version.to_s) < Gem::Version.new('7')
-          'migration_6.rb.erb'
-        else
-          'migration.rb.erb'
-        end
+      def primary_key
+        options[:primary_key] || class_name.singularize.constantize.primary_key
+      rescue StandardError
+        'id'
       end
 
       def singularized_table_name

data/lib/generators/hoardable/templates/install.rb.erb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+class InstallHoardable < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
+  def up
+    execute(
+      <<~SQL
+        DO $$
+        BEGIN
+          IF NOT EXISTS (
+            SELECT 1 FROM pg_type t WHERE t.typname = 'hoardable_operation'
+          ) THEN
+            CREATE TYPE hoardable_operation AS ENUM ('update', 'delete', 'insert');
+          END IF;
+        END
+        $$;
+        CREATE OR REPLACE FUNCTION hoardable_source_set_id() RETURNS trigger
+          LANGUAGE plpgsql AS
+        $$
+        DECLARE
+          _pk information_schema.constraint_column_usage.column_name%TYPE;
+          _id _pk%TYPE;
+        BEGIN
+          SELECT c.column_name
+            FROM information_schema.table_constraints t
+            JOIN information_schema.constraint_column_usage c
+            ON c.constraint_name = t.constraint_name
+            WHERE c.table_name = TG_TABLE_NAME AND t.constraint_type = 'PRIMARY KEY'
+            LIMIT 1
+            INTO _pk;
+          EXECUTE format('SELECT $1.%I', _pk) INTO _id USING NEW;
+          NEW.hoardable_id = _id;
+          RETURN NEW;
+        END;$$;
+        CREATE OR REPLACE FUNCTION hoardable_version_prevent_update() RETURNS trigger
+          LANGUAGE plpgsql AS
+        $$BEGIN
+          RAISE EXCEPTION 'updating a version is not allowed';
+          RETURN NEW;
+        END;$$;
+      SQL
+    )
+  end
+  def down
+    execute(
+      <<~SQL
+        DROP TYPE IF EXISTS hoardable_operation;
+        DROP FUNCTION IF EXISTS hoardable_version_prevent_update();
+        DROP FUNCTION IF EXISTS hoardable_source_set_id();
+      SQL
+    )
+  end
+end

data/lib/generators/hoardable/templates/migration.rb.erb

@@ -2,21 +2,25 @@
 
 class Create<%= class_name.singularize.delete(':') %>Versions < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
   def change
-    create_enum :hoardable_operation, %w[update delete insert]
+    add_column :<%= table_name %>, :hoardable_id, :<%= foreign_key_type %>
+    add_index :<%= table_name %>, :hoardable_id
     create_table :<%= singularized_table_name %>_versions, id: false, options: 'INHERITS (<%= table_name %>)' do |t|
       t.jsonb :_data
       t.tsrange :_during, null: false
       t.uuid :_event_uuid, null: false, index: true
-      t.enum :_operation, enum_type: 'hoardable_operation', null: false, index: true
-      t.<%= foreign_key_type %> :hoardable_source_id, null: false, index: true
+      t.column :_operation, :hoardable_operation, null: false, index: true
     end
     reversible do |dir|
       dir.up do
         execute(
           <<~SQL
+            UPDATE <%= table_name %> SET hoardable_id = <%= primary_key %>;
             CREATE TRIGGER <%= singularized_table_name %>_versions_prevent_update
               BEFORE UPDATE ON <%= singularized_table_name %>_versions FOR EACH ROW
               EXECUTE PROCEDURE hoardable_version_prevent_update();
+            CREATE TRIGGER <%= table_name %>_set_hoardable_id
+              BEFORE INSERT ON <%= table_name %> FOR EACH ROW
+              EXECUTE PROCEDURE hoardable_source_set_id();
           SQL
         )
       end
@@ -25,14 +29,17 @@ class Create<%= class_name.singularize.delete(':') %>Versions < ActiveRecord::Mi
           <<~SQL
             DROP TRIGGER <%= singularized_table_name %>_versions_prevent_update
               ON <%= singularized_table_name %>_versions;
+            DROP TRIGGER <%= table_name %>_set_hoardable_id
+              ON <%= table_name %>;
           SQL
         )
       end
     end
-    add_index(:<%= singularized_table_name %>_versions, :id, unique: true)
+    change_column_null :<%= table_name %>, :hoardable_id, false
+    add_index(:<%= singularized_table_name %>_versions, :<%= primary_key %>, unique: true)
     add_index(
       :<%= singularized_table_name %>_versions,
-      %i[_during hoardable_source_id],
+      %i[_during hoardable_id],
       name: 'idx_<%= singularized_table_name %>_versions_temporally'
     )
   end

data/lib/hoardable/associations.rb

@@ -12,7 +12,6 @@ module Hoardable
       include HasOne
       include BelongsTo
       include HasRichText
-      include HasOneAttached
     end
   end
 end

data/lib/hoardable/belongs_to.rb

@@ -21,7 +21,7 @@ module Hoardable
         define_method("trashed_#{name}") do
           source_reflection = reflections[name.to_s]
           source_reflection.version_class.trashed.only_most_recent.find_by(
-            hoardable_source_id: source_reflection.foreign_key
+            hoardable_id: source_reflection.foreign_key
           )
         end
 

data/lib/hoardable/database_client.rb

@@ -13,30 +13,25 @@ module Hoardable
     delegate :version_class, to: :source_record
 
     def insert_hoardable_version(operation, &block)
-      version = version_class.insert(initialize_version_attributes(operation), returning: :id)
-      version_id = version[0]['id']
+      version = version_class.insert(initialize_version_attributes(operation), returning: source_primary_key.to_sym)
+      version_id = version[0][source_primary_key]
       source_record.instance_variable_set('@hoardable_version', version_class.find(version_id))
       source_record.run_callbacks(:versioned, &block)
     end
 
-    def find_or_initialize_hoardable_event_uuid
-      Thread.current[:hoardable_event_uuid] ||= ActiveRecord::Base.connection.query('SELECT gen_random_uuid();')[0][0]
-    end
-
-    def hoardable_version_source_id
-      @hoardable_version_source_id ||= query_hoardable_version_source_id
+    def source_primary_key
+      source_record.class.primary_key
     end
 
-    def query_hoardable_version_source_id
-      primary_key = source_record.class.primary_key
-      version_class.where(primary_key => source_record.read_attribute(primary_key)).pluck('hoardable_source_id')[0]
+    def find_or_initialize_hoardable_event_uuid
+      Thread.current[:hoardable_event_uuid] ||= ActiveRecord::Base.connection.query('SELECT gen_random_uuid();')[0][0]
     end
 
     def initialize_version_attributes(operation)
-      source_record.attributes_before_type_cast.without('id').merge(
+      source_attributes_without_primary_key.merge(
         source_record.changes.transform_values { |h| h[0] },
         {
-          'hoardable_source_id' => source_record.id,
+          'hoardable_id' => source_record.id,
           '_event_uuid' => find_or_initialize_hoardable_event_uuid,
           '_operation' => operation,
           '_data' => initialize_hoardable_data.merge(changes: source_record.changes),
@@ -45,6 +40,10 @@ module Hoardable
       )
     end
 
+    def source_attributes_without_primary_key
+      source_record.attributes_before_type_cast.without(source_primary_key)
+    end
+
     def initialize_temporal_range
       ((previous_temporal_tsrange_end || hoardable_source_epoch)..Time.now.utc)
     end

data/lib/hoardable/engine.rb

@@ -105,11 +105,5 @@ module Hoardable
         require_relative 'encrypted_rich_text' if SUPPORTS_ENCRYPTED_ACTION_TEXT
       end
     end
-
-    initializer 'hoardable.active_storage' do
-      ActiveSupport.on_load(:active_storage_attachment) do
-        require_relative 'attachment'
-      end
-    end
   end
 end

data/lib/hoardable/finder_methods.rb

@@ -7,18 +7,18 @@ module Hoardable
   # with the class method +hoardable+.
   module FinderMethods
     def find_one(id)
-      super(hoardable_source_ids([id])[0])
+      super(hoardable_ids([id])[0])
     end
 
     def find_some(ids)
-      super(hoardable_source_ids(ids))
+      super(hoardable_ids(ids))
     end
 
     private
 
-    def hoardable_source_ids(ids)
+    def hoardable_ids(ids)
       ids.map do |id|
-        version_class.where(hoardable_source_id: id).select(primary_key).ids[0] || id
+        version_class.where(hoardable_id: id).select(primary_key).ids[0] || id
       end
     end
   end

data/lib/hoardable/has_many.rb

@@ -5,7 +5,7 @@ module Hoardable
   module HasMany
     extend ActiveSupport::Concern
 
-    # An +ActiveRecord+ extension that allows looking up {VersionModel}s by +hoardable_source_id+ as
+    # An +ActiveRecord+ extension that allows looking up {VersionModel}s by +hoardable_id+ as
     # if they were {SourceModel}s when using {Hoardable#at}.
     module HasManyExtension
       def scope
@@ -15,9 +15,13 @@ module Hoardable
       private
 
       def hoardable_scope
-        if Hoardable.instance_variable_get('@at') &&
-           (hoardable_source_id = @association.owner.hoardable_source_id)
-          @association.scope.rewhere(@association.reflection.foreign_key => hoardable_source_id)
+        if Hoardable.instance_variable_get('@at') && (hoardable_id = @association.owner.hoardable_id)
+          if @association.reflection.is_a?(ActiveRecord::Reflection::ThroughReflection)
+            @association.reflection.source_reflection.instance_variable_set(
+              '@active_record_primary_key', 'hoardable_id'
+            )
+          end
+          @association.scope.rewhere(@association.reflection.foreign_key => hoardable_id)
         else
           @association.scope
         end

data/lib/hoardable/scopes.rb

@@ -62,10 +62,10 @@ module Hoardable
       scope :at, lambda { |datetime|
         raise(CreatedAtColumnMissingError, @klass.table_name) unless @klass.column_names.include?('created_at')
 
-        include_versions.where(id: version_class.at(datetime).select('id')).or(
+        include_versions.where(id: version_class.at(datetime).select(@klass.primary_key)).or(
           exclude_versions
             .where("#{table_name}.created_at < ?", datetime)
-            .where.not(id: version_class.select(:hoardable_source_id).where(DURING_QUERY, datetime))
+            .where.not(id: version_class.select(:hoardable_id).where(DURING_QUERY, datetime))
         ).hoardable
       }
     end

data/lib/hoardable/source_model.rb

@@ -32,8 +32,6 @@ module Hoardable
       include Scopes
 
       around_update(if: [HOARDABLE_CALLBACKS_ENABLED, HOARDABLE_VERSION_UPDATES]) do |_, block|
-        next if self.is_a?(Hoardable::Attachment)
-
         hoardable_client.insert_hoardable_version('update', &block)
       end
 
@@ -53,7 +51,7 @@ module Hoardable
         dependent: nil,
         class_name: version_class.to_s,
         inverse_of: :hoardable_source,
-        foreign_key: :hoardable_source_id
+        foreign_key: :hoardable_id
       )
     end
 
@@ -62,7 +60,7 @@ module Hoardable
     #
     # @return [Boolean]
     def trashed?
-      versions.trashed.only_most_recent.first&.hoardable_source_id == id
+      !self.class.exists?(self.class.primary_key => id)
     end
 
     # Returns a boolean of whether the record is actually a +version+ cast as an instance of the
@@ -70,7 +68,7 @@ module Hoardable
     #
     # @return [Boolean]
     def version?
-      !!hoardable_client.hoardable_version_source_id
+      hoardable_id != id
     end
 
     # Returns the +version+ at the supplied +datetime+ or +time+, or +self+ if there is none.
@@ -100,13 +98,8 @@ module Hoardable
       version.is_a?(version_class) ? version.revert! : self
     end
 
-    # Returns the +hoardable_source_id+ that represents the original {SourceModel} record’s ID. Will
-    # return nil if the current {SourceModel} record is not an instance of a {VersionModel} cast as
-    # {SourceModel}.
-    #
-    # @return [Integer, nil]
-    def hoardable_source_id
-      hoardable_client.hoardable_version_source_id || id
+    def hoardable_id
+      read_attribute('hoardable_id')
     end
 
     delegate :version_class, to: :class

data/lib/hoardable/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Hoardable
-  VERSION = '0.11.0'
+  VERSION = '0.12.2'
 end

data/lib/hoardable/version_model.rb

@@ -7,6 +7,13 @@ module Hoardable
     extend ActiveSupport::Concern
 
     class_methods do
+      # This is needed to allow {FinderMethods} to work with the version class.
+      #
+      # @!visibility private
+      def version_class
+        self
+      end
+
       # This is needed to omit the pseudo row of 'tableoid' when using +ActiveRecord+’s +insert+.
       #
       # @!visibility private
@@ -20,6 +27,7 @@ module Hoardable
       belongs_to(
         :hoardable_source,
         inverse_of: :versions,
+        foreign_key: :hoardable_id,
         class_name: superclass.model_name
       )
 
@@ -37,7 +45,7 @@ module Hoardable
       # Returns only trashed +versions+ that are currently orphans.
       scope :trashed, lambda {
         left_outer_joins(:hoardable_source)
-          .where(superclass.table_name => { id: nil })
+          .where(superclass.table_name => { superclass.primary_key => nil })
           .where(_operation: 'delete')
       }
 
@@ -78,7 +86,7 @@ module Hoardable
 
       transaction do
         hoardable_source.tap do |reverted|
-          reverted.update!(hoardable_source_attributes.without('id'))
+          reverted.update!(hoardable_source_attributes.without(self.class.superclass.primary_key))
           reverted.instance_variable_set(:@hoardable_version, self)
           reverted.run_callbacks(:reverted)
         end
@@ -113,24 +121,16 @@ module Hoardable
       _data&.dig('changes')
     end
 
-    # Returns the ID of the {SourceModel} that created this {VersionModel}
-    def hoardable_source_id
-      read_attribute('hoardable_source_id')
-    end
-
     private
 
     def insert_untrashed_source
       superscope = self.class.superclass.unscoped
-      superscope.insert(hoardable_source_attributes.merge('id' => hoardable_source_id))
-      superscope.find(hoardable_source_id)
+      superscope.insert(hoardable_source_attributes.merge(superscope.primary_key => hoardable_id))
+      superscope.find(hoardable_id)
     end
 
     def hoardable_source_attributes
-      @hoardable_source_attributes ||=
-        attributes_before_type_cast
-        .without('hoardable_source_id')
-        .reject { |k, _v| k.start_with?('_') }
+      attributes_before_type_cast.without(self.class.column_names - self.class.superclass.column_names)
     end
   end
 end

data/lib/hoardable.rb

@@ -15,6 +15,5 @@ require_relative 'hoardable/has_many'
 require_relative 'hoardable/belongs_to'
 require_relative 'hoardable/has_one'
 require_relative 'hoardable/has_rich_text'
-require_relative 'hoardable/has_one_attached'
 require_relative 'generators/hoardable/migration_generator'
 require_relative 'generators/hoardable/install_generator'

data/sig/hoardable.rbs

@@ -8,6 +8,7 @@ module Hoardable
   HOARDABLE_CALLBACKS_ENABLED: ^(untyped) -> untyped
   HOARDABLE_SAVE_TRASH: ^(untyped) -> untyped
   HOARDABLE_VERSION_UPDATES: ^(untyped) -> untyped
+  SUPPORTS_ENCRYPTED_ACTION_TEXT: untyped
   self.@context: Hash[untyped, untyped]
   self.@config: untyped
   self.@at: nil
@@ -17,12 +18,15 @@ module Hoardable
   def self.at: (untyped datetime) -> untyped
   def self.logger: -> untyped
 
+  class Engine
+  end
+
   module FinderMethods
     def find_one: (untyped id) -> untyped
     def find_some: (untyped ids) -> untyped
 
     private
-    def hoardable_source_ids: ([untyped] ids) -> Array[untyped]
+    def hoardable_ids: ([untyped] ids) -> Array[untyped]
   end
 
   module Scopes
@@ -44,15 +48,13 @@ module Hoardable
   end
 
   class DatabaseClient
-    @hoardable_version_source_id: untyped
-
     attr_reader source_record: SourceModel
     def initialize: (SourceModel source_record) -> void
     def insert_hoardable_version: (untyped operation) -> untyped
+    def source_primary_key: -> untyped
     def find_or_initialize_hoardable_event_uuid: -> untyped
-    def hoardable_version_source_id: -> untyped
-    def query_hoardable_version_source_id: -> untyped
     def initialize_version_attributes: (untyped operation) -> untyped
+    def source_attributes_without_primary_key: -> untyped
     def initialize_temporal_range: -> Range
     def initialize_hoardable_data: -> untyped
     def assign_hoardable_context: (:event_uuid | :meta | :note | :whodunit key) -> nil
@@ -71,7 +73,7 @@ module Hoardable
     def at: (untyped datetime) -> SourceModel?
     def version_at: (untyped datetime) -> untyped
     def revert_to!: (untyped datetime) -> SourceModel?
-    def hoardable_source_id: -> untyped
+    def hoardable_id: -> untyped
 
     private
     def hoardable_client: -> DatabaseClient
@@ -82,18 +84,16 @@ module Hoardable
   end
 
   module VersionModel
-    @hoardable_source_attributes: untyped
-
     def revert!: -> untyped
     def untrash!: -> untyped
     def changes: -> untyped
-    def hoardable_source_id: -> untyped
 
     private
     def insert_untrashed_source: -> untyped
     def hoardable_source_attributes: -> untyped
 
     public
+    def version_class: -> VersionModel
     def scope_attributes: -> untyped
   end
 
@@ -108,15 +108,15 @@ module Hoardable
   end
 
   module Associations
-    @hoardable_association_overrider: Overrider
+    include HasRichText
+    include BelongsTo
+    include HasOne
+    include HasMany
+  end
 
-    def belongs_to: (*untyped args) -> nil
-    def has_one: (*untyped args) -> nil
+  module HasMany
     def has_many: (*untyped args) -> untyped
 
-    private
-    def hoardable_association_overrider: -> Overrider
-
     module HasManyExtension
       @scope: untyped
       @association: bot
@@ -126,14 +126,21 @@ module Hoardable
       private
       def hoardable_scope: -> untyped
     end
+  end
 
-    class Overrider
-      attr_reader klass: Associations
-      def initialize: (Associations klass) -> void
-      def override_belongs_to: (untyped name) -> untyped
-      def override_has_one: (untyped name) -> untyped
-      def override_has_many: (untyped name) -> untyped
-    end
+  module BelongsTo
+    def belongs_to: (*untyped args) -> nil
+
+    private
+    def hoardable_override_belongs_to: (untyped name) -> untyped
+  end
+
+  module HasOne
+    def has_one: (*untyped args) -> nil
+  end
+
+  module HasRichText
+    def has_rich_text: (untyped name, ?encrypted: false, ?hoardable: false) -> nil
   end
 
   class MigrationGenerator
@@ -141,7 +148,7 @@ module Hoardable
 
     def create_versions_table: -> untyped
     def foreign_key_type: -> String
-    def migration_template_name: -> String
+    def primary_key: -> String
     def singularized_table_name: -> untyped
   end
 
@@ -150,4 +157,12 @@ module Hoardable
     def create_migration_file: -> untyped
     def self.next_migration_number: (untyped dir) -> untyped
   end
+
+  class RichText
+    include Model
+  end
+
+  class EncryptedRichText
+    include Model
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hoardable
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.12.2
 platform: ruby
 authors:
 - justin talbott
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-10-09 00:00:00.000000000 Z
+date: 2022-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -106,12 +106,10 @@ files:
 - Rakefile
 - lib/generators/hoardable/install_generator.rb
 - lib/generators/hoardable/migration_generator.rb
-- lib/generators/hoardable/templates/functions.rb.erb
+- lib/generators/hoardable/templates/install.rb.erb
 - lib/generators/hoardable/templates/migration.rb.erb
-- lib/generators/hoardable/templates/migration_6.rb.erb
 - lib/hoardable.rb
 - lib/hoardable/associations.rb
-- lib/hoardable/attachment.rb
 - lib/hoardable/belongs_to.rb
 - lib/hoardable/database_client.rb
 - lib/hoardable/encrypted_rich_text.rb
@@ -120,7 +118,6 @@ files:
 - lib/hoardable/finder_methods.rb
 - lib/hoardable/has_many.rb
 - lib/hoardable/has_one.rb
-- lib/hoardable/has_one_attached.rb
 - lib/hoardable/has_rich_text.rb
 - lib/hoardable/model.rb
 - lib/hoardable/rich_text.rb

data/lib/generators/hoardable/templates/functions.rb.erb

@@ -1,16 +0,0 @@
-# frozen_string_literal: true
-
-class InstallHoardable < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
-  def up
-    execute(
-      <<~SQL
-        CREATE OR REPLACE FUNCTION hoardable_version_prevent_update() RETURNS trigger
-          LANGUAGE plpgsql AS
-        $$BEGIN
-          RAISE EXCEPTION 'updating a version is not allowed';
-          RETURN NEW;
-        END;$$;
-      SQL
-    )
-  end
-end

data/lib/generators/hoardable/templates/migration_6.rb.erb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-class Create<%= class_name.singularize.delete(':') %>Versions < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
-  def change
-    reversible do |dir|
-      dir.up do
-        execute <<~SQL
-          DO $$
-          BEGIN
-              IF NOT EXISTS (
-                SELECT 1 FROM pg_type t
-                WHERE t.typname = 'hoardable_operation'
-              ) THEN
-                CREATE TYPE hoardable_operation AS ENUM ('update', 'delete', 'insert');
-              END IF;
-          END
-          $$;
-        SQL
-      end
-    end
-    create_table :<%= singularized_table_name %>_versions, id: false, options: 'INHERITS (<%= table_name %>)' do |t|
-      t.jsonb :_data
-      t.tsrange :_during, null: false
-      t.uuid :_event_uuid, null: false, index: true
-      t.column :_operation, :hoardable_operation, null: false, index: true
-      t.<%= foreign_key_type %> :hoardable_source_id, null: false, index: true
-    end
-    reversible do |dir|
-      dir.up do
-        execute(
-          <<~SQL
-            CREATE TRIGGER <%= singularized_table_name %>_versions_prevent_update
-              BEFORE UPDATE ON <%= singularized_table_name %>_versions FOR EACH ROW
-              EXECUTE PROCEDURE hoardable_version_prevent_update();
-          SQL
-        )
-      end
-      dir.down do
-        execute(
-          <<~SQL
-            DROP TRIGGER <%= singularized_table_name %>_versions_prevent_update
-              ON <%= singularized_table_name %>_versions;
-          SQL
-        )
-      end
-    end
-    add_index(:<%= singularized_table_name %>_versions, :id, unique: true)
-    add_index(
-      :<%= singularized_table_name %>_versions,
-      %i[_during hoardable_source_id],
-      name: 'idx_<%= singularized_table_name %>_versions_temporally'
-    )
-  end
-end

data/lib/hoardable/attachment.rb

@@ -1,16 +0,0 @@
-# frozen_string_literal: true
-
-module Hoardable
-  # A {Hoardable} subclass of {ActiveStorage::Attachment}
-  class Attachment < ActiveStorage::Attachment
-    include Model
-
-    class CreateOne < ActiveStorage::Attached::Changes::CreateOne
-      private
-
-      def build_attachment
-        Attachment.new(record: record, name: name, blob: blob)
-      end
-    end
-  end
-end

data/lib/hoardable/has_one_attached.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-module Hoardable
-  # Provides temporal version awareness for +ActionText+.
-  module HasOneAttached
-    extend ActiveSupport::Concern
-
-    class_methods do
-      def has_one_attached(name, **options, &block)
-        hoardable = options.delete(:hoardable)
-        super(name, **options, &block)
-        return unless hoardable
-
-        'ActiveStorage::Attachment'.constantize
-        reflection_options = reflections["#{name}_attachment"].options
-        reflection_options[:class_name] = reflection_options[:class_name].sub(/ActiveStorage/, 'Hoardable')
-
-        generated_association_methods.class_eval <<-CODE, __FILE__, __LINE__ + 1
-          # frozen_string_literal: true
-          def #{name}=(attachable)
-            attachment_changes["#{name}"] =
-              if attachable.nil?
-                ActiveStorage::Attached::Changes::DeleteOne.new("#{name}", self)
-              else
-                Hoardable::Attachment::CreateOne.new("#{name}", self, attachable)
-              end
-          end
-        CODE
-      end
-    end
-  end
-  private_constant :HasOneAttached
-end