diff --git a/src/lgr_advanced/lgr_editor/templates/lgr_editor/codepoint_list.html b/src/lgr_advanced/lgr_editor/templates/lgr_editor/codepoint_list.html
index 9a385e8b..2de63e49 100644
--- a/src/lgr_advanced/lgr_editor/templates/lgr_editor/codepoint_list.html
+++ b/src/lgr_advanced/lgr_editor/templates/lgr_editor/codepoint_list.html
@@ -185,7 +185,7 @@
diff --git a/src/lgr_advanced/lgr_editor/templates/lgr_editor/references.html b/src/lgr_advanced/lgr_editor/templates/lgr_editor/references.html
index 365fe278..75154fe2 100644
--- a/src/lgr_advanced/lgr_editor/templates/lgr_editor/references.html
+++ b/src/lgr_advanced/lgr_editor/templates/lgr_editor/references.html
@@ -24,12 +24,15 @@
{% trans 'Existing references' %}
{% if not is_set %}
-
-
-
+
|
{% endif %}
diff --git a/src/lgr_advanced/lgr_editor/templates/lgr_editor/tags.html b/src/lgr_advanced/lgr_editor/templates/lgr_editor/tags.html
index 8d259b3d..03b029ac 100644
--- a/src/lgr_advanced/lgr_editor/templates/lgr_editor/tags.html
+++ b/src/lgr_advanced/lgr_editor/templates/lgr_editor/tags.html
@@ -44,12 +44,15 @@
{% trans 'Existing tags' %}
{% if not is_set %}
-
-
-
+
|
{% endif %}
diff --git a/src/lgr_advanced/lgr_editor/views/codepoints/codepoint.py b/src/lgr_advanced/lgr_editor/views/codepoints/codepoint.py
index 011208d2..d12bfdba 100644
--- a/src/lgr_advanced/lgr_editor/views/codepoints/codepoint.py
+++ b/src/lgr_advanced/lgr_editor/views/codepoints/codepoint.py
@@ -380,8 +380,7 @@ class CodePointDeleteView(LGREditMixin, CodePointMixin, View):
Delete a codepoint from an LGR.
"""
- # TODO - only accept POST request
- def get(self, request, *args, **kwargs):
+ def post(self, request, *args, **kwargs):
char = self.lgr.get_char(self.codepoint)
try:
@@ -403,8 +402,7 @@ class VariantDeleteView(LGREditMixin, CodePointMixin, View):
Delete a variant of a codepoint from an LGR.
"""
- # TODO - only accept POST request
- def get(self, request, *args, **kwargs):
+ def post(self, request, *args, **kwargs):
var_slug = self.kwargs['var_slug']
var_cp, var_when, var_not_when = slug_to_var(var_slug)
diff --git a/src/lgr_advanced/lgr_editor/views/create.py b/src/lgr_advanced/lgr_editor/views/create.py
index 80379df0..f9bb71d3 100644
--- a/src/lgr_advanced/lgr_editor/views/create.py
+++ b/src/lgr_advanced/lgr_editor/views/create.py
@@ -223,7 +223,6 @@ def setup(self, request, *args, **kwargs):
super().setup(request, *args, **kwargs)
self.lgr_pk = self.kwargs['lgr_pk']
- # TODO make that a post
- def get(self, request, *args, **kwargs):
+ def post(self, request, *args, **kwargs):
LgrModel.get_object(request.user, self.lgr_pk).delete()
return redirect('lgr_advanced_mode')
diff --git a/src/lgr_advanced/lgr_editor/views/reference.py b/src/lgr_advanced/lgr_editor/views/reference.py
index c81fbcaa..5419439c 100644
--- a/src/lgr_advanced/lgr_editor/views/reference.py
+++ b/src/lgr_advanced/lgr_editor/views/reference.py
@@ -185,7 +185,7 @@ class DeleteReferenceView(LGREditMixin, View):
Delete a reference from an LGR.
"""
- def get(self, request, *args, **kwargs):
+ def post(self, request, *args, **kwargs):
ref_id = self.kwargs['ref_id']
logger.debug("Delete reference %s'", ref_id)
diff --git a/src/lgr_advanced/lgr_editor/views/tag.py b/src/lgr_advanced/lgr_editor/views/tag.py
index 4225086a..d6ca31a3 100644
--- a/src/lgr_advanced/lgr_editor/views/tag.py
+++ b/src/lgr_advanced/lgr_editor/views/tag.py
@@ -90,7 +90,7 @@ class DeleteTagView(LGREditMixin, View):
Delete a tag from an LGR.
"""
- def get(self, request, *args, **kwargs):
+ def post(self, request, *args, **kwargs):
tag_id = self.kwargs['tag_id']
logger.debug("Delete tag %s'", tag_id)
diff --git a/src/lgr_advanced/templates/lgr_advanced/index.html b/src/lgr_advanced/templates/lgr_advanced/index.html
index 70ca6d95..6aec3d5c 100644
--- a/src/lgr_advanced/templates/lgr_advanced/index.html
+++ b/src/lgr_advanced/templates/lgr_advanced/index.html
@@ -34,13 +34,18 @@
{% trans "LGRs" %}
@@ -49,14 +54,19 @@
{% trans "LGR sets" %}
{% for lgr in local_lgrs.list %}
-
-
- {% blocktrans with lgr_name=lgr.name %}View {{ lgr_name }}{% endblocktrans %}
-
- {% blocktrans %}View embedded LGRs{% endblocktrans %}
-
-
-
+
diff --git a/src/lgr_idn_table_review/icann_tools/templates/lgr_idn_table_review_icann/icann_mode.html b/src/lgr_idn_table_review/icann_tools/templates/lgr_idn_table_review_icann/icann_mode.html
index 185bf7ad..97e5d97f 100644
--- a/src/lgr_idn_table_review/icann_tools/templates/lgr_idn_table_review_icann/icann_mode.html
+++ b/src/lgr_idn_table_review/icann_tools/templates/lgr_idn_table_review_icann/icann_mode.html
@@ -33,14 +33,18 @@
{% for reports_grouped in report_ids %}
{% with report=reports_grouped.list|first %}
-
-
- {% blocktrans with report.report_id as report_id %}{{ report_id }}{% endblocktrans %}
-
-
-
-
+
{% endwith %}
{% empty %}
diff --git a/src/lgr_idn_table_review/idn_tool/templates/lgr_idn_table_review_tool/list_reports.html b/src/lgr_idn_table_review/idn_tool/templates/lgr_idn_table_review_tool/list_reports.html
index d615084b..33a4df0f 100644
--- a/src/lgr_idn_table_review/idn_tool/templates/lgr_idn_table_review_tool/list_reports.html
+++ b/src/lgr_idn_table_review/idn_tool/templates/lgr_idn_table_review_tool/list_reports.html
@@ -21,18 +21,16 @@
-
{% with report=reports_grouped.list|first %}
{% with report_id=report.report_id %}
-
- {{ report_id }}
-
-
-
- {{ report|display_expiration }}
+
{% endwith %}
{% endwith %}
diff --git a/src/lgr_idn_table_review/templates/lgr_idn_table_review/list_report_files.html b/src/lgr_idn_table_review/templates/lgr_idn_table_review/list_report_files.html
index e037fd2b..b789ddf5 100644
--- a/src/lgr_idn_table_review/templates/lgr_idn_table_review/list_report_files.html
+++ b/src/lgr_idn_table_review/templates/lgr_idn_table_review/list_report_files.html
@@ -37,14 +37,18 @@
{% endif %}
-
-
-
-
-
-
+
|
{% endfor %}
diff --git a/src/lgr_models/templates/lgr_models/_report_list.html b/src/lgr_models/templates/lgr_models/_report_list.html
index 675bffd8..fa51cc05 100644
--- a/src/lgr_models/templates/lgr_models/_report_list.html
+++ b/src/lgr_models/templates/lgr_models/_report_list.html
@@ -5,14 +5,18 @@
{% for report in reports %}
-
-
- {% trans 'Download' %} {{ report.filename }}
-
-
-
-
+
{{ report|display_expiration }}
{% endfor %}
diff --git a/src/lgr_session/views.py b/src/lgr_session/views.py
index bbf91ebd..36af540b 100644
--- a/src/lgr_session/views.py
+++ b/src/lgr_session/views.py
@@ -56,8 +56,7 @@ def get(self, request, *args, **kwargs):
class DeleteReportFileView(LGRSessionView):
- # TODO make that a post
- def get(self, request, *args, **kwargs):
+ def post(self, request, *args, **kwargs):
if self.pk:
self.session.storage_delete_report_file(self.pk)
if self.report_id: