Merge pull request #102 from allink/amal/feat/legacy-redirect-last-ch…

…ar-adjustment legacy redirect: removed the deletion of the last character as it led…
allink · Aug 17, 2020 · 122658e · 122658e
2 parents 63adda1 + a1e8d8a
commit 122658e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -18,6 +18,9 @@ new features or plugins
 #### FIXES
 general bugfixes
 
+## v2.6.3
+#### FIXES
+- removed Query filter from legacy redirect that removed the last character as it led to multiple matches
 
 ## v2.6.2
 #### FIXES

diff --git a/allink_core/__init__.py b/allink_core/__init__.py
@@ -1,6 +1,6 @@
 import os
 
-__version__ = '2.6.2'
+__version__ = '2.6.3'
 
 # Cheeky setting that allows each template to be accessible by two paths.
 # Eg: the template 'allink/templates/allink/base.html' can be accessed via both

diff --git a/allink_core/core_apps/allink_legacy_redirect/middleware.py b/allink_core/core_apps/allink_legacy_redirect/middleware.py
@@ -27,7 +27,7 @@ def __call__(self, request):  # noqa TODO this needs to be refactored anyway, so
         link = None
         try:
             link = AllinkLegacyLink.objects.get(Q(old=request.path) | Q(
-                old=request.path + '/') | Q(old=request.path[:-1]) | Q(old=request.get_full_path()), active=True)
+                old=request.path + '/') | Q(old=request.get_full_path()), active=True)
 
             # if user is logged in, skip redirect
             if link.redirect_when_logged_out and request.user.is_authenticated: