Search

Ideas that are…

Search Ideas


37 ideas match your query.:

Then how would you call this from a helper method?

#314·Dennis HackethalOP, over 1 year ago·CriticismCriticized2

Hiccdown methods should live in their own, separate modules. How about they are called ‘renderers’?

ruby
module ProductsRenderer
def self.index vc, # …
vc.some_helper_method
end
end
#313·Dennis HackethalOP, over 1 year agoArchived

That would be mixing class methods an instance methods in Rails helper modules, which typically only contain instance methods. Not idiomatic Rails usage.

#312·Dennis HackethalOP, over 1 year ago·Criticism

If so, there might be a way to bind them to the view_context. Or I could definitely pass the view_context explicitly as the first parameter:

So instead of

ruby
@helper_module.instance_method(@action_name).bind_call(view_context)

I would do

ruby
@helper_module.send(@action_name, view_context)

And the parameter list of each Hiccdown method would start accordingly:

ruby
module ProductsHelper
def self.index vc #, …
vc.some_helper_method
end
def some_helper_method
# …
end
end
#310·Dennis HackethalOP revised over 1 year ago·Original #307·Criticism

If so, there might be a way to bind them to the view_context. Or I could definitely pass the view_context explicitly as the first parameter:

So instead of

ruby
@helper_module.instance_method(@action_name).bind_call(view_context)

I would do

ruby
@helper_module.send(@action_name, view_context)

And the parameter list of each Hiccdown method would start accordingly:

ruby
module ProductsHelper
def self.index vc #, …
# …
end
end
#308·Dennis HackethalOP revised over 1 year ago·Original #307·CriticismCriticized1

If so, there might be a way to bind them to the view_context. Or I could definitely pass the view_context explicitly as the first parameter.

#307·Dennis HackethalOP, over 1 year ago·CriticismCriticized1

Does that mean they wouldn’t have access to the view_context? If so, calling helper methods from inside these class methods wouldn’t be possible.

#305·Dennis HackethalOP revised over 1 year ago·Original #304·CriticismCriticized1

Does that mean they wouldn’t have the view_context? If so, calling helper methods from inside these class methods wouldn’t be possible.

#304·Dennis HackethalOP, over 1 year ago·CriticismCriticized1

Hiccdown methods should live in Rails helpers as class methods. That way, the problem described in #302 is solved – methods can be referenced unambiguously:

ruby
ProductsHelper.index
StoresHelper.index
#303·Dennis HackethalOP, over 1 year ago·Criticized3

Hiccdown methods should live in Rails helpers as instance methods.

#302·Dennis HackethalOP revised over 1 year ago·Original #300·Criticized2

That isn’t a good idea because Hiccdown methods often share the same conventional names (index, show, etc), which can and does lead to conflict.

#301·Dennis HackethalOP, over 1 year ago·Criticism

Hiccdown methods should live in Rails helpers.

#300·Dennis HackethalOP, over 1 year ago·Criticized1