Opened 15 years ago

Closed 15 years ago

#10634 closed (invalid)

Confusing differential exceptions: get_or_create vs get with psycopg2

Reported by: fergusferrier Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: Keywords: psycopg2 get_or_create get misleading
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


I'm using Psycopg2. I hope the models here are somewhat self-explanatory in structure.

I made the simple mistake of forgetting the boolean 'created' in a get_or_create...

   usr = EventsUser.objects.get_or_create(user = request.user)

Then running the following code [NB. usr is not an instance of EventsUser at this point]...

    events_user = usr,
    event = event,
    assoctype = 'EML_HR'

gives thus:

Traceback (most recent call last):
  File "/home/django/domains/", line 418, in test_set_reminder_unset
    response = self.client.get(reverse('Events.views.SetReminders', args=[]), {'day': '1', 'hour': '1'})
  File "/home/django/domains/", line 278, in get
    response = self.request(**r)
  File "/home/django/domains/", line 91, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "/home/django/domains/", line 35, in decorator
    return f(request, *args, **kwargs)
  File "/home/django/domains/", line 319, in SetReminders
    assoctype = 'EML_HR'
  File "/home/django/domains/", line 120, in get_or_create
    return self.get_query_set().get_or_create(**kwargs)
  File "/home/django/domains/", line 365, in get_or_create
    return self.get(**kwargs), False
  File "/home/django/domains/", line 337, in get
    num = len(clone)
  File "/home/django/domains/", line 161, in __len__
    self._result_cache = list(self.iterator())
  File "/home/django/domains/", line 281, in iterator
    for row in self.query.results_iter():
  File "/home/django/domains/", line 256, in results_iter
    for rows in self.execute_sql(MULTI):
  File "/home/django/domains/", line 2098, in execute_sql
    cursor.execute(sql, params)
  File "build/bdist.linux-x86_64/egg/psycopg2/", line 96, in getquoted
    pobjs = [adapt(o) for o in self._seq]
ProgrammingError: can't adapt

When one replaces the second get_or_create with a plain get, you are returned

    ValueError: Cannot assign "(<EventsUser: EventsUser object>, False)": "Reminder.events_user" must be a "EventsUser" instance.

which is a much more useful exception. I would expect get_or_create to return a similar.

Change History (1)

comment:1 by Jacob, 15 years ago

Resolution: invalid
Status: newclosed

You're passing the wrong type (a 2-tuple) into the database binding, so you're getting a database error. If we typecheck everything coming in we're going to make queries slower, so this is a "you need to get it right in your own code" situation.

Note: See TracTickets for help on using tickets.
Back to Top