cafe-grader-web Commit - r102:8fbc279e8875

cafe-grader-web

Commit `r102:8fbc279e8875` public

parent | child

Description:

[web] added message feature git-svn-id: http://theory.cpe.ku.ac.th/grader/web/trunk@194 6386c4cd-e34a-4fa8-8920-d93eb39b512e

Commit status:

[Not Reviewed]

References:

default

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r102:8fbc279e8875 - Sat, 26 Apr 2008 02:55:53 - 16 files changed: 279 inserted, 1 deleted

app/controllers/messages_controller.rb created 644 ¶ +73

@@ -0,0 +1,73
	1	+ class MessagesController < ApplicationController
	2	+
	3	+ before_filter :authenticate
	4	+
	5	+ verify :method => :post, :only => ['create'],
	6	+ :redirect_to => { :action => 'list' }
	7	+
	8	+ before_filter :only => ['console','show'] do \|controller\|
	9	+ controller.authorization_by_roles(['admin'])
	10	+ end
	11	+
	12	+ def list
	13	+ @user = User.find(session[:user_id])
	14	+ @messages = Message.find_all_sent_by_user(@user)
	15	+ end
	16	+
	17	+ def console
	18	+ @user = User.find(session[:user_id])
	19	+ @messages = Message.find_all_system_unreplied_messages
	20	+ end
	21	+
	22	+ def show
	23	+ @message = Message.find(params[:id])
	24	+ end
	25	+
	26	+ def create
	27	+ user = User.find(session[:user_id])
	28	+ @message = Message.new(params[:message])
	29	+ @message.sender = user
	30	+ if !@message.save
	31	+ render :action => 'list' and return
	32	+ else
	33	+ flash[:notice] = 'New message posted'
	34	+ redirect_to :action => 'list'
	35	+ end
	36	+ end
	37	+
	38	+ def reply
	39	+ user = User.find(session[:user_id])
	40	+ @message = Message.new(params[:r_message])
	41	+ @message.sender = user
	42	+ if !@message.save
	43	+ render :action => 'show' and return
	44	+ else
	45	+ flash[:notice] = 'Message replied'
	46	+ rep_msg = @message.replying_message
	47	+ rep_msg.replied = true
	48	+ rep_msg.save
	49	+ redirect_to :action => 'console'
	50	+ end
	51	+ end
	52	+
	53	+ protected
	54	+ def build_replying_message_hierarchy(user)
	55	+ @all_messages = {}
	56	+
	57	+
	58	+ # manually build replies hierarchy (to improve efficiency)
	59	+ [@messages, @replied_messages].each do \|collection\|
	60	+ collection.each do \|m\|
	61	+ @all_messages[m.id] = {:msg => m, :replies => []}
	62	+ end
	63	+ end
	64	+
	65	+ @all_messages.each do \|m\|
	66	+ rep_id = m.replying_message_id
	67	+ if @all_messages[rep_id]!=nil
	68	+ @all_messages[rep_id][:replies] << m
	69	+ end
	70	+ end
	71	+ end
	72	+
	73	+ end

app/helpers/messages_helper.rb created 644 ¶ +2

@@ -0,0 +1,2
	1	+ module MessagesHelper
	2	+ end

app/models/message.rb created 644 ¶ +60

@@ -0,0 +1,60
	1	+ class Message < ActiveRecord::Base
	2	+
	3	+ belongs_to :sender, :class_name => "User"
	4	+ belongs_to :receiver, :class_name => "User"
	5	+
	6	+ belongs_to :replying_message, :class_name => "Message"
	7	+
	8	+ # commented manually do it
	9	+ #
	10	+ #has_many :replied_messages, {
	11	+ # :class_name => "Message",
	12	+ # :foreign_key => "replying_message_id"
	13	+ #}
	14	+ #
	15	+
	16	+ attr_accessor :replied_messages
	17	+
	18	+ def self.find_all_sent_by_user(user)
	19	+ messages = user.messages
	20	+ replied_messages = user.replied_messages
	21	+ Message.build_replying_message_hierarchy messages, replied_messages
	22	+ return messages
	23	+ end
	24	+
	25	+ def self.find_all_system_unreplied_messages
	26	+ self.find(:all,
	27	+ :conditions => 'ISNULL(receiver_id) ' +
	28	+ 'AND (ISNULL(replied) OR replied=0)',
	29	+ :order => 'created_at')
	30	+ end
	31	+
	32	+ def self.build_replying_message_hierarchy(*args)
	33	+ # manually build replies hierarchy (to improve efficiency)
	34	+ all_messages = {}
	35	+
	36	+ args.each do \|collection\|
	37	+ collection.each do \|m\|
	38	+ all_messages[m.id] = m
	39	+ m.replied_messages = []
	40	+ end
	41	+ end
	42	+
	43	+ all_messages.each_value do \|m\|
	44	+ rep_id = m.replying_message_id
	45	+ if all_messages[rep_id]!=nil
	46	+ all_messages[rep_id].add_replied_message(m)
	47	+ end
	48	+ end
	49	+ end
	50	+
	51	+ def add_replied_message(m)
	52	+ if @replied_messages==nil
	53	+ @replied_messages = [m]
	54	+ else
	55	+ @replied_messages << m
	56	+ end
	57	+ @replied_messages
	58	+ end
	59	+
	60	+ end

