radiant-forum-extension 2.1.6 → 3.0.0.rc3

data/README.md

@@ -2,9 +2,9 @@
 
 This is a tidy but comprehensive forum implementation that plugs into radiant and supports both discussion forums and page comments. 
 
-This new (as I write) version 2 is a complete rewrite with a lot of internal clarification in preparation for rails 3. It's in late beta at the moment: bug fixes, documentation and packaging for public release.
+This new (as I write) version 2 is a complete rewrite with a lot of internal clarification in preparation for rails 3. It's only a release candidate at this stage: I'm working on bug fixes, documentation and packaging for public release. There is also a developing demo and documentation site that you can use as a starting point. Code is at [https://github.com/spanner/radiant-forum-demo](https://github.com/spanner/radiant-forum-demo) and the site will be public soon.
 
-The forum is designed to be pick-and-mixed with other radiant extensions, so it's very focused on being an excellent forum and doesn't do anything else. I hope you will find it's very good at what it does do:
+The forum is designed to be pick-and-mixed with other radiant extensions, so it's focused on being a good forum and doesn't do anything else. I hope you will find it's good at what it does do:
 
 * forums, topics, posts etc
 * page comments
@@ -17,14 +17,20 @@ The forum is designed to be pick-and-mixed with other radiant extensions, so it'
 
 I've tried to keep it simple and tidy both inside and out. This is helped by the fact that all the user-management is in the reader extension, so you should find this is easy to adapt and extend. The admin interface on the other hand is pretty basic: we expect most administration to happen through the public pages but there does need to be at least a dashboard to give a better overview. That will come soon.
 
-As far as I know this is the best forum software available for rails. I look forward to finding out that it's not.
+As far as I know this is the only decent forum software available for rails. If I can't find a better alternative, I will spin this off as a separate rails 3 engine that works alongside radiant but does not require it.
+
+## Demo and documentation
+
+You can find a standard radiant forum installation at [https://github.com/spanner/radiant-forum-demo](https://github.com/spanner/radiant-forum-demo). It serves many purposes: integration test target, reference platform, documentation and demo site and starting point for larger projects. There is some way to go on the documentation front but it should already serve as a useful starting point.
 
 ## Status
 
-This code has been surviving in the world for four or five years (and more, since some it originally came from Beast), but this version is about 75% new and you know what that means: new bugs. Github issues are always welcome.
+This code has been surviving in the world for five or six years (and more, since some it originally came from Beast), but this version is about 75% new and you know what that means: new bugs. Github issues are always welcome.
 
 ## Latest
 
+* Tidied up and made compatible with radiant 1
+* Group-visibility folded in
 * Inline editing with editable-for-interval setting
 * Punymce-based wysiwig editing.
 * Page comments data structure simplified. Migration required.
@@ -34,7 +40,7 @@ This code has been surviving in the world for four or five years (and more, sinc
 * Basic admin interface as well as readerland administration
 * Editable interval for messages
 
-This version is not wholly multi-site compatible at the moment, since it relies heavily on configuration items that are not yet site-scoped. They will be soon.
+This version is not multi-site compatible. It relies heavily on configuration items that are not yet site-scoped. That's next on the list after radiant 1 comes out.
 
 ## Still to do
 
@@ -46,13 +52,9 @@ This version is not wholly multi-site compatible at the moment, since it relies
 
 Radiant 0.9.2 (we're using the new configuration interface) with the [reader](http://github.com/spanner/radiant-reader-extension) extension. Reader has a few requirements of its own so it's best to install that one first, make sure it's testing clean and then install the forum.
 
-We also require [will_paginate](http://github.com/mislav/will_paginate/) and [paperclip](http://github.com/thoughtbot/paperclip/) as gems but you will already have those.
-
 	sudo rake gems:install
 	
 should get everything you need. 
-
-The forum is compatible with multi_site but you should use [sites](https://github.com/spanner/radiant-sites-extension) instead if you want forums and readers site-scoped.
 	
 ## Administration
 
@@ -62,6 +64,10 @@ The forum is easy to use and almost entirely separate from your page hierarchy.
 
 There is now a configuration panel for the forum and you shouldn't have to do any console-tinkering.
 
+## Changing forum text
+
+Every string used in the public-facing forum (and reader) pages comes from the locale file. They are labelled functionally rather than literally, with keys like `forum_extension.no_search_results`. To change the wording, either edit or override the locale.
+
 ## Layouts
 
 Forum pages have their own controllers but they are presented inside your normal radiant layouts. Use the configuration interface to choose the layout you want to use. Your layout can work in the usual way: all we do is define page parts for you to include with `<r:content part="something" />`. These are the parts available:
@@ -93,13 +99,11 @@ Have a look at the included sample layout for a starting point.
 
 ## Speed and caching
 
-This version of the forum is designed to work from behind radiant's main cache. There is no context-specific material in any of the pages or page parts visible to a normal browsing user and the comment and reply forms are normally retrieved by ajax calls after the pages have loaded. For that you need to use the provided (jquery-based) forum.js, or write your own equivalent (or for a more robust but less helpful forum, skip the ajax and let people click through to the comment or reply form).
-
-The intention is to make this behaviour configurable but at the moment I'm finding the forum pages too slow if uncached. Most of the wait comes from the shared_layouts/radius template machinery but there are some slow queries too. With optimisation a live forum should be viable.
+Forum and reader views are not cached, by default, but commented pages are. It is possible to cache the forum if you omit any dynamic content (such as the reply form), but at some cost to ease of use.
 
 ## Private discussion
 
-If you install the [reader_group](http://github.com/spanner/radiant-reader_group-extension) and [group_forum](http://github.com/spanner/radiant-group_forum-extension) extensions then your forums can be made visible only to designated groups, as with pages and other groupable items.
+Group functionality has recently been folded into the reader and forum extensions, so you don't need to install a whole tree of tiny changes any more. The interface hasn't quite settled down, but the functionality is solid: associate a forum with a group, and all of its contents are visible only to members of that group.
 
 ## Searching the forum
 
@@ -128,15 +132,15 @@ The forum comes with some jquery-based scripting to handle inline administration
 
 ## Smilies
 
-Included here is a redcloth extension to handle emoticons of the type inserted by the punymce toolbar we're using. Since we have control of both ends I've customised the set to make accidental emoticons less likely. There will soon be an option to disable them completely.
+Included here is a redcloth extension to handle emoticons of the type inserted by the punymce toolbar we're using. I've customised the set to make accidental emoticons less likely. There will soon be an option to disable them completely.
 
 ## Bugs
 
-In the short term, quite likely. [Github issues](http://github.com/spanner/radiant-forum-extension/issues), please, or for little things an email or github message is fine.
+In the short term, very likely. [Github issues](http://github.com/spanner/radiant-forum-extension/issues), please, or for little things an email or github message is fine.
 
 ## Author & Copyright
 
 * William Ross, for spanner. will at spanner.org
-* Originally based on dear old Beast, currently not visible at [http://beast.caboo.se](http://beast.caboo.se)
+* Originally based on dear old Beast
 * Copyright 2007-11 spanner ltd
 * released under the same terms as Rails and/or Radiant

data/Rakefile

@@ -1,19 +1,3 @@
-begin
-  require 'jeweler'
-  Jeweler::Tasks.new do |gem|
-    gem.name = "radiant-forum-extension"
-    gem.summary = %Q{Forum and Comment Extension for Radiant CMS}
-    gem.description = %Q{Nice clean forums and page comments for inclusion in your radiant site. Derived long ago from beast. Requires the reader extension and share_layouts.}
-    gem.email = "will@spanner.org"
-    gem.homepage = "http://github.com/spanner/radiant-forum-extension"
-    gem.authors = ["spanner"]
-    gem.add_dependency "radiant", ">= 0.9.0"
-    gem.add_dependency 'radiant-reader-extension', "~> 1.3.0"
-  end
-rescue LoadError
-  puts "Jeweler (or a dependency) not available. This is only required if you plan to package forum as a gem."
-end
-
 # In rails 1.2, plugins aren't available in the path until they're loaded.
 # Check to see if the rspec plugin is installed first and require
 # it if it is.  If not, use the gem version.

data/app/controllers/forum_base_controller.rb

@@ -4,41 +4,64 @@ class ForumBaseController < ReaderActionController
   radiant_layout { |c| Radiant::Config['forum.layout'] || Radiant::Config['reader.layout'] }
   before_filter :require_login_unless_public
   before_filter :establish_context
+  before_filter :require_visibility_to_reader
   helper :forum, :reader
 
 protected
 
+  def set_cache_header
+    if Radiant.config['forum.cached?']
+      expires_in (Radiant.config['forum.cache_duration'] || 60).to_i.minutes, :public => true
+    else
+      expires_now
+    end
+  end
+  
   def require_login_unless_public
     return false unless Radiant::Config['forum.public?'] || require_reader && require_activated_reader
   end
   
+  def require_visibility_to_reader
+    if @page && !@page.visible_to?(current_reader)
+      flash[:error] = t("page_not_public")
+      redirect_to reader_permission_denied_url
+      return false
+    end
+    
+    if @forum && !@forum.visible_to?(current_reader)
+      flash[:error] = "forum_not_public"
+      redirect_to reader_permission_denied_url
+      return false
+    end
+  end
+  
   def establish_context
     @reader = Reader.find(params[:reader_id]) unless params[:reader_id].blank?
     @topic = Topic.visible_to(current_reader).find(params[:topic_id]) unless params[:topic_id].blank?
     @forum = Forum.visible_to(current_reader).find(params[:forum_id]) unless params[:forum_id].blank?
     @page = Page.visible_to(current_reader).find(params[:page_id]) unless params[:page_id].blank?
   end
-
+  
   def redirect_to_post
     if (@post.page)
       redirect_to "#{@post.page.url}?#{WillPaginate::ViewHelpers.pagination_options[:param_name]}=#{@post.page_when_paginated}##{@post.dom_id}"
     elsif @post.first?
-      redirect_to forum_topic_path(@post.topic.forum, @post.topic)
+      redirect_to topic_path(@post.topic)
     else
       post_location = {WillPaginate::ViewHelpers.pagination_options[:param_name] => @post.page_when_paginated, :anchor => @post.dom_id}
-      redirect_to forum_topic_url(@post.topic.forum, @post.topic, post_location)
+      redirect_to topic_path(@post.topic, post_location)
     end
   end
 
   def redirect_to_topic
-    redirect_to forum_topic_path(@forum, @topic)
+    redirect_to topic_path(@topic)
   end
   
   def redirect_to_page_or_topic
     if @page
       redirect_to @page.url
     elsif @topic
-      redirect_to forum_topic_url(@topic.forum, @topic)
+      redirect_to topic_path(@topic)
     end
   end
 
@@ -51,15 +74,14 @@ protected
   def render_page_or_feed(template_name = action_name)
     respond_to do |format|
       format.html { 
-        expires_now
+        set_cache_header
         render :action => template_name 
       }
       format.rss { 
-        expires_now
-        render :action => template_name, :layout => 'feed' 
+        render :action => template_name, :layout => 'feed'
       }
       format.js { 
-        render :action => template_name, :layout => false 
+        render :action => template_name, :layout => false
       }
     end
   end
@@ -72,7 +94,7 @@ protected
     respond_to do |format|
       format.html { 
         expires_now
-        flash[:error] = t('topic_locked')
+        flash[:error] = t('forum_extension.topic_locked')
         redirect_to_page_or_topic 
       }
       format.js { render :partial => 'topics/locked' }

data/app/controllers/posts_controller.rb

@@ -9,12 +9,16 @@ class PostsController < ForumBaseController
   def index
     @term = params[:q]
     posts = Post.visible_to(current_reader)
-    posts = posts.containing(@term) unless @term.blank?
+    posts = posts.matching(@term) unless @term.blank?
     posts = posts.from_reader(@reader) if @reader
     posts = posts.in_forum(@forum) if @forum
     posts = posts.in_topic(@topic) if @topic
     @posts = posts.paginate(pagination_parameters)
-    render_page_or_feed
+    respond_to do |format|
+      format.html
+      format.js { render :partial => 'search_results' }
+      format.rss { render :layout => 'feed' }
+    end
   end
 
   def show
@@ -26,15 +30,24 @@ class PostsController < ForumBaseController
 
   def new
     unless @post.topic || @post.page
-      @forum ||= Forum.find_by_name(Radiant::Config['forum.default_forum'])
-      @post.topic = @forum.topics.new
+      if @forum ||= Forum.find_by_name(Radiant::Config['forum.default_forum'])
+        @post.topic = @forum.topics.new
+      else
+        @post.topic = Topic.new
+      end
     end
     respond_to do |format|
-      format.html { expires_now }
+      format.html {
+        @inline = false
+        render
+      }
       format.js {
+        @inline = true
         if @post.page
+          # page comment form is usually brought in by xhr so that page itself can be cached
           render :partial => 'pages/add_comment', :layout => false
         else
+          # this is a much less common case, but you can cache topic pages if you really want to
           render :partial => 'topics/reply', :layout => false
         end
       }
@@ -49,33 +62,43 @@ class PostsController < ForumBaseController
       format.js { render :partial => 'post' }
     end
   rescue ActiveRecord::RecordInvalid
-    flash[:error] = t("validation_failure")
+    flash[:error] = t("forum_extension.validation_failure")
     respond_to do |format|
       format.html { render :action => 'new' }
-      format.js { render :template => 'posts/new', :layout => false }
+      format.js { render :partial => 'form' }
     end
   end
 
   def edit
     respond_to do |format| 
-      format.html { expires_now }
-      format.js { render :template => 'posts/edit', :layout => false }
+      format.html {
+        @inline = false
+        render
+      }
+      format.js { 
+        @inline = true
+        if current_reader.is_moderator?(@post) || @post.editable_by?(current_reader)
+          render :partial => 'form' 
+        else
+          render :partial => 'ineditable' 
+        end
+      }
     end
   end
   
   def update
     @post.attributes = params[:post]
     @post.save!
-    Radiant::Cache.clear if @post.page
+    Radiant::Cache.clear if @post.page || (@post.topic && Radiant.config['forum.cached?'])
     respond_to do |format|
       format.html { redirect_to_post }
       format.js { render :partial => 'post' }
     end
   rescue ActiveRecord::RecordInvalid
-    flash[:error] = t("validation_failure")
+    flash[:error] = t("forum_extension.validation_failure")
     respond_to do |format|
       format.html { render :action => 'edit' }
-      format.js { render :template => 'posts/edit', :layout => false }
+      format.js { render :partial => 'form' }
     end
   end
 
@@ -89,13 +112,13 @@ class PostsController < ForumBaseController
   def destroy
     if @post.first?
       @post.topic.destroy
-      flash[:notice] = t("topic_removed")
+      flash[:notice] = t("forum_extension.topic_removed")
       redirect_to_forum
     else
       @post.destroy
       respond_to do |format|
         format.html { 
-          flash[:notice] = t("post_removed")
+          flash[:notice] = t("forum_extension.post_removed")
           redirect_to_page_or_topic 
         }
         format.js { render :partial => 'post' }

data/app/helpers/forum_helper.rb

@@ -1,19 +1,15 @@
-require 'sanitize'
-require "sanitize/config/forum"
-
 module ForumHelper
+  include ReaderHelper
+  mattr_accessor :forums_found
   
-  def feed_link(url)
-    link_to image_tag('/images/furniture/feed_14.png', :alt => t('rss_feed')), url, :class => "rssfeed"
+  def using_forums?
+    @forums_found = Forum.count > 1 unless defined? @forums_found
+    @forums_found
   end
-
-  def clean_textilize(text) # adding smilies to the default reader method
-    if text.blank?
-      ""
-    else
-      textilized = RedCloth.new(text, [ :hard_breaks ])
-      textilized.hard_breaks = true if textilized.respond_to?("hard_breaks=")
-      Sanitize.clean(textilized.to_html(:textile, :smilies), Sanitize::Config::FORUM)
+  
+  def feed_link(url)
+    if Radiant.config['forum.offer_rss?']
+      link_to image_tag('/images/furniture/feed_14.png', :alt => t('forum_extension.rss_feed')), url, :class => "rssfeed"
     end
   end
   
@@ -28,16 +24,27 @@ module ForumHelper
     end
   end
   
+  def link_to_forum(forum, options={})
+    title = options.delete(:title) || forum.title 
+    link_to title, forum_url(forum), options
+  end
+
+  def link_to_topic(topic, options={})
+    title = options.delete(:title) || topic.title 
+    link_to title, topic_url(topic), options
+  end
+
   def link_to_post(post, options={})
-    link_to post.holder.title, paginated_post_url(post), options
+    title = options.delete(:title) || post.holder.title 
+    link_to title, paginated_post_url(post), options
   end
 
   def edit_link(post)
-    link_to t('edit'), edit_topic_post_url(post.topic, post), :class => 'edit_post', :id => "edit_post_#{post.id}", :title => t("edit_post")
+    link_to t('forum_extension.edit'), edit_topic_post_url(post.topic, post), :class => 'edit_post', :id => "edit_post_#{post.id}", :title => t("forum_extension.edit_post")
   end
 
   def remove_link(post)
-    link_to t('delete'), topic_post_url(post.topic, post), :method => 'delete', :class => 'delete_post', :id => "delete_post_#{post.id}", :title => t("remove_post"), :confirm => t('really_remove_post')
+    link_to t('forum_extension.delete'), topic_post_url(post.topic, post), :method => 'delete', :class => 'delete_post', :id => "delete_post_#{post.id}", :title => t("remove_post"), :confirm => t('forum_extension.really_remove_post')
   end
   
   def friendly_date(datetime)

data/app/models/forum.rb

@@ -1,22 +1,26 @@
 class Forum < ActiveRecord::Base
   has_site if respond_to? :has_site
+  has_groups
+  
   has_many :topics, :dependent => :destroy
 
   default_scope :order => 'name ASC'
   named_scope :imported, :conditions => "old_id IS NOT NULL"
   validates_presence_of :name
   
-  # other extensions can attach chains here to limit access
-  def self.visible_to(reader)
-    self.scoped
-  end
-  
   def dom_id
     "forum_#{self.id}"
   end
   
-  def visible_to?(reader=nil)
-    return true if reader || Radiant::Config['forum.public?']
+  # chains the visible_to? method created during the has_groups call.
+  def visible_to_with_configuration?(reader=nil)
+    Rails.logger.warn "Forum#visible_to_with_configuration?"
+    return true if (reader || visible_by_default?) && visible_to_without_configuration?(reader)
+  end
+  alias_method_chain :visible_to?, :configuration
+  
+  def visible_by_default?
+    !!Radiant.config['forum.public?']
   end
 
 end

data/app/models/post.rb

@@ -11,11 +11,11 @@ class Post < ActiveRecord::Base
   accepts_nested_attributes_for :attachments, :allow_destroy => true
   validates_presence_of :reader, :body
 
+  before_save :update_search_text
   after_create :notify_holder_of_creation
   after_destroy :notify_holder_of_destruction
-  
+
   default_scope :order => "posts.created_at DESC"
-  
   named_scope :comments, :conditions => "page_id IS NOT NULL"
   named_scope :non_comments, :conditions => "page_id IS NULL"
   named_scope :imported, :conditions => "old_id IS NOT NULL"
@@ -32,18 +32,26 @@ class Post < ActiveRecord::Base
       self.length  # replacing a SQL shortcut that omits the distinct clause
     end
   end
-  named_scope :containing, lambda { |term|
-    { 
-      :conditions => ["posts.body LIKE :term OR topics.name LIKE :term", {:term => "%#{term}%"}],
-      :joins => "LEFT OUTER JOIN topics on posts.topic_id = topics.id"
+  # adapted from the usual scope defined in has_groups, since here visibility is set at the forum level
+  named_scope :visible_to, lambda { |reader| 
+    conditions = "topics.id IS NULL OR forums.id IS NULL OR pp.group_id IS NULL"
+    if reader && reader.group_ids.any?
+      ids = reader.group_ids
+      conditions = ["#{conditions} OR pp.group_id IN(#{ids.map{"?"}.join(',')})", *ids]
+    end
+    {
+      :joins => "INNER JOIN topics on posts.topic_id = topics.id LEFT OUTER JOIN forums on topics.forum_id = forums.id LEFT OUTER JOIN permissions as pp on pp.permitted_id = forums.id AND pp.permitted_type = 'Forum'",
+      :conditions => conditions,
+      :group => column_names.map { |n| self.table_name + '.' + n }.join(','),
+      :readonly => false
+    } 
+  }
+  named_scope :matching, lambda { |term|
+    {
+      :conditions => ["posts.search_text LIKE ?", "%#{stopped(term)}%"] 
     }
   }
 
-  # other extensions can attach chains here to limit access
-  def self.visible_to(reader)
-    self.scoped
-  end
-
   def self.in_forum(forum)
     in_topics(forum.topics)
   end
@@ -51,11 +59,19 @@ class Post < ActiveRecord::Base
   def holder
     page || topic
   end
+  
+  def title
+    holder.title if holder
+  end
     
   def comment?
     !!page
   end
   
+  def reply?
+    !comment? && !first?
+  end
+  
   def page_when_paginated
     holder.page_for(self)
   end
@@ -94,13 +110,21 @@ class Post < ActiveRecord::Base
   
   def editable_by?(reader=nil)
     return false unless reader
-    still_editable? && reader && (reader.id == reader_id)
+    still_editable? && reader.id == reader_id
   end
 
   def visible_to?(reader=nil)
-    return true if reader || Radiant::Config['forum.public?']
+    if topic && !topic.visible_to?(reader)
+      false
+    elsif page && !page.visible_to?(reader)
+      false
+    elsif !reader && !Radiant::Config['forum.public?']
+      false
+    else
+      true
+    end
   end
-  
+
   # so that page comments can be rendered from a radius tag
   def body_html
     if body
@@ -131,4 +155,19 @@ class Post < ActiveRecord::Base
     "post_#{self.id}"
   end
 
+private
+
+  # this is  rather crude, but it's database-agnostic, doesn't require 
+  # any external indexing, and works well enough for most purposes.
+  # for a more satisfactory search, tell sphinx to index the search_text field.
+
+  def update_search_text
+    self.search_text = self.class.stopped("#{self.title} #{self.body}")
+  end
+
+  def self.stopped(text="")
+    stops = I18n.t('forum_extension.stopwords').split.join('|')
+    text.downcase.gsub(/\b(#{stops})\b/, '').gsub(/(\W+)/, ' ')
+  end
+
 end

data/app/models/post_attachment.rb

@@ -40,7 +40,7 @@ class PostAttachment < ActiveRecord::Base
     :path => ":rails_root/:class/:id/:basename:no_original_style.:extension"      # attachments can only be accessed through the PostAttachments controller, in case file security is required
 
   attr_protected :file_file_name, :file_content_type, :file_file_size
-  validates_attachment_presence :file, :message => t('error.no_file')
+  validates_attachment_presence :file, :message => t('forum_extension.error.no_file')
   validates_attachment_content_type :file, :content_type => Radiant::Config["forum.attachment.content_types"].split(', ') if Radiant::Config.table_exists? && !Radiant::Config["forum.attachment.content_types"].blank?
   validates_attachment_size :file, :less_than => Radiant::Config["forum.attachment.max_size"].to_i.megabytes if Radiant::Config.table_exists? && Radiant::Config["forum.attachment.max_size"]
 

data/app/models/topic.rb

@@ -1,9 +1,7 @@
 class Topic < ActiveRecord::Base
   has_site if respond_to? :has_site
   has_comments
-  
   belongs_to :forum
-  belongs_to :replied_by, :class_name => 'Reader'
 
   validates_presence_of :name
   validates_uniqueness_of :old_id, :allow_nil => true
@@ -14,18 +12,33 @@ class Topic < ActiveRecord::Base
   named_scope :latest, lambda { |count|
     { :order => 'replied_at DESC', :limit => count }
   }
-
-  # other extensions can attach chains here to limit access
-  def self.visible_to(reader)
-    self.scoped
-  end
+  # adapted from the usual scope defined in has_groups, since here visibility is set at the forum level
+  named_scope :visible_to, lambda { |reader| 
+    conditions = "forums.id IS NULL OR pp.group_id IS NULL"
+    if reader && reader.group_ids.any?
+      ids = reader.group_ids
+      conditions = ["#{conditions} OR pp.group_id IN(#{ids.map{"?"}.join(',')})", *ids]
+    end
+    {
+      :joins => "LEFT OUTER JOIN forums on topics.forum_id = forums.id LEFT OUTER JOIN permissions as pp on pp.permitted_id = forums.id AND pp.permitted_type = 'Forum'",
+      :conditions => conditions,
+      :group => column_names.map { |n| self.table_name + '.' + n }.join(','),
+      :readonly => false
+    } 
+  }
 
   def dom_id
     "topic_#{self.id}"
   end
 
   def visible_to?(reader=nil)
-    return true if reader || Radiant::Config['forum.public?']
+    if forum && !forum.visible_to?(reader)
+      false
+    elsif !reader && !Radiant::Config['forum.public?']
+      false
+    else
+      true
+    end
   end
   
   def reader

data/app/views/admin/dashboard/_forum_dashboard.html.haml

@@ -0,0 +1,51 @@
+- include_stylesheet 'admin/forum_dashboard'
+
+#recent_comments.dashboard_module
+  .header
+    %h2
+      = t('forum_extension.pages_with_latest_comments')
+  .blockcontent
+    - latest = Page.last_commented(6)
+    - if latest.any?
+      - latest.each do |page|
+        .page
+          %h3
+            = link_to("#{icon} #{node_title}", edit_admin_page_url(page), :title => page.url, :class => 'page')
+            = link_to("→", page.path, :class => 'public')
+          %p.minor
+            =t('forum_extension.comment_count_from', :count => page.posts.count)
+            = link_to topic.replied_by.name, reader_url(topic.replied_by)
+            = friendly_date(topic.replied_at)
+    - else
+      %p.minor
+        =t('dashboard_extension.no_show')
+        
+#recent_topics.dashboard_module
+  .header
+    %h2
+      = t('forum_extension.latest_forum_activity')
+  .blockcontent
+    - latest = Topic.latest(4)
+    - if latest.any?
+      - latest.each do |topic|
+        - post = topic.posts.last
+        %h3
+          = link_to_post(post, :class => 'topic')
+          %span.context
+            - if topic.has_replies?
+              = t('forum_extension.new_reply_from')
+              = link_to topic.replied_by.name, reader_url(topic.replied_by)
+              = friendly_date(topic.replied_at)
+            - else
+              = t('forum_extension.started_by')
+              = link_to topic.reader.name, reader_url(topic.reader)
+              = friendly_date(topic.created_at)
+        %p.excerpt
+          = truncate_words(scrub_html(post.body), 16)
+      %p.minor
+        =t('forum_extension.administer_by_browsing')
+    - else
+      %p.minor
+        =t('dashboard_extension.no_show')
+        
+