0

標準のアカウント情報に加えて、配送情報を含むユーザーモデルがあります。

ユーザーの情報を更新できる3つの異なるビューがあります。

  • ウィザード
  • 設定
  • アカウント

私はRESTモデルを使用する必要があることを知っているので、Updateアクションを介してこれらすべての更新を実行しようとしています。1つの落とし穴は、ウィザードビューで、StripeAPIを呼び出す必要があることです。だから私はUpdateコントローラーでそのStripe呼び出しを持っています。しかし、ユーザーが[設定]ビューまたは[アカウント]ビューでアカウントを更新しようとすると、更新コントローラーがStripe APIを呼び出していますが、これは明らかに私が意図していることではありません。

だから私がやろうとしたのは、ウィザードのパラメーター(:total_value)の1つが渡されているかどうかを確認する条件を挿入し、渡されている場合はStripeに課金することです。そうでない場合は、ユーザーデータを更新するだけです。しかし、私はそれを機能させることができませんでした。それは単にユーザーを更新しているだけで、Stripeに課金していません。

これは、これらの更新を分離する正しい方向ですか?または、Usersコントローラー内でカスタムアクションを作成する必要がありますか?これはどのように作動しますか?

users_controller

def update
  if !params[:total_value].nil?
    if @user.update_attributes(params[:user])
      # flash[:success] = "Profile updated"
      sign_in @user
      # Value input by the user (in dollars)
      @userPrice = params[:user][:total_value]
      # Send full price in cents to Stripe
      @stripePrice = @userPrice.to_i * 100

      Stripe::Charge.create(
        :amount => @stripePrice,
        :currency => "usd",
        :card => params[:user][:stripe_card_token],
        :description => "Charge for service"
      )  
      redirect_to success_path
      flash[:success] = "Hurray!"
    else
      flash.now[:notice] = "Can not be saved, please enter information."
      render :new
    end
  elsif 
    if @user.update_attributes(params[:user])
      sign_in @user
      flash[:success] = "Profile updated"
      redirect_to account_path
    else
      flash[:error] = "Error"
    end
  end    
 end

ウィザード/show.html.erb

  <%= form_for(@user) do |f| %>
  <%= render 'shared/error_messages', object: f.object %>
        <%= f.text_field :total_value, placeholder: "0", :class => 'amount' %>

        <%= f.label :name %>
        <%= f.text_field :name, required: "required" %>

        <%= f.label :address_1, "Address" %>
        <%= f.text_field :address_1, required: "required" %>

        <%= f.label :address_2, "Address line 2" %>
        <%= f.text_field :address_2 %>

        <%= f.label :city, "City" %>
        <%= f.text_field :city, required: "required" %>

        <%= f.label :state, "State" %>
        <%= f.text_field :state, required: "required" %>

        <%= f.label :zip, "Zip code" %>
        <%= f.text_field :zip, required: "required" %>

        <%= f.hidden_field :stripe_card_token %>

        <%= label_tag :card_number, "Credit Card Number " %>  
        <%= text_field_tag :card_number, nil, name: nil %>  

        <%= label_tag :card_code, "Security Code on Card (CVV)" %>  
        <%= text_field_tag :card_code, nil, name: nil %>  
       <%= label_tag :card_month, "Card Expiration" %>  
        <%= select_month nil, {add_month_numbers_true: true}, {name: nil, id: "card_month"}%>  
        <%= select_year nil, {start_year: Date.today.year, end_year: Date.today.year+15}, {name: nil, id: "card_year"}%>  
        <%= f.submit "Update your shipping information", class: "btn btn-large btn-primary" %>

    <% end %>

users / shipping.html.erb

      <%= form_for(@user) do |f| %>
        <%= render 'shared/error_messages', object: f.object %>
            <%= f.label :name %>
            <%= f.text_field :name %>

            <%= f.label :address_1, "Address" %>
            <%= f.text_field :address_1 %>

            <%= f.label :address_2, "Address line 2" %>
            <%= f.text_field :address_2 %>

            <%= f.label :city, "City" %>
            <%= f.text_field :city %>

            <%= f.label :state, "State" %>
            <%= f.text_field :state %>

            <%= f.label :zip, "Zip code" %>
            <%= f.text_field :zip %>  

            <%= f.submit "Update your shipping information", class: "btn btn-large btn-primary" %>
      <% end %>

users / account.html.erb

  <%= form_for(@user) do |f| %>
    <%= render 'shared/error_messages', object: f.object %>
        <%= f.label :name %>
        <%= f.text_field :name %>

        <%= f.label :email %>
        <%= f.text_field :email %>

        <%= f.label :password %>
        <%= f.password_field :password %>

        <%= f.label :password_confirmation, "Confirm Password" %>
        <%= f.password_field :password_confirmation %>

        <%= f.submit "Save changes", class: "btn btn-large btn-primary" %>

  <% end %>
4

1 に答える 1

0

2 つの問題があります。まず、次のことです。

if !params[:total_value].nil?

する必要があります

unless params[:user][:total_value].nil? (個人的には好きunlessですif !...

ニシャントのコメントで述べたように。次にelsif、何もチェックせずに、その条件の終了があります。だけでelse十分です。

于 2013-01-13T06:08:16.813 に答える