Ticket #2332: is_authenticated.diff

File is_authenticated.diff, 13.5 KB (added by Gary Wilson <gary.wilson@…>, 18 years ago)

is_authenticated implementation with doc changes and NO deprecation warnings in is_anonymous

  • django/views/generic/create_update.py

     
    2020            the form wrapper for the object
    2121    """
    2222    if extra_context is None: extra_context = {}
    23     if login_required and request.user.is_anonymous():
     23    if login_required and not request.user.is_authenticated():
    2424        return redirect_to_login(request.path)
    2525
    2626    manipulator = model.AddManipulator(follow=follow)
    … …  
    3939            # No errors -- this means we can save the data!
    4040            new_object = manipulator.save(new_data)
    4141
    42             if not request.user.is_anonymous():
     42            if request.user.is_authenticated():
    4343                request.user.message_set.create(message="The %s was created successfully." % model._meta.verbose_name)
    4444
    4545            # Redirect to the new object: first by trying post_save_redirect,
    … …  
    8686            the original object being edited
    8787    """
    8888    if extra_context is None: extra_context = {}
    89     if login_required and request.user.is_anonymous():
     89    if login_required and not request.user.is_authenticated():
    9090        return redirect_to_login(request.path)
    9191
    9292    # Look up the object to be edited
    … …  
    113113        if not errors:
    114114            object = manipulator.save(new_data)
    115115
    116             if not request.user.is_anonymous():
     116            if request.user.is_authenticated():
    117117                request.user.message_set.create(message="The %s was updated successfully." % model._meta.verbose_name)
    118118
    119119            # Do a post-after-redirect so that reload works, etc.
    … …  
    162162            the original object being deleted
    163163    """
    164164    if extra_context is None: extra_context = {}
    165     if login_required and request.user.is_anonymous():
     165    if login_required and not request.user.is_authenticated():
    166166        return redirect_to_login(request.path)
    167167
    168168    # Look up the object to be edited
    … …  
    180180
    181181    if request.method == 'POST':
    182182        object.delete()
    183         if not request.user.is_anonymous():
     183        if request.user.is_authenticated():
    184184            request.user.message_set.create(message="The %s was deleted." % model._meta.verbose_name)
    185185        return HttpResponseRedirect(post_delete_redirect)
    186186    else:

  • django/contrib/auth/models.py

     
    125125    def is_anonymous(self):
    126126        "Always returns False. This is a way of comparing User objects to anonymous users."
    127127        return False
     128   
     129    def is_authenticated(self):
     130        """Always return True.
     131        This is a way to tell if the user is logged in when using request.user
     132        in views or user in templates.
     133        """
     134        return True
    128135
    129136    def get_full_name(self):
    130137        "Returns the first_name plus the last_name, with a space in between."
    … …  
    293300
    294301    def is_anonymous(self):
    295302        return True
     303   
     304    def is_authenticated(self):
     305        return False

  • django/contrib/auth/decorators.py

     
    1717        return _checklogin
    1818    return _dec
    1919
    20 login_required = user_passes_test(lambda u: not u.is_anonymous())
     20login_required = user_passes_test(lambda u: u.is_authenticated())
    2121login_required.__doc__ = (
    2222    """
    2323    Decorator for views that checks that the user is logged in, redirecting

  • django/contrib/comments/templatetags/comments.py

     
    114114        comment_list = get_list_function(**kwargs).order_by(self.ordering + 'submit_date').select_related()
    115115
    116116        if not self.free:
    117             if context.has_key('user') and not context['user'].is_anonymous():
     117            if context.has_key('user') and context['user'].is_authenticated():
    118118                user_id = context['user'].id
    119119                context['user_can_moderate_comments'] = Comment.objects.user_is_moderator(context['user'])
    120120            else:

  • django/contrib/comments/views/karma.py

     
    1515    rating = {'up': 1, 'down': -1}.get(vote, False)
    1616    if not rating:
    1717        raise Http404, "Invalid vote"
    18     if request.user.is_anonymous():
     18    if not request.user.is_authenticated():
    1919        raise Http404, _("Anonymous users cannot vote")
    2020    try:
    2121        comment = Comment.objects.get(pk=comment_id)

  • django/contrib/comments/views/comments.py

     
    6363                validator_list=get_validator_list(8),
    6464            ),
    6565        ])
    66         if not user.is_anonymous():
     66        if user.is_authenticated():
    6767            self["username"].is_required = False
    6868            self["username"].validator_list = []
    6969            self["password"].is_required = False

  • django/contrib/comments/templates/comments/form.html

     
    22{% if display_form %}
    33<form {% if photos_optional or photos_required %}enctype="multipart/form-data" {% endif %}action="/comments/post/" method="post">
    44
    5 {% if user.is_anonymous %}
     5{% if user.is_authenticated %}
     6<p>{% trans "Username:" %} <strong>{{ user.username }}</strong> (<a href="/accounts/logout/">{% trans "Log out" %}</a>)</p>
     7{% else %}
    68<p><label for="id_username">{% trans "Username:" %}</label> <input type="text" name="username" id="id_username" /><br />{% trans "Password:" %} <input type="password" name="password" id="id_password" /> (<a href="/accounts/password_reset/">{% trans "Forgotten your password?" %}</a>)</p>
    7 {% else %}
    8 <p>{% trans "Username:" %} <strong>{{ user.username }}</strong> (<a href="/accounts/logout/">{% trans "Log out" %}</a>)</p>
    99{% endif %}
    1010
    1111{% if ratings_optional or ratings_required %}

  • django/contrib/flatpages/views.py

     
    2222    f = get_object_or_404(FlatPage, url__exact=url, sites__id__exact=settings.SITE_ID)
    2323    # If registration is required for accessing this page, and the user isn't
    2424    # logged in, redirect to the login page.
    25     if f.registration_required and request.user.is_anonymous():
     25    if f.registration_required and not request.user.is_authenticated():
    2626        from django.contrib.auth.views import redirect_to_login
    2727        return redirect_to_login(request.path)
    2828    if f.template_name:

  • django/contrib/admin/views/decorators.py

     
    4646    member, displaying the login page if necessary.
    4747    """
    4848    def _checklogin(request, *args, **kwargs):
    49         if not request.user.is_anonymous() and request.user.is_staff:
     49        if request.user.is_authenticated() and request.user.is_staff:
    5050            # The user is valid. Continue to the admin page.
    5151            if request.POST.has_key('post_data'):
    5252                # User must have re-authenticated through a different window

  • django/contrib/admin/templates/admin/base.html

     
    2020        <div id="branding">
    2121        {% block branding %}{% endblock %}
    2222        </div>
    23         {% if not user.is_anonymous %}{% if user.is_staff %}
     23        {% if user.is_authenticated and user.is_staff %}
    2424        <div id="user-tools">{% trans 'Welcome,' %} <strong>{% if user.first_name %}{{ user.first_name|escape }}{% else %}{{ user.username }}{% endif %}</strong>. {% block userlinks %}<a href="doc/">{% trans 'Documentation' %}</a> / <a href="password_change/">{% trans 'Change password' %}</a> / <a href="logout/">{% trans 'Log out' %}</a>{% endblock %}</div>
    25         {% endif %}{% endif %}
     25        {% endif %}
    2626        {% block nav-global %}{% endblock %}
    2727    </div>
    2828    <!-- END Header -->

  • docs/request_response.txt

     
    106106    A ``django.contrib.auth.models.User`` object representing the currently
    107107    logged-in user. If the user isn't currently logged in, ``user`` will be set
    108108    to an instance of ``django.contrib.auth.models.AnonymousUser``. You
    109     can tell them apart with ``is_anonymous()``, like so::
     109    can tell them apart with ``is_authenticated()``, like so::
    110110
    111         if request.user.is_anonymous():
     111        if request.user.is_authenticated():
     112            # Do something for logged-in users.
     113        else:
    112114            # Do something for anonymous users.
    113         else:
    114             # Do something for logged-in users.
    115115
    116116    ``user`` is only available if your Django installation has the
    117117    ``AuthenticationMiddleware`` activated. For more, see

  • docs/authentication.txt

     
    9595custom methods:
    9696
    9797    * ``is_anonymous()`` -- Always returns ``False``. This is a way of
    98       comparing ``User`` objects to anonymous users.
     98      differentiating ``User`` and ``AnonymousUser`` objects. Do not use
     99      this method for determining if the user has been authenticated.  For
     100      that, use ``is_authenticated()``.
    99101
     102    * ``is_authenticated()`` -- Always returns ``True``. This is a way to
     103      tell if the user has been authenticated when using ``request.user`` in
     104      views or ``user`` in templates.
     105
    100106    * ``get_full_name()`` -- Returns the ``first_name`` plus the ``last_name``,
    101107      with a space in between.
    102108
    … …  
    219225
    220226    * ``id`` is always ``None``.
    221227    * ``is_anonymous()`` returns ``True`` instead of ``False``.
     228    * ``is_authenticated()`` returns ``False`` instead of ``True``.
    222229    * ``has_perm()`` always returns ``False``.
    223230    * ``set_password()``, ``check_password()``, ``save()``, ``delete()``,
    224231      ``set_groups()`` and ``set_permissions()`` raise ``NotImplementedError``.
    … …  
    254261``request.user`` in views. ``request.user`` will give you a ``User`` object
    255262representing the currently logged-in user. If a user isn't currently logged in,
    256263``request.user`` will be set to an instance of ``AnonymousUser`` (see the
    257 previous section). You can tell them apart with ``is_anonymous()``, like so::
     264previous section). You can tell them apart with ``is_authenticated()``, like so::
    258265
    259     if request.user.is_anonymous():
     266    if request.user.is_authenticated():
     267        # Do something for authenticated users.
     268    else:
    260269        # Do something for anonymous users.
    261     else:
    262         # Do something for logged-in users.
    263270
    264271.. _request objects: http://www.djangoproject.com/documentation/request_response/#httprequest-objects
    265272.. _session documentation: http://www.djangoproject.com/documentation/sessions/
    … …  
    323330~~~~~~~~~~~
    324331
    325332The simple, raw way to limit access to pages is to check
    326 ``request.user.is_anonymous()`` and either redirect to a login page::
     333``request.user.is_authenticated()`` and either redirect to a login page::
    327334
    328335    from django.http import HttpResponseRedirect
    329336
    330337    def my_view(request):
    331         if request.user.is_anonymous():
     338        if not request.user.is_authenticated():
    332339            return HttpResponseRedirect('/login/?next=%s' % request.path)
    333340        # ...
    334341
    335342...or display an error message::
    336343
    337344    def my_view(request):
    338         if request.user.is_anonymous():
     345        if not request.user.is_authenticated():
    339346            return render_to_response('myapp/login_error.html')
    340347        # ...
    341348
    … …  
    439446permission ``polls.can_vote``::
    440447
    441448    def my_view(request):
    442         if request.user.is_anonymous() or not request.user.has_perm('polls.can_vote'):
     449        if not request.user.is_authenticated() or not request.user.has_perm('polls.can_vote'):
    443450            return HttpResponse("You can't vote in this poll.")
    444451        # ...
    445452
    … …  
    605612The currently logged-in user, either a ``User`` instance or an``AnonymousUser``
    606613instance, is stored in the template variable ``{{ user }}``::
    607614
    608     {% if user.is_anonymous %}
     615    {% if user.is_authenticated %}
     616        <p>Welcome, {{ user.username }}. Thanks for logging in.</p>   
     617    {% else %}
    609618        <p>Welcome, new user. Please log in.</p>
    610     {% else %}
    611         <p>Welcome, {{ user.username }}. Thanks for logging in.</p>
    612619    {% endif %}
    613620
    614621Permissions
Back to Top