ユーザー編集プロフィールのトラブル

Question

私の上司のウェブサイトはRubyOnRailsアプリです。私の上司は、ユーザーが自分のプロフィールを編集する方法（パスワードの変更、電子メールの変更など）を依頼しました。

すでにユーザー作成システムと認証システムがありました。

プロファイル、ビュー、モデルを編集するための新しいアクションを作成しました。

編集システムを機能させるためにさまざまな方法を試しましたが、うまくいきませんでした。 プロセスが完了して（フォームを送信してコントローラのインデックスにリダイレクトする）、ユーザーの値で何も編集されないことがあります。ここ

はError Logある：ここ

Processing AccountController#edit (for MyIP at Today_Date Current_Hour) [PUT] 
    Parameters: { 
     "user_edit"=>{ 
      "city"=>"My City", 
      "country"=>"My Country", 
      "newsletter_subscription"=>"0", 
      "password_confirmation"=>"[FILTERED]", 
      "street"=>"My Street", 
      "last_name"=>"My Last Name", 
      "password"=>"[FILTERED]", 
      "login"=>"My.Login", 
      "first_name"=>"My First Name", 
      "email"=>"myemail@mydomain.com" 
     }, 
     "commit"=>"confirm_edit", 
     "action"=>"edit", 
     "_method"=>"put", 
     "authenticity_token"=>"My_Too_Long_Authenticity_Token", 
     "local"=>"fr", 
     "controller"=>"account" 
    } 

