active_metadata 0.5.2 → 0.6.0

data/lib/active_metadata/persistence/{active_record/attachment.rb → attachment.rb}

@@ -1,4 +1,4 @@
-module ActiveMetadata::Persistence::ActiveRecord::Attachment
+module ActiveMetadata::Persistence::Attachment
 
   def self.included(receiver)
     receiver.send :include, InstanceMethods
@@ -7,20 +7,20 @@ module ActiveMetadata::Persistence::ActiveRecord::Attachment
   module InstanceMethods
 
     def save_attachment_for(field, file, starred=false)
-      attachment = Attachment.create! :document_class => metadata_class, :document_id => metadata_id, :label => field, :attach => file,
+      attachment = ActiveMetadata::Attachment.create! :document_class => metadata_class, :document_id => metadata_id, :label => field, :attach => file,
                                       :starred => starred, :created_by => current_user_id
       reload_attachments_cache_for field 
       self.send(:send_notification, field, "", attachment.attach.original_filename, :attachment_message, current_user_id)
     end
 
-    def attachments_for(field, order_by="attach_updated_at DESC")
+    def attachments_for(field, order_by="updated_at DESC")
       Rails.cache.fetch(attachments_cache_key(field), :expires_in => ActiveMetadata::CONFIG['cache_expires_in'].minutes) do
         fetch_attachments_for field, nil, order_by
       end  
     end
 
     def delete_attachment(id)
-      a = Attachment.find(id)
+      a = ActiveMetadata::Attachment.find(id)
       filename = a.attach.original_filename
       a.destroy      
       reload_attachments_cache_for a.label
@@ -28,7 +28,7 @@ module ActiveMetadata::Persistence::ActiveRecord::Attachment
     end
 
     def update_attachment(id, newfile, starred=nil)
-      a = Attachment.find(id)
+      a = ActiveMetadata::Attachment.find(id)
       old_filename = a.attach.original_filename
       a.attach = newfile
       a.updated_by = current_user_id
@@ -45,7 +45,7 @@ module ActiveMetadata::Persistence::ActiveRecord::Attachment
     end
 
     def find_attachment_by_id(id)
-      Attachment.find(id)
+      ActiveMetadata::Attachment.find(id)
     end
 
     # not cached
@@ -54,12 +54,12 @@ module ActiveMetadata::Persistence::ActiveRecord::Attachment
     end
 
     def star_attachment(id)
-      n = Attachment.find(id)
+      n = ActiveMetadata::Attachment.find(id)
       update_attachment id,n.attach,true
     end
 
     def unstar_attachment(id)
-      n = Attachment.find(id)
+      n = ActiveMetadata::Attachment.find(id)
       update_attachment id,n.attach,false
     end
 
@@ -69,10 +69,10 @@ module ActiveMetadata::Persistence::ActiveRecord::Attachment
       Rails.cache.write(attachments_cache_key(field),fetch_attachments_for(field), :expires_in => ActiveMetadata::CONFIG['cache_expires_in'].minutes )     
     end  
     
-    def fetch_attachments_for(field, starred=nil, order_by="attach_updated_at DESC")
+    def fetch_attachments_for(field, starred=nil, order_by="updated_at DESC")
       conditions = {:document_class => metadata_class, :document_id => metadata_id,:label => field}
       conditions[:starred] = starred if !starred.nil?
-      Attachment.all(:conditions => conditions , :order => order_by)
+      ActiveMetadata::Attachment.all(:conditions => conditions , :order => order_by)
     end  
 
   end

data/lib/active_metadata/persistence/{active_record/history.rb → history.rb}

@@ -1,4 +1,4 @@
-module ActiveMetadata::Persistence::ActiveRecord::History
+module ActiveMetadata::Persistence::History
 
   def self.included(receiver)
     receiver.send :include, InstanceMethods
@@ -8,8 +8,8 @@ module ActiveMetadata::Persistence::ActiveRecord::History
 
     def save_history
       self.changes.each do |key, value|
