log_book 0.1.7 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 844141903822ef7b4da7514701dfb0b3d4463189
-  data.tar.gz: 6d389d3383574dbe3e676419bf46724f178482d9
+SHA256:
+  metadata.gz: 3df7903b6702205b16c36791b7f8ebcfc1612f3dcb0042bfbc77bbde66050ff0
+  data.tar.gz: fdbdd4bdb6424a69ba602a0233e7611a2c3a53aeea65368ee0974f3b6e66275a
 SHA512:
-  metadata.gz: a375456188f8cb54db495328f8260bd92adadcce1446a8454ae84bfaceedab837520cc475fbd1e7c4d46feab2f7d4748684e5781e5a2ae8201d81150979f27e0
-  data.tar.gz: a38628f1726647304d3b31ee46fbba09857f71a6cb2e25e686f1a836ed36ddb9539a063fb60d65c8206ce2d307012c264a5d21a0f9731bbfe65f2c3eb20c795e
+  metadata.gz: 88f94471b1bb9b83c11abd1b6ba3cc7c5f0c6b440a7d49014a033ada08b87945be21c15a3f41bfef9e2fc8ea145dc9941aa21ed2cdcf311167dee4dcca2a96eb
+  data.tar.gz: 6931310a5f20887d3936b14d8a59836dda15366e5a046c7be0ac9ebda5802c2650ef944bd1e217e1a303b51af294c39fc5feaf654d7f497614fdf1cf374e07e7

data/.gitignore

@@ -3,7 +3,6 @@
 .bundle
 .config
 .yardoc
-Gemfile.lock
 InstalledFiles
 _yardoc
 coverage

data/.travis.yml

@@ -0,0 +1,9 @@
+language: ruby
+rvm:
+  - 2.3.1
+  - 2.5.1
+
+matrix:
+  fast_finish: true
+  allow_failures:
+  - rvm: 2.5.1

data/Gemfile.lock

