tylerrick-bcms_blog 1.0.5

data/app/views/partials/_blog_post.html.haml

@@ -0,0 +1,91 @@
+%style
+  :sass
+    .blog_post.first
+      h2, h2 a
+        font: normal 30px/28px 'arial black', arial, sans-serif !important
+      border-bottom: 1px solid #ccc
+
+    .blog_post
+      border: 0px solid gray
+      margin-bottom: 20px
+
+      h2
+        margin-bottom: 0
+      h2, h2 a
+        font: normal 20px/23px 'arial black', arial, sans-serif !important
+        text-decoration: none !important
+      h2 a:hover
+        text-decoration: underline !important
+
+      .image
+        float: left
+        border: 1px solid #ccc
+        margin: 10px
+        margin-top: 0
+
+      .date
+        color: #666
+      .read_more
+        font-weight: bold
+
+      .comment + .comment
+        border-top: 1px dashed #ccc
+
+    .clear
+      clear: both
+
+:ruby
+  # _counter is defined only if we pass :collection to the partial
+  if defined?(blog_post_counter)
+    showing_individual_post = false
+  else
+    showing_individual_post = true
+    blog_post_counter = 0
+  end
+
+  if blog_post_counter == 0
+    max_width = 250
+    css_class = 'first'
+  else
+    max_width = 75
+    css_class = ''
+  end
+
+.blog_post.clear{:id => "blog_post_#{blog_post.id}", :class => css_class}
+  - if blog_post.attachment
+    -# .image= image_tag_with_max_size blog_post.attachment.file_path, blog_post.attachment.full_file_location, :width => max_width
+    .image= image_tag blog_post.attachment.file_path
+
+  %h2= link_to h(blog_post.name), href = _blog_post_path(blog_post)
+
+  .date= blog_post.published_at.to_s(:long)
+
+  .body
+    - if showing_individual_post or blog_post.summary.blank?
+      = blog_post.body
+    - else
+      = blog_post.summary
+      %p.read_more= link_to 'Read More »', href
+
+  - if showing_individual_post || blog_post_counter == 0
+    .meta
+      - unless blog_post.category_id.blank?
+        Posted in #{link_to h(blog_post.category_name), _blog_path(blog_post.blog, 'posts_in_category', :category => blog_post.category_name)}
+        %strong |
+      - if blog_post.tags.any?
+        Tags:
+        %span.tags
+          = blog_post.tags.map{|t| link_to(h(t.name), _blog_path(blog_post.blog, 'posts_with_tag', :tag => t.name)) }.join(", ")
+        %strong |
+        = link_to h(pluralize(blog_post.comments_count, "Comment")), "#{_blog_post_path(blog_post)}#comments"
+  %br.clear/
+
+  - comments = blog_post.comments.reject(&:new_record?)
+  - if showing_individual_post and comments.any?
+    %h2 Comments
+    - comments.each_with_index do |comment, i|
+      %div{:class => "comment #{'first' if i == 0}"}
+        = h comment.body
+        %p
+          \—#{comment.url.present? ? link_to(h(comment.author), comment.url) : h(comment.author)}
+

data/app/views/portlets/blog_post/_form.html.erb

@@ -0,0 +1,3 @@
+<%= f.cms_text_field :name %>
+<%= f.cms_drop_down :blog_id, Blog.all.map{|b| [b.name, b.id.to_s]} %>
+<%= f.cms_text_area :template, :default_value => @block.class.default_template %>

data/app/views/portlets/blog_post/render.html.erb