-        next if ActiveMetadata::CONFIG['history_skip_fields'].include?(key)        
-        History.create! :value => value[1],:document_class => metadata_class, :document_id => metadata_id,:label => key.to_s, :created_by => current_user_id
+        next if ActiveMetadata::CONFIG['history_skip_fields'].include?(key)
+        ActiveMetadata::History.create! :value => value[1],:document_class => metadata_class, :document_id => metadata_id,:label => key.to_s, :created_by => current_user_id
         invalidate_history_cache_for key.to_s
         self.send(:send_notification, key, value[0], value[1], :history_message,current_user_id) 
       end
@@ -28,7 +28,7 @@ module ActiveMetadata::Persistence::ActiveRecord::History
     end  
 
     def fetch_histories_for field, order
-      History.all(:conditions => {:document_class => metadata_class, :document_id => metadata_id,:label => field}, :order => order)
+      ActiveMetadata::History.all(:conditions => {:document_class => metadata_class, :document_id => metadata_id,:label => field}, :order => order)
     end  
 
 

data/lib/active_metadata/persistence/{active_record/note.rb → note.rb}

@@ -1,4 +1,4 @@
-module ActiveMetadata::Persistence::ActiveRecord::Note
+module ActiveMetadata::Persistence::Note
 
   def self.included(receiver)
     receiver.send :include, InstanceMethods
@@ -7,13 +7,13 @@ module ActiveMetadata::Persistence::ActiveRecord::Note
   module InstanceMethods
     
     def create_note_for(field, note, starred=false)
-      Note.create! :document_id => metadata_id, :document_class => metadata_class, :label => field.to_s,:note => note, :created_by => current_user_id, :starred => starred
+      ActiveMetadata::Note.create! :document_id => metadata_id, :document_class => metadata_class, :label => field.to_s,:note => note, :created_by => current_user_id, :starred => starred
       reload_notes_cache_for field
       self.send(:send_notification, field, "", note, :note_message, current_user_id) 
     end
 
     def update_note(id, note, starred=nil)
-      n = Note.find(id)
+      n = ActiveMetadata::Note.find(id)
       old_value = n.note
       attributes =  {:note => note, :updated_by => current_user_id, :updated_at => Time.now.utc}
       #mass assign starred inly if provided
@@ -32,7 +32,7 @@ module ActiveMetadata::Persistence::ActiveRecord::Note
     end
 
     def find_note_by_id(id)
-      Note.find(id)
+      ActiveMetadata::Note.find(id)
     end      
     
     def create_notes_for(field,notes)
@@ -40,7 +40,7 @@ module ActiveMetadata::Persistence::ActiveRecord::Note
     end
 
     def delete_note(id)
-      n = Note.find(id)
+      n = ActiveMetadata::Note.find(id)
       old_value = n.note
       n.destroy
       reload_notes_cache_for n.label
@@ -57,12 +57,12 @@ module ActiveMetadata::Persistence::ActiveRecord::Note
     end
 
     def star_note(id)
-      n = Note.find(id)
+      n = ActiveMetadata::Note.find(id)
       update_note id,n.note,true
     end
 
     def unstar_note(id)
-      n = Note.find(id)
+      n = ActiveMetadata::Note.find(id)
       update_note id,n.note,false
     end
 
@@ -77,7 +77,7 @@ module ActiveMetadata::Persistence::ActiveRecord::Note
       unless starred.nil?
         conditions[:starred] = starred
       end
-      Note.all(:conditions => conditions, :order => order_by)
+      ActiveMetadata::Note.all(:conditions => conditions, :order => order_by)
     end  
         
   end

data/lib/active_metadata/persistence/persistence.rb

@@ -1,9 +1,19 @@
 module ActiveMetadata::Persistence
-  
-  require "active_metadata/persistence/active_record" 
-  
+
+  require "active_metadata/persistence/note"
+  require "active_metadata/persistence/history"
+  require "active_metadata/persistence/watcher"
+  require "active_metadata/persistence/attachment"
+
   def self.included(receiver)
