active_metadata 0.6.9 → 0.7.0

data/lib/active_metadata/persistence/attachment.rb

@@ -52,7 +52,12 @@ module ActiveMetadata::Persistence::Attachment
       ActiveMetadata::Attachment.find(id)
     end
 
-    # not cached
+    # return all the starred notes for the current model and for any field
+    # datas does not come from cache
+    def starred_attachments
+      fetch_attachments_for nil, true
+    end
+
     def starred_attachments_for(field)
       fetch_attachments_for field, true
     end
@@ -75,8 +80,9 @@ module ActiveMetadata::Persistence::Attachment
     end
 
     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?
+      conditions = {:document_class => metadata_class, :document_id => metadata_id}
+      conditions[:label] = field unless field.nil?
+      conditions[:starred] = starred unless starred.nil?
       ActiveMetadata::Attachment.all(:conditions => conditions, :order => order_by)
     end
 

data/lib/active_metadata/persistence/note.rb

@@ -56,16 +56,27 @@ module ActiveMetadata::Persistence::Note
       notes_for(field).size
     end
 
-    # not cached
+    # return all the starred notes for the current model and for any field
+    # datas does not come from cache
+    def starred_notes
+      fetch_notes_for nil, true
+    end
+
+    # return all the starred notes for a particular field
+    # datas does not come from cache
     def starred_notes_for(field)
       fetch_notes_for field, true
     end
 
+    # star a note
+    # reload the cache calling update
     def star_note(id)
       n = ActiveMetadata::Note.find(id)
       update_note id, n.note, true
     end
 
+    # unstar a note
+    # reload the cache calling update
     def unstar_note(id)
       n = ActiveMetadata::Note.find(id)
       update_note id, n.note, false
@@ -78,10 +89,9 @@ module ActiveMetadata::Persistence::Note
     end
 
     def fetch_notes_for(field, starred=nil, order_by="updated_at DESC")
-      conditions = {:label => field, :document_class => metadata_class, :document_id => metadata_id}
-      unless starred.nil?
-        conditions[:starred] = starred
-      end
+      conditions = {:document_class => metadata_class, :document_id => metadata_id}
+      conditions[:label] = field  unless field.nil?
+      conditions[:starred] = starred  unless starred.nil?
       ActiveMetadata::Note.all(:conditions => conditions, :order => order_by)
     end
 

data/lib/active_metadata/streamable.rb

@@ -1,9 +1,16 @@
 module ActiveMetadata::Streamable
 
-  def stream_for(field, order_by = :created_at)
+  # return the streamables items by field in an ordered array
+  # ActiveMetadata::CONFIG['streamables'] defines what models will be retrieved
+  def stream_for(field, order_by = :updated_at)
     sort_stream(collect_stream_data(field), order_by)
   end
 
+  # same as #stream_for but not filtered by field
+  def stream_all_starred order_by = :updated_at
+    sort_stream(collect_starred_stream_data, order_by)
+  end
+
   private
   def sort_stream stream, order_by
     stream.sort{ |a,b| b.send(order_by) <=> a.send(order_by) }
@@ -17,6 +24,14 @@ module ActiveMetadata::Streamable
     res
   end
 
+  def collect_starred_stream_data
+    res = []
+    ActiveMetadata::CONFIG['streamables'].each do |model|
+      res.concat self.send("starred_#{model.to_s.pluralize}".to_sym).collect { |el| el }
+    end
+    res
+  end
+
   def stream_collect_method model
     model.to_s == 'note' ? 'notes_for' : 'attachments_for'
   end

data/lib/active_metadata/version.rb

@@ -1,3 +1,3 @@
 module ActiveMetadata
-  VERSION = "0.6.9"
+  VERSION = "0.7.0"
 end

data/spec/lib/attachments_spec.rb

@@ -176,6 +176,21 @@ describe ActiveMetadata do
 
     end
 
+    describe "#starred_attachments" do
+
+      it "should return all the starred attachments for any model field" do
+        @document.save_attachment_for(:name, @attachment, true)
+        @document.save_attachment_for(:title, @attachment, true)
+        @document.save_attachment_for(:title, @attachment2 )
+
+        starred = @document.starred_attachments
+        starred.count.should eq 2
+        starred.find{|att| att.attach.instance_read(:file_name) == "pdf_test_2.pdf"}.should be_nil
+      end
+
+    end
+
+
 
   end
 

data/spec/lib/notes_spec.rb

@@ -204,5 +204,19 @@ describe ActiveMetadata do
 
     end
 
+    describe "#starred_notes" do
+
+      it "should return all the starred notes for any model field" do
+        @document.create_note_for :name, "starred note for name", true
+        @document.create_note_for :title, "starred note for title", true
+        @document.create_note_for :title, "not starred note for title"
+
+        starred = @document.starred_notes
+        starred.count.should eq 2
+        starred.find{|note| note.note == "not starred note for title"}.should be_nil
+      end
+
+    end
+
   end
 end

data/spec/lib/streamable_spec.rb

@@ -131,5 +131,35 @@ describe ActiveMetadata::Streamable do
 
   end
 
+  describe "#stream_all_starred" do
+
+    context "given 2 notes (only one starred) and 2 attachments (only one starred)" do
+
+      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}"), i.odd? )
+          @document.create_note_for(:name, "note_#{i}", i.odd? )
+        end
+      end
+
+      it "should return 2 elements" do
+        @document.stream_all_starred.count.should eq 2
+      end
+
+      it "should return only the starred items" do
+        @document.stream_all_starred.collect{|el| el.starred? }.count.should eq 2
+      end
+
+      it "should return the starred stream ordered by UPDATED_AT DESC" do
+        starred = @document.stream_all_starred
+        starred.first.should be_kind_of ActiveMetadata::Note
+        @document.update_attachment starred.last.id, starred.last.attach.to_file
+        @document.stream_all_starred.first.should be_kind_of ActiveMetadata::Attachment
+      end
+
+    end
+
+  end
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_metadata
 version: !ruby/object:Gem::Version
-  version: 0.6.9
+  version: 0.7.0
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-23 00:00:00.000000000Z
+date: 2012-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &70324883600380 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -22,10 +22,15 @@ dependencies:
         version: 3.2.0
   type: :runtime
   prerelease: false
-  version_requirements: *70324883600380
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 3.2.0
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &70324883599960 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -33,10 +38,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70324883599960
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
-  requirement: &70324883599500 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -44,10 +54,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70324883599500
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &70324883599080 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -55,21 +70,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70324883599080
-- !ruby/object:Gem::Dependency
-  name: sqlite3-ruby
-  requirement: &70324883598660 !ruby/object:Gem::Requirement
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: *70324883598660
 - !ruby/object:Gem::Dependency
   name: cucumber-rails
-  requirement: &70324883598240 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -77,10 +86,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70324883598240
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: ci_reporter
-  requirement: &70324883597820 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -88,10 +102,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70324883597820
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: paperclip
-  requirement: &70324883596860 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -99,7 +118,12 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70324883596860
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: First implementation will write metadata on mongodb
 email:
 - acampolonghi@gmail.com
@@ -235,7 +259,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3812328665944082884
+      hash: -2284098148828860317
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -244,10 +268,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3812328665944082884
+      hash: -2284098148828860317
 requirements: []
 rubyforge_project: active_metadata
-rubygems_version: 1.8.10
+rubygems_version: 1.8.19
 signing_key: 
 specification_version: 3
 summary: Add metadata to fields in an active record model