[patch] Fixtures for postgresql and problems with colors.

[anonymous]

I have this fixture (yaml format):

- model: sites.Site
  pk: 1
  fields:
    domain: localhost:8000
    name: localhost:8000

I try to load this fixture:

> ./manage.py loaddata sites
Loading 'sites' fixtures...
Installing yaml fixture 'sites' from ...
Installed 1 object(s) from 1 fixture(s)
Traceback (most recent call last):
  File "./manage.py", line 12, in <module>
    execute_manager(settings)
  File "/usr/local/lib/python2.5/site-packages/django/core/management.py", line 1668, in execute_manager
    execute_from_command_line(action_mapping, argv)
  File "/usr/local/lib/python2.5/site-packages/django/core/management.py", line 1582, in execute_from_command_line
    action_mapping[action](args[1:], int(options.verbosity))
  File "/usr/local/lib/python2.5/site-packages/django/core/management.py", line 1412, in load_data
    cursor.execute(line)
  File "/usr/local/lib/python2.5/site-packages/django/db/backends/util.py", line 12, in execute
    return self.cursor.execute(sql, params)
psycopg2.ProgrammingError: syntax error at or near " "
LINE 1: SELECT setval('django_site_id_seq', ( [3...
        ^

If I use patch show-sql.diff (SELECT is with colors):

> ./manage.py loaddata sites
Loading 'sites' fixtures...
Installing yaml fixture 'sites' from ...
Installed 1 object(s) from 1 fixture(s)
SELECT setval('django_site_id_seq', (SELECT max("id") FROM "django_site"));
'\x1b[33mSELECT\x1b[0m setval(\'\x1b[32;1mdjango_site_id_seq\x1b[0m\', (\x1b[33mSELECT\x1b[0m max(\x1b[32;1m"id"\x1b[0m) \x1b[33mFROM\x1b[0m \x1b[1m"django_site"\x1b[0m));'
Traceback (most recent call last):
  ...

If I use patch no-colors-for-sql.diff, loading is ok.

I use postgresql 8.2.3, psycopg 2.0.5 and python 2.5.1 rc1 (Ubuntu 6.04 Feisty Fawn), but I also tried psycopg 1.1.21 and/or python 2.4.4 (four unsuccessful attempts and two successful attempts - patch is only for psycopg2).

[anonymous]

Maybe this part of django can be relevant (django.core.management.py, 49-51):

# Disable terminal coloring on Windows, Pocket PC, or if somebody's piping the output.
if sys.platform == 'win32' or sys.platform == 'Pocket PC' or not sys.stdout.isatty():
    disable_termcolors()

Attempt with pipe (and without patches):

./manage.py loaddata sites | cat -
Loading 'sites' fixtures...
Installing yaml fixture 'sites' from ...
Installed 1 object(s) from 1 fixture(s)

[anonymous]

Marked #4012 as a duplication of this.
no_colors_for_sql_pg1.diff is a patch for the psycopg 1.x layer.

[Vinay Sajip <vinay_sajip@…>]

Fix for missing disable_termcolors() in load_data

[Vinay Sajip <vinay_sajip@…>]

The problem appears to be caused by a missing call to disable_termcolors() in load_data(). Fixed in patch: loaddata_nocolors.diff.

[Vinay Sajip <vinay_sajip@…>]

Hope it's OK to set this as Accepted, since it seems to be a clear-cut bug (raised by another user). Don't feel it needs any documentation or unit test.

[Russell Keith-Magee]

(In [5102]) Fixed #3954 -- Disabled termcolors when loading fixtures. Well spotted, Vijay Sajip.

[Vinay Sajip <vinay_sajip@…>]

Errr... that's "Vinay", not "Vijay". Thanks for committing the fix!

[Russell Keith-Magee]

Replying to Vinay Sajip <vinay_sajip@yahoo.co.uk>:

Errr... that's "Vinay", not "Vijay". Thanks for committing the fix!

Humble apologies. For what it's worth, I got your name right in the AUTHORS file :-)