-    receiver.send :include, ActiveMetadata::Persistence::ActiveRecord
+    receiver.send :include, InstanceMethods
+  end
+
+  module InstanceMethods
+    include ActiveMetadata::Persistence::Note
+    include ActiveMetadata::Persistence::History
+    include ActiveMetadata::Persistence::Watcher
+    include ActiveMetadata::Persistence::Attachment
   end
 
 end

data/lib/active_metadata/persistence/{active_record/watcher.rb → watcher.rb}

@@ -1,4 +1,4 @@
-module ActiveMetadata::Persistence::ActiveRecord::Watcher
+module ActiveMetadata::Persistence::Watcher
 
   def self.included(receiver)
     receiver.send :include, InstanceMethods
@@ -7,21 +7,21 @@ module ActiveMetadata::Persistence::ActiveRecord::Watcher
   module InstanceMethods
     def create_watcher_for(field, owner)
       raise RuntimeError, "The object id MUST be valued" unless self.id
-      Watcher.create! :document_class => metadata_class, :document_id => metadata_id, :label => field, :owner_id => owner.id
+      ActiveMetadata::Watcher.create! :document_class => metadata_class, :document_id => metadata_id, :label => field, :owner_id => owner.id
     end                      
 
     def watchers_for(field)
-      Watcher.all(:conditions => {:document_class => metadata_class, :label => field, :document_id => metadata_id})
+      ActiveMetadata::Watcher.all(:conditions => {:document_class => metadata_class, :label => field, :document_id => metadata_id})
     end
 
     def delete_watcher_for(field, owner)
-      Watcher.where(:document_id => metadata_id, :label => field, :owner_id => owner.id).each do |watcher|
+      ActiveMetadata::Watcher.where(:document_id => metadata_id, :label => field, :owner_id => owner.id).each do |watcher|
         watcher.destroy
       end   
     end    
     
     def is_watched_by(field,owner)
-      Watcher.where(:document_class => metadata_class, :document_id => metadata_id, :label => field, :owner_id => owner.id).empty? ? false : true
+      ActiveMetadata::Watcher.where(:document_class => metadata_class, :document_id => metadata_id, :label => field, :owner_id => owner.id).empty? ? false : true
     end                    
 
     def send_notification(field, old_value, new_value, type=:default_message, created_by=nil)   

data/lib/active_metadata/streamable.rb

@@ -0,0 +1,25 @@
+module ActiveMetadata::Streamable
+
+  def stream_for field
+    sort_stream(collect_stream_data(field))
+  end
+
+  private
+  def sort_stream stream
+    stream.sort{|a,b| b.updated_at <=> a.updated_at}
+  end
+
+  def collect_stream_data field
+    res = []
+    ActiveMetadata::CONFIG['streamables'].each do |model|
+      res.concat self.send(stream_collect_method(model.to_s),field).collect { |el| el }
+    end
+    res
+  end
+
+  def stream_collect_method model
+    model.to_s == 'note' ? 'notes_for' : 'attachments_for'
+  end
+
+
+end

data/lib/active_metadata/version.rb

@@ -1,3 +1,3 @@
 module ActiveMetadata
-  VERSION = "0.5.2"
+  VERSION = "0.6.0"
 end

data/lib/active_metadata.rb

@@ -1,7 +1,7 @@
+require "active_metadata/engine"
+
 module ActiveMetadata
-  require 'engine' if defined?(Rails) && Rails::VERSION::MAJOR == 3
   require 'active_metadata/base'
-  require 'application_controller'
   require "active_metadata/version"
   require "active_metadata/form_helper"
-end
+end

data/lib/{rake/ci.rb → tasks/active_metadata_tasks.rake}

@@ -1,15 +1,18 @@
+require 'rspec/core/rake_task'
+require 'cucumber/rake/task'
+
 begin
   require 'ci/reporter/rake/rspec'
   require 'ci/reporter/rake/cucumber'
 
