has_messages 0.4.0 → 0.4.1

data/CHANGELOG.rdoc

@@ -1,5 +1,11 @@
 == master
 
+== 0.4.1 / 2010-03-07
+
+* Add a generator for db migrations
+* Release gems via rake-gemcutter instead of rubyforge
+* By default, sort messages newest first [Alexander Lang]
+
 == 0.4.0 / 2009-04-19
 
 * Add dependency on Rails 2.3

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2006-2009 Aaron Pfeifer
+Copyright (c) 2006-2010 Aaron Pfeifer
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.rdoc

@@ -33,6 +33,17 @@ demonstrating a reference implementation of these features.
 
 == Usage
 
+=== Installation
+
++has_messages+ requires additional database tables to work.  You can generate
+a migration for these tables like so:
+
+  script/generate has_messages
+
+Then simply migrate your database:
+
+  rake db:migrate
+
 === Adding message support
 
   class User < ActiveRecord::Base

data/Rakefile

@@ -1,15 +1,17 @@
+require 'rubygems'
+require 'rake'
 require 'rake/testtask'
 require 'rake/rdoctask'
 require 'rake/gempackagetask'
-require 'rake/contrib/sshpublisher'
 
 spec = Gem::Specification.new do |s|
   s.name              = 'has_messages'
-  s.version           = '0.4.0'
+  s.version           = '0.4.1'
   s.platform          = Gem::Platform::RUBY
-  s.summary           = 'Demonstrates a reference implementation for sending messages between users.'
+  s.summary           = 'Demonstrates a reference implementation for sending messages between users in ActiveRecord'
+  s.description       = s.summary
   
-  s.files             = FileList['{app,db,lib,test}/**/*'] + %w(CHANGELOG.rdoc init.rb LICENSE Rakefile README.rdoc) - FileList['test/app_root/{log,log/*,script,script/*}']
+  s.files             = FileList['{app,generators,lib,test}/**/*'] + %w(CHANGELOG.rdoc init.rb LICENSE Rakefile README.rdoc) - FileList['test/app_root/{log,log/*,script,script/*}']
   s.require_path      = 'lib'
   s.has_rdoc          = true
   s.test_files        = Dir['test/**/*_test.rb']
@@ -53,20 +55,27 @@ Rake::RDocTask.new(:rdoc) do |rdoc|
   rdoc.options << '--line-numbers' << '--inline-source'
   rdoc.rdoc_files.include('README.rdoc', 'CHANGELOG.rdoc', 'LICENSE', 'lib/**/*.rb', 'app/**/*.rb')
 end
-  
+
+desc 'Generate a gemspec file.'
+task :gemspec do
+  File.open("#{spec.name}.gemspec", 'w') do |f|
+    f.write spec.to_ruby
+  end
+end
+
 Rake::GemPackageTask.new(spec) do |p|
   p.gem_spec = spec
-  p.need_tar = true
-  p.need_zip = true
 end
 
 desc 'Publish the beta gem.'
 task :pgem => [:package] do
+  require 'rake/contrib/sshpublisher'
   Rake::SshFilePublisher.new('aaron@pluginaweek.org', '/home/aaron/gems.pluginaweek.org/public/gems', 'pkg', "#{spec.name}-#{spec.version}.gem").upload
 end
 
 desc 'Publish the API documentation.'
 task :pdoc => [:rdoc] do
+  require 'rake/contrib/sshpublisher'
   Rake::SshDirPublisher.new('aaron@pluginaweek.org', "/home/aaron/api.pluginaweek.org/public/#{spec.name}", 'rdoc').upload
 end
 
@@ -75,15 +84,8 @@ task :publish => [:pgem, :pdoc, :release]
 
 desc 'Publish the release files to RubyForge.'
 task :release => [:gem, :package] do
-  require 'rubyforge'
-  
-  ruby_forge = RubyForge.new.configure
-  ruby_forge.login
+  require 'rake/gemcutter'
   
