diff --git a/core/templates/core/file_delete_confirm.jinja b/core/templates/core/file_delete_confirm.jinja deleted file mode 100644 index 155ac62a..00000000 --- a/core/templates/core/file_delete_confirm.jinja +++ /dev/null @@ -1,54 +0,0 @@ -{% extends "core/file.jinja" %} - -{% block title %} - {% trans %}Delete confirmation{% endtrans %} -{% endblock %} - -{% if is_fragment %} - - {# Don't display tabs and errors #} - {% block tabs %} - {% endblock %} - {% block errors %} - {% endblock %} - -{% endif %} - -{% block file %} -

{% trans %}Delete confirmation{% endtrans %}

- - {% if next %} - {% set action = current + "?next=" + next %} - {% else %} - {% set action = current %} - {% endif %} - -
- {% csrf_token %} - -

{% trans obj=object %}Are you sure you want to delete "{{ obj }}"?{% endtrans %}

- - - - -
- -{% endblock %} - - - - diff --git a/core/urls.py b/core/urls.py index 7e4b4e74..8d6d08a2 100644 --- a/core/urls.py +++ b/core/urls.py @@ -21,7 +21,7 @@ # Place - Suite 330, Boston, MA 02111-1307, USA. # # -from django.urls import path, re_path, register_converter +from django.urls import path, register_converter from django.views.generic import RedirectView from com.views import NewsListView @@ -193,27 +193,11 @@ urlpatterns = [ name="user_gift_delete", ), # File views - re_path(r"^file/$", FileListView.as_view(), name="file_list"), - re_path( - r"^file/(?P[0-9]+)/$", - FileView.as_view(), - name="file_detail", - ), - re_path( - r"^file/(?P[0-9]+)/edit/$", - FileEditView.as_view(), - name="file_edit", - ), - re_path( - r"^file/(?P[0-9]+)/prop/$", - FileEditPropView.as_view(), - name="file_prop", - ), - re_path( - r"^file/(?P[0-9]+)/delete/$", - FileDeleteView.as_view(), - name="file_delete", - ), + path(r"file/", FileListView.as_view(), name="file_list"), + path("file//", FileView.as_view(), name="file_detail"), + path("file//edit/", FileEditView.as_view(), name="file_edit"), + path("file//prop/", FileEditPropView.as_view(), name="file_prop"), + path("file//delete/", FileDeleteView.as_view(), name="file_delete"), path("file/moderation/", FileModerationView.as_view(), name="file_moderation"), path( "file//moderate/", FileModerateView.as_view(), name="file_moderate" diff --git a/core/views/files.py b/core/views/files.py index 886e0ba9..eeccb86b 100644 --- a/core/views/files.py +++ b/core/views/files.py @@ -342,33 +342,18 @@ class FileView(CanViewMixin, DetailView, FormMixin): return kwargs -class FileDeleteView(AllowFragment, CanEditPropMixin, DeleteView): +class FileDeleteView(CanEditPropMixin, DeleteView): model = SithFile pk_url_kwarg = "file_id" - template_name = "core/file_delete_confirm.jinja" - context_object_name = "file" + template_name = "core/delete_confirm.jinja" def get_success_url(self): self.object.file.delete() # Doing it here or overloading delete() is the same, so let's do it here if "next" in self.request.GET: return self.request.GET["next"] if self.object.parent is None: - return reverse( - "core:file_list", - ) - return reverse( - "core:file_detail", - kwargs={ - "file_id": self.object.parent.id, - }, - ) - - def get_context_data(self, **kwargs): - kwargs = super().get_context_data(**kwargs) - kwargs["next"] = self.request.GET.get("next", None) - kwargs["previous"] = self.request.GET.get("previous", None) - kwargs["current"] = self.request.path - return kwargs + return reverse("core:file_list") + return reverse("core:file_detail", kwargs={"file_id": self.object.parent.id}) class FileModerationView(AllowFragment, ListView):