-  namespace :ci do
+  namespace :ci do :environment
 
-    ENV["DATABASE_ENV"] = 'test'
+    ENV["RAILS_ENV"] ||= 'test'
 
     namespace :setup do
-      
+
       @reports_dir = ENV['CI_REPORTS'] || 'features/reports'
-      
+
       task :cucumber_report_cleanup do
         rm_rf @reports_dir
       end
@@ -20,10 +23,10 @@ begin
       end
 
     end
-    
+
     task "spec" => ["rspec_run"]
     task "cucumber" => ["cucumber_run"]
-    
+
   end
 
   task "ci" => ["db:migrate", "ci:spec", "ci:cucumber"]
@@ -36,3 +39,21 @@ RSpec::Core::RakeTask.new(:rspec_run => ["ci:setup:rspec"]) do |t|
 end
 
 Cucumber::Rake::Task.new(:cucumber_run => ["ci:setup:cucumber"])
+
+
+namespace :active_metadata do
+
+  desc "Install the active_metadata gem requirements file ***TASK IS ON ALPHA STAGE***"
+  task :install do
+
+    FileUtils.cp File.expand_path('../../../config/active_metadata.yml',__FILE__), File.expand_path('config/')
+    puts "Installed active_metadata.yml"
+
+    puts "Copying migrations"
+    ts = Time.now.utc.strftime('%Y%m%d%H%M%S')
+    FileUtils.cp File.expand_path('../../../db/migrate/02_active_metadata_migrations.rb',__FILE__), File.expand_path("db/migrate/#{ts}_active_metadata_migrations.rb")
+    puts "run rake db:migrate to complete the gem installation"
+
+  end
+
+end

data/lib/tasks/cucumber.rake

@@ -0,0 +1,65 @@
+# IMPORTANT: This file is generated by cucumber-rails - edit at your own peril.
+# It is recommended to regenerate this file in the future when you upgrade to a 
+# newer version of cucumber-rails. Consider adding your own code to a new file 
+# instead of editing this one. Cucumber will automatically load all features/**/*.rb
+# files.
+
+
+unless ARGV.any? {|a| a =~ /^gems/} # Don't load anything when running the gems:* tasks
+
+vendored_cucumber_bin = Dir["#{Rails.root}/vendor/{gems,plugins}/cucumber*/bin/cucumber"].first
+$LOAD_PATH.unshift(File.dirname(vendored_cucumber_bin) + '/../lib') unless vendored_cucumber_bin.nil?
+
+begin
+  require 'cucumber/rake/task'
+
+  namespace :cucumber do
+    Cucumber::Rake::Task.new({:ok => 'db:test:prepare'}, 'Run features that should pass') do |t|
+      t.binary = vendored_cucumber_bin # If nil, the gem's binary is used.
+      t.fork = true # You may get faster startup if you set this to false
+      t.profile = 'default'
+    end
+
+    Cucumber::Rake::Task.new({:wip => 'db:test:prepare'}, 'Run features that are being worked on') do |t|
+      t.binary = vendored_cucumber_bin
+      t.fork = true # You may get faster startup if you set this to false
+      t.profile = 'wip'
+    end
+
+    Cucumber::Rake::Task.new({:rerun => 'db:test:prepare'}, 'Record failing features and run only them if any exist') do |t|
+      t.binary = vendored_cucumber_bin
+      t.fork = true # You may get faster startup if you set this to false
+      t.profile = 'rerun'
+    end
+
+    desc 'Run all features'
+    task :all => [:ok, :wip]
+
+    task :statsetup do
+      require 'rails/code_statistics'
+      ::STATS_DIRECTORIES << %w(Cucumber\ features features) if File.exist?('features')
+      ::CodeStatistics::TEST_TYPES << "Cucumber features" if File.exist?('features')
+    end
+  end
+  desc 'Alias for cucumber:ok'
+  task :cucumber => 'cucumber:ok'
+
+  task :default => :cucumber
+
+  task :features => :cucumber do
+    STDERR.puts "*** The 'features' task is deprecated. See rake -T cucumber ***"
+  end
+
+  # In case we don't have ActiveRecord, append a no-op task that we can depend upon.
+  task 'db:test:prepare' do
+  end
+
+  task :stats => 'cucumber:statsetup'
+rescue LoadError
+  desc 'cucumber rake task not available (cucumber not installed)'
+  task :cucumber do
+    abort 'Cucumber rake task is not available. Be sure to install cucumber as a gem or plugin'
+  end
+end
+
+end

