Простой validates_length_of приводит к неожиданному NoMethodError - PullRequest
0 голосов
/ 13 августа 2010

У меня есть форма сообщения, которая использует validates_length_of в модели, чтобы ограничить длину сообщения.

validates_length_of   :content, :maximum => 1024

Я «проверил» это, введя около 5 тысяч символов и отправил форму. Но я получаю это NoMethodError:

NoMethodError in Messages#create

Showing app/views/messages/_messages.html.erb where line #1 raised:

You have a nil object when you didn't expect it!
You might have expected an instance of Array.
The error occurred while evaluating nil.each
Extracted source (around line #1):

1: <%- for message in @messages -%>
2:   <div class="message">
3:     <%= link_to message.title, message %>
....

Сообщения # индекс:

def index
  @title = "chat"
  @messages = Message.paginate :page => params[:page], :per_page => 8, :order => "updated_at DESC" 
  respond_to do |format|
    format.html
  end
end

Сообщения # создать:

def create
  @message = Message.new(params[:message])
  respond_to do |format|
    if @message.save
      record_post_time
      flash[:notice] = "Created \"#{@message.title}\""
      format.html { redirect_to(messages_url) }
    else
      format.html { render :action => "index" }
    end
  end
end

_messages.html.erb: (очень уродливо и нуждается в серьезном рефакторе)

<%- for message in @messages -%>
  <div class="message">
    <%= link_to message.title, message %>

    <%- if message.name.empty? -%>

    <%- else -%>   
      <span class="name">
        by 
        <%- if message.email.blank? -%>
          <%=h message.name %>
        <%- else -%>
          <a href="mailto:<%= message.email %>"><%=h message.name %></a>
        <%- end -%>
      </span>
    <%- end -%>

    <span class="time">
      active &#32;<%= timeago(message.updated_at) %>
    </span>

    <%- if message.comments.empty? -%>
      <span class="reply">
        <%= link_to 'reply', :controller => 'messages', :action => 'show', :id => message %>
      </span>
    <% else %>
      <span class="reply">
        <%= link_to pluralize(message.comments.count, 'reply'), :controller => 'messages', :action => 'show', :id => message %>
      </span>
    <%- end -%>

    <p><%= sanitize message.content,
 :tags => %w(a embed img object p param),
 :attributes => %w(allowfullscreen allowscriptaccess href name src type value) %></p>

    <% unless controller.controller_name == "tags" %>
      <%- unless message.tag_list.nil? || message.tag_list.empty? -%>
        <% message.tags.each do |t| %>
          <div class="tag"><%= link_to t.name.titleize, tag_path(t) %></div>
        <% end %>
      <%- end -%>
    <% end %>

    <%- unless message.comments.empty? -%>
      <div class="comments">
        <%= render :partial => message.firstcomments %>
        <%- if message.comments.count >= 4 -%>
          <%= link_to 'more...', :action => 'show', :id => message %>
        <%- end -%>
      </div>
    <%- end -%>
  </div>
<%- end -%>

Я не уверен, что именно здесь происходит. Любая помощь будет оценена (даже с частичным рефакторингом). Спасибо за чтение моего вопроса.

1 Ответ

1 голос
/ 13 августа 2010

Привет. Когда вы ввели 5k или около того символов, Message становится недопустимой записью, поэтому при создании сработало else условие

else
  format.html { render :action => "index" }
end

, которое вы отображаете здесь render не выполняет код внутри действияпросто отображает текстовый файл с вашими текущими данными. Вы не назначили @messages в вашем create действии (я полагаю, вы также не сделали этого в new), поэтому nil при рендеринге indexВы должны инициализировать его внутри new или create или изменить render 'index' на render 'new'

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...