change_logger 0.0.3 → 0.0.4

data/Manifest

@@ -1,7 +1,7 @@
 CHANGELOG
 MIT-LICENSE
 Manifest
-README
+README.rdoc
 Rakefile
 cucumber.yml.bac
 features/log_changes.feature

data/README.rdoc

@@ -0,0 +1,29 @@
+= change_logger
+
+change_logger is based off of https://github.com/airblade/paper_trail. Instead of storing the whole updated object in a text column, it creates a record for each changed value. Not sure if it's a better way of tracking changes, but it's how I wanted the changes stored.
+
+== Download
+
+Github: http://github.com/danengle/change_logger
+
+Gem:
+  gem install change_logger
+	
+== Usage
+
+Run the generator to create the migration
+  rails g change_logger
+  rake db:migrate
+
+In any model that you want to track changes, add acts_as_change_logger below all of the association declarations so that it can properly hook into any has_and_belongs_to_many associations. Use the ignore option to not save changes for certain attributes.
+
+  class User < ActiveRecord::Base
+    has_many :things
+    has_and_belongs_to_many :permissions
+		
+    acts_as_change_logger :ignore => [:crypted_password, :password_salt]
+  end
+	
+== Notes on tests
+
+There are cucumber stories that pass, but I don't seem to be using them quite how they are supposed to be. It is testable completely outside of the rails environment, but seems to akwardly incorporate the parts needed to make suitable tests. If you're able to give me some pointers on how better to setup the tests, please do.

data/Rakefile

@@ -4,7 +4,7 @@ require 'rake'
 require 'echoe'
 require 'env'
 
-Echoe.new('change_logger', '0.0.3') do |p|
+Echoe.new('change_logger', '0.0.4') do |p|
   p.description = "A gem for tracking what changes and who did it"
   p.url = "http://github.com/danengle/awesome_tables"
   p.author = "Dan Engle"

data/change_logger.gemspec

@@ -2,26 +2,23 @@
 
 Gem::Specification.new do |s|
   s.name = %q{change_logger}
-  s.version = "0.0.3"
+  s.version = "0.0.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 1.2") if s.respond_to? :required_rubygems_version=
   s.authors = ["Dan Engle"]
-  s.cert_chain = ["/Users/danengle/.ssh/gem-public_cert.pem"]
-  s.date = %q{2010-12-05}
+  s.date = %q{2011-03-09}
   s.description = %q{A gem for tracking what changes and who did it}
   s.email = %q{engle.68 @nospam@ gmail.com}
