opinio 0.4 → 0.4.1

data/app/views/opinio/comments/create.js.erb

@@ -1,11 +1,15 @@
 $('#no_comments').fadeOut()
-<% if @reply %>
-  if($('#comment_<%= @comment.commentable_id %> ul').length == 0)
-    $('#comment_<%= @comment.commentable_id %>').append('<ul id="comment_<%= @comment.commentable_id %>_replies" class="replies"></ul>')
-  $('#comment_<%= @comment.commentable_id %>_replies').append("<%= escape_javascript( render @comment, :locals => {:reply => @reply} ) %>")
-  $('#commentable_id').val('<%= @comment.commentable.commentable_id %>')
-  $('#commentable_type').val('<%= @comment.commentable.commentable_type %>')
+<% if @comment.valid? %>
+  <% if @reply %>
+    if($('#comment_<%= @comment.commentable_id %> ul').length == 0)
+      $('#comment_<%= @comment.commentable_id %>').append('<ul id="comment_<%= @comment.commentable_id %>_replies" class="replies"></ul>')
+    $('#comment_<%= @comment.commentable_id %>_replies').append("<%= escape_javascript( render @comment, :locals => {:reply => @reply} ) %>")
+    $('#commentable_id').val('<%= @comment.commentable.commentable_id %>')
+    $('#commentable_type').val('<%= @comment.commentable.commentable_type %>')
+  <% else %>
+    $('#comments').prepend("<%= escape_javascript( render @comment, :locals => {:reply => @reply} ) %>")
+  <% end %>
 <% else %>
-  $('#comments').prepend("<%= escape_javascript( render @comment, :locals => {:reply => @reply} ) %>")
+  $('#comments').prepend("<%= escape_javascript(flash[:notice]) %>");
+  $('#comments').prepend("<%= escape_javascript(flash[:error]) %>");
 <% end %>
-

data/lib/opinio/opinio_model.rb

@@ -61,6 +61,8 @@ module Opinio
 
         validates_presence_of :commentable
 
+        validates :owner, :presence => true, :associated => true
+
         scope :owned_by, lambda {|owner| where('owner_id = ?', owner.id) }
 
         send :include, Opinio::OpinioModel::InstanceMethods

data/lib/opinio/shared_examples.rb

@@ -39,6 +39,12 @@ shared_examples_for :opinio do
     comment.errors[:commentable].should be_present
   end
 
+  it "and it should insist on having an owner" do
+    c = Comment.new(:body => "The Comment !")
+    c.commentable = @post
+    c.save.should == false
+  end
+
 end
 
 

data/lib/opinio/version.rb

@@ -1,5 +1,5 @@
 module Opinio
   class Version
-    VERSION = '0.4'
+    VERSION = '0.4.1'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opinio
 version: !ruby/object:Gem::Version
-  version: '0.4'
+  version: 0.4.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-07 00:00:00.000000000Z
+date: 2012-02-23 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &2156948880 !ruby/object:Gem::Requirement
+  requirement: &2153246840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: '3'
   type: :runtime
   prerelease: false
-  version_requirements: *2156948880
+  version_requirements: *2153246840
 - !ruby/object:Gem::Dependency
   name: kaminari
-  requirement: &2156948260 !ruby/object:Gem::Requirement
+  requirement: &2153246340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2156948260
+  version_requirements: *2153246340
 - !ruby/object:Gem::Dependency
   name: jquery-rails
-  requirement: &2156947720 !ruby/object:Gem::Requirement
+  requirement: &2153245840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,7 +43,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2156947720
+  version_requirements: *2153245840
 description: Opinio is an engine used to add comments functionallity to rails 3 applications.
 email:
 - luiz.felipe.gp@gmail.com