django - AttributeError: 'NoneType' object has no attribute 'first_name'

w_lpz picture w_lpz · Jun 20, 2016 · Viewed 15.5k times · Source

Would appreciate some help understanding whenever I try to setup my client I am getting the following error: AttributeError: 'NoneType' object has no attribute 'first_name'.

Here is the log:

Environment:


Request Method: GET
Request URL: http://192.168.33.10:8000/podfunnel/clientsetup/

Django Version: 1.9
Python Version: 2.7.6
Installed Applications:
('producer',
 'django.contrib.admin',
 'django.contrib.sites',
 'registration',
 'django.contrib.auth',
 'django.contrib.contenttypes',
 'django.contrib.sessions',
 'django.contrib.messages',
 'django.contrib.staticfiles',
 'storages',
 'django_extensions',
 'randomslugfield',
 'adminsortable2',
 'crispy_forms')
Installed Middleware:
('django.contrib.sessions.middleware.SessionMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.csrf.CsrfViewMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'django.contrib.auth.middleware.SessionAuthenticationMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware',
 'django.middleware.clickjacking.XFrameOptionsMiddleware',
 'django.middleware.security.SecurityMiddleware')



Traceback:

File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py" in get_response
  149.                     response = self.process_exception_by_middleware(e, request)

File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py" in get_response
  147.                     response = wrapped_callback(request, *callback_args, **callback_kwargs)

File "/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py" in view
  68.             return self.dispatch(request, *args, **kwargs)

File "/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py" in dispatch
  88.         return handler(request, *args, **kwargs)

File "/home/vagrant/fullcast_project/producer/views/pod_funnel.py" in get
  110.         initial_values['first_name'] = client.first_name

Exception Type: AttributeError at /podfunnel/clientsetup/
Exception Value: 'NoneType' object has no attribute 'first_name'

It has to do in my ClientSetupView under y views.py:

class ClientSetupView(View):
form_class = ClientSetupForm
template_name = 'pod_funnel/forms.html'

# In the get we manage the request to get the form and prefill it if necessary
def get(self, request, *args, **kwargs):
    # We need to check first if the current user has a client and podcast setup,
    # if so, prepopulate. Otherwise get empty form.
    initial_values = {}
    user = request.user

    if Client.objects.filter(user=user).exists():

        client = CLient.objects.filter(user=user).first()

        initial_values['first_name'] = client.first_name
        initial_values['last_name'] = client.last_name

        podcast = Podcast.objects.filter(client=client).first()
        request.session['client_id'] = client.id

        if podcast:
            initial_values['podcast_name'] = podcast.name
            # lets store podcast_id in session so we can retrieve it directly in post
            request.session['podcast_id'] = podcast.id

    form = self.form_class(initial=initial_values)
    return render(request, self.template_name, {'form': form})

# In the the post we manage updates to the data
def post(self, request, *args, **kwargs):
    form = self.form_class(request.POST)

    if form.is_valid():
        # lets get the data
        first_name = form.cleaned_data.get('first_name')
        last_name = form.cleaned_data.get('last_name')
        podcast_name = form.cleaned_data.get('podcast_name')

        # First we see if we have a client_id in session, if so we retrieve and update
        # otherwise we create a new one.
        client_id = request.session.get('client_id', None)
        if client_id is not None:
            request.session.delete('client_id')
            client = Client.objects.get(id=client_id)
        else:
            client = Client()
        client.first_name = first_name
        client.last_name = last_name
        client.company_name = podcast_name
        client.save()

        # Now we update or create the associated podcast.
        podcast_id = request.session.get('podcast_id', None)
        if podcast_id is not None:
            request.session.delete('podcast_id')
            podcast = Podcast.objects.get(id=podcast_id)
        else:
            podcast = Podcast()
        podcast.client = client
        podcast.name = podcast_name
        podcast.save()

        success, message_list = update_or_create_preset_for_podcast(podcast)

        # in any case we can continue with additional podcast setup, but we need
        # to attempt to create the preset one more time at some point before the
        # user can create the first productions.
        return HttpResponseRedirect(reverse('podfunnel:podcastsetup'))

    return render(request, self.template_name, {'form': form})

Answer

Moses Koledoye picture Moses Koledoye · Jun 20, 2016

The line were you check if client is not None should come immediately after reading from the database as first may return None if there are no items:

client = Client.objects.filter(user=user).first()
if client is not None:
    initial_values['first_name'] = client.first_name
    initial_values['last_name'] = client.last_name

Or you can use exists to test if there is at least one client for that user before fetching first:

if Client.objects.filter(user=user).exists():
    client = Client.objects.filter(user=user).first()
    initial_values['first_name'] = client.first_name
    initial_values['last_name'] = client.last_name

The exists approach is more efficient:

Returns True if the QuerySet contains any results, and False if not. This tries to perform the query in the simplest and fastest way possible, but it does execute nearly the same query as a normal QuerySet query.