Added the ability to edit configuration options. More are coming later.

Added contributors to the readme file.

Author: radar

README.markdown

@@ -118,4 +118,8 @@ Also you should set `STANDALONE = false` inside _config/environment.rb_.
 
 If you wish to use a different authentication system other than Restful Authentication then remove the user model (*app/models/user.rb*) and replace it with your own whilst not forgetting to `include Rboard::UserExtension`, the AuthenticatedSystem module (*lib/authenticated_system.rb*) and the AuthenticatedTestHelper module (*lib/authenticated_test_helper.rb*)
 
-
+## Contributors
+  
+  Thomas Sinclair
+  Bodaniel Jeanes
+  Tore Darell

TODO.html

@@ -1,4 +1,4 @@
-{\rtf1\ansi\ansicpg1252\cocoartf949\cocoasubrtf430
+{\rtf1\ansi\ansicpg1252\cocoartf949\cocoasubrtf460
 {\fonttbl\f0\fnil\fcharset0 LucidaGrande;}
 {\colortbl;\red255\green255\blue255;\red255\green3\blue0;\red255\green0\blue0;}
 \margl1440\margr1440\margb1800\margt1800\vieww10500\viewh9600\viewkind0
@@ -177,9 +177,11 @@ Visual representation of unread topics (including previously read topics with ne
 	\uc0\u9702 	Show all new(/newly replied to) posts and mark all forums read links\
 \pard\tx220\tx720\pardeftab720\li720\fi-720\sl380\ql\qnatural\pardirnatural
 \cf0 \expnd0\expndtw0\kerning0
-	\'95	Ability to mark forums as active (shown/hidden)\
-	\'95	Ability to mark forums as open (allows new posts)\
-	\'95    Customise how many sub forums are shown on the index page\
+	\'95	Ability to mark forums as active (shown/hidden) (mark[oz])\
+	\'95	Ability to mark forums as open (allows new posts) (mark[oz])\
+	\'95	\cf2 \expnd0\expndtw0\kerning0
+Customise how many sub forums are shown on the index page (mark[oz])\cf0 \expnd0\expndtw0\kerning0
+\
 \pard\tx560\tx1120\tx1680\tx2240\tx2800\tx3360\tx3920\tx4480\tx5040\tx5600\tx6160\tx6720\pardeftab720\sl380\sa220\ql\qnatural\pardirnatural
 
 \b \cf0 \expnd0\expndtw0\kerning0

TODO.rtf

@@ -24,7 +24,7 @@ def can_not_manage
     end
 
     def find_sections
-      @sections = ["categories", "forums", "users", "groups", "ranks", "themes"].select do |name|
+      @sections = ["categories", "forums", "users", "groups", "ranks", "themes", "configurations"].select do |name|
         current_user.can?("manage_#{name}")
       end
     end

app/controllers/admin/application_controller.rb

@@ -0,0 +1,19 @@
+class Admin::ConfigurationsController < Admin::ApplicationController
+  before_filter :find_configuration, :only => [:edit, :update]
+  
+  def index
+    @configurations = Configuration.all
+  end
+  
+  
+  def update_all
+    configurations = Configuration.find_all_by_key(params[:configurations].keys)
+    
+    configurations.each do |configuration|
+      configuration.update_attributes(params[:configurations][configuration.id])
+    end
+    flash[:notice] = t(:configuration_settings_updated)
+    redirect_to admin_configurations_path  
+  end
+  
+end

app/controllers/admin/configurations_controller.rb

@@ -0,0 +1,2 @@
+module Admin::ConfigurationsHelper
+end

app/helpers/admin/configurations_helper.rb

@@ -1,2 +1,12 @@
 class Configuration < ActiveRecord::Base
+  default_scope :order => :title
+  
+  def self.[](key)
+    Configuration.find_by_key!(key).value
+  rescue ActiveRecord::RecordNotFound
+    raise NotFound, t(:Configuration_not_found, :key => key)
+  end
+  
+  
+  class NotFound < Exception; end
 end

app/models/configuration.rb

@@ -0,0 +1,29 @@
+<%= link_to t(:Administration_Section), admin_root_path %> -> <%= t(:Configuration_Settings) %>
+<h2><%= t(:Configuration_Settings) %></h2>
+<%= link_to t(:New, :thing => "Group"), new_admin_group_path %>
+<% form_for :configurations, :url => update_all_admin_configurations_path, :html => { :method => :put } do %>
+  <table cellspacing='0' cellpadding='2' class='list-table' width='100%' rules='groups'>
+    <thead>
+      <tr>
+        <td>Title</td>
+        <td>Value</td>
+        <td>Submit</td>
+      </tr>
+    </thead>
+    <tbody>
+      <% for configuration in @configurations %>
+        <tr>
+          <td class='lalign'><strong><%= configuration.title %></strong><br>
+              <%= configuration.description %>
+          </td>
+          <% fields_for "configurations[]", configuration do |c| %>
+            <td>
+              <%= c.text_field :value %>
+            </td>
+          <% end %>
+          <td><%= submit_tag "Update" %></td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+<% end %>

app/views/admin/configurations/index.html.erb

@@ -3,7 +3,7 @@
     <h1><%= link_to(h(forum.title), forum_path(forum)) %></h1>
     <%= h(forum.description) %>
     <% if !forum.children.empty? %>
-      <br /><%= t(:Subforums) %>: <% forum.children.each do |forum| %> <%= link_to(h(forum.title), forum_path(forum)) %> <% end %>
+      <br /><%= t(:Subforums) %>: <% forum.children(Configuration['subforums_display'].to_i).each do |forum| %> <%= link_to(h(forum.title), forum_path(forum)) %> <% end %>
     <% end %>
   </td>
   <td align='center'><%= forum.topics.size %></td>

app/views/forums/_forum.html.erb

@@ -181,6 +181,9 @@
    :category_permission_denied => "You are not allowed to access that category",
    :Category_specific_permissions => "Category-specific permissions",
    :Child_of => "Child of",
+   :Configuration_not_found => "The configuration option {{key}} does not exist.",
+   :Configuration_Settings => "Configuration Settings",
+   :configuration_settings_updated => "The configuration settings were updated.",
    :Confirm_Password => "Confirm Password",
    :confirm_deletion => "Are you sure you want to delete this {{thing}}?",
    :confirm_user_ban => "Are you sure you want to ban this user?",
@@ -409,6 +412,8 @@
    :Sticky => "Sticky",
    :Sticky_this_topic => "Sticky this topic",
    :Subforums => "Subforums",
+   :subforums_display => "Number of Sub-forums to Display",
+   :subforums_display_description => "Limits the number of sub-forums listed underneath a forum listing. Defaults to 3.",
    :Subforums_For => "Subforums For",
    :Subscribe => "Subscribe",
    :Subscriptions => "Subscriptions",

config/locales/en.rb

@@ -15,6 +15,8 @@
     end
 
     admin.chronic 'chronic', :controller => 'chronic'
+    
+    admin.resources :configurations, :collection => { :update_all => :put }
     admin.resources :forums, :member => { :move_up => :put, :move_down => :put, :move_to_top => :put, :move_to_bottom => :put } do |forum|
       forum.resources :permissions
     end

config/routes.rb

@@ -0,0 +1,10 @@
+class AddDescriptionToConfigurations < ActiveRecord::Migration
+  def self.up
+    add_column :configurations, :description, :text
+    add_column :configurations, :title, :string
+  end
+
+  def self.down
+    
+  end
+end

db/migrate/20090607023038_add_description_to_configurations.rb

@@ -0,0 +1,8 @@
+class AddCanManageConfigurationsToPermissions < ActiveRecord::Migration
+  def self.up
+    add_column :permissions, :can_manage_configurations, :boolean, :default => false
+  end
+
+  def self.down
+  end
+end

db/migrate/20090607043542_add_can_manage_configurations_to_permissions.rb

@@ -9,7 +9,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20090606004748) do
+ActiveRecord::Schema.define(:version => 20090607043542) do
 
   create_table "banned_ips", :force => true do |t|
     t.string   "ip"
@@ -29,6 +29,8 @@
   create_table "configurations", :force => true do |t|
     t.string "key"
     t.string "value"
+    t.text   "description"
+    t.string "title"
   end
 
   create_table "edits", :force => true do |t|
@@ -159,6 +161,7 @@
     t.boolean "can_delete_topics",                :default => false
     t.boolean "can_manage_moderations",           :default => false
     t.boolean "can_read_others_private_messages", :default => false
+    t.boolean "can_manage_configurations",        :default => false
   end
 
   create_table "posts", :force => true do |t|

db/schema.rb

@@ -89,7 +89,7 @@ task :install => :environment do
                                         :can_see_category => true)
 
    puts "Generating some configuration options..."
-   Configuration.create(:key => "default_group_id", :value => registered_group.id)
+   Configuration.create(:key => "subforums_display", :title => I18n.t(:subforums_display), :value => 3, :description => I18n.t(:subforums_display_description))
 
    if SEARCHING
      puts "Configuring thinking sphinx..."

lib/tasks/install.rake

@@ -257,4 +257,8 @@ ul#admin-navlist, ul#moderator-navlist {
 
 .list-table td {
   text-align: center; 
+}
+
+.lalign { 
+  text-align:left !important;
 }

public/themes/blue/style.css

@@ -0,0 +1,5 @@
+subforums_display:
+  key: subforums_display
+  value: "1"
+  description: Limit the number of subforums displayed on the forums listing.
+  title: Number of Subforums to Display

spec/fixtures/configurations.yml

@@ -0,0 +1,13 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe Configuration do
+  fixtures :configurations
+  it "should be able to find a configuration with a valid key" do
+    Configuration['subforums_display'].should_not be_nil
+  end
+  
+  it "should not be able to find a configuration with an invalid key" do
+    lambda { Configuration['invalid_key'] }.should raise_error(Configuration::NotFound, "The configuration option invalid_key does not exist.")
+  end
+
+end