-  %w(gem tgz zip).each do |ext|
-    file = "pkg/#{spec.name}-#{spec.version}.#{ext}"
-    puts "Releasing #{File.basename(file)}..."
-    
-    ruby_forge.add_release(spec.rubyforge_project, spec.name, spec.version, file)
-  end
+  Rake::Gemcutter::Tasks.new(spec)
+  Rake::Task['gem:push'].invoke
 end

data/generators/has_messages/USAGE

@@ -0,0 +1,5 @@
+Usage:
+
+  script/generate has_messages
+
+This will create migrations that will add the proper tables to store messages.

data/generators/has_messages/has_messages_generator.rb

@@ -0,0 +1,9 @@
+class HasMessagesGenerator < Rails::Generator::Base
+  def manifest
+    record do |m|
+      m.migration_template '001_create_messages.rb', 'db/migrate', :migration_file_name => 'create_messages'
+      m.sleep 1
+      m.migration_template '002_create_message_recipients.rb', 'db/migrate', :migration_file_name => 'create_message_recipients'
+    end
+  end
+end

data/lib/has_messages.rb

@@ -40,13 +40,13 @@ module HasMessages
                   :as => :sender,
                   :class_name => 'Message',
                   :conditions => {:hidden_at => nil},
-                  :order => 'messages.created_at ASC'
+                  :order => 'messages.created_at DESC'
       has_many  :received_messages,
                   :as => :receiver,
                   :class_name => 'MessageRecipient',
                   :include => :message,
                   :conditions => ['message_recipients.hidden_at IS NULL AND messages.state = ?', 'sent'],
-                  :order => 'messages.created_at ASC'
+                  :order => 'messages.created_at DESC'
       
       include HasMessages::InstanceMethods
     end

data/test/app_root/db/migrate/002_migrate_has_messages_to_version_2.rb

@@ -1,12 +1,12 @@
 class MigrateHasMessagesToVersion2 < ActiveRecord::Migration
   def self.up
-    ActiveRecord::Migrator.new(:up, "#{Rails.root}/../../db/migrate", 0).migrations.each do |migration|
+    ActiveRecord::Migrator.new(:up, "#{Rails.root}/../../generators/has_messages/templates", 0).migrations.each do |migration|
       migration.migrate(:up)
     end
   end
   
   def self.down
-    ActiveRecord::Migrator.new(:up, "#{Rails.root}/../../db/migrate", 0).migrations.each do |migration|
+    ActiveRecord::Migrator.new(:down, "#{Rails.root}/../../generators/has_messages/templates", 0).migrations.each do |migration|
       migration.migrate(:down)
     end
   end

data/test/factory.rb

@@ -37,7 +37,8 @@ module Factory
     attributes[:sender] = create_user unless attributes.include?(:sender)
     attributes.reverse_merge!(
       :subject => 'New features',
-      :body => 'Lots of new things to talk about... come to the meeting tonight to find out!'
+      :body => 'Lots of new things to talk about... come to the meeting tonight to find out!',
+      :created_at => Time.current + Message.count
     )
   end
   

data/test/functional/has_messages_test.rb

@@ -50,11 +50,11 @@ class UserWithUnsentMessages < ActiveSupport::TestCase
   end
   
   def test_should_have_unsent_messages
-    assert_equal [@first_draft, @second_draft], @user.unsent_messages
+    assert_equal [@second_draft, @first_draft], @user.unsent_messages
   end
   
   def test_should_include_unsent_messages_in_messages
-    assert_equal [@sent_message, @first_draft, @second_draft], @user.messages
+    assert_equal [@second_draft, @first_draft, @sent_message], @user.messages
   end
 end
 
@@ -63,6 +63,7 @@ class UserWithSentMessages < ActiveSupport::TestCase
     @user = create_user
     @to = create_user(:login => 'you')
     @draft = create_message(:sender => @user)
+    @draft.save!
     
     @first_sent_message = create_message(:sender => @user, :to => @to)
     @first_sent_message.deliver
