cafe-grader-web Commit - r18:d8813c40713d

cafe-grader-web

parent | child

Description:

add_email_to_user, fix empty problem for_in_place_editing git-svn-id: http://theory.cpe.ku.ac.th/grader/web/trunk@42 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

r18:d8813c40713d - Fri, 08 Feb 2008 07:41:41 - 5 files changed: 42 inserted, 4 deleted

db/migrate/012_add_email_to_users.rb created 644 ¶ +9

			@@ -0,0 +1,9
		1	+ class AddEmailToUsers < ActiveRecord::Migration
		2	+ def self.up
		3	+ add_column :users, :email, :string
		4	+ end
		5	+
		6	+ def self.down
		7	+ remove_column :users, :email
		8	+ end
		9	+ end

app/controllers/users_controller.rb ¶ +2 -1

                 before_filter :authenticate
                 verify :method => :post, :only => [:chg_passwd],
                        :redirect_to => { :action => :index }
                 in_place_edit_for :user, :full_name
-            -   in_place_edit_for :user, :alias
+            +   in_place_edit_for :user, :alias_for_editing
+            +   in_place_edit_for :user, :email_for_editing
                 def index
                   @user = User.find(session[:user_id])
                 end
                 def chg_passwd

app/models/user.rb ¶ +25 -1

                 end
                 def admin?
                   self.roles.detect {|r| r.name == 'admin' }
                 end
-            - #  protected
+            +   def email_for_editing
+            +     if self.email!=nil
+            +       self.email
+            +     else
+            +       "unknown"
+            +     end
+            +   end
+            +
+            +   def email_for_editing=(e)
+            +     self.email=e
+            +   end
+            +
+            +   def alias_for_editing
+            +     if self.alias!=nil
+            +       self.alias
+            +     else
+            +       "unknown"
+            +     end
+            +   end
+            +
+            +   def alias_for_editing=(e)
+            +     self.alias=e
+            +   end
+            +
+            +   protected
                   def encrypt_new_password
                     return if password.blank?
                     self.salt = (10+rand(90)).to_s
                     self.hashed_password = encrypt(password,salt)
                   end

app/views/users/index.html.haml ¶ +4 -1

                   %td.uinfo= @user.login
                 %tr
                   %th.uinfo Full name
                   %td.uinfo= in_place_editor_field :user, 'full_name', {}, :rows => 1
                 %tr
                   %th.uinfo Alias
-            -     %td.uinfo= in_place_editor_field :user, 'alias', {}, :rows => 1
+            +     %td.uinfo= in_place_editor_field :user, 'alias_for_editing', {}, :rows => 1
+            +   %tr
+            +     %th.uinfo E-mail
+            +     %td.uinfo= in_place_editor_field :user, 'email_for_editing', {}, :rows => 1
                 %tr
                   %th.uinfo Password
                   %td.uinfo
                     - form_tag :action => 'chg_passwd', :method => 'post' do
                       %table
                         %tr

db/schema.rb ¶ +2 -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 => 11) do
+            + ActiveRecord::Schema.define(:version => 12) do
                 create_table "languages", :force => true do |t|
                   t.string "name",        :limit => 10
                   t.string "pretty_name"
                   t.string "ext",         :limit => 10
                 end
                 create_table "users", :force => true do |t|
                   t.string "login",           :limit => 10
                   t.string "full_name"
                   t.string "hashed_password"
                   t.string "salt",            :limit => 5
                   t.string "alias"
+            +     t.string "email"
                 end
                 add_index "users", ["login"], :name => "index_users_on_login", :unique => true
               end

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

	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