@@ -0,0 +1,33 @@
+<% page_title @blog_post.name %>
+<%= render :partial => "partials/blog_post", :object => @blog_post %>
+
+<style>
+  @import url('/stylesheets/cms/form_layout.css');
+</style>
+
+<div class="blog_comment_form">
+  <% form_for @blog_comment, :url => cms_handler_path(@portlet, "create_comment") do |f| %>
+    <%= f.hidden_field :post_id %>
+    <%= f.error_messages %>
+    <div class="fields text_fields">
+      <%= f.label :author, 'Name *' %>
+      <%= f.text_field :author %>
+    </div>
+    <div class="fields text_fields clear">
+      <%= f.label :email, 'E-mail address' %>
+      <%= f.text_field :email %>
+    </div>
+    <div class="fields text_fields clear">
+      <%= f.label :url, 'Web site' %>
+      <%= f.text_field :url %>
+    </div>
+    <div class="fields text_editor_fields clear">
+      <%= f.label :body, 'Body *' %>
+      <br/>
+      <%= f.text_area :body, :size => "50x5", :style => 'height: auto'  %>
+    </div>
+    <div class="buttons clear">
+      <%= submit_tag "Submit comment" %>
+    </div>
+  <% end %>
+</div>

data/db/migrate/20090415000000_create_blogs.rb

@@ -0,0 +1,45 @@
+require 'pp'
+
+Page # trigger auto-loading
+# At the time of this writing, these associations are missing :dependent => :destroy
+class Page
+  has_many :page_routes, :dependent => :destroy
+end
+class PageRoute
+  has_many :requirements, :class_name => "PageRouteRequirement", :dependent => :destroy
+  has_many :conditions,   :class_name => "PageRouteCondition",   :dependent => :destroy
+end
+
+class CreateBlogs < ActiveRecord::Migration
+  def self.up
+    create_versioned_table :blogs do |t|
+      t.string :name
+      t.string :format
+      t.text :template
+    end
+    
+    create_table :blog_group_memberships do |t|
+      t.integer :blog_id
+      t.integer :group_id
+    end
+    
+    ContentType.create!(:name => "Blog", :group_name => "Blog")
+  end
+
+  def self.down
+    puts "Destroying portlets, pages, page_routes..."
+    pp (portlets = BlogPostPortlet.all).map(&:connected_pages).flatten.each(&:destroy)
+    pp portlets.each(&:destroy)
+
+    #Blog.all.map(&:connected_pages).flatten.map(&:page_routes).flatten.each(&:destroy)
+    pp Blog.all.map(&:connected_pages).flatten.each(&:destroy)
+
+    ContentType.destroy_all(:name => "Blog")
+    Connector.destroy_all(:connectable_type => "Blog")
+
+    drop_table :blog_versions
+    drop_table :blogs
+
+    drop_table :blog_group_memberships
+  end
+end

data/db/migrate/20090415000001_create_blog_posts.rb

@@ -0,0 +1,25 @@
+class CreateBlogPosts < ActiveRecord::Migration
+  def self.up
+    create_versioned_table :blog_posts do |t|
+      t.integer :blog_id
+      t.integer :author_id
+      t.integer :category_id
+      t.string :name
+      t.string :slug
+      t.text :summary
+      t.text :body, :size => (64.kilobytes + 1)
+      t.integer :comments_count
+      t.datetime :published_at
+    end
+    CategoryType.create!(:name => "Blog Post")
+    ContentType.create!(:name => "BlogPost", :group_name => "Blog")
+  end
+
+  def self.down
+    ContentType.destroy_all(:name => 'BlogPost')
+    CategoryType.destroy_all(:name => "Blog Post")
+
+    drop_table :blog_post_versions
+    drop_table :blog_posts
+  end
+end

data/db/migrate/20090415000002_create_blog_comments.rb

@@ -0,0 +1,19 @@
+class CreateBlogComments < ActiveRecord::Migration
+  def self.up
+    create_versioned_table :blog_comments do |t|
+      t.integer :post_id
+      t.string :author
+      t.string :email
+      t.string :url
+      t.string :ip
+      t.text :body
+    end
+    ContentType.create!(:name => "BlogComment", :group_name => "Blog")    
+  end
+
+  def self.down
+    ContentType.destroy_all(:name => 'BlogComment')
+    drop_table :blog_comment_versions
+    drop_table :blog_comments
+  end
+end

data/db/migrate/20090415000003_add_attachment_to_blog_posts.rb

