Метод блока Rails Presenter

Потому что, если вы собираетесь использовать команду yield, вы не должны указывать &block, так как теперь вы фактически получаете блок в качестве параметра, используя обычный синтаксис параметра.

class UserPresenter
  def initialize(user, vc)
    @user = user
    @vc   = vc
  end

  def linkify() # <-- Remove &block
    vc.link_to("foo") do
      yield
    end
  end
end

# ...
# Somewhere else, assuming you have access to @presenter which is an instance of
# UserPresenter
# ...

def show
  @presenter.linkify do
    # ...
    # do my view stuff here
    # ...
  end
end

show()

# Now, if your "View" is nothing but a block that needs to get passed in
# then you'd do this...

def show(&block)
  @presenter.linkify do
    block.call()
  end
end

# This would be used this way:

show(lambda {  
  # ...
  # View stuff here
  # ..
}) 

Как указано в ответе на лакросс. Неправильный контекст представления является корнем этой причины. Я попытался обойти вашу ситуацию. И вот как в итоге получилось:

Я создал вспомогательный метод в ApplicationHelper:

module ApplicationHelper
  def link(presenter)
    presenter.linkify(self) do
      yield
    end
  end
end

изменено linkify() на:

def linkify(vc)
  vc.link_to("foo") do
    yield
  end
end

это означает, что нет необходимости иметь vc в конструкторе класса ведущего, или вы можете обновить vc в методе link, определенном в помощнике (по вашему выбору).

представления теперь выглядят примерно так:

presenter_from_view.html.slim:

-@presenter = UserPresenter.new(@user, self)
=link @presenter do
  p 123

presenter_from_controller.html.slim:

=link @presenter do
  p 123

Я согласен, может быть, это не то, как вы хотели, чтобы ваше решение было сделано. Но я не мог получить более чистую работу для этого. Однако здесь вам не нужно беспокоиться о передаче self в представлениях, где бы вы ни использовали link @presenter do.. (что может стать слишком трудным для написания кода, когда вы используете linkify в нескольких представлениях). угадать).

P.S.: Все ваши спецификации сейчас проходят. И, если вам нужен измененный код, я могу отправить его в ваш репозиторий в отдельной ветке. Дай мне знать.

Из документации Слима о помощниках и захвате:

module Helpers
  def headline(&block)
    if defined?(::Rails)
      # In Rails we have to use capture!
      "<h1>#{capture(&block)}</h1>"
    else
      # If we are using Slim without a framework (Plain Tilt),
      # this works directly.
      "<h1>#{yield}</h1>"
    end
  end
end

Можете ли вы попробовать использовать захват следующим образом?

def linkify(&block)
  result = capture(&block)
  vc.link_to("foo") do
    result
  end
end

Неправильный контекст представления вызывает эту проблему. Просто измените UserPresenter#initialize, чтобы он не принимал контекст представления, инициализируйте презентер в контроллере и вместо этого передайте правильный контекст представления из представления, например так:

= @presenter.linkify(self) do
  p "123"

Какую ошибку вы получаете? Просто глядя на код...

В этом методе

def linkify()
  #
  # HERE IS THE PROBLEM
  #
  vc.link_to("foo") do
    yield
  end
end

где определяется vc?

Я думаю, вы имеете в виду @vc, которая является переменной экземпляра, которую вы инициализируете.

Также в качестве примечания... пустые () в linkify() избыточны в рубиновом методе без переменных. Вы можете их устранить.

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

Кажется, я понял, ПОЧЕМУ это не работает. Когда вы передаете view_context в контроллер, он визуализирует view_context один раз, когда вы передаете его ведущему, а затем вы снова, когда вы фактически визуализируете представление.

def initialize(user, vc)
  @user = user
  @vc = vc
end

# When called from the view:
@presenter = UserPresenter.new(@user, self)
# You're passing the view directly in "as is" and renders as expected.

# However, when you pass it in from the controller:
@presenter = UserPresent.new(@user, view_context)
# you're essentially rendering the view_context in the controller, and then again
# once it renders at the end of your action. That's why you're getting this:
#   "<p>123</p><a href="foo"><p>123</p></a>"

Вам нужно либо отправлять себя из представления каждый раз, когда вы вызываете linkify, либо вы можете использовать вспомогательный метод приложения, который всегда будет иметь контекст представления.