Hello,
I need to write a function that will return a link only if the current
user is the owner. Here is my code...
1. application_helper.rb
2.
3. def link_to_if_owned(owner_id, anchor_text, where_to_go)
4. if current_user.id == owner_id # current user is owner
5. "#{link_to anchor_text, where_to_go}"
6. else
7. anchor
8. end
9. end
And here''s how I call it in the view...
1. <%= link_to_if_owned(@car.owner_id, "Add Gas",
":controller =>
:cars, :action => :add_gas, :car_id => @car.id")%>
The downside is that the actual HTML rendered is a bad link (this is
what I get)...
1. <a href=":controller => :cars, :action => :add_gas, :car_id
=>
@car.id">Add Gas</a>
How can I make it so the link works?
--
Posted via http://www.ruby-forum.com/.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Talk" group.
To post to this group, send email to
rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
To unsubscribe from this group, send email to
rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
-~----------~----~----~----~------~----~------~--~---
Frederick Cheung
2008-Jul-08 16:02 UTC
Re: Conditional "link_to" helper function - AYUDAME POR FAVOR
On 8 Jul 2008, at 16:51, Joe Peck wrote:> > Hello, > > I need to write a function that will return a link only if the current > user is the owner. Here is my code... > > 1. application_helper.rb > 2. > 3. def link_to_if_owned(owner_id, anchor_text, where_to_go) > 4. if current_user.id == owner_id # current user is owner > 5. "#{link_to anchor_text, where_to_go}"That''s unnecessary: just link_to ... is enough> > 6. else > 7. anchor > 8. end > 9. end > > And here''s how I call it in the view... > > 1. <%= link_to_if_owned(@car.owner_id, "Add Gas", ":controller => > :cars, :action => :add_gas, :car_id => @car.id")%> >don''t pass the link parameters in a string (after all you wouldn''t with a ''normal'' link_to would you). Fred> The downside is that the actual HTML rendered is a bad link (this is > what I get)... > > 1. <a href=":controller => :cars, :action => :add_gas, :car_id => > @car.id">Add Gas</a> > > How can I make it so the link works? > -- > Posted via http://www.ruby-forum.com/. > > >--~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
Joe Peck
2008-Jul-08 18:07 UTC
Re: Conditional "link_to" helper function - AYUDAME POR FAVO
How can I pass it if I don''t know how many parameters there are going to be each time? Let''s say I want to use it like this: <%= link_to_if_owned(@car.owner_id, "Add Gas", ":controller => :cars, :action => :add_gas, :car_id => @car.id")%> But also like this: <%= link_to_if_owned(@car.owner_id, "List Cars", ":controller => :cars, :action => :list")%> -- Posted via http://www.ruby-forum.com/. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
Frederick Cheung
2008-Jul-08 18:10 UTC
Re: Conditional "link_to" helper function - AYUDAME POR FAVO
On 8 Jul 2008, at 19:07, Joe Peck wrote:> > How can I pass it if I don''t know how many parameters there are > going to > be each time? Let''s say I want to use it like this: > > <%= link_to_if_owned(@car.owner_id, "Add Gas", ":controller => > :cars, :action => :add_gas, :car_id => @car.id")%> > > But also like this: > > <%= link_to_if_owned(@car.owner_id, "List Cars", ":controller => > :cars, :action => :list")%>That doesn''t make any difference. :controller => :cars, :action => :add_gas, :car_id => @car.id is only one parameter (they''re collected into a single hash. try it). You don''t need to change your helper at all, just how you are calling it. (if you ever need to capture a variable number of arguments, use *args. You don''t need it here though) Fred> > > -- > Posted via http://www.ruby-forum.com/. > > >--~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
Joe Peck
2008-Jul-08 18:21 UTC
Re: Conditional "link_to" helper function - AYUDAME POR FAVO
Thanks, yeah, I tested it right after I wrote my comment. Turns out it works already. Thanks again for the advice. -- Posted via http://www.ruby-forum.com/. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---