@@ -0,0 +1,173 @@
+PATH
+  remote: .
+  specs:
+    log_book (0.6.3)
+      acts-as-taggable-on (>= 8.0)
+      rails (>= 6.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actioncable (7.0.2.2)
+      actionpack (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      nio4r (~> 2.0)
+      websocket-driver (>= 0.6.1)
+    actionmailbox (7.0.2.2)
+      actionpack (= 7.0.2.2)
+      activejob (= 7.0.2.2)
+      activerecord (= 7.0.2.2)
+      activestorage (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      mail (>= 2.7.1)
+      net-imap
+      net-pop
+      net-smtp
+    actionmailer (7.0.2.2)
+      actionpack (= 7.0.2.2)
+      actionview (= 7.0.2.2)
+      activejob (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      mail (~> 2.5, >= 2.5.4)
+      net-imap
+      net-pop
+      net-smtp
+      rails-dom-testing (~> 2.0)
+    actionpack (7.0.2.2)
+      actionview (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      rack (~> 2.0, >= 2.2.0)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.2.0)
+    actiontext (7.0.2.2)
+      actionpack (= 7.0.2.2)
+      activerecord (= 7.0.2.2)
+      activestorage (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      globalid (>= 0.6.0)
+      nokogiri (>= 1.8.5)
+    actionview (7.0.2.2)
+      activesupport (= 7.0.2.2)
+      builder (~> 3.1)
+      erubi (~> 1.4)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.1, >= 1.2.0)
+    activejob (7.0.2.2)
+      activesupport (= 7.0.2.2)
+      globalid (>= 0.3.6)
+    activemodel (7.0.2.2)
+      activesupport (= 7.0.2.2)
+    activerecord (7.0.2.2)
+      activemodel (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+    activestorage (7.0.2.2)
+      actionpack (= 7.0.2.2)
+      activejob (= 7.0.2.2)
+      activerecord (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      marcel (~> 1.0)
+      mini_mime (>= 1.1.0)
+    activesupport (7.0.2.2)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+    acts-as-taggable-on (9.0.1)
+      activerecord (>= 6.0, < 7.1)
+    assert_difference (1.0.0)
+      activesupport (>= 3.0.0)
+    builder (3.2.4)
+    concurrent-ruby (1.1.9)
+    crass (1.0.6)
+    digest (3.1.0)
+    erubi (1.10.0)
+    globalid (1.0.0)
+      activesupport (>= 5.0)
+    i18n (1.10.0)
+      concurrent-ruby (~> 1.0)
+    io-wait (0.2.1)
+    loofah (2.14.0)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    mail (2.7.1)
+      mini_mime (>= 0.1.1)
+    marcel (1.0.2)
+    method_source (1.0.0)
+    mini_mime (1.1.2)
+    minitest (5.15.0)
+    mocha (1.13.0)
+    net-imap (0.2.3)
+      digest
+      net-protocol
+      strscan
+    net-pop (0.1.1)
+      digest
+      net-protocol
+      timeout
+    net-protocol (0.1.2)
+      io-wait
+      timeout
+    net-smtp (0.3.1)
+      digest
+      net-protocol
+      timeout
+    nio4r (2.5.8)
+    nokogiri (1.13.3-x86_64-darwin)
+      racc (~> 1.4)
+    racc (1.6.0)
+    rack (2.2.3)
+    rack-test (1.1.0)
+      rack (>= 1.0, < 3)
+    rails (7.0.2.2)
+      actioncable (= 7.0.2.2)
+      actionmailbox (= 7.0.2.2)
+      actionmailer (= 7.0.2.2)
+      actionpack (= 7.0.2.2)
+      actiontext (= 7.0.2.2)
+      actionview (= 7.0.2.2)
+      activejob (= 7.0.2.2)
+      activemodel (= 7.0.2.2)
+      activerecord (= 7.0.2.2)
+      activestorage (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      bundler (>= 1.15.0)
+      railties (= 7.0.2.2)
+    rails-dom-testing (2.0.3)
+      activesupport (>= 4.2.0)
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.4.2)
+      loofah (~> 2.3)
+    railties (7.0.2.2)
+      actionpack (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+      method_source
+      rake (>= 12.2)
+      thor (~> 1.0)
+      zeitwerk (~> 2.5)
+    rake (13.0.6)
+    sqlite3 (1.4.2)
+    strscan (3.0.1)
+    thor (1.2.1)
+    timeout (0.2.0)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    websocket-driver (0.7.5)
+      websocket-extensions (>= 0.1.0)
+    websocket-extensions (0.1.5)
+    zeitwerk (2.5.4)
+
+PLATFORMS
+  x86_64-darwin-19
+
+DEPENDENCIES
+  assert_difference (>= 1.0)
+  bundler (>= 2.0)
+  log_book!
+  minitest (>= 5.1)
+  mocha (>= 1.13)
+  rake (>= 0.13)
+  sqlite3 (>= 1.0)
+
+BUNDLED WITH
+   2.2.32

data/README.md

@@ -1,6 +1,6 @@
 # LogBook
 
-Storing an events log book.
+Ruby on Rails, Automatically keeping record of all Model Changes
 
 ## Installation
 
@@ -23,6 +23,7 @@ As the Model should be loaded after ActiveReccord has established the connection
 ### ActsOnTaggableOn dependency
 
     rails generate acts_as_taggable_on:migration
+    # rake acts_as_taggable_on_engine:install:migrations # for version '~> 4.0' or superior
     rake db:migrate
 
 ## Usage
@@ -50,8 +51,50 @@ If you want to include _who executes the action_ use the special attribute `log_
     my_model.log_book_historian = current_user
     my_model.save!
 
+If you want to _mute_ a model change:
+
+    my_model.log_book_mute = true
+    my_model.save! # No LogBook::Event will be generated
+
+If you want to _mute_ LogBook globally:
+
+    LogBook.mute = true
+    my_model.save! # No LogBook::Event will be generated
+
+If you want to _ignore_ some fields from the changes Event:
+
+    class MyModel < ActiveRecord::Base
+      log_book :ignore => [:my_counter]
+    end
+
+    my_model.update_atttibtes!(:my_counter => 9) # No LogBook::Event will be generated
+
+If you want _LogBook::Events_ to be destroyed on _Model_ destroy:
+
+    class MyModel < ActiveRecord::Base
+      log_book :dependent => :destroy
+    end
+
+In other case the _LogBook::Events_ will remain after _Model_ destroyed.
+
+## Rails Integration
+
+Check this example project to see how LogBook is integrated:
+
+- [Skeleton](https://github.com/fguillen/Skeleton)
+
+Or this commit in a more modern Rails project:
+
+- [RailsSkeleton](https://github.com/fguillen/RailsSkeleton/commit/4c16c6402254bf4c44b7589b404fc22c746bae15)
+
 ## TODO
 
+Use block configuration instead of `model.log_book_historian` do something like:
+
+    LogBook.conf(:log_book_historian => user) do
+      model.save!
+    end
+
 
 ## Sate of the art
 

data/lib/generators/log_book/templates/create_log_book_events.rb

@@ -1,17 +1,20 @@
-class CreateLogBookEvents < ActiveRecord::Migration
+class CreateLogBookEvents < ActiveRecord::Migration[4.2]
   def self.up
     create_table :log_book_events do |t|
       t.integer :historian_id
       t.string :historian_type
       t.integer :historizable_id
       t.string :historizable_type
-      t.text :text, :null => false
+      t.text :differences, :limit => 16777215 # mediumtext
 
-      t.timestamps
+      t.timestamps :null => false
     end
+
+    add_index :log_book_events, [:historizable_id, :historizable_type, :created_at], :name => "index_log_book_events_on_historizable_and_created_at"
+    add_index :log_book_events, [:created_at]
   end
 
   def self.down
     drop_table :log_book_events
   end
-end
+end

data/lib/log_book/event.rb

@@ -1,14 +1,12 @@
 class LogBook::Event < ::ActiveRecord::Base
   self.table_name = "log_book_events"
 
-  attr_accessible :historian, :historizable, :text, :tag_list
-
   acts_as_taggable
 
-  belongs_to :historian, :polymorphic => true
+  belongs_to :historian, :polymorphic => true, :optional => true
   belongs_to :historizable, :polymorphic => true
 
-  validates :text, :presence => true
-
   scope :by_recent, -> { order("id desc") }
+
+  serialize :differences, JSON
 end

data/lib/log_book/plugin.rb

@@ -5,7 +5,6 @@ module LogBook::Plugin
   end
 
   module ClassMethods
-
     def log_book(opts = {})
       after_create :log_book_event_on_create
       after_update :log_book_event_on_update
@@ -14,24 +13,36 @@ module LogBook::Plugin
       has_many :log_book_events, :class_name => "LogBook::Event", :as => :historizable, :dependent => (opts[:dependent] || :nullify)
 
       attr_accessor :log_book_historian
+      attr_accessor :log_book_mute
+      cattr_accessor :log_book_options
+
+      self.log_book_options = opts
+      self.log_book_options[:ignore] ||= []
+      self.log_book_options[:ignore] << :updated_at # ignoring noisy field
     end
   end
 
   module InstanceMethods
     def log_book_event_on_create
-      LogBook.created(self.log_book_historian, self)
+      LogBook.created(self.log_book_historian, self) if !self.log_book_mute
     end
 
     def log_book_event_on_update
-      LogBook.updated(self.log_book_historian, self) unless changes.empty?
+      # TODO: this line of code is duplicated
+      if ActiveRecord::VERSION::STRING.to_f >= 5.1
+        clean_changes = saved_changes.select { |k,v| !self.log_book_options[:ignore].include? k.to_sym }
+      else
+        clean_changes = changes.select { |k,v| !self.log_book_options[:ignore].include? k.to_sym }
+      end
+      LogBook.updated(self.log_book_historian, self) if !clean_changes.empty? and !self.log_book_mute
     end
 
     def log_book_event_on_destroy
-      LogBook.destroyed(self.log_book_historian, self)
+      LogBook.destroyed(self.log_book_historian, self) if !self.log_book_mute
     end
   end
 end
 
 ActiveSupport.on_load(:active_record) do
   include LogBook::Plugin
-end
+end

data/lib/log_book/utils.rb

@@ -1,16 +1,27 @@
 module LogBook::Utils
   def self.pretty_changes(model)
+    # TODO: this line of code is duplicated
+    if ActiveRecord::VERSION::STRING.to_f >= 5.1
+      clean_changes = model.saved_changes.select { |k,v| !model.log_book_options[:ignore].include? k.to_sym }
+    else
+      clean_changes = model.changes.select { |k,v| !model.log_book_options[:ignore].include? k.to_sym }
+    end
+
     result =
-      model.changes.reject { |k,v| k == "updated_at" || k =~ /password/ || k == "perishable_token" || k == "persistence_token" }.map do |k,v|
+      clean_changes.map do |k,v|
         old_value = v[0]
         new_value = v[1]
 
         old_value = old_value.to_s( :localdb ) if old_value.instance_of? ActiveSupport::TimeWithZone
         new_value = new_value.to_s( :localdb ) if new_value.instance_of? ActiveSupport::TimeWithZone
 
-        "#{k}[#{old_value} -> #{new_value}]"
-      end.join( ", " )
+        {
+          "key" => k,
+          "before" => old_value,
+          "after" => new_value
+        }
+      end
 
     result
   end
-end
+end

data/lib/log_book/version.rb

@@ -1,3 +1,3 @@
 module LogBook
-  VERSION = "0.1.7"
+  VERSION = "0.7.1"
 end

data/lib/log_book.rb

@@ -1,6 +1,5 @@
 require "active_record"
 require "active_support/core_ext/module"
-require "protected_attributes"
 require "acts-as-taggable-on"
 require_relative "log_book/version"
 require_relative "log_book/plugin"
@@ -13,32 +12,44 @@ module LogBook
     :destroy => "destroy"
   }
 
-  def self.event(historian, historizable, text, tag_list)
+  @@muted = false
+
+  def self.event(historian, historizable, differences, tag_list)
+    return if @@muted
+
     tag_list_composed = []
     tag_list_composed << scope_tag(historian)   if historian
     tag_list_composed << kind_tag(historizable) if historizable
     tag_list_composed += [tag_list].flatten     if tag_list
-    
+
     LogBook::Event.create!(
       :historian => historian,
       :historizable => historizable,
-      :text => text,
+      :differences => differences,
       :tag_list => tag_list_composed
     )
   end
 
+  def self.muted=(value)
+    @@muted = value
+  end
+
+  def self.muted
+    @@muted
+  end
+
   private
 
   def self.created(historian, historizable)
-    LogBook.event(historian, historizable, "#{historizable.class.name} created", LogBook::OPERATIONS[:create])
+    LogBook.event(historian, historizable, nil, LogBook::OPERATIONS[:create])
   end
 
   def self.updated(historian, historizable)
-    LogBook.event(historian, historizable, "#{historizable.class.name} updated [#{LogBook::Utils.pretty_changes(historizable)}]", LogBook::OPERATIONS[:update])
+    LogBook.event(historian, historizable, LogBook::Utils.pretty_changes(historizable), LogBook::OPERATIONS[:update])
   end
 
   def self.destroyed(historian, historizable)
-    LogBook.event(historian, historizable, "#{historizable.class.name} destroyed", LogBook::OPERATIONS[:destroy])
+    LogBook.event(historian, historizable, nil, LogBook::OPERATIONS[:destroy])
   end
 
   def self.scope_tag(historian)
@@ -49,4 +60,3 @@ module LogBook
     historizable.class.name.underscore
   end
 end
-

data/log_book.gemspec

@@ -18,14 +18,13 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_runtime_dependency "rails", "~> 4.1.4"
-  spec.add_runtime_dependency "protected_attributes"
-  spec.add_runtime_dependency "acts-as-taggable-on", "3.3.0"
+  spec.add_runtime_dependency "rails", ">= 6.0"
+  spec.add_runtime_dependency "acts-as-taggable-on", ">= 8.0"
 
-  spec.add_development_dependency "bundler", "~> 1.6.3"
-  spec.add_development_dependency "rake"
-  spec.add_development_dependency "mocha"
-  spec.add_development_dependency "sqlite3"
-  spec.add_development_dependency "assert_difference"
-  spec.add_development_dependency "minitest", "~> 5.4.0"
+  spec.add_development_dependency "bundler", ">= 2.0"
+  spec.add_development_dependency "rake", ">= 0.13"
+  spec.add_development_dependency "mocha", ">= 1.13"
+  spec.add_development_dependency "sqlite3", ">= 1.0"
+  spec.add_development_dependency "assert_difference", ">= 1.0"
+  spec.add_development_dependency "minitest", ">= 5.1"
 end

data/test/log_book_test.rb

@@ -12,54 +12,101 @@ class LogBookTest < MiniTest::Test
 
   def test_event
     assert_difference "LogBook::Event.count", 1 do
-      LogBook.event(@user, @item, "Item wadus", LogBook::OPERATIONS[:create])
+      LogBook.event(@user, @item, ["DIFFERENCE"], LogBook::OPERATIONS[:create])
     end
 
-    log_book = LogBook::Event.last
-    assert_equal(@user, log_book.historian)
-    assert_equal(@item, log_book.historizable)
-    assert_equal("Item wadus", log_book.text)
-    assert_equal(["user", "item", "create"].sort, log_book.tag_list.sort)
+    log_book_event = LogBook::Event.last
+    assert_equal(@user, log_book_event.historian)
+    assert_equal(@item, log_book_event.historizable)
+    assert_equal(["DIFFERENCE"], log_book_event.differences)
+    assert_equal(["user", "item", "create"].sort, log_book_event.tag_list.sort)
   end
 
   def test_event_with_nils
     assert_difference "LogBook::Event.count", 1 do
-      LogBook.event(nil, nil, "Item wadus", nil)
+      LogBook.event(nil, nil, ["DIFFERENCE"], nil)
     end
 
-    log_book = LogBook::Event.last
-    assert_equal(nil, log_book.historian)
-    assert_equal(nil, log_book.historizable)
-    assert_equal("Item wadus", log_book.text)
-    assert_equal([], log_book.tag_list)
+    log_book_event = LogBook::Event.last
+    assert_nil(log_book_event.historian)
+    assert_nil(log_book_event.historizable)
+    assert_equal(["DIFFERENCE"], log_book_event.differences)
+    assert_equal([], log_book_event.tag_list)
   end
 
   def test_created
     item = Item.new(:title => "Item Title")
     item.log_book_historian = @user
 
-    LogBook.expects(:event).with(@user, item, "Item created", LogBook::OPERATIONS[:create])
+    LogBook.expects(:event).with(@user, item, nil, LogBook::OPERATIONS[:create])
+
+    item.save!
+  end
+
+  def test_created_when_muted
+    item = Item.new(:title => "Item Title")
+    item.log_book_mute = true
+
+    LogBook.expects(:event).never
 
     item.save!
   end
 
   def test_updated
-    LogBook.expects(:event).with(@user, @item, "Item updated [title[Item Title -> Other Title]]", LogBook::OPERATIONS[:update])
+    differences = [
+      {
+        "key" => "title",
+        "before" => "Item Title",
+        "after" => "Other Title"
+      }
+    ]
+
+    LogBook.expects(:event).with(@user, @item, differences, LogBook::OPERATIONS[:update])
+
+    @item.update!(:title => "Other Title")
+  end
+
+  def test_updated_when_muted
+    LogBook.expects(:event).never
+
+    @item.log_book_mute = true
+    @item.update!(:title => "Other Title")
+  end
+
+  def test_updated_with_ignore_fields
+    differences = [
+      {
+        "key" => "title",
+        "before" => "Item Title",
+        "after" => "Other Title"
+      }
+    ]
 
-    @item.update_attributes!(:title => "Other Title")
+    item_with_opts = ItemWithOpts.create!(:title => "Item Title", :my_counter => 0)
+    LogBook.expects(:event).with(@user, item_with_opts, differences, LogBook::OPERATIONS[:update])
+
+    item_with_opts.log_book_historian = @user
+    item_with_opts.update!(:title => "Other Title", :my_counter => 10)
   end
 
   def test_item_destroyed
-    LogBook.expects(:event).with(@user, @item, "Item destroyed", LogBook::OPERATIONS[:destroy])
+    LogBook.expects(:event).with(@user, @item, nil, LogBook::OPERATIONS[:destroy])
 
     @item.destroy
   end
 
+  def test_item_destroyed_when_muted
+    LogBook.expects(:event).never
+
+    @item.log_book_mute = true
+    @item.destroy
+  end
+
   def test_has_many_log_book_events
     LogBook::Event.destroy_all
 
-    log_book_event_1 = LogBook.event(@user, @item, "Item wadus", LogBook::OPERATIONS[:create])
-    log_book_event_2 = LogBook.event(@user, @item, "Item wadus", LogBook::OPERATIONS[:update])
+    log_book_event_1 = LogBook.event(@user, @item, nil, LogBook::OPERATIONS[:create])
+    log_book_event_2 = LogBook.event(@user, @item, nil, LogBook::OPERATIONS[:update])
 
     @item.reload
 
@@ -69,7 +116,7 @@ class LogBookTest < MiniTest::Test
   end
 
   def test_log_book_events_nullify_on_historizable_destroy
-    log_book_event = LogBook.event(@user, @item, "Item wadus", LogBook::OPERATIONS[:create])
+    log_book_event = LogBook.event(@user, @item, nil, LogBook::OPERATIONS[:create])
 
     @item.destroy
 
@@ -77,16 +124,26 @@ class LogBookTest < MiniTest::Test
 
     assert_equal(true, log_book_event.historizable_id.nil?)
     assert_equal(true, log_book_event.historizable.nil?)
-    assert_equal("Item", log_book_event.historizable_type)
+    assert_equal(true, log_book_event.historizable_type.nil?)
   end
 
   def test_log_book_events_destroy_on_historizable_destroy
     item_with_opts = ItemWithOpts.new
-    log_book_event = LogBook.event(@user, item_with_opts, "Item wadus", LogBook::OPERATIONS[:create])
+    log_book_event = LogBook.event(@user, item_with_opts, nil, LogBook::OPERATIONS[:create])
 
     item_with_opts.destroy
 
     assert_equal(false, LogBook::Event.exists?(log_book_event.id))
   end
-end
 
+  def test_created_when_global_muted
+    item = Item.new(:title => "Item Title")
+    LogBook.muted = true
+
+    LogBook::Event.expects(:create!).never
+
+    item.save!
+
+    LogBook.muted = false
+  end
+end

data/test/models.rb

@@ -6,5 +6,5 @@ class Item < ActiveRecord::Base
 end
 
 class ItemWithOpts < Item
-  log_book(:dependent => :destroy)
+  log_book(:dependent => :destroy, :ignore => [:my_counter])
 end

data/test/plugin_test.rb

@@ -19,7 +19,7 @@ class PluginTest < MiniTest::Test
   def test_event_on_update
     @item.log_book_historian = @user
     LogBook.expects(:updated).with(@user, @item)
-    @item.update_attributes(:title => "Other Title")
+    @item.update!(:title => "Other Title")
   end
 
   def test_event_on_destroy
@@ -28,4 +28,3 @@ class PluginTest < MiniTest::Test
     @item.destroy
   end
 end
-

data/test/schema.rb

@@ -5,23 +5,28 @@ ActiveRecord::Schema.define :version => 0 do
 
   create_table :items, :force => true do |t|
     t.string :title
+    t.integer :my_counter
   end
 
   # acts-as-taggable-on
-  # https://github.com/mbleigh/acts-as-taggable-on/tree/master/db/migrate
-  #
-  create_table :tags do |t|
+  # https://github.com/mbleigh/acts-as-taggable-on/blob/6e1837762f0d60edc9b6c2d92b5a9435f404173f/spec/internal/db/schema.rb
+  create_table :tags, force: true do |t|
     t.string :name
+    t.integer :taggings_count, default: 0
+    t.string :type
   end
+  add_index 'tags', ['name'], name: 'index_tags_on_name', unique: true
 
-  create_table :taggings do |t|
+  create_table :taggings, force: true do |t|
     t.references :tag
-    t.references :taggable, :polymorphic => true
-    t.references :tagger, :polymorphic => true
-    t.string :context, :limit => 128
+    t.references :taggable, polymorphic: true
+    t.references :tagger, polymorphic: true
+    t.string :context, limit: 128
 
     t.datetime :created_at
   end
 
-  add_column :tags, :taggings_count, :integer, default: 0
+  add_index 'taggings',
+            ['tag_id', 'taggable_id', 'taggable_type', 'context', 'tagger_id', 'tagger_type'],
+            unique: true, name: 'taggings_idx'
 end

data/test/test_helper.rb

@@ -1,9 +1,13 @@
+
 require "active_record"
 require "active_support/core_ext/module"
 require "assert_difference"
 require "acts-as-taggable-on"
+
+# require "minitest/unit"
 require "minitest/autorun"
-require "mocha/setup"
+require "mocha/mini_test"
+
 require_relative "../lib/log_book"
 
 FileUtils.rm("#{File.dirname(__FILE__)}/db/log_book.sqlite", :force => true)

metadata

@@ -1,141 +1,127 @@
 --- !ruby/object:Gem::Specification
 name: log_book
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.7.1
 platform: ruby
 authors:
 - Fernando Guillen
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2014-07-15 00:00:00.000000000 Z
+date: 2022-03-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 4.1.4
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 4.1.4
-- !ruby/object:Gem::Dependency
-  name: protected_attributes
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: acts-as-taggable-on
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: '8.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: '8.0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.3
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.3
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.13'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.13'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: assert_difference
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 5.4.0
+        version: '5.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 5.4.0
+        version: '5.1'
 description: Storing an events log book
 email:
 - fguillen.mail@gmail.com
@@ -143,9 +129,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .ruby-version
+- ".gitignore"
+- ".travis.yml"
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -167,24 +154,23 @@ homepage: https://github.com/fguillen/LogBook
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.0.3
-signing_key: 
+rubygems_version: 3.2.22
+signing_key:
 specification_version: 4
 summary: Storing an events log book
 test_files:

data/.ruby-version

@@ -1 +0,0 @@
-2.0.0-p247