data/spec/active_metadata_yml.rb

@@ -0,0 +1,25 @@
+require "spec_helper"
+
+describe "active_metadata.yml" do
+
+  context "given the active_metadata.yml parsed" do
+
+    before do
+      @envs = ['test','development']
+      @config = YAML.load(File.read("config/active_metadata.yml"))
+    end
+
+    it "should have a streamables config" do
+      test_envs{|env| @config[env]['streamables'].should_not be_nil}
+    end
+
+
+  end
+
+  def test_envs &block
+    @envs.each do |env|
+      yield env
+    end
+  end
+
+end

data/spec/controllers/active_metadata/attachments_controller_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe ActiveMetadata::AttachmentsController do
+
+  context "given 2 attachments for @document#name" do
+
+    render_views
+
+    before(:each) do
+      @document = Document.create! { |d| d.name = "John" }
+      (1..2).each do |i|
+        @document.save_attachment_for(:name,test_pdf("pdf_test_#{i}"))
+     end
+    end
+
+    describe "GET 'index'" do
+
+      it "should success" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        response.should be_success
+      end
+
+      it "should assign attachments" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        assigns(:attachments).should_not be_nil
+        assigns(:attachments).size.should eq 2
+      end
+
+      it "should display 3 notes" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        response.body.should match(/pdf_test_1.pdf/)
+        response.body.should match(/pdf_test_2.pdf/)
+      end
+
+    end
+
+  end
+
+
+end

data/spec/controllers/active_metadata/notes_controller_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe ActiveMetadata::NotesController do
+
+  context "given 3 notes @document#name" do
+
+    render_views
+
+    before(:each) do
+      @document = Document.create! { |d| d.name = "John" }
+      (1..3).each do |i|
+        @document.create_note_for(:name, "note#{i}")
+      end
+    end
+
+    describe "GET 'index'" do
+
+      it "should success" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        response.should be_success
+      end
+
+      it "should assign notes" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        assigns(:notes).should_not be_nil
+        assigns(:notes).size.should eq 3
+      end
+
+      it "should display 3 notes" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        response.body.should match(/note1/)
+        response.body.should match(/note2/)
+        response.body.should match(/note3/)
+      end
+
+    end
+
+  end
+
+
+end

data/spec/controllers/active_metadata/stream_controller_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe ActiveMetadata::StreamController do
+
+  context "given 1 note and 1 attachment for @document#name" do
+
+    render_views
+
+    before(:each) do
+      @document = Document.create! { |d| d.name = "John" }
+      @document.save_attachment_for(:name,test_pdf("pdf_test"))
+      @document.create_note_for(:name, "nota per name john")
+    end
+
+    describe "GET 'index'" do
+
+      it "should success" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        response.should be_success
+      end
+
+      it "should assign document" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        assigns(:document).should_not be_nil
+        assigns(:document).id.should eq @document.id
+      end
+
+      it "should assign stream" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        assigns(:stream).should_not be_nil
+        assigns(:stream).size.should eq 2
+      end
+
+      it "should render both note and the attachment" do
+        get 'index', :model_name => 'document', :model_id => @document.id, :field_name => 'name'
+        response.body.should match(/pdf_test.pdf/)
+        response.body.should match(/nota per name john/)
+      end
+
+    end
+
+  end
+
+
+
+end