@@ -0,0 +1,23 @@
+class AddAttachmentToBlogPosts < ActiveRecord::Migration
+  def self.up
+    change_table :blog_posts do |t|
+      t.belongs_to :attachment
+      t.integer :attachment_version
+    end
+    change_table :blog_post_versions do |t|
+      t.belongs_to :attachment
+      t.integer :attachment_version
+    end
+  end
+
+  def self.down
+    change_table :blog_posts do |t|
+      t.remove :attachment
+      t.remove :attachment_version
+    end
+    change_table :blog_post_versions do |t|
+      t.remove :attachment
+      t.remove :attachment_version
+    end
+  end
+end

data/doc/README_FOR_APP

@@ -0,0 +1,2 @@
+Use this README file to introduce your application and point to useful places in the API for learning more.
+Run "rake doc:app" to generate API documentation for your models, controllers, helpers, and libraries.

data/doc/migrate_to_20100427.rb

@@ -0,0 +1,68 @@
+# Ideally, we would create a new migration whenever we wanted to make changes to an existing
+# migration that is already in circulation, instead of simply editing the existing migrations
+# and publishing the new version, because doing rake db:migrate:redo VERSION=20090415000000,
+# etc. on each updated migration is not only a pain but will probably result in more data being
+# lost than is strictly necessary -- assuming the migrations work correctly at all.
+#
+# On the other hand, there's something pretty compelling about "fixing" the migration so that
+# it doesn't have any useless steps that are just going to get reverted by the next migration.
+#
+# There is also the problem of model dependencies: you can't safely remove a model in the same
+# commit that you use that model in a migration (f.e., Model.destroy_all). Even if you remove it
+# in the following commit, it's only safe if people check out every intermediate version and run
+# db:migrate at each version, instead of simply checking out the tip version and then running
+# db:migrate. And people don't do that.
+#
+# Anyway, I guess we assume that there are only a few brave souls out there who are already
+# using bcms_blog and hope for the best. Here is the "missing migration"; hopefully it will be
+# helpful to you.
+#
+# Run this script instead of re-running the changed migrations with rake db:migrate:redo so that
+# your blog data is not destroyed.
+#
+# Usage: ./script/runner 'require "/path/to/bcms_blog/doc/migrate_to_20100427.rb"'
+
+require 'pp'
+
+Page # trigger auto-loading
+# At the time of this writing, these associations are missing :dependent => :destroy
+class Page
+  has_many :page_routes, :dependent => :destroy
+end
+class PageRoute
+  has_many :requirements, :class_name => "PageRouteRequirement", :dependent => :destroy
+  has_many :conditions,   :class_name => "PageRouteCondition",   :dependent => :destroy
+end
+
+class MigrateTo20100427 < ActiveRecord::Migration
+  def self.up
+    drop_table :blog_group_membership_versions
+
+    PageRouteOption.all.each {|a| a.destroy unless a.page_route }
+
+    puts "Destroying portlets, pages, page_routes left over from old version of bcms_blog..."
+    puts "(*Not* destroying any existing Blogs, pages on which Blogs are connected, or BlogPosts)"
+    portlets = [BlogPostPortlet, BlogPostsInCategoryPortlet, BlogPostsWithTagPortlet, BlogPostsInDayPortlet, BlogPostsInMonthPortlet, BlogPostsInYearPortlet]
+    #pp portlets.map(&:all).flatten.map(&:connected_pages).flatten.map(&:page_routes).flatten.each(&:destroy)
+    pp portlets.map(&:all).flatten.map(&:connected_pages).flatten.each(&:destroy)
+
+    # Something like this might not work if they have moved their Blog page to a different section or to no section, so we better let users resolve this manually...
+    #Blog.all.map(&:connected_pages).flatten.each do |page|
+    #  page.hidden = true
+    #  page.save!
+    #  page.section.hidden = false
+    #  page.section.save!
+    #end
+
+    puts "Calling after_create on each Blog in the system..."
+    Blog.all.each do |blog|
+      puts "#{blog}..."
+      blog.send :after_create
+    end
+  end
+
+  def self.down
+  end
+end
+
+MigrateTo20100427.up

