Ticket #34066: patch.diff

django/contrib/auth/forms.py

@@ -163 +163 @@
         super().__init__(*args, **kwargs)
         password = self.fields.get("password")
         if password:
-            password.help_text = password.help_text.format("../password/")
+            password.help_text = password.help_text.format(
+                f"../../{self.instance.pk}/password/"
+            )
         user_permissions = self.fields.get("user_permissions")
         if user_permissions:
             user_permissions.queryset = user_permissions.queryset.select_related(

tests/auth_tests/test_forms.py

@@ -1 +1 @@
 import datetime
 import re
+import urllib.parse
 from unittest import mock
 
 from django.contrib.auth.forms import (
@@ -22 +23 @@
 from django.forms import forms
 from django.forms.fields import CharField, Field, IntegerField
 from django.test import SimpleTestCase, TestCase, override_settings
+from django.urls import reverse
 from django.utils import translation
 from django.utils.text import capfirst
 from django.utils.translation import gettext as _
@@ -867 +869 @@
         # value to render correctly
         self.assertEqual(form.initial["password"], form["password"].value())
 
+    @override_settings(ROOT_URLCONF="auth_tests.urls_admin")
+    def test_bug_34066_link_to_password_reset_in_helptext(self):
+        user = User.objects.get(username="testclient")
+        form = UserChangeForm(data={}, instance=user)
+        passsword_help_text = form.fields["password"].help_text
+        matches = re.search('<a href="(.*?)">', passsword_help_text)
+
+        # Get URL to UserChangeForm in admin via to_field (instead of pk)
+        admin_user_change_url = f"""{reverse(
+            "admin:%s_%s_change" % (user._meta.app_label, user._meta.model_name),
+            args=(user.username,),
+        )}?to_field=username"""
+
+        # join relative url found in password's help_text to admin_user_change_url
+        joined_url = urllib.parse.urljoin(admin_user_change_url, matches.group(1))
+
+        # Get URL to PasswordChangeForm in admin
+        pw_change_url = reverse(
+            "admin:%s_%s_password_change"
+            % (user._meta.app_label, user._meta.model_name),
+            args=(user.pk,),
+        )
+        # assert joined_url and pw_change_url are ideential
+        self.assertEqual(joined_url, pw_change_url)
+
     def test_custom_form(self):
         class CustomUserChangeForm(UserChangeForm):
             class Meta(UserChangeForm.Meta):