Как откатиться к определенной версии в django-reversion?

Question

[nurzhannogerbek]

Здравствуйте! Помогите пожалуйста разобраться.

Использую приложение django-reversion в своем проекте. Проблема в следующем: вывожу историю изменений в виде списка. Пытаюсь откатиться к определенной версии, но выдает ошибку. Из ошибки можно предположить, что проблема вызывается когда хочу установить комментарий set_comment. Что делаю не так и как исправить проблему?

views.py:

from reversion.views import RevisionMixin
from django.views import View
from django.shortcuts import get_object_or_404, redirect
from reversion.models import Version
import reversion

class ArticleRevert(RevisionMixin, View):
    def get(self, request, *args, **kwargs):
        article=get_object_or_404(Article, pk=self.kwargs.get('pk'))
        revision=get_object_or_404(Version.objects.get_for_object(article), pk=self.kwargs.get('article_reversion_id')).revision
        reversion.set_comment("Был совершен откат к версии #{}".format(revision.id))
        revision.revert()
        return redirect('article_list')

ERROR:

Traceback (most recent call last):
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/django/core/handlers/exception.py", line 41, in inner
    response = get_response(request)
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/django/core/handlers/base.py", line 187, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/django/core/handlers/base.py", line 185, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/django/views/generic/base.py", line 68, in view
    return self.dispatch(request, *args, **kwargs)
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/reversion/views.py", line 43, in do_revision_view
    return func(request, *args, **kwargs)
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/django/views/generic/base.py", line 88, in dispatch
    return handler(request, *args, **kwargs)
  File "/Applications/Projects/web/project/article/views.py", line 165, in get
    reversion.set_comment("Был совершен откат к версии #{}".format(reversion.id))
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/reversion/revisions.py", line 122, in set_comment
    _update_frame(comment=comment)
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/reversion/revisions.py", line 87, in _update_frame
    _local.stack = _local.stack[:-1] + (_current_frame()._replace(**kwargs),)
  File "/Users/nurzhan_nogerbek/Virtualenvs/py2714/lib/python2.7/site-packages/reversion/revisions.py", line 53, in _current_frame
    raise RevisionManagementError("There is no active revision for this thread")
RevisionManagementError: There is no active revision for this thread

Answer 1

[Александр Брагин]

Судя по исходнику, при использовании RevisionMixin для GET/HEAD/OPTIONS запросов механизм "версионности" игнорируется.
Собственно set_comment напрямую зависит от его активности.

Если нужно "откат" завернуть в новую ревизию, то, по идее, вам нужно поменять get на post.

Если уж очень хочется использовать get (что, в принципе, не лучшая практика для запросов, влияющих на состояние бд), то ничего не мешает вместо RevisionMixin использовать декоратор reversion.create_revision в CBV явно.

Comments

[nurzhannogerbek]

Вот оно как. Думаю нужен метод post и наследовать не от View. Как бы вы решили эту задачу?

P.S. Все же считаю, что RevisionMixin нужен в моем конкретном случаи, так как я пытаюсь установить комментарий (set_comment). В FBV для этих целей использовал бы @reversion.create_revision().

[Александр Брагин]

С django-reversion не работал, поэтому могу основываться только на беглом изучении исходников/доки.

Но если нужно "откат" завернуть в новую ревизию, то, по идее, вам нужно поменять get на post.

Наследовать от View (или его подклассов) так или иначе надо, если планируете использовать CBV. Если уж очень хочется использовать get (что, в принципе, не лучшая практика для запросов, влияющих на состояние бд), то ничего не мешает вместо RevisionMixin использовать декоратор reversion.create_revision в CBV явно.

[nurzhannogerbek]

Попробовал ваш последний совет и заработало. Хоть метод и рабочий, но немного грязный или как вы считаете? Также я тоже считаю что, если происходит влияющих на состояние БД, то лучше использовать post запрос. Проблема в том что если поменять get на post в нижестоящем коде то вылетает исключение:

LevelName: WARNING | Message: Method Not Allowed (GET): /article/326/reversions/21/article_revert/
LevelName: WARNING | Message: "GET /article/326/reversions/21/article_revert/ HTTP/1.1" 405 0

class ArticleRevert(View):

    @reversion.create_revision()
    def get(self, request, *args, **kwargs):
        #  Здесь остальной код

[Александр Брагин]

NogerbekNurzhan, А чего в нем грязного?:) Вполне себе нормальное использование декоратора:)

Method Not Allowed (GET): /article/326/reversions/21/article_revert/ - возникает из-за того, что вы заменили get на post во вьюхе, но по прежнему отправляете GET (переход по ссылке скорее всего). Посмотрите как работают CBV вьюхи (дока)

[nurzhannogerbek]

Александр Брагин, да вы правы, у меня происходит переход по ссылки. Решил использовать ссылку так как при откате происходит переадресация. В итоге остановился на следующем варианте:

class ArticleRevert(RedirectView):

    @reversion.create_revision()
    def get_redirect_url(self, *args, **kwargs):
        # Здесь Код
        return reverse(здесь url адрес)

P.S. Можете пожалуйста ваши комментарии вынести в ответ, я отмечу ответ как правильный.