NoMethodError (Attempt to call private method): 
    app/controllers/account_controller.rb:90:in `edit´ 
    thin (1.2.7) lib/thin/connection.rb:76:in `pre_process´ 
    thin (1.2.7) lib/thin/connection.rb:74:in `catch´ 
    thin (1.2.7) lib/thin/connection.rb:74:in `pre_process´ 
    thin (1.2.7) lib/thin/connection.rb:57:in `process´ 
    thin (1.2.7) lib/thin/connection.rb:42:in `receive_data´ 
    eventmachine (0.12.10) lib/eventmachin.rb:256:in `run_machine´ 
    eventmachine (0.12.10) lib/eventmachin.rb:256:in `run´ 
    thin (1.2.7) lib/thin/backends/base.rb:57:in `start´ 
    thin (1.2.7) lib/thin/server.rb:156:in `start´ 
    thin (1.2.7) lib/thin/controllers/controller.rb:80:in `start´ 
    thin (1.2.7) lib/thin/runner.rb:177:in `send´ 
    thin (1.2.7) lib/thin/runner.rb:177:in `run_command´ 
    thin (1.2.7) lib/thin/runner.rb:143:in `run!´ 
    thin (1.2.7) bin/thin:6 
    /usr/local/bin/thin:19:in `load´ 
    /usr/local/bin/thin:19 

Rendering /var/rails/MySite/releases/MyApp/public/500.html 
(500 Internal Server Error) 

は、コントローラの一部である：ここ

1 require "erb" 
2 require 'net/http' 
3 require 'net/https' 
4 class AccountController < ApplicationController 
5  include ERB::Util 
6  layout "store" 
7  
68  def edit_profile 
69   @page_title = t(:account_edit_profile) 
70   if logged_in? 
71    if !session.nil? && session[:dump_user] 
72     @user = retrieveFromSession(:dump_user) 
73    else 
74     @user = current_user 
75    end 
76    begin 
77     rescue ActiveRecord::RecordNotFound 
78      record_not_found 
79      redirect_to(:controller=>:account, :action => :index) 
80    end 
81   else 
82    flash[:notice] = t(:permission_denied) 
83    redirect_to(:controller=>:account, :action => :index) 
84   end 
86  end 
85  
86  def edit 
87  @page_title = t(:account_edit_profile) 
88   if logged_in? 
89    if is_active_module? :symbol_private 
90     @user = User.find_by_id!(current_user.id).update(params[:user_edit]) 
91     if @user.valid? 
92      session[:dump_user] = Marshal.dump(@user) 
93      redirect_to(:controller=>:account, :action => :edit_confirm) 
94     else 
95      render :action => 'edit_profile'; 
96     end 
97    else 
98     flash[:notice] = t(:permission_denied) 
99     redirect_to(:controller=>:account, :action => :profile) 
100   end 
101  else 
102   flash[:notice] = t(:permission_denied) 
103   redirect_to(:controller=>:account, :action => :profile) 
104  end 
105 end 
106 
107 def edit_confirm 
108  if logged_in? 
109   begin 
110    user = retrieveFromSession(:dump_user) 
111    session[:dump_user] = Marshal.dump(user) 
112    if is_active_module? :symbol_private 
113     finalize_edit 
114    end 
115    rescue ActiveRecord::RecordNotSaved 
116     flash[:notice] = t :something_went_wrong 
117     redirect_to(:controller=>:account, :action => :profile) 
118   end 
119  else 
120   flash[:notice] = t(:permission_denied) 
121   redirect_to(:controller=>:account, :action => :index) 
122  end 
123 end 
124 
125 def finalize_edit 
126  user=(!user.nil?) ? user : retrieveFromSession(:dump_user) 
127  user.save! 
128   flash[:notice] = t(:profil_successfully_edited) 
129   session[:dump_user]=nil 
130   redirect_to(:controller=>:account, :action => :profile) 
131 end 
132 
133 end 

"はedit_profile" するための図である。

1 <div id="title_store"> 
2  <%= t(:account_edit_profile)%> 
3 </div> 
4 <div id="paragraph_store"> 
5  <div> 
6   <% 
7    labelled_form_for :user_edit, @user, :url =>{:action =>"edit"}, 
8    :html=>{:multipart=>true, :method=>:put} do |@form| 
9   %> 
10    <% field_set_tag t(:edit_profile) do %> 
11     <%= @form.text_field :login, :value=>(@user.login), :disabled=>"disabled"%> 
12     <%= @form.text_field :email, :value=>(@user.email) %> 
13     <%= @form.password_field :password %> 
14     <%= @form.password_field :password_confirmation %> 
15     <%= @form.text_field :first_name, :value=>(@user.first_name) %> 
16     <%= @form.text_field :last_name, :value=>(@user.last_name) %> 
17     <%= @form.text_field :street, :value=>(@user.street) %> 
18     <%= @form.text_field :city, :value=>(@user.city) %> 
19     <%= @form.text_field :country, :value=>(@user.country) %> 
20     <%= @form.check_box :newsletter_subscription, :checked => params[:newsletter_subscription]%> 
21     <div id="envoie"> 
22      <div id="retour"> 
23       <%= link_to t(:cancel), :controller=>:account, :action=>:profile %> 
24      </div> 
25      <div class="submit"> 
26       <%= submit_tag t(:confirm_edit) %> 
27      </div> 
28     </div> 
29    <%end%> 
30   <%end%> 
31  </div> 
32 </div> 

ビューが内部でありますhtml/head/bodyとviのレンダリングに必要なその他の項目があるレイアウトのew。

未使用のため、ヘルパーを投稿していません。

モデルに関して、私はそれを作っていませんでした。私はそれを理解していません。フォームを送信するために必要な値がある部分だけを理解して保存します。ここで

がモデルである：

1  require 'digest/sha1' 
2  require 'shared/element' 
3  require 'exceptions' 
4  require 'rubygems' 
5  require 'net/ldap' 
6 
7  class UserEdit < ActiveRecord::Base 
8   has_and_belongs_to_many :roles 
9   belongs_to :abonnement 
10  belongs_to :secretariat 
11  has_many :news 
12  has_many :items 
13  has_many :absences 
14  has_many :opener, :foreign_key => "opened_by_id", :class_name =>'Task' 
15  has_many :closer, :foreign_key => "closed_by_id",:class_name =>'Task' 
16  has_many :attributor, :foreign_key => "attributed_to_id",:class_name =>'Task' 
17  has_many :static_pages 
18   
19  attr_accessible :login,:password,:password_confirmation, :email,:is_ldap_user, :secretariat_id, :newsletter_subscription 
20  attr_accessible :first_name, :last_name, :street, :city, :country 
21   
22  validates_presence_of  :first_name 
23  validates_presence_of  :last_name 
24  validates_presence_of  :street 
25  validates_presence_of  :city 
26  validates_presence_of  :country 
27  
28  validates_presence_of  :password,     :if => :password_changed? 
29  validates_presence_of  :password_confirmation,  :if => :password_changed? 
30  validates_length_of  :password, :within => 4..40, :if => :password_changed? 
31  validates_confirmation_of :password,     :if => :password_changed? 
32  validates_length_of  :login, :within => 3..20 
33  validates_length_of  :email, :maximum => 50 
34  validates_uniqueness_of :login, :case_sensitive => false 
35  validates_uniqueness_of :email, :case_sensitive => false, :if => Proc.new{|u| configatron.email_uniqueness} 
36  validates_format_of :email, :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i 
37 
38  
39  before_save :encrypt_password, :fix_is_ldap_user_attribute 
40  before_destroy :check_godness 
41   
42  extend Element 
43   
44  def self.authenticate(login, password) 
45   u = find_by_login(login) 
46   if !u 
47    raise WrongLoginPassword 
48   end 
49    
50   if(is_active_module?:ldap) 
51    unless u.authenticated?(password) || is_in_ldap?(login,password) 
52     raise LDAPOnlyUser 
53    end 
54   else 
55    raise WrongLoginPassword unless !u.nil? && u.authenticated?(password) 
56   end 
57    
58   if !u.activated_at 
59    raise InactiveAccount 
60   elsif u.roles.find_by_title('banned') 
61    raise BannedAccount 
62   else 
63    return u 
64   end 
65  end 
66   
67  def self.encrypt(password, salt) 
68   Digest::SHA1.hexdigest("--#{salt}--#{password}--") 
69  end 
70   
71  def encrypt(password) 
72   self.class.encrypt(password, salt) 
73  end 
74   
75  def authenticated?(password) 
76   crypted_password == encrypt(password) 
77  end 
78   
79  def remember_token? 
80   remember_token_expires_at && Time.now.utc remember_token_expires_at 
81  end 
82   
83  def remember_me 
84   self.remember_token_expires_at = 2.weeks.from_now.utc 
85   self.remember_token   = encrypt("#{email}--#{remember_token_expires_at}") 
86   save(false) 
87  end 
88   
89  def forget_me 
90   self.remember_token_expires_at = nil 
91   self.remember_token   = nil 
92   save(false) 
93  end 
94   
95  def self.content_columns 
96   @content_columns = super.delete_if do |value| 
97    (value.human_name.downcase == "crypted password") || 
98    (value.human_name.downcase == "salt") || 
99    (value.human_name.downcase =~ /.*token.*/) || 
100    (value.human_name.downcase == 'password reset code') || 
101    (value.human_name.downcase == 'is ldap user') || 
102    (value.human_name.downcase == 'updated at') || 
103    (value.human_name.downcase == "activation code") 
104   end 
105  end 
106   
107  def activate 
108   @activated = true 
109   self.activated_at=Time.now.utc 
110   self.activation_code=nil 
111   true 
112  end 
113   
114  def recently_activated? 
115   @activated 
116  end 
117   
118  def recently_lost_password? 
119   @lost_password 
120  end 
121   
122  def password_reset? 
123   @password_reset 
124  end 
125   
126  def is_admin? 
127   (self.roles.map{ |role| role.title.downcase}.include? 'admin') || is_god? 
128  end 
129   
130  def is_god? 
131   self.roles.map{ |role| role.title.downcase}.include? 'god' 
132  end 
133   
134  def is_secretary? 
135   self.roles.map{ |role| role.title.downcase}.include? 'secretary' 
136  end 
137   
138  def make_password_reset_code 
139   @lost_password = true 
140   self.password_reset_code = Digest::SHA1.hexdigest(Time.now.to_s.split(//).sort_by {rand}.join) 
141  end 
142   
143  def reset_password 
144   self.password = Digest::SHA1.hexdigest(Time.now.to_s.split(//).sort_by {rand}.join).slice(0,8) 
145   self.password_confirmation = self.password 
146   self.password_reset_code = nil 
147   @password_reset = self.password 
148   self.save!  
149  end 
150   
151  def available_roles 
152   if (self.roles.map{ |role| role.title.downcase}.include? 'god') 
153    Role.find(:all) 
154   else 
155    Role.find(:all, 
156     :conditions=>["title != ?",'god']) 
157   end 
158  end 
159  
160  protected 
161  
162  def encrypt_password 
163   return if password.blank? 
164   self.salt = Digest::SHA1.hexdigest("--#{Time.now.to_s}--#{login}--") if new_record? 
165   self.crypted_password = encrypt(password) 
166  end 
167   
168  def password_required? 
169   if(is_active_module?(:ldap) && is_ldap_user == true) 
170    return false 
171   else 
172    (crypted_password.blank? || !password.blank?) 
173   end 
174  end 
175   
176  def make_activation_code 
177   self.activation_code = Digest::SHA1.hexdigest(Time.now.to_s.split(//).sort_by {rand}.join) 
178  end 
179   
180  private 
181  def check_godness 
182   raise UnautorizedDeletion if is_god? 
183  end 
184   
185  def is_valid_secretariat? 
186   return true unless is_active_module? :ipmc 
187   return true if self.secretariat_id.nil? 
188   if Secretariat.find_by_id(self.secretariat_id).nil? 
189    errors.add :secretariat_id, :invalid 
190    false 
191   else 
192    true 
193   end 
194  end 
195   
196  def fix_is_ldap_user_attribute 
197   self.is_ldap_user = true if is_ldap_user.nil? && is_active_module?(:ldap) 
198  end 
199  
200  def self.is_in_ldap?(login ,password) 
201   ldap = Net::LDAP.new 
202   ldap.host = configatron.ldap.host 
203   ldap.port = configatron.ldap.port 
204   ldap.encryption :simple_tls if configatron.ldap.encryption 
205   ldap.auth configatron.ldap.base_dn, configatron.ldap.password 
206   begin 
207    result = ldap.bind_as(
208        :base => configatron.ldap.base_dn, 
209        :filter=>"(uid=#{login})", 
210        :password => password 
211    ) 
212    rescue SystemExit 
213     return false 
214   end 
215   if result 
216    return true 
217   else 
218    return false 
219   end 
220  end 
221   
222 end 

私は完全に失われていると私は間違っているのか理解していません。モデルの90％を理解していないし、ウェブ上で私を助けることができるものが見つからなかった。

私のエラーが表示されたり、問題を解決する方法がわかっている場合、またはユーザーが自分のプロフィールを編集できるようにする簡単な方法があれば、助けてください。

Answer 1

[OK]を、私は私の編集システムの作業を行うための方法を見つけました。

私のエラーは、私が100％理解できなかったシステムを使用しようとしていたことでした。

最後に私はそれを簡単に作った。コントローラーで

：私は<%= render :partial => "form"%>で_form.rhmlと呼ばれるビューで

 
    def edit_profile 
     if logged_in? 
      @page_title = t(:account_edit_profile) 
      @user = current_user 
      if request.post? 
       if @user.update_attributes(params[:user_edit]) 
        flash[:notice] = t(:profil_succesfully_edited) 
        redirect_to :controller => :account, :action => :profile 
       end 
      end 
     else 
      flash[:notice] = t(:must_logged_in) 
      redirect_to :controller => :index 
     end 
    end 

。あなたの答えのために

<%= error_messages_for :user_edit %> 
<% form_for :user_edit, @user, :html=>{:multipart=>true, :method=>:post} do |f| %> 
    <p> 
     <%= f.label t(:login) %> 
     <%= f.text_field :login %> 
    </p> 
    <p> 
     <%= f.label t(:email) %> 
     <%= f.text_field :email %> 
    </p> 
    <p> 
     <%= f.label t(:first_name) %> 
     <%= f.text_field :first_name %> 
    </p> 
    <p> 
     <%= f.label t(:last_name) %> 
     <%= f.text_field :last_name %> 
    </p> 
    <p> 
     <%= f.label t(:street) %> 
     <%= f.text_field :street %> 
    </p> 
    <p> 
     <%= f.label t(:city) %> 
     <%= f.text_field :city %> 
    </p> 
    <p> 
     <%= f.label t(:country) %> 
     <%= f.text_field :country %> 
    </p> 
    <p class="checkbox"> 
     <%= f.check_box :newsletter_subscription, :checked => params[:newsletter_subscription] %> 
     <%= f.label t(:newsletter_subscription) %> 
    </p> 
    <div id="envoie"> 
     <div id="retour"> 
      <%= link_to t(:cancel), :controller=>:account, :action=>:profile %> 
     </div> 
     <div class="submit"> 
      <%= f.submit t(:confirm_edit) %> 
     </div> 
    </div> 
<% end %> 

ありがとう： はまた、このフォームで私はそれを書きました。

そして、私はこのポストははい、私は知っているが、それはビューからフォームによって呼び出されるモデルだI.

Answer 2

私が見る最初の奇妙なことは、Userモデルとモデルを使用していることです。あなたが投稿したコードにUserEditという名前が付いています。

Answer 3

第二に奇妙なことは、あなたが持っている自分の形である：do |@form| ないに@ -variableする必要があること。 do |f|

また、例えば、:value=>(@user.login),のような明示的に値を渡す必要はありません.-フォームが@userのためのものであることを既に指定しており、フォームは自動的にそれらの値を取得する必要がありますそれだけで。すべてを単純化してみて、それが役立つかどうかを見てください（少なくとも可読性に役立ちます）。

次に、バックトレースエラーの原因となっている行に。実際のエラーなどが原因となっている操作を参照するために、2つのラインにそれを分割してみてください。

@user = User.find_by_id(current_user.id) 
@user.update(params[:user_edit]) 

しかし...本当に、あなただけCURRENT_USERと同じであるユーザーオブジェクトを探しているなら...現在のユーザーを使用して検索をスキップしてみませんか？結局、あなたがcurrent_userを持っていれば、ユーザーオブジェクトをリロードする必要はありません。

だから私は完全にそのラインを交換するお勧めします：

@user = current_user 
@user.update(params[:user_edit])