5

Hago esto en mis funciones edit, update, destroy

@post = Post.friendly.find(params[:id])
if @post.user_id == current_user.id

hay alguna forma de optimizar y hacer una sola función , antes lo hacia de esta manera:

before_action :set_user_post, only: [:edit, :update, :destroy]

private
def set_user_post
  @post = current_user.posts.find_by(id: params[:id])
end

Pero agregue la gema friendly_id y lo modifique asi:

private
def set_user_post
  @post = current_user.posts.friendly.find(id: params[:id])
end

Pero me da error.

2
  • Hola Carlos, bienvenido a S.O. en español. Cuando agregues preguntas, también incluye los errores generados, no solamente que te dio un error. De todas formas creo que el error se genera porque estas usando find(id: params[:id]) cuando debería ser find(params[:id]) Commented el 14 ago. 2016 a las 17:45
  • Si, gracias, ese era el error, ya lo solucione. Commented el 15 ago. 2016 a las 3:18

2 respuestas 2

3

El error se genera porque estas usando find(id: params[:id]) cuando debería ser find(params[:id]).

1

Solucion

begin
    @post = current_user.posts.friendly.find(params[:id])
  rescue ActiveRecord::RecordNotFound
    redirect_to @post
  end

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.