app/views/messages/_message.html.haml created 644 ¶ +7

@@ -0,0 +1,7
	1	+ .message
	2	+ .stat
	3	+ = "#{message.sender.full_name} at #{message.created_at}"
	4	+ %div{:class => (!defined?(reply) or (reply==false)) ? "body" : "reply-body"}
	5	+ = simple_format(message.body)
	6	+ - if message.replied_messages.length!=0
	7	+ = render :partial => 'message', :collection => message.replied_messages, :locals => {:reply => true}

app/views/messages/_short_message.html.haml created 644 ¶ +6

@@ -0,0 +1,6
	1	+ %tr
	2	+ %td=h short_message.sender.full_name
	3	+ %td= "#{short_message.created_at}"
	4	+ %td=h truncate(short_message.body)
	5	+ %td
	6	+ = link_to "[reply]", :action => 'show', :id => short_message.id

app/views/messages/console.html.haml created 644 ¶ +13

@@ -0,0 +1,13
	1	+ = user_title_bar(@user)
	2	+
	3	+ %h1 Console: active messages
	4	+
	5	+ = link_to '[all messages]', :action => 'list_all'
	6	+
	7	+ %table
	8	+ %tr
	9	+ %th From
	10	+ %th When
	11	+ %th Message
	12	+ %th
	13	+ = render :partial => "short_message", :collection => @messages

app/views/messages/list.html.haml created 644 ¶ +14

@@ -0,0 +1,14
	1	+ = user_title_bar(@user)
	2	+
	3	+ %h3 Your Messages
	4	+
	5	+ - form_for 'message', nil, :url => { :action => 'create'} do \|f\|
	6	+ %p
	7	+ %b New message
	8	+ = submit_tag "Post"
	9	+ %br/
	10	+ = f.text_area :body, :rows => 5, :cols => 100
	11	+
	12	+ %hr/
	13	+
	14	+ = render :partial => 'message', :collection => @messages, :locals => {:reply => false}

app/views/messages/show.html.haml created 644 ¶ +13

@@ -0,0 +1,13
	1	+ %h3 Message
	2	+
	3	+ .message
	4	+ .stat
	5	+ = "#{@message.sender.full_name} at #{@message.created_at}"
	6	+ .body= simple_format(@message.body)
	7	+
	8	+ %h3 Your reply:
	9	+ - form_for 'r_message', nil, :url => { :action => 'reply'} do \|f\|
	10	+ = f.text_area :body, :rows => 5, :cols => 100
	11	+ = f.hidden_field :receiver_id, {:value => @message.sender_id }
	12	+ = f.hidden_field :replying_message_id, {:value => @message.id }
	13	+ = submit_tag "Post"

db/migrate/032_create_messages.rb created 644 ¶ +17

@@ -0,0 +1,17
	1	+ class CreateMessages < ActiveRecord::Migration
	2	+ def self.up
	3	+ create_table :messages do \|t\|
	4	+ t.column "sender_id", :integer
	5	+ t.column "receiver_id", :integer
	6	+ t.column "replying_message_id", :integer
	7	+ t.column "body", :text
	8	+ t.column "replied", :boolean # this is for efficiency
	9	+
	10	+ t.timestamps
	11	+ end
	12	+ end
	13	+
	14	+ def self.down
	15	+ drop_table :messages
	16	+ end
	17	+ end

test/fixtures/messages.yml created 644 ¶ +7

@@ -0,0 +1,7
	1	+ # Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
	2	+
	3	+ # one:
	4	+ # column: value
	5	+ #
	6	+ # two:
	7	+ # column: value

test/functional/messages_controller_test.rb created 644 ¶ +8

@@ -0,0 +1,8
	1	+ require File.dirname(__FILE__) + '/../test_helper'
	2	+
	3	+ class MessagesControllerTest < ActionController::TestCase
	4	+ # Replace this with your real tests.
	5	+ def test_truth
	6	+ assert true
	7	+ end
	8	+ end

test/unit/message_test.rb created 644 ¶ +8

@@ -0,0 +1,8
	1	+ require File.dirname(__FILE__) + '/../test_helper'
	2	+
	3	+ class MessageTest < ActiveSupport::TestCase
	4	+ # Replace this with your real tests.
	5	+ def test_truth
	6	+ assert true
	7	+ end
	8	+ end

app/helpers/application_helper.rb ¶ +2

                   user = User.find(session[:user_id])
                   if (user!=nil) and (user.admin?)
                     # admin menu
                     menu_items << "<b>Administrative task:</b> "
                     append_to menu_items, '[Announcements]', 'announcements', 'index'
+            +       append_to menu_items, '[Msg console]', 'messages', 'console'
                     append_to menu_items, '[Problem admin]', 'problems', 'index'
                     append_to menu_items, '[User admin]', 'user_admin', 'index'
                     append_to menu_items, '[User stat]', 'user_admin', 'user_stat'
                     #append_to menu_items, '[Graders]', 'graders', 'list'
                     append_to menu_items, '[Site config]', 'configurations', 'index'
                     menu_items << "<br/>"
                   end
                   # main page
                   append_to menu_items, '[Main]', 'main', 'list'
+            +     append_to menu_items, '[Messages]', 'messages', 'list'
                   append_to menu_items, '[Tasks]', 'tasks', 'list'
                   append_to menu_items, '[Submissions]', 'main', 'submission'
                   append_to menu_items, '[Test]', 'test', 'index'
                   append_to menu_items, '[Settings]', 'users', 'index'
                   append_to menu_items, '[Log out]', 'main', 'login'

app/models/user.rb ¶ +10

               class User < ActiveRecord::Base
                 has_and_belongs_to_many :roles
                 has_many :test_requests, :order => "submitted_at DESC"
+            +   has_many :messages,
+            +            :class_name => "Message",
+            +            :foreign_key => "sender_id",
+            +            :order => 'created_at DESC'
+            +
+            +   has_many :replied_messages,
+            +            :class_name => "Message",
+            +            :foreign_key => "receiver_id",
+            +            :order => 'created_at DESC'
+            +
                 belongs_to :site
                 validates_presence_of :login
                 validates_presence_of :full_name
                 validates_length_of :full_name, :minimum => 1

db/schema.rb ¶ +11 -1

               # to create the application database on another system, you should be using db:schema:load, not running
               # all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations
               # you'll amass, the slower it'll run and the greater likelihood for issues).
               #
               # It's strongly recommended to check this file into your version control system.
-            - ActiveRecord::Schema.define(:version => 31) do
+            + ActiveRecord::Schema.define(:version => 32) do
                 create_table "announcements", :force => true do |t|
                   t.string   "author"
                   t.text     "body"
                   t.boolean  "published"
                   t.datetime "created_at"
                 create_table "languages", :force => true do |t|
                   t.string "name",        :limit => 10
                   t.string "pretty_name"
                   t.string "ext",         :limit => 10
                 end
+            +   create_table "messages", :force => true do |t|
+            +     t.integer  "sender_id"
+            +     t.integer  "receiver_id"
+            +     t.integer  "replying_message_id"
+            +     t.text     "body"
+            +     t.boolean  "replied"
+            +     t.datetime "created_at"
+            +     t.datetime "updated_at"
+            +   end
+            +
                 create_table "problems", :force => true do |t|
                   t.string  "name",           :limit => 30
                   t.string  "full_name"
                   t.integer "full_score"
                   t.date    "date_added"
                   t.boolean "available"

public/stylesheets/application.css ¶ +28

@@ -188,6 +188,34
188		188
189	div.pub-info, div.pub-info p {	189	div.pub-info, div.pub-info p {
190	text-align: right;	190	text-align: right;
191	font-style: italic;	191	font-style: italic;
192	font-size: 9px;	192	font-size: 9px;
193	}	193	}
		194	+
		195	+ /******************
		196	+ [Messages
		197	+ ******************/
		198	+
		199	+ div.message {
		200	+ padding-top: 5px;
		201	+ padding-left: 10px;
		202	+ }
		203	+
		204	+ div.message div.body {
		205	+ border: 1px solid green;
		206	+ background: #eeffee;
		207	+ padding-left: 5px;
		208	+ }
		209	+
		210	+ div.message div.reply-body {
		211	+ border: 1px solid black;
		212	+ background: #ffeeee;
		213	+ padding-left: 5px;
		214	+ }
		215	+
		216	+ div.message div.stat {
		217	+ font-size: 10px;
		218	+ color: white;
		219	+ background: green;
		220	+ font-weight: bold;
		221	+ }

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r102:8fbc279e8875 - Sat, 26 Apr 2008 02:55:53 - 16 files changed: 279 inserted, 1 deleted

Sign in to your account

Commit r102:8fbc279e8875 public

Author

r102:8fbc279e8875 - Sat, 26 Apr 2008 02:55:53 - 16 files changed: 279 inserted, 1 deleted

Commit `r102:8fbc279e8875` public