2016-10-17 2 views
0

У меня есть приложение, в котором пользователь имеет портфолио. Портфолио может иметь много позиций, и каждая позиция может иметь много движений.Ошибка при передаче значения nil в объекте Rails

Мой портфель показать страницу построен так:

<p> 
    <strong>Name:</strong> 
    <%= @portfolio.name %> 
</p> 

<h1>Positions</h1> 
<div class = 'table'> 
<table> 
    <thead> 
    <tr> 
     <th>Name</th> 
     <th>Quantity</th> 
     <th>Stock</th> 
     <th colspan="3"></th> 
    </tr> 
    </thead> 
<tbody id = "positions"> 
    <%= render @positions %> 
</tbody> 
</table> 
</div> 

<h3>New Position</h3> 

<%= render 'positions/form' %> 

<%= link_to 'Edit', edit_portfolio_path(@portfolio) %> | 
<%= link_to 'Back', portfolios_path %> 

Моя позиция парциальное построена:

<tr> 
    <td class="col-md-1"><%= position.name %></td> 
    <td class="col-md-1"><%= position.quantity %></td> 
    <td class="col-md-1"><%= position.ticker %></td> 
    <td> 
    <%= form_for [@portfolio, position, @movement] do |f| %> 
    <div id = "movement-errors"> 
     <% if @movement.errors.any? %> 
     <%= render 'shared/error_messages', object: @movement %> 
     <% end %> 
    </div> 

    <div class="field"> 
     <%= f.label :quantity %> 
     <%= f.number_field :quantity, class: 'form-control' %> 
    </div> 
    <div class="actions"> 
     <%= f.submit class: 'btn btn-primary' %> 
    </div> 
    <% end %> 
    </td> 
</tr> 

И мои общие ошибки парциальное является:

<% if object.errors.any? %> 
    <div id = "error_explanation"> 
    <div class = "alert alert-danger"> 
     The form contains <%= pluralize(object.errors.count, "error") %>. 
    </div> 
    <ul> 
     <% object.errors.full_messages.each do |msg| %> 
     <li><%= msg %></li> 
     <% end %> 
    </ul> 
    </div> 
<% end %> 

Когда я представляю значение nil в форме движения, я хочу, чтобы ошибки всплывали и сообщали пользователю сообщение. У меня проверка проверки наличия количества.

class Movement < ApplicationRecord 
    belongs_to :position 
    validates :quantity, presence: true 

И мои движения контроллера # создать действие выглядит

class MovementsController < ApplicationController 
    before_action :load_portfolio_and_position 

def create 
    @movement = Movement.new(movement_params) 
    @movement.position_id = @position.id 
    @movement.update_price 
    @movement.date = DateTime.now 
    @movement.trade ='buy' 

    respond_to do |format| 
    if @movement.save 
     @position.update_attribute(:quantity, (@movement.quantity + @position.quantity)) 
     format.html { redirect_to @portfolio, notice: 'Movement was successfully created.' } 
    format.js 
    else 
     format.html { render template: 'portfolios/show' } 
     format.json { render json: @movement.errors, status: :unprocessable_entity } 
     format.js 
    end 
    end 
end 

Как она стоит прямо сейчас, когда я подам пустую форму, я получаю эту ошибку, используя better_errors перл:

' nil 'не является объектом, совместимым с ActiveModel. Он должен реализовать: to_partial_path.

Линия рендеринга @positions выделена на странице моего портфолио. Я думал, что реализую частичный путь. Является ли мой синтаксис неправильным?

+0

Возможно, я слепой, но я не вижу, где вы устанавливаете @portfolio в своем контроллере. – David

+0

Я отредактировал, чтобы показать мой контроллер движения before_action. Я захватываю портфолио и размещаю объекты перед всеми действиями. –

+0

Захватываете ли вы объект '@ position' в методе' load_portfolio_and_position'? –

ответ

0

При передаче объекта частичное использование объекта: атрибут, то частичные ссылки, которые возражают по имени частичного

например

Когда вы звоните

<%= render 'shared/error_messages', object: @movement %> 

общий/error_messages частичная будет локальная переменная error_messages, который ссылается @movement

Когда я только прохождение одного объекта к частичному мне нравится использовать «как»

eg

<%= render 'shared/error_messages', object: @movement , as: :movement %> 

Это позволяет указать имя локальной переменной. Если я передаю несколько объектов, я бы использовал описанный метод @ 7urkm3n.

Смежные вопросы