Дата в URL повтор, не разобрался?

Question

[bismoney]

Так и не смог разобраться.... прошу помощи друзья.

Сейчас есть routes.rb

resources :post_categories, path: '', only: :show do
   resources :posts, path: '', only: :show
end

И обрабочик в контроллере post_controller, где мы вытягиваем категорию по post_category_id и сам пост ассоц по id

def show
@posts = PostCategory.find_by!(seo_url: params[:post_category_id]).posts.find(params[:id])
end

Тем самым получая путь
site.com/category_name/id

Сейчас понадобилось получить такой путь
site.com/category_name/year/month/day/id

Где year/month/day это дата поста id

В routes.rb прописываю такую конструкцию

resources :post_categories, path: '', only: :show do
   resources :posts, path: '/:year/:month/:day', only: :show, constraints: { year: /\d{4}/, month: /\d{2}/, day: /\d{2}/ }
end

Но хоть убейся не могу понять как обработать эти параметры в контроллере...
params[:year], params[:month], params[:day]

Помогите кому не трудно.

Answer 1

[vsuhachev]

У вас какое-то масло масляное. Для того чтобы получить Post достаточно передать его id. Зачем вам в url еще категория и дата мне непонятно. Обычно, если используется url вида site.com/category_name/year/month/day, то он выдает группу объектов, отфильтрованную по категории и дате. Если вам нужно такое поведение - добавляйте в PostsController метод index и заводите такой роут на него.

Comments

[bismoney]

Нет, мне нужно получить url пути к посту именно дату этого поста. Вы таких примеров никогда не видели где дата поста в урл?). Нет эта дата мне не нужна для сортировки по дате. Да можно получить пост передавая его только его id, но в таком случае поста будет открываться даже так 2022/22/11/ID а это уже куча дублей, так как проверки даты поста нет, а id есть.

[bismoney]

Вот яркий пример нашел, в ленте такой url используется
lenta.ru/news/2015/11/17/siria_migrants

[bismoney]

Так же вот к примеру ведомости, www.vedomosti.ru/realty/articles/2015/11/17/617107...

И к примеру если я сделаю так "/22/", то получю 404 ошибку, так как в этот месяц, день и год небыло поста с id 617107-trebovaniya-stroitelyam.
www.vedomosti.ru/realty/articles/2015/22/17/617107...

Описал как мог :)

[vsuhachev]

OK, если так надо, то я бы сделал так:

В модели Post

scope :by_publish_date, -> (date) { where(publish_date: date) }
scope :by_category_seo, -> (category_seo) { joins(:post_category).where(post_categories: { seo_url: category_seo}) }

В PostController

def show
  date = Time.new(params[:year], params[:month], params[:day])
  @post = Post.by_category_seo(params[:post_category_id]).by_publish_date(date).find(params[:id])
end

[vsuhachev]

Ну или еще вариант - искать в БД просто по id и потом проконтролировать равны ли у найденного объекта дата публикации и категория тому что пришло в url. Если нет то выдать 404.

[bismoney]

vsuhachev: Ох огромное спасибо за решения, сейчас опробую, даже готов скинуть на вебманьку 200р за решение) не просто же так... я реально потратил уже 2 сутки на поиск.
А как в случае второго будет выглядеть в контроллере сравнение, можешь подсказать? Спасибо.

[vsuhachev]

Как то так будет выглядеть

def show
  @post = Post.includes(:post_category).find(params[:id])
  date = Time.new(params[:year], params[:month], params[:day])

  if @post.post_category.seo_url != params[:post_category_id] || @post.publish_date != date
    raise ActionController::RoutingError.new('Not Found')
  end
end

[bismoney]

vsuhachev: Спасибо огромнейшее... первый осмыслил, передала чуток под себя и все заработало) Я очень вам благодарен. Спасибо что не прошли мимо, а мне все-таки надо подтянуть запросы, так как на них я стоял, а ваш пример сразу все сдвинул с места.

[bismoney]

vsuhachev: Вернее не так поставил вопрос, я вывел ссылку на пост так

= link_to 'Show', post_category_post_path(post.post_category.seo_url, Date.parse(post.publish_date).strftime("%Y"), Date.parse(post.publish_date).strftime("%m"), Date.parse(post.publish_date).strftime("%d"), post)

А есть возможно вывести ее короче?

[vsuhachev]

Ну создайте хэлпер в ApplicationHelper, любой код можно спрятать в функцию для краткости

def post_path(post)
  post_category_post_path(
    post.post_category.seo_url, 
    post.publish_date.year, post.publish_date.month, post.publish_date.day,
    post.id
  )
end

[bismoney]

vsuhachev: Спасибо огромное!)))))