data/lib/bcms_blog/routes.rb

@@ -0,0 +1,9 @@
+module Cms::Routes
+  def routes_for_bcms_blog
+    namespace(:cms) do |cms|      
+      cms.content_blocks :blogs
+      cms.content_blocks :blog_posts
+      cms.content_blocks :blog_comments
+    end
+  end
+end

data/lib/bcms_blog.rb

@@ -0,0 +1 @@
+require 'bcms_blog/routes'

data/rails/init.rb

@@ -0,0 +1,4 @@
+gem_root = File.expand_path(File.join(File.dirname(__FILE__), ".."))
+Cms.add_to_rails_paths gem_root
+Cms.add_generator_paths gem_root, "db/migrate/[0-9]*_*.rb"
+ApplicationHelper.module_eval { include Cms::BlogHelper }

data/test/factories.rb

@@ -0,0 +1,31 @@
+if ActiveRecord::Base.connection.table_exists?("blogs")
+
+Factory.define :group do |m|
+  m.sequence(:name) {|n| "TestGroup#{n}" }
+end
+
+Factory.define :group_type do |m|
+  m.sequence(:name) {|n| "TestGroupType#{n}" }
+end
+
+Factory.define :user do |m|
+  m.first_name "Test"
+  m.last_name "User"
+  m.sequence(:login) {|n| "test_#{n}" }
+  m.email {|a| "#{a.login}@example.com" }
+  m.password "password"
+  m.password_confirmation {|a| a.password }
+end
+
+Factory.define :blog do |m|
+  m.sequence(:name) {|n| "TestBlog#{n}"}
+end
+
+Factory.define :blog_post do |b|
+  b.sequence(:name) { |n| "BlogPost#{n}" }
+  b.blog { Factory.create(:blog) }
+  b.sequence(:body) { |n| "Lorem ipsum #{n}" }
+  b.author { User.find(:first) }
+end
+
+end

data/test/functional/blog_post_test.rb

@@ -0,0 +1,41 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class BlogPostTest < ActionController::TestCase
+  tests Cms::ContentController
+  
+  def setup
+    create_baseline_data
+  end
+  
+  def teardown
+    destroy_baseline_data
+  end
+
+  def test_show_post
+    get :show, :path => ["blog", "post"],
+      :year => @first_post.year,
+      :month => @first_post.month,
+      :day => @first_post.day,
+      :slug => @first_post.slug
+    #log @response.body
+    assert_response :success
+    assert_select "title", @first_post.name
+    assert_select ".blog_post", 1
+
+    assert_select "#blog_post_#{@first_post.id}" do
+      assert_select "h2 a", @first_post.name
+      assert_select "p.body", @first_post.body
+      assert_select "p.meta a", "General"
+      assert_select "p.meta a", "0 Comments"
+    end
+    
+  end
+
+  def test_non_existent_slug_should_return_404
+    get :show, :path => ["blog", "post"],
+      :year => 2005, :month => 6, :day => 14,
+      :slug => "not-here"
+    assert_response :not_found
+  end
+  
+end

data/test/functional/blog_test.rb

