Edgewall Software

Opened 7 years ago

Last modified 7 years ago

#12838 closed defect

Set default schema — at Initial Version

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Milestone: 1.2.3
Component: database backend Version:
Severity: normal Keywords: postgres
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

The issue described in comment:11:ticket:12643 can be easily reproduced by running the tests using a database URI that doesn't specify a schema (postgres://tracuser:tracpassword@localhost:5432/trac), and the following patch:

  • trac/test.py 

    diff --git a/trac/test.py b/trac/test.py
index a37342048..ea5657021 100755
    a b def get_dburi():  
    284284        # Assume the schema 'tractest' for PostgreSQL
    285285        if scheme == 'postgres' and \
    286286                not db_prop.get('params', {}).get('schema'):
    287             dburi += ('&' if '?' in dburi else '?') + 'schema=tractest'
     287            pass
     288            # dburi += ('&' if '?' in dburi else '?') + 'schema=tractest'
    288289        elif scheme == 'sqlite' and db_prop['path'] != ':memory:' and \
    289290                not db_prop.get('params', {}).get('synchronous'):
    290291            # Speed-up tests with SQLite database

Do we want to set the public schema by default?

Change History (0)

Note: See TracTickets for help on using tickets.