tamed_beast 0.0.1

data/Gemfile

@@ -0,0 +1,4 @@
+source "http://rubygems.org"
+
+# Specify your gem's dependencies in tamed_beast.gemspec
+gemspec

data/README

@@ -0,0 +1,12 @@
+Welcome to Tamed Beast!
+
+Tamed beast is an attempt to tame savage_beast under Rails3. We worked with savage_beast under rails 2.x, but we find ourselves less than comfortable using it...
+And it came the day to migrate to Rails3 and - hey - how do we migrate savage_beast to Rails3? What about all our posts and forums?
+Damn.
+So we started writing our own bulletin board solution, trying to be compatible with the savage_beast data model.
+
+TODO:
+	* Add moderators / manage moderatorship roles
+  * Add views
+  * Add search feature
+  * Improve tests

data/Rakefile

@@ -0,0 +1,2 @@
+require 'bundler'
+Bundler::GemHelper.install_tasks

data/app/controllers/forums_controller.rb

@@ -0,0 +1,60 @@
+class ForumsController < ApplicationController
+	before_filter :login_required, :except => [:index, :show]
+  before_filter :find_or_initialize_forum, :except => :index
+	
+  #before_filter :admin?, :except => [:show, :index]
+
+  def index
+    @forums = Forum.order('position')
+    # reset the page of each forum we have visited when we go back to index
+    session[:forum_page] = nil
+    respond_to do |format|
+      format.html
+      format.xml { render :xml => @forums }
+    end
+  end
+
+  def show
+    respond_to do |format|
+      format.html do
+        # keep track of when we last viewed this forum for activity indicators
+        (session[:forums] ||= {})[@forum.id] = Time.now.utc if !current_user.nil?
+        (session[:forum_page] ||= Hash.new(1))[@forum.id] = params[:page].to_i if params[:page]
+
+        @topics = @forum.topics.paginate :page => params[:page]
+        User.find(:all, :conditions => ['id IN (?)', @topics.collect { |t| t.replied_by }.uniq]) unless @topics.blank?
+      end
+      format.xml { render :xml => @forum }
+    end
+  end
+
+  def create
+    @forum.attributes = params[:forum]
+    @forum.save!
+    respond_to do |format|
+      format.html { redirect_to @forum }
+      format.xml  { head :created, :location => forum_url(@forum, :format => :xml) }
+    end
+  end
+
+  def update
+    @forum.update_attributes!(params[:forum])
+    respond_to do |format|
+      format.html { redirect_to @forum }
+      format.xml  { head 200 }
+    end
+  end
+  
+  def destroy
+    @forum.destroy
+    respond_to do |format|
+      format.html { redirect_to forums_path }
+      format.xml  { head 200 }
+    end
+  end
+  
+  protected
+    def find_or_initialize_forum
+      @forum = params[:id] ? Forum.find(params[:id]) : Forum.new
+    end
+end

data/app/controllers/posts_controller.rb

