RoR: неопределенный метод "url_for" для nil: NilClass - PullRequest
1 голос
/ 22 августа 2010

У меня есть стандартное приложение Rails.

Когда создается Совет, я хотел бы создать Сообщение для каждого Пользователя, который интересуется этим Советом.

Звучит просто, не так ли?Это должно быть ...

Итак, мы начинаем с Tip Observer:

class TipObserver < ActiveRecord::Observer
  def after_save(tip)
    # after the tip is saved, we'll create some messages to inform the users
    users = User.interested_in(tip) # get the relevant users
    users.each do |u|
      m = Message.new
      m.recipient = u
      link_to_tip = tip_path(tip)
      m.body = "Hello #{u.name}, a new tip: #{link_to_tip}"
      m.save!
    end
  end
end

Ошибки:

tip_observer.rb:13:in `after_save': undefined method `tip_path' for #<TipObserver:0xb75ca17c> (NoMethodError)

Хорошо, поэтому TipObserver необходим доступ к UrlWriterметоды.Это должно быть довольно просто исправить, верно?

class TipObserver < ActiveRecord::Observer
  include ActionController::UrlWriter

Теперь он работает (!) И выводит:

Hello dave18, a new tip: /tips/511

Отлично, что работает !!Ну, вроде как, на самом деле мы хотим, чтобы это была ссылка для кликов.Опять же, это должно быть просто, верно?

link_to_tip = link_to tip.name, tip_path(tip)

Ошибки:

tip_observer.rb:13:in `after_save': undefined method `link_to' for #<TipObserver:0xb75f7708> (NoMethodError)

Хорошо, поэтому на этот раз TipObserver необходим доступ к UrlHelper методам.Это должно быть довольно просто исправить, верно?

class TipObserver < ActiveRecord::Observer
  include ActionController::UrlWriter
  include ActionView::Helpers::UrlHelper

Ошибки:

whiny_nil.rb:52:in `method_missing': undefined method `url_for' for nil:NilClass (NoMethodError)

Хорошо, кажется, добавление, которое мешало объявлению url_for.Давайте попробуем включить в другом порядке:

class TipObserver < ActiveRecord::Observer
  include ActionView::Helpers::UrlHelper
  include ActionController::UrlWriter

Ошибки:

url_rewriter.rb:127:in `merge': can't convert String into Hash (TypeError)

Хм, очевидного пути к этому нет.Но после прочтения некоторых умных засорений можно предположить, что Sweepers такие же, как Observers , но имеют доступ к помощникам URL.Итак, давайте преобразуем Observer в Sweeper и удалим UrlHelper и UrlWriter.

class TipObserver < ActionController::Caching::Sweeper
  observe Tip
  #include ActionView::Helpers::UrlHelper
  #include ActionController::UrlWriter

Хорошо, это позволяет ему работать, но вот вывод:

Hello torey39, a new tip:

Итак, ошибки нет, но URL не генерируется.Дальнейшее исследование с помощью консоли показывает, что:

tip_path => nil

и, следовательно:

tip_path(tip) => nil

Хорошо, я понятия не имею, как решить эту проблему, так что, возможно, мы можем атаковать это с другогонаправление.Если мы переместим содержимое в шаблон erb и представим Message.body как представление - это дает два преимущества - во-первых, содержимое «Просмотр» помещается в правильное расположение, и это может помочь нам избежать этих проблем с _ _path.

Итак, давайте изменим метод after_save:

def after_save(tip)
  ...
  template_instance = ActionView::Base.new(Rails::Configuration.new.view_path)
  m.body = template_instance.render(:partial => "messages/tip", :locals => { 
      :user=>user, 
      :tip=>tip
    })
  m.save!
end

Ошибки:

undefined method `url_for' for nil:NilClass (ActionView::TemplateError)

Отлично, но теперь мы снова вернулись к этому кровавому url_for.Так что на этот раз жаловаться на ActionView.Давайте попробуем исправить это тогда:

def after_save(tip)
  ...
  template_instance = ActionView::Base.new(Rails::Configuration.new.view_path)
  template_instance.extend ActionController::UrlWriter

Ошибки:

undefined method `default_url_options' for ActionView::Base:Class

Отлично, так что, что бы мы ни делали, мы получим ошибки.Я попробовал много разных способов присвоения default_url_options внутри template_instance, но безуспешно.

Пока что это не выглядит очень "Railsy", на самом деле это кажется совершенно трудным.

Итак, мой вопрос:

  • Пытаюсь ли я получить квадратный колышек в круглой дыре?Если да, то как мне адаптировать архитектуру для обеспечения этой функциональности?Я не могу поверить, что это не то, что существует на других веб-сайтах.
  • Должен ли я отказаться от попыток использовать Observer или Sweeper?
  • Должен ли я пытаться создавать новые сообщения с помощью MessagesController,и если да, то как я могу вызывать MessagesController напрямую и несколько раз из Observer / Sweeper?

Любые советы, советы или предложения будут с благодарностью приняты, я бьюсь головой об этот кирпичстена в течение нескольких дней и медленно теряет желание жить.

tia

Кит

1 Ответ

2 голосов
/ 22 августа 2010

Ну, вы правы, что ваш подход не очень Rails-ы.Вы пытаетесь смешивать методы модели, контроллера и представления так, как они не предназначены, и это всегда немного шатко.

Если бы я пошел по вашему пути, я бы, наверное, сдался на link_to проблема и (я признаю, что это не «путь Rails») вручную закодировал HTML-код для ссылки.Так что link_to_tip = link_to tip.name, tip_path(tip) становится link_to_tip = '<a href="#{tip_path(tip)}">#{tip.name}</a> - быстрое и грязное решение, если вы ищете его; -)

Но, по моему опыту, Rails довольно хорош, пока вы не захотите сделать что-то нестандартноепуть.Тогда он может укусить вас: -)

Проблема в том, что вы пишете и храните текст в своей модели сообщений, которого там быть не должно.Модель сообщения должна belong_to Tips, а представление должно отвечать за представление текста сообщения, включая ссылку на подсказку.Если Сообщение может быть о чем-то отличном от Tips, вы можете создать полиморфную ассоциацию в модели Message следующим образом:

belongs_to :source, :polymorphic => true

Модель Tip будет включать:

has_many :messages, :as => :source

Тогда высделайте это (используя ваш код в качестве примера):

m = Message.new
m.source = tip
m.save!

Представление, которое отображает сообщение, отвечает за создание ссылки, например:

<%= "Hello #{u.name}, a new tip: #{link_to m.source.name, tip_path(m.source)}" %>
...