2

Проблема заключается в том, что понятия не имею как записать логику. Имеется страничка тех. поддержки, пользователь оставляет заявку, в личном кабинете у него доступна табличка с его обращением, где есть кнопки "просмотреть", "редактировать" и "удалить". Но нужно сделать так, чтобы кнопка "удалить" была доступна только 30 минут от времени создания заявки. Мне бы понять, где это прописать, и как. Мысль есть прописать в контролере, но логику пока до конца не могу разобрать.

if (request.created_at <= DateTime.now+1.minute &&  DateTime.now+1.minute > DateTime.now)
            = button_to 'Удалить', request_path(request),method: :delete, title: 'Удаление', class: 'btn btn-danger', data: { confirm: 'Вы уверены, что хотите удалить?' } do
              %i.fa.fa-trash-o
            -#= link_to request_path(request), method: :delete, title: 'Удаление', class: 'btn btn-danger', data: { confirm: 'Вы уверены, что хотите удалить?' } do
          - else
            = button_to 'Удалить', request_path(request),method: :delete, title: 'Удаление', class: 'btn btn-danger', disable: true, data: { confirm: 'Вы уверены, что хотите удалить?' } do
              %i.fa.fa-trash-o

На данный момент дошел лишь до этого, но что-то запутался, и, возможно, допущены ошибки и неправильная логика. Желательно простенький пример для рассмотрения и понимания.

2 ответа 2

1

Несколько шагов:

1. Назовем модель иначе, чем Request

На всякий случай. Потому что request и response в рельсах - зарезервированные имена для обращения к запросу и ответу от сервера. Если использовать Request для модели - явных ошибок от рельс вы не увидите, но потом можете внезапно столкнуться с проблемами.

Я называю такие модели SiteRequest, и использую site_request для хранения отдельной записи модели.

2. Проверять время нам нужно в двух местах

Во-первых, во вьюхе при выводе кнопки. Показывать кнопку, если с момента создания заявки прошло менее 30 минут. Но есть еще одно место:

Кейс: 28 минут с момента создания заявки. Страница загрузилась, кнопка на ней видна. Пользователь кликнул по ней спустя 3 минуты. Заявку удаляем или нет? Ведь в момент клика по кнопке - прошла уже 31 минута с момента создания.

Если учесть этот кейс - нужно добавить условие на 30 минут и в экшн удаления заявки site_requests#destroy. Одно и то же условие в двух разных местах на сайте.

3. Сохраним условие в одном общем месте

Чтобы не писать и там, и там "Если прошло меньше 30 минут с момента создания задания" - запишем это условие, например, в методе модели:

class SiteRequest < ApplicationRecord
  # Ваш существующий код модели

  def allow_to_destroy?
    created_at + 30.minutes <= DateTime.now
  end
end

4. Используем метод модели для проверки

Во-первых - во вьюхе (написано на slim):

- if site_request.allow_to_destroy?
  = link_to 'Удалить', site_request, method: :delete,
            title: 'Удаление',
            class: 'btn btn-danger',
            data: { confirm: 'Вы уверены, что хотите удалить?' } do
    i.fa.fa-trash-o

Во-вторых - в экшне destroy:

class SiteRequestsController < ApplicationController
  def destroy
    site_request = SiteRequest.find(params[:id])
    if site_request.allow_to_destroy?
      if site_request.destroy
        flash[:success] = 'Заявка успешно удалена'
      else
        flash[:error] = 'Заявку не удалось удалить'
      end
    else
      flash[:error] = 'С момента создания заявки прошло 30 минут, вы не можете ее удалить'
    end
    redirect_to :site_requests
  end
end
6
  • Благодарю за ответ, но ошибка остается одна, что метод не может быть определен
    – an0mality
    6 апр 2017 в 11:35
  • В каком месте у вас ошибка? И как она звучит в оригинале? Undefined method? 6 апр 2017 в 11:37
  • Прошу меня простить. Ошибку нашел сам. не верно прописал во вьюхе метод. Но проблема в то, что даже после нажатия кнопки даже спустя 30 минут контролер удаляет запись
    – an0mality
    6 апр 2017 в 11:47
  • У вас там код if site_request.allow_to_destroy? такой? И если такой - что хранится в site_request? Должна одиночная запись модели SiteRequest. Но у вас, судя по ошибке, там лежит что-то другое. 6 апр 2017 в 11:50
  • 1
    Уже поправил говорю же. И вроде как кнопка стала пропадать. Сижу тестирую, благодарю за помощь. Использовал целую запись строки, и внимания не обратил, что не так
    – an0mality
    6 апр 2017 в 12:02
-2

Если заявка сохраняется в БД: - То необходимо создать условие появление кнопки удалить и редактировать. Для этого советую почитать документацию к библиотеке Time . А дальше, простые математические и логические операции с колонкой "created_at" относящейся к конкретной заявке. - Логику лучше всего писать в Helper , что бы не засорять вьюхи.

Код не стал специально писать, так как информация в вопросе не способствует погружению в бизнес логику.

4

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.