@@ -0,0 +1,69 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class BlogTest < ActionController::TestCase
+  tests Cms::ContentController
+  
+  def setup
+    create_baseline_data
+  end
+  
+  def teardown
+    destroy_baseline_data
+  end
+  
+  def test_list_of_blog_posts
+    get :show
+    #log @response.body
+    assert_response :success
+    assert_select ".blog_post", 5
+    
+    assert_select "#blog_post_#{@first_post.id}" do
+      assert_select "h2 a", @first_post.name
+      assert_select "p.body", @first_post.body
+      assert_select "p.meta a", "General"
+      assert_select "p.meta a", "0 Comments"
+    end
+    
+    assert_select "#blog_post_#{@foo_post_1.id}" do
+      assert_select "h2 a", @foo_post_1.name
+      assert_select "p.body", @foo_post_1.body
+      assert_select "p.meta .tags a", "foo"
+      assert_select "p.meta .tags a", "stuff"
+    end
+  end
+  
+  def test_list_of_tagged_blog_posts
+    get :show, :category => "General"
+    #log @response.body
+    assert_response :success
+    assert_select ".blog_post", 3
+  end
+  
+  def test_list_of_categorized_blog_posts
+    get :show, :tag => "foo"
+    #log @response.body
+    assert_response :success
+    assert_select ".blog_post", 2
+  end
+  
+  def test_list_of_blog_posts_in_day
+    get :show, :path => ["blog", "posts_in_day"],
+        :year => 2008, :month => 7, :day => 5
+    assert_response :success
+    assert_select ".blog_post", 2
+  end
+  
+  def test_list_of_blog_posts_in_month
+    get :show, :path => ["blog", "posts_in_month"],
+        :year => 2008, :month => 7
+    assert_response :success
+    assert_select ".blog_post", 3
+  end
+  
+  def test_list_of_blog_posts_in_year
+    get :show, :path => ["blog", "posts_in_year"],
+        :year => 2008
+    assert_response :success
+    assert_select ".blog_post", 4
+  end
+end

data/test/functional/cms/blog_posts_controller_test.rb

@@ -0,0 +1,43 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class Cms::BlogPostsControllerTest < ActionController::TestCase
+  def setup
+    create_non_admin_user
+    login_as(@user)
+  end
+
+  def test_access_denied_on_create_if_blog_not_user_editable
+    @editable = Factory.create(:blog, :groups => [@group])
+    @non_editable = Factory.create(:blog)
+    post :create, :blog_post => { :blog_id => @non_editable.id }
+    assert @response.body.include?("AccessDenied")
+  end
+  
+  def test_access_denied_on_update_if_blog_not_user_editable
+    @editable = Factory.create(:blog, :groups => [@group])
+    @non_editable = Factory.create(:blog)
+    @blog_post = Factory.create(:blog_post, :blog => @non_editable)
+    put :update, :id => @blog_post, :blog_post => { :name => "Foo" }
+    assert @response.body.include?("AccessDenied")
+  end
+  
+  def test_no_access_if_no_editable_blogs
+    @blog = Factory.create(:blog)
+    get :index
+    assert_template "no_access"
+  end
+  
+  def test_index_shouldnt_show_non_editable_posts
+    @editable = Factory.create(:blog, :groups => [@group])
+    @non_editable = Factory.create(:blog)
+    @blog_post = Factory.create(:blog_post, :name => "Non-editable", :blog => @non_editable)
+    get :index
+    assert !@response.body.include?("Non-editable")
+  end
+  
+  def test_create_sets_author
+    @blog = Factory.create(:blog, :groups => [@group])
+    post :create, :blog_post => { :blog_id => @blog.id }
+    assert_equal @user, assigns(:block).author
+  end
+end

data/test/functional/cms/blogs_controller_test.rb

@@ -0,0 +1,12 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class Cms::BlogsControllerTest < ActionController::TestCase
+  def test_list_blog_posts_as_non_admin
+    create_non_admin_user
+    login_as(@user)
+    
+    get :index
+    assert_response :success
+    assert @response.body.include? "Sorry, this section is restricted to administrators."
+  end
+end

data/test/performance/browsing_test.rb

@@ -0,0 +1,9 @@
+require 'test_helper'
+require 'performance_test_help'
+
+# Profiling results for each test method are written to tmp/performance.
+class BrowsingTest < ActionController::PerformanceTest
+  def test_homepage
+    get '/'
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,104 @@
+ENV["RAILS_ENV"] = "test"
+ENV['BACKTRACE'] = "YES PLEASE"
+require File.expand_path(File.dirname(__FILE__) + "/../config/environment")
+require 'test_help'
+
+unless $database_initialized
+  $database_initialized = true
+  
+  # Empty the database and load in the default seed data for browsercms
+  # and the blog module
+  `rake db:test:purge`
+  `rake db:migrate`
+
+  # Publish all pages, as they are drafts after migrating
+  Page.find(:all).each(&:publish!)
+end
+
+class ActiveSupport::TestCase
+  require File.dirname(__FILE__) + '/test_logging'
+  include TestLogging  
+  # Transactional fixtures accelerate your tests by wrapping each test method
+  # in a transaction that's rolled back on completion.  This ensures that the
+  # test database remains unchanged so your fixtures don't have to be reloaded
+  # between every test method.  Fewer database queries means faster tests.
+  #
+  # Read Mike Clark's excellent walkthrough at
+  #   http://clarkware.com/cgi/blosxom/2005/10/24#Rails10FastTesting
+  #
+  # Every Active Record database supports transactions except MyISAM tables
+  # in MySQL.  Turn off transactional fixtures in this case; however, if you
+  # don't care one way or the other, switching from MyISAM to InnoDB tables
+  # is recommended.
+  #
+  # The only drawback to using transactional fixtures is when you actually 
+  # need to test transactions.  Since your test is bracketed by a transaction,
+  # any transactions started in your code will be automatically rolled back.
+  self.use_transactional_fixtures = true
+
+  # Instantiated fixtures are slow, but give you @david where otherwise you
+  # would need people(:david).  If you don't want to migrate your existing
+  # test cases which use the @david style and don't mind the speed hit (each
+  # instantiated fixtures translates to a database query per test method),
+  # then set this back to true.
+  self.use_instantiated_fixtures  = false
+
+  # Setup all fixtures in test/fixtures/*.(yml|csv) for all tests in alphabetical order.
+  #
+  # Note: You'll currently still have to declare fixtures explicitly in integration tests
+  # -- they do not yet inherit this setting
+  fixtures :all
+
+  # Add more helper methods to be used by all tests here...
+  def create_baseline_data
+    # Find the seed data items
+    @blog = Blog.find_by_name("My Blog")
+    @blog_page = Page.find_by_path("/")
+    @blog_post_page = Page.find_by_path("/blog/post")
+    @category_type = CategoryType.find_by_name("Blog Post")
+    
+    # For some reason this is necessary otherwise the relevant page routes aren't loaded when
+    # the tests are run via "rake" (as opposed to running them directly). I don't know exactly
+    # why this is the case.
+    ActionController::Routing::Routes.load!
+
+    @stuff = Category.create!(:name => "Stuff", :category_type => @category_type)
+    @general = Category.create!(:name => "General", :category_type => @category_type)
+    
+    @first_post = Factory(:blog_post, :blog => @blog, :category => @general,
+      :published_at => Time.utc(2008, 7, 5, 6), :publish_on_save => true)
+    
+    @foo_post_1 = Factory(:blog_post, :blog => @blog, :category => @stuff,
+      :published_at => Time.utc(2008, 7, 5, 12), :tag_list => "foo stuff", :publish_on_save => true)
+    
+    @foo_post_2 = Factory(:blog_post, :blog => @blog, :category => @general,
+      :published_at => Time.utc(2008, 7, 21), :publish_on_save => true)
+    
+    @bar_post_1 = Factory(:blog_post, :blog => @blog, :category => @stuff,
+      :published_at => Time.utc(2008, 9, 2), :tag_list => "foo stuff", :publish_on_save => true)
+    
+    @bar_post_2 = Factory(:blog_post, :blog => @blog, :category => @general,
+      :published_at => Time.utc(2009, 3, 18), :publish_on_save => true)
+  end
+  
+  def destroy_baseline_data
+    Category.destroy_all
+    BlogPost.destroy_all
+  end
+  
+  def create_non_admin_user
+    @group = Factory(:group, :name => "Test", :group_type => Factory(:group_type, :name => "CMS User", :cms_access => true))
+    @group.permissions << Permission.find_by_name("edit_content")
+    @group.permissions << Permission.find_by_name("publish_content")
+    @group.save!
+    
+    @user = Factory(:user)
+    @user.groups << @group
+    @user.save!
+  end
+  
+  def login_as(user)
+    @request.session[:user_id] = user ? user.id : nil
+  end
+  
+end