-  s.extra_rdoc_files = ["CHANGELOG", "README", "lib/app/models/change_log.rb", "lib/change_logger.rb", "lib/change_logger/acts_as_change_logger.rb", "lib/change_logger/railtie.rb", "lib/change_logger/whodunnit.rb", "lib/generators/change_logger_generator.rb", "lib/generators/templates/create_change_logs.rb"]
-  s.files = ["CHANGELOG", "MIT-LICENSE", "Manifest", "README", "Rakefile", "cucumber.yml.bac", "features/log_changes.feature", "features/step_definitions/log_changes_steps.rb", "features/support/env.rb", "features/support/hooks.rb", "lib/app/models/change_log.rb", "lib/change_logger.rb", "lib/change_logger/acts_as_change_logger.rb", "lib/change_logger/railtie.rb", "lib/change_logger/whodunnit.rb", "lib/generators/change_logger_generator.rb", "lib/generators/templates/create_change_logs.rb", "spec/factories.rb", "spec/migrate/20101201_init_test_db.rb", "spec/models/permission.rb", "spec/models/user.rb", "change_logger.gemspec"]
+  s.extra_rdoc_files = ["CHANGELOG", "README.rdoc", "lib/app/models/change_log.rb", "lib/change_logger.rb", "lib/change_logger/acts_as_change_logger.rb", "lib/change_logger/railtie.rb", "lib/change_logger/whodunnit.rb", "lib/generators/change_logger_generator.rb", "lib/generators/templates/create_change_logs.rb"]
+  s.files = ["CHANGELOG", "MIT-LICENSE", "Manifest", "README.rdoc", "Rakefile", "cucumber.yml.bac", "features/log_changes.feature", "features/step_definitions/log_changes_steps.rb", "features/support/env.rb", "features/support/hooks.rb", "lib/app/models/change_log.rb", "lib/change_logger.rb", "lib/change_logger/acts_as_change_logger.rb", "lib/change_logger/railtie.rb", "lib/change_logger/whodunnit.rb", "lib/generators/change_logger_generator.rb", "lib/generators/templates/create_change_logs.rb", "spec/factories.rb", "spec/migrate/20101201_init_test_db.rb", "spec/models/permission.rb", "spec/models/user.rb", "change_logger.gemspec"]
   s.homepage = %q{http://github.com/danengle/awesome_tables}
-  s.rdoc_options = ["--line-numbers", "--inline-source", "--title", "Change_logger", "--main", "README"]
+  s.rdoc_options = ["--line-numbers", "--inline-source", "--title", "Change_logger", "--main", "README.rdoc"]
   s.require_paths = ["lib"]
   s.rubyforge_project = %q{change_logger}
-  s.rubygems_version = %q{1.3.7}
-  s.signing_key = %q{/Users/danengle/.ssh/gem-private_key.pem}
+  s.rubygems_version = %q{1.5.0}
   s.summary = %q{A gem for tracking what changes and who did it}
 
   if s.respond_to? :specification_version then
-    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
     s.specification_version = 3
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then

data/lib/change_logger/acts_as_change_logger.rb

@@ -2,6 +2,7 @@ module ChangeLogger
   module ActsAsChangeLogger
     ACTIONS = {
       :create => 'CREATED',
+      :update => 'UPDATED',
       :delete => 'DELETED'
     }
     def self.included(base)
@@ -11,12 +12,18 @@ module ChangeLogger
     module ClassMethods
       def acts_as_change_logger(options = {})
         send :include, InstanceMethods
-        cattr_accessor :ignore
+        
+        cattr_accessor :ignore, :track_templates
         self.ignore = (options[:ignore] || []).map &:to_s
-        self.ignore.push('id', 'created_at', 'updated_at')
+        self.ignore.push('id', 'revision', 'created_at', 'updated_at')
+        self.track_templates = (options[:track_templates] || []).map &:to_s
+        
+        attr_accessor :template_changed
+        after_update :record_template_change
         
         has_many :change_logs, :as => :item, :order => 'change_logs.created_at desc'
         after_create :record_object_creation
+        before_update :increment_revision
         after_update :record_attribute_updates
         before_destroy :record_object_destruction
         self.reflect_on_all_associations(:has_and_belongs_to_many).each do |reflection|
@@ -31,12 +38,35 @@ module ChangeLogger
     
     module InstanceMethods
       
+      def increment_revision
+        self.increment(:revision) if self.respond_to?(:revision)
+      end
+      
+      def record_template_change
+        self.template_changed = {} if self.template_changed.nil?
+        if self.template_changed.values.include?(true)
+          self.template_changed.keys.each do |relation|
+            record_change("#{relation}_template", ACTIONS[:update], self.send(relation).to_yaml)
+          end
+        end
+      end
+      
       def record_association_add(object)
-        record_change(object.class.to_s, ACTIONS[:create], object.id) if self.persisted?
+        if self.class.track_templates.include?(object.class.to_s.tableize)
+          self.template_changed = {} if self.template_changed.nil?
+          self.template_changed[object.class.to_s.tableize.to_sym] = true          
+        else
+          record_change(object.class.to_s, ACTIONS[:create], object.id) if self.persisted?
+        end
       end
       
       def record_association_remove(object)
-        record_change(object.class.to_s, object.id, ACTIONS[:delete]) if self.persisted?
+        if self.class.track_templates.include?(object.class.to_s.tableize)
+          self.template_changed = {} if self.template_changed.nil?
+          self.template_changed[object.class.to_s.tableize.to_sym] = true          
+        else
+          record_change(object.class.to_s, object.id, ACTIONS[:delete]) if self.persisted?
+        end
       end
       
       def record_object_creation
@@ -64,13 +94,16 @@ module ChangeLogger
       private
       
       def record_change(attribute_name, old_val, new_val)        
-        self.change_logs.create(
+        change_log = self.change_logs.new(
           :attribute_name => attribute_name,
           :old_value => old_val,
           :new_value => new_val,
           :changed_by => whodunnit
         )
+        change_log.revision = self.revision if self.respond_to?(:revision)
+        change_log.save
       end
+            
     end
   end
 end

data/lib/change_logger/whodunnit.rb

@@ -5,16 +5,10 @@ module ChangeLogger
       base.before_filter :set_whodunnit
     end
     
-    protected
-    
-    def whodunnit
-      current_user rescue nil
-    end
-    
     private
     
     def set_whodunnit
-      ::ChangeLogger.whodunnit = whodunnit
+      ::ChangeLogger.whodunnit = current_user.login rescue nil
     end
   end
 end

data/lib/generators/templates/create_change_logs.rb

@@ -4,7 +4,7 @@ class CreateChangeLogs < ActiveRecord::Migration
       t.integer :item_id, :null => false
       t.string :item_type, :null => false
       t.string :attribute_name, :old_value, :new_value
-      t.string :changed_by, :null => false
+      t.string :changed_by
       t.datetime :created_at
     end
     add_index :change_logs, [:item_type, :item_id]

metadata

@@ -1,41 +1,16 @@
 --- !ruby/object:Gem::Specification 
 name: change_logger
 version: !ruby/object:Gem::Version 
-  prerelease: false
-  segments: 
-  - 0
-  - 0
-  - 3
-  version: 0.0.3
+  prerelease: 
+  version: 0.0.4
 platform: ruby
 authors: 
 - Dan Engle
 autorequire: 
 bindir: bin
-cert_chain: 
-- |
-  -----BEGIN CERTIFICATE-----
-  MIIDMjCCAhqgAwIBAgIBADANBgkqhkiG9w0BAQUFADA/MREwDwYDVQQDDAhlbmds
-  ZS42ODEVMBMGCgmSJomT8ixkARkWBWdtYWlsMRMwEQYKCZImiZPyLGQBGRYDY29t
-  MB4XDTEwMTAxMTAwNTI1M1oXDTExMTAxMTAwNTI1M1owPzERMA8GA1UEAwwIZW5n
-  bGUuNjgxFTATBgoJkiaJk/IsZAEZFgVnbWFpbDETMBEGCgmSJomT8ixkARkWA2Nv
-  bTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBAKf6nWhdrlzcbME1sHRD
-  wbobJ/PfkZZeMmeOI00ij7gtpJ8/GTAHhDSazzNeQUaUJ01+g82QtRyDtXrsosj6
-  vV1HIGAIGESLcOi8vDqnUGZrmY/o3pO7cgYXbHjT9Jn/Uoqt9IXhE97Bf1mYyTvp
-  DznUjccAp+z/8N6xwOEgXqSCfMcfH1zyuurRo13PkJvEKkLAW+PsdA28xovauUXo
-  j2H8+myZLGi5KFeN13UnOo8x0t/McNYkjGaQcVBrIqqX4fle4LNakUfeyJoweXew
-  pAlIjByYtMDQT21fxQLthLayx35nuMXgMWC6+Evzx4U2M7D5GedSxbjZT3uzsHNR
-  pvkCAwEAAaM5MDcwCQYDVR0TBAIwADAdBgNVHQ4EFgQUt/Ysk6uU3ZrPbcD6kshs
-  ghx0mCYwCwYDVR0PBAQDAgSwMA0GCSqGSIb3DQEBBQUAA4IBAQB9gUK+9jS5asdp
-  cpDlY89APPt9dogKZbt7a8KRJu3qlVfcdW+/naxNOQdRtXspdUlsNdYUGJjJnV0K
-  hZ5TzDCKkH09nhpShLnTbwyB2srwRrMAZWpqvD2f6Muh8H5JB1lxe5RE1v1yZH7d
-  gokx0zsfVuIz2je+OeAz6qmfmiFQnG79vvSaQl6XsDdFsNf51Hdizt7fR1vC7MKr
-  5ZNu0jDZxdAIsmHVdvJK7tMZWq8097ZatwhhE9ReA/ZoMBy4WK7k1G32K2AykCk7
-  in+c1BznEvMTAcQnwGUyalIFnODl9PB5tDyYYUJdPCPxe/YxL9csB3jNDxe6HrZK
-  2MS0NMYj
-  -----END CERTIFICATE-----
+cert_chain: []
 
-date: 2010-12-05 00:00:00 -06:00
+date: 2011-03-09 00:00:00 -06:00
 default_executable: 
 dependencies: []
 
@@ -47,7 +22,7 @@ extensions: []
 
 extra_rdoc_files: 
 - CHANGELOG
-- README
+- README.rdoc
 - lib/app/models/change_log.rb
 - lib/change_logger.rb
 - lib/change_logger/acts_as_change_logger.rb
@@ -59,7 +34,7 @@ files:
 - CHANGELOG
 - MIT-LICENSE
 - Manifest
-- README
+- README.rdoc
 - Rakefile
 - cucumber.yml.bac
 - features/log_changes.feature
@@ -89,7 +64,7 @@ rdoc_options:
 - --title
 - Change_logger
 - --main
-- README
+- README.rdoc
 require_paths: 
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement 
@@ -97,22 +72,17 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 0
       version: "0"
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 1
-      - 2
       version: "1.2"
 requirements: []
 
 rubyforge_project: change_logger
-rubygems_version: 1.3.7
+rubygems_version: 1.5.0
 signing_key: 
 specification_version: 3
 summary: A gem for tracking what changes and who did it

data.tar.gz.sig

@@ -1 +0,0 @@
-�Ы���{V�?��@��,�õݭ��8���kD[!�����h7cy`�&�;���/E��%�Fc�r9�̕)���p�Ls��P{�H_=���,re�:A����3y|�I��z�)|@��c&�?0����%$B���Do��04�8\D�֪	�k5�ly˄aY1�Hx(.[y\�#ɔ�j�jT^cp-�TF��\R�W��:��Ys#�r�
�y�����=yG�ѵb|xt+).T��E@.��my��uR