@@ -72,11 +73,11 @@ class UserWithSentMessages < ActiveSupport::TestCase
   end
   
   def test_should_have_sent_messages
-    assert_equal [@first_sent_message, @second_sent_message], @user.sent_messages
+    assert_equal [@second_sent_message, @first_sent_message], @user.sent_messages
   end
   
   def test_should_include_sent_messages_in_messages
-    assert_equal [@draft, @first_sent_message, @second_sent_message], @user.messages
+    assert_equal [@second_sent_message, @first_sent_message, @draft], @user.messages
   end
 end
 
@@ -95,7 +96,7 @@ class UserWithReceivedMessages < ActiveSupport::TestCase
   end
   
   def test_should_have_received_messages
-    assert_equal [@first_sent_message, @second_sent_message], @user.received_messages.map(&:message)
+    assert_equal [@second_sent_message, @first_sent_message], @user.received_messages.map(&:message)
   end
 end
 
@@ -122,7 +123,7 @@ class UserWithHiddenMessagesTest < ActiveSupport::TestCase
   end
   
   def test_should_not_include_hidden_messages_in_messages
-    assert_equal [@unsent_message, @sent_message], @user.messages
+    assert_equal [@sent_message, @unsent_message], @user.messages
   end
   
   def test_should_not_include_hidden_messages_in_unsent_messages

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: has_messages
 version: !ruby/object:Gem::Version 
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors: 
 - Aaron Pfeifer
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-04-19 00:00:00 -04:00
+date: 2010-03-07 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -22,7 +22,7 @@ dependencies:
       - !ruby/object:Gem::Version 
         version: 0.7.0
     version: 
-description: 
+description: Demonstrates a reference implementation for sending messages between users in ActiveRecord
 email: aaron@pluginaweek.org
 executables: []
 
@@ -31,30 +31,22 @@ extensions: []
 extra_rdoc_files: []
 
 files: 
-- app/models
-- app/models/message_recipient.rb
 - app/models/message.rb
-- db/migrate
-- db/migrate/001_create_messages.rb
-- db/migrate/002_create_message_recipients.rb
+- app/models/message_recipient.rb
+- generators/has_messages/USAGE
+- generators/has_messages/has_messages_generator.rb
+- generators/has_messages/templates/001_create_messages.rb
+- generators/has_messages/templates/002_create_message_recipients.rb
 - lib/has_messages.rb
-- test/factory.rb
-- test/test_helper.rb
-- test/functional
-- test/functional/has_messages_test.rb
-- test/unit
-- test/unit/message_test.rb
 - test/unit/message_recipient_test.rb
-- test/app_root
-- test/app_root/db
-- test/app_root/db/migrate
+- test/unit/message_test.rb
 - test/app_root/db/migrate/002_migrate_has_messages_to_version_2.rb
 - test/app_root/db/migrate/001_create_users.rb
-- test/app_root/config
-- test/app_root/config/environment.rb
-- test/app_root/app
-- test/app_root/app/models
 - test/app_root/app/models/user.rb
+- test/app_root/config/environment.rb
+- test/test_helper.rb
+- test/factory.rb
+- test/functional/has_messages_test.rb
 - CHANGELOG.rdoc
 - init.rb
 - LICENSE
@@ -62,6 +54,8 @@ files:
 - README.rdoc
 has_rdoc: true
 homepage: http://www.pluginaweek.org
+licenses: []
+
 post_install_message: 
 rdoc_options: []
 
@@ -82,11 +76,11 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: pluginaweek
-rubygems_version: 1.3.1
+rubygems_version: 1.3.5
 signing_key: 
-specification_version: 2
-summary: Demonstrates a reference implementation for sending messages between users.
+specification_version: 3
+summary: Demonstrates a reference implementation for sending messages between users in ActiveRecord
 test_files: 
-- test/functional/has_messages_test.rb
-- test/unit/message_test.rb
 - test/unit/message_recipient_test.rb
+- test/unit/message_test.rb
+- test/functional/has_messages_test.rb