Как исправить мою вложенную модель перенаправления формы - PullRequest
Новая
       11

Как исправить мою вложенную модель перенаправления формы

1 голос
/ 28 августа 2011

У меня есть процесс регистрации, который идет: 

user = User.new
user.email = ...
user.password = ...
user.profile = Profile.new
user.profile.save
user.save

В моем приложении я запускаю процесс регистрации на домашней странице через InfoController для обработки статических страниц. Поскольку форма на главной странице смешивает модели user и profile, я использую вложенную форму модели. Однако, когда я отправляю форму, я перенаправлен не туда. Может кто-нибудь помочь мне понять, что я сделал не так?

Файл Routes.rb: 

match '/login' => "sessions#new", :as => "login"
match '/signup' => 'profiles#new', :as => "signup"
match 'skip/signup', :to => 'info#signupskip'
match 'skip/profiles/new', :to => 'profiles#newskip'

root :to => 'info#home'
root :to => "questions#index"
resources :users
resources :profiles
resources :info
resource :session
resources :session

ProfilesController: 

class ProfilesController < ApplicationController
  before_filter :authenticate

  def new
    @profile = Profile.new
  end

  def name
    puts "#{user.profile.first_name} #{user.profile.last_name}"
  end

  def create
    @profile = Profile.new(params[:profile])
    if @profile.save
      redirect_to profile_path, :notice => 'User successfully added.'
    else
      render :action => 'new'
    end
  end
  ...

UsersController: 

class UsersController < ApplicationController
  before_filter :authenticate, :only => [:edit, :update]

  def new
    @user = User.new
  end

  def index
    @user = User.all
  end

  def create
    @user = User.new(params[:user])
    if @user.save
      redirect_to profile_path, :notice => 'User successfully added.'
    else
      render :action => 'new'
    end
  end

И, наконец, форма с вложенными моделями: 

  <%= form_for(:profile, :url => 'signup', :html => {:id => 'homepage'}) do |f| %>
    <p class="hometext">I'm&nbsp;</p>
    <div>
      <%= f.label :first_name, :placeholder => 'First name' %>
      <%= f.text_field :first_name, :size=> 8, :id => "profile[first_name]" %>
    </div>
    <div>
      <label for="profile[last_name]">Last name</label>
      <%= f.text_field :last_name, :size=> 8, :id => "profile[last_name]" %>
    </div>
    <%= f.fields_for :user do |f| %>
    <p class="hometext">.&nbsp;My&nbsp;email&nbsp;is&nbsp;
      <div>
        <label for="user[email]">Email</label>
        <%= f.text_field :email, :size=> 13, :id => "user[email]" %>
      </div>
    <% end %>
    <p class="hometext">.&nbsp;I want to&nbsp;</p>
    <div>
      <label for="user[goal]">ex: be President</label>
      <%= f.text_field :goal, :size=> 13, :id => "user[goal]" %>
    </div>
    <p class="hometext">when I grow up.&nbsp;</p>
    <div id="button">
      <%= submit_tag 'Join', :class => 'button orange' %>
    </div>
  <% end %>

1 Ответ

2 голосов
/ 31 августа 2011

Следующий маршрут неверен:

match '/signup' => 'profiles#new', :as => "signup"

В вашем form_for вы указываете, что он для :profile.По умолчанию Rails будет POST это до profiles#create.Поскольку вы передаете :url => 'signup', форма на самом деле POST преобразуется в /signup, который отображается по вышеупомянутому маршруту в profiles#new.Однако действие new просто устанавливает форму - это действие, с которого вы впервые начали.При этом правильный маршрут должен быть следующим:

match '/signup' => 'profiles#create', :as => "signup"

На самом деле, если вы хотите сделать его намного лучше, он должен быть следующим:

post '/signup' => 'profiles#create', :as => "signup"

Далее, поскольку вы используете именованный маршрут (передавая as в маршруте), вы должны использовать :url => signup_path в вашем form_for.

КакСкажу немного: я не совсем уверен, как выглядят ваши модели, но я бы, наверное, согласился с Клебером.Кажется более интуитивным, что вместо этого должно быть form_for :user.

...