Changes between Initial Version and Version 1 of Ticket #16919, comment 4


Ignore:
Timestamp:
Dec 17, 2011, 2:51:34 PM (12 years ago)
Author:
Ethan Jucovy

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #16919, comment 4

    initial v1  
    33Currently the `password_reset_confirm` view does not provide "user" as a template context variable, nor even "uidb36" and "token".  Since the form also doesn't have the user object stored on a GET request, this means that there's no way for the template to say "{% if validlink %} Hello, {{ user.username }} -- reset your password here {% endif %}" -- short of forking the view, or some pretty hacky middleware that re-parses the request URL and re-fetches the user from the given uid+token.
    44
     5If this patch were accepted, that would be possible, like so: "{% if validlink %} Hello, {{ form.user.username }} -- reset your password here {% endif %}"
     6
    57I see that the "needs_tests" flag is set on this ticket .. what sort of test would be required for this patch to be merged?
Back to Top