@@ -0,0 +1,98 @@
+class PostsController < ApplicationController
+  before_filter :find_post,      :except => [:index, :create]
+  before_filter :login_required, :except => [:index, :show]
+  #@@query_options = { :select => "#{Post.table_name}.*, #{Topic.table_name}.title as topic_title, #{Forum.table_name}.name as forum_name", :joins => "inner join #{Topic.table_name} on #{Post.table_name}.topic_id = #{Topic.table_name}.id inner join #{Forum.table_name} on #{Topic.table_name}.forum_id = #{Forum.table_name}.id" }
+
+  def index
+    #conditions = []
+    #[:user_id, :forum_id, :topic_id].each { |attr| conditions << Post.send(:sanitize_sql, ["#{Post.table_name}.#{attr} = ?", params[attr]]) if params[attr] }
+    #conditions = conditions.empty? ? nil : conditions.collect { |c| "(#{c})" }.join(' AND ')
+    #@posts = Post.paginate @@query_options.merge(:conditions => conditions, :page => params[:page], :count => {:select => "#{Post.table_name}.id"}, :order => post_order)
+    #@users = User.find(:all, :select => 'distinct *', :conditions => ['id in (?)', @posts.collect(&:user_id).uniq]).index_by(&:id)
+    #render_posts_or_xml
+
+    #@posts = Post.where(:forum_id => params[:forum_id]).where(:topic_id => params[:topic_id]).order('created_at desc').paginate(:page => params[:page])
+  end
+
+  def show
+    respond_to do |format|
+      format.html { redirect_to forum_topic_path(@post.forum_id, @post.topic_id) }
+      format.xml  { render :xml => @post.to_xml }
+    end
+  end
+
+  def create
+    @topic = Topic.find_by_id_and_forum_id(params[:topic_id],params[:forum_id])
+    @forum = @topic.forum
+    @post  = @topic.posts.build(params[:post])
+    @post.user = current_user
+    @post.save!
+    email_users_in_topic(@topic)
+    respond_to do |format|
+      format.html do
+        redirect_to forum_topic_path(:forum_id => params[:forum_id], :id => params[:topic_id], :anchor => @post.dom_id, :page => params[:page] || '1')
+      end
+      format.xml { head :created, :location => post_url(:forum_id => params[:forum_id], :topic_id => params[:topic_id], :id => @post, :format => :xml) }
+    end
+  rescue ActiveRecord::RecordInvalid
+    flash[:bad_reply] = 'Please post something at least...'[:post_something_message]
+    respond_to do |format|
+      format.html do
+        redirect_to forum_topic_path(:forum_id => params[:forum_id], :id => params[:topic_id], :anchor => 'reply-form', :page => params[:page] || '1')
+      end
+    end
+  end
+
+  def edit
+    respond_to do |format| 
+      format.html
+    end
+  end
+
+  def update
+    @post.attributes = params[:post]
+    @post.save!
+  rescue ActiveRecord::RecordInvalid
+    flash[:bad_reply] = 'An error occurred'[:error_occured_message]
+  ensure
+    respond_to do |format|
+      format.html do
+        redirect_to forum_topic_path(:forum_id => params[:forum_id], :id => params[:topic_id], :anchor => @post.dom_id, :page => params[:page] || '1')
+      end
+      format.xml { head 200 }
+    end
+  end
+
+  def destroy
+    @post.destroy
+    #flash[:notice] = "Post of '{title}' was deleted."[:post_deleted_message, @post.topic.title]
+    respond_to do |format|
+      format.html do
+        redirect_to(@post.topic.frozen? ? 
+                    forum_path(params[:forum_id]) :
+                    forum_topic_path(:forum_id => params[:forum_id], :id => params[:topic_id], :page => params[:page]))
+      end
+      format.xml { head 200 }
+    end
+  end
+
+  protected
+  def post_order
+    "#{Post.table_name}.created_at#{params[:forum_id] && params[:topic_id] ? nil : " desc"}"
+  end
+
+  def find_post			
+    @post = Post.find_by_id_and_topic_id_and_forum_id(params[:id], params[:topic_id], params[:forum_id]) || raise(ActiveRecord::RecordNotFound)
+  end
+
+  def email_users_in_topic(topic)
+    sent = []
+    topic.posts.find(:all, :conditions => ["user_id <> ?", current_user.id]).each do |post|
+      unless sent.include?(post.user.id)
+        UserMailer.send_later(:deliver_forum_notification, post.user, topic) if post.user.notify_forum?
+        sent << post.user_id
+      end 
+    end
+    UserMailer.send_later(:deliver_forum_notification_admin, topic)
+  end
+end

data/app/controllers/topics_controller.rb

@@ -0,0 +1,94 @@
+class TopicsController < ApplicationController
+  before_filter :find_forum_and_topic, :except => :index
+  before_filter :login_required , :only => [:new, :create, :edit, :update, :destroy]
+
+  def index
+    respond_to do |format|
+      format.html { redirect_to forum_path(params[:forum_id]) }
+      format.xml do
+        @topics = Topic.paginate_by_forum_id(params[:forum_id], :order => 'sticky desc, replied_at desc', :page => params[:page])
+        render :xml => @topics.to_xml
+      end
+    end
+  end
+
+  def new
+    @topic = Topic.new
+  end
+
+  def show
+    respond_to do |format|
+      format.html do
+        # keep track of when we last viewed this topic for activity indicators
+        (session[:topics] ||= {})[@topic.id] = Time.now.utc if !current_user.nil?
+        # authors of topics don't get counted towards total hits
+        @topic.hit! unless current_user.nil? and @topic.user == current_user
+        @posts = @topic.posts.paginate :page => params[:page]
+        User.find(:all, :conditions => ['id IN (?)', @posts.collect { |p| p.user_id }.uniq]) unless @posts.nil?
+        @post   = Post.new
+      end
+      format.xml do
+        render :xml => @topic.to_xml
+      end
+    end
+  end
+
+  def create
+    topic_save, post_save = false, false
+   
+    Topic.transaction do
+      @topic  = Topic.new(:title => params[:topic][:title])
+      @topic.forum_id = @forum.id
+      assign_protected
+      if @topic.save
+        topic_save = true
+        @post       = Post.new(:body => params[:topic][:body])
+        @post.topic_id = @topic.id
+        @post.forum_id = @topic.forum_id
+        @post.user  = current_user
+        @topic.body = @post.body
+        post_save = true if @post.save
+      end
+    end
+
+    if topic_save && post_save
+      respond_to do |format| 
+        format.html { redirect_to forum_topic_path(@forum, @topic) }
+        format.xml  { head :created, :location => topic_url(:forum_id => @forum, :id => @topic, :format => :xml) }
+      end
+    else
+      render :action => "new"
+    end
+  end
+
+  def update
+    @topic.attributes = params[:topic]
+    assign_protected
+    @topic.save!
+    respond_to do |format|
+      format.html { redirect_to forum_topic_path(@forum, @topic) }
+      format.xml  { head 200 }
+    end
+  end
+
+  def destroy
+    @topic.destroy
+    respond_to do |format|
+      format.html { redirect_to forum_path(@forum) }
+      format.xml  { head 200 }
+    end
+  end
+
+  protected
+  def assign_protected
+    @topic.user = current_user if @topic.new_record?
+    #return unless admin?
+    #@topic.sticky, @topic.locked = params[:topic][:sticky], params[:topic][:locked] 
+    @topic.forum_id = params[:forum_id] if params[:forum_id]
+  end
+
+  def find_forum_and_topic
+    @forum = Forum.find(params[:forum_id])
+    @topic = @forum.topics.find(params[:id]) if params[:id]
+  end
+end

data/app/models/forum.rb

@@ -0,0 +1,23 @@
+class Forum < ActiveRecord::Base
+  acts_as_list
+
+  validates_presence_of :name
+
+  has_many :topics, :order => 'sticky desc, replied_at desc', :dependent => :delete_all
+  has_one  :recent_topic, :class_name => 'Topic', :order => 'sticky desc, replied_at desc'
+
+  # this is used to see if a forum is "fresh"... we can't use topics because it puts
+  # stickies first even if they are not the most recently modified
+  has_many :recent_topics, :class_name => 'Topic', :order => 'replied_at DESC'
+  has_one  :recent_topic,  :class_name => 'Topic', :order => 'replied_at DESC'
+
+  has_many :posts,     :order => "#{Post.table_name}.created_at DESC", :dependent => :delete_all
+  has_one  :recent_post, :order => "#{Post.table_name}.created_at DESC", :class_name => 'Post'
+
+  format_attribute :description
+  
+  # retrieves forums ordered by position
+  def self.find_ordered(options = {})
+    self.order('position')
+  end
+end

data/app/models/post.rb

@@ -0,0 +1,26 @@
+class Post < ActiveRecord::Base
+  def self.per_page() 25 end
+
+  belongs_to :forum
+  belongs_to :user
+  belongs_to :topic
+
+  format_attribute :body
+  before_create { |r| r.forum_id = r.topic.forum_id }
+  after_create  :update_cached_fields
+  after_destroy :update_cached_fields
+
+  validates_presence_of :user_id, :body, :topic_id
+  attr_accessible :body	
+	
+  def editable_by?(user)
+    user && (user.id == user_id)
+  end
+  
+  protected
+    # using count isn't ideal but it gives us correct caches each time
+    def update_cached_fields
+      Forum.update_all ['posts_count = ?', Post.count(:id, :conditions => {:forum_id => forum_id})], ['id = ?', forum_id]
+      topic.update_cached_post_fields(self)
+    end
+end

data/app/models/topic.rb

@@ -0,0 +1,97 @@
+class Topic < ActiveRecord::Base
+  validates_presence_of :forum_id, :user_id, :title
+  before_create  :set_default_replied_at_and_sticky
+  before_update  :check_for_changing_forums
+  after_save     :update_forum_counter_cache
+  before_destroy :update_post_user_counts
+  after_destroy  :update_forum_counter_cache
+
+  belongs_to :forum
+  belongs_to :user
+  belongs_to :last_post, :class_name => "Post", :foreign_key => 'last_post_id'
+  
+  has_many :posts,     :order => "#{Post.table_name}.created_at", :dependent => :delete_all
+  has_one  :recent_post, :order => "#{Post.table_name}.created_at DESC", :class_name => 'Post'
+  
+  has_many :voices, :through => :posts, :source => :user, :uniq => true
+  belongs_to :replied_by_user, :foreign_key => "replied_by", :class_name => "User"
+
+  attr_accessible :title
+  attr_accessor :body
+	
+	def hit!
+    self.class.increment_counter :hits, id
+  end
+
+  def sticky?
+    sticky == 1
+  end
+
+  def views
+    hits
+  end
+
+  def paged?
+    posts_count > Post.per_page
+  end
+  
+  def last_page
+    [(posts_count.to_f / Post.per_page).ceil.to_i, 1].max
+  end
+
+  def editable_by?(user)
+    user && (user.id == user_id) 
+  end
+  
+  def update_cached_post_fields(post)
+    # these fields are not accessible to mass assignment
+    remaining_post = post.frozen? ? recent_post : post
+    if remaining_post
+      self.class.update_all(['replied_at = ?, replied_by = ?, last_post_id = ?, posts_count = ?', 
+        remaining_post.created_at, remaining_post.user_id, remaining_post.id, posts.count], ['id = ?', id])
+    else
+      self.destroy
+    end
+  end
+  
+  protected
+    def set_default_replied_at_and_sticky
+      self.replied_at = Time.now.utc
+      self.sticky   ||= 0
+    end
+
+    def set_post_forum_id
+      Post.update_all ['forum_id = ?', forum_id], ['topic_id = ?', id]
+    end
+
+    def check_for_changing_forums
+      old = Topic.find(id)
+      @old_forum_id = old.forum_id if old.forum_id != forum_id
+      true
+    end
+    
+    # using count isn't ideal but it gives us correct caches each time
+    def update_forum_counter_cache
+      forum_conditions = ['topics_count = ?', Topic.count(:id, :conditions => {:forum_id => forum_id})]
+      # if the topic moved forums
+      if !frozen? && @old_forum_id && @old_forum_id != forum_id
+        set_post_forum_id
+        Forum.update_all ['topics_count = ?, posts_count = ?', 
+          Topic.count(:id, :conditions => {:forum_id => @old_forum_id}),
+          Post.count(:id,  :conditions => {:forum_id => @old_forum_id})], ['id = ?', @old_forum_id]
+      end
+      # if the topic moved forums or was deleted
+      if frozen? || (@old_forum_id && @old_forum_id != forum_id)
+        forum_conditions.first << ", posts_count = ?"
+        forum_conditions       << Post.count(:id, :conditions => {:forum_id => forum_id})
+      end
+      # User doesn't have update_posts_count method in SB2, as reported by Ryan
+			#@voices.each &:update_posts_count if @voices
+      Forum.update_all forum_conditions, ['id = ?', forum_id]
+      @old_forum_id = @voices = nil
+    end
+    
+    def update_post_user_counts
+      @voices = voices.to_a
+    end
+end

data/app/views/forums/_form.html.erb

@@ -0,0 +1,21 @@
+<p style="float:right; margin-top:0">
+</p>
+
+<p id="forum_name">
+
+<table border="0" cellspacing="0" cellpadding="0" class="noborder nopad wide">
+  <td>
+    <label><%= 'Title'[:title_title] %></label><br />
+    <%= form.text_field :name, :class => "primary" %>
+  </td>
+  <td style="text-align:right">
+    <label><%= 'Position'[:position_title] %></label><br />
+    <%= form.text_field :position, :size => 5 %>
+
+  </td>
+</table>
+
+</p>
+<p id="forum_descripion">
+<label><%= 'Description'[:description_title] %></label><br />
+<%= form.text_area :description, :rows => 7 %></p>