PostgresqlPatch: pgsql_patch_12.diff

scripts/trac-admin

@@ -27 +27 @@
-import time
-import cmd
-import shlex
-import sqlite
-import StringIO
+import traceback
-
-from trac import perm
-from trac import util
@@ -96 +96 @@
-            return 0
-        return 1
-        
-
+, db_str
-        try:
-create=1
+1, db_str
-            return self.__env
-        except Exception, e:
-            print 'Failed to create environment.', e
+            print traceback.print_exc()
-            sys.exit(1)
-
-    def db_open(self):
@@ -115 +116 @@
-
-    def db_execsql (self, sql, cursor=None):
-        data = []
+        row = None
-        if not cursor:
-            cnx=self.db_open()
-            cursor = cnx.cursor()
@@ -122 +124 @@
-            cnx = None
-        cursor.execute(sql)
-        while 1:
-
+
+
+
+
-            if row == None:
-                break
-            data.append(row)
@@ -455 +460 @@
-
-    ## Initenv
-    _help_initenv = [('initenv', 'Create and initialize a new environment interactively'),
-
+ <dbms> [dbms options]
-                      'Create and initialize a new environment from arguments')]
-
-    def do_initdb(self, line):
@@ -488 +493 @@
-        dt = trac.siteconfig.__default_templates_dir__
-        prompt = 'Templates directory [%s]> ' % dt
-        returnvals.append(raw_input(prompt) or dt)
+        print
+        print " Please enter the database in which you wish to store Trac data."
+        print " Default is 'sqlite', but others are supported:"
+        print " 'sqlite'  - SQLite       http://www.sqlite.org"
+        print " 'pypgsql' - PostgreSQL   http://www.postgresql.org"
+        print
+        ddb = "sqlite"
+        prompt =  'database system [%s]> ' % ddb
+        dbms = raw_input(prompt) or ddb
+        returnvals.append(dbms)
+        
+        # PostgreSQL - additional questions
+        if dbms == "pypgsql":
+            print
+            print " Please enter the host of your PostgreSQL database."
+            print " Default is 'localhost'"
+            print
+            host = "localhost"
+            prompt = "PostgreSQL host [%s]> " % host
+            returnvals.append(raw_input(prompt) or host)
+            
+            print
+            print " Please enter the port of your PostgreSQL database."
+            print " Default is '5432'"
+            print
+            port = "5432"
+            prompt = "PostgreSQL port [%s]> " % port
+            returnvals.append(raw_input(prompt) or port)
+            
+            print
+            print " Please enter the name of your PostgreSQL database."
+            print " Default is 'trac'"
+            print " Note: This database should not exist, as trac-admin will" \
+                  " attempt to create it."
+            print
+            name = "trac"
+            prompt = "PostgreSQL database [%s]> " % name
+            returnvals.append(raw_input(prompt) or name)
+            
+            print
+            print " Please enter the user of your PostgreSQL database."
+            print " Default is 'trac'"
+            print
+            user = "trac"
+            prompt = "PostgreSQL user [%s]> " % user
+            returnvals.append(raw_input(prompt) or user)
+            
+            print
+            print " Please enter the password of your PostgreSQL database."
+            print " Default is 'trac'"
+            print
+            password = "trac"
+            prompt = "PostgreSQL password [%s]> " % password
+            returnvals.append(raw_input(prompt) or password)
+            
-        return returnvals
-         
-    def do_initenv(self, line):
@@ -498 +558 @@
-        project_name = None
-        repository_dir = None
-        templates_dir = None
+        db_str = None
-        if len(arg) == 1:
-            returnvals = self.get_initenv_args()
-            project_name = returnvals[0]
-            repository_dir = returnvals[1]
-            templates_dir = returnvals[2]
-
+
+
-            print 'Wrong number of arguments to initenv %d' % len(arg)
-            return
-        else:
-            project_name = arg[0]
-            repository_dir = arg[1]
-            templates_dir = arg[2]
+            db_str = arg[3]
+        
+        # sqlite-specific stuff
+        if db_str.lower() == "sqlite":
+            db_str = 'sqlite:db/trac.db'
+        
+        # postgres-specific stuff
+        if db_str.lower() == "pypgsql":
+            if len(arg) == 1:
+                host = returnvals[4]
+                port = returnvals[5]
+                database = returnvals[6]
+                user = returnvals[7]
+                password = returnvals[8]
+            elif len(arg) != 9:
+                print 'Wrong number of arguments to initenv %d' % len(arg)
+                print 'For PostgreSQL (after pypgsql): <host> <port> <database>' \
+                      ' <user> <password>'
+                return
+            else:
+                host = arg[4]
+                port = arg[5]
+                database = arg[6]
+                user = arg[7]
+                password = arg[8]
+            db_str = "pypgsql:host='%s:%s',database='%s',user='%s'," \
+                     "password='%s'" \
+                     % (host, port, database, user, password)
+            createdb_str = "createdb --host=%s --port=%s --owner=%s " \
+                           "--username=%s %s\n" \
+                           % (host, port, user, user, database)
+            print createdb_str
+            try:
+                # TODO: this is not cross-platform.
+                # for Windows, look at win32pipe.popen()
+                os.popen(createdb_str)
+            except Exception, e:
+                print "Error creating PostgreSQL database: %s" % e
+                print "command: %s" % createdb_str
+        
-        from svn import util, repos, core
-        core.apr_initialize()
-        pool = core.svn_pool_create(None)
@@ -528 +630 @@
-            return
-        try:
-            print 'Creating and Initializing Project'
-
+
+
+
-            cnx = self.__env.get_db_cnx()
-            print ' Inserting default data'
-            self.__env.insert_default_data()
@@ -541 +645 @@
-            print '  project.name'
-            self.__env.set_config('project', 'name', project_name)
-            self.__env.save_config()
+            
-            # Add a few default wiki pages
-            print ' Installing wiki pages'
-            cursor = cnx.cursor()
@@ -550 +655 @@
-            sync.sync(cnx, rep, fs_ptr, pool)
-        except Exception, e:
-            print 'Failed to initialize database.', e
+            print traceback.print_exc()
-            sys.exit(2)
-
-
@@ -671 +777 @@
-                self.do_help ('wiki')
-        except Exception, e:
-            print 'Wiki %s failed:' % arg[0], e
+            print traceback.print_exc()
+            
-
-    def _do_wiki_list(self):
-        data = self.db_execsql('SELECT name,max(version),time'

setup.py

@@ -199 +199 @@
-      author_email="info@edgewall.com",
-      license=LICENSE,
-      url=URL,
-
+
+
-      data_files=[(_p('share/trac/templates'), glob('templates/*')),
-                  (_p('share/trac/htdocs'), glob(_p('htdocs/*.*')) + [_p('htdocs/README')]),
-                  (_p('share/trac/htdocs/css'), glob(_p('htdocs/css/*'))),

trac/db_default.py

@@ -37 +37 @@
-## Default data
-##
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-
-CREATE TABLE session (
-         sid             text,
-         username        text,
-         var_name        text,
-         var_value       text,
-         UNIQUE(sid,var_name)
-);
-
-CREATE INDEX node_change_idx    ON node_change(rev);
-CREATE INDEX ticket_change_idx  ON ticket_change(ticket, time);
-CREATE INDEX wiki_idx           ON wiki(name,version);
-CREATE INDEX session_idx        ON session(sid,var_name);
-"""
-
-##
-## Default Reports
-##
@@ -286 +261 @@
-  FROM ticket t,enum p
-  WHERE p.name=t.priority AND p.type='priority'
-  ORDER BY (milestone IS NULL), milestone DESC, (status = 'closed'), 
-modified
+changetime
-"""),
-#----------------------------------------------------------------------------
-('My Tickets',

trac/dbms/db_pypgsql.py

@@ +1 @@
+# -*- coding: iso8859-1 -*-
+#
+# Copyright (C) 2005 Edgewall Software
+# Copyright (C) 2005 Christopher Lenz <cmlenz@gmx.de>
+#
+# Trac is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation; either version 2 of the
+# License, or (at your option) any later version.
+#
+# Trac is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+# General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+#
+# Author: Christopher Lenz <cmlenz@gmx.de>
+
+import os
+from pyPgSQL import PgSQL
+import string
+
+from trac import db_default
+
+from trac.db import ConnectionWrapper
+
+from trac.util import TracError
+
+class PyPgSQLConnection(ConnectionWrapper):
+    """
+    Connection wrapper for PostgreSQL using the PyPgSQL driver.
+    """
+
+    __slots__ = ['cnx']
+
+    def __init__(self, dbargs, env, create=0):
+        if create:
+            self.create_db(env.path, dbargs)
+        
+        # TODO: check to see if db exists, or throw error (see sqlite module)
+
+        args = self.parse_args(dbargs)
+                
+        cnx = PgSQL.connect("", **args)
+        ConnectionWrapper.__init__(self, cnx)
+
+    def get_last_id(self, table, field):
+        cursor = self.cursor()
+        sql = "SELECT %s FROM %s " \
+              "WHERE %s = CURRVAL('%s_%s_seq')" \
+              % (field, table, field, table, field)
+        cursor.execute(sql)
+        id = cursor.fetchone()[0]
+        cursor.close()
+        return id
+
+    def create_db(self, path, dbargs):
+        
+        # TODO: make the database
+        # Are we able to do this without su - postgres ??
+        # Should we move what's in trac-admin to here?  Probably...
+        
+        args = self.parse_args(dbargs)
+        
+        # get a connection
+        cnx = PgSQL.connect("", **args)
+        
+        # populate the default schema
+        schema = db_default.schema
+        sql = ""
+        
+        for tables in schema:
+            table = tables[0]
+            fields = tables[1]
+            sql_fields = []
+            unique = []
+            sql += "CREATE TABLE %s (\n" % table
+            
+            for field in fields:
+                name = field[0]
+                type = field[1]
+                size = field[2]
+                pk   = field[3]
+                
+                if type == 'auto':
+                    type = 'serial'
+                    
+                if pk == 'u':
+                    unique.append(name)
+                
+                sql_fields.append(" %s %s" % (name, type))
+                
+            if len(unique):
+                sql_fields.append(" CONSTRAINT %s_pkey PRIMARY KEY(%s)" % 
+                    (table, ",".join(unique)))
+            sql += ",\n".join(sql_fields)
+            sql += "\n);\n\n"
+            
+        cursor = cnx.cursor()
+        cursor.execute(sql)
+        cnx.commit()
+        
+        cursor.close()
+        cnx.close()
+
+    def parse_args(self, dbargs):
+        """very crude code for parsing the arguments in connect_params"""
+        
+        kargs = {}
+        args = dbargs.split(",")
+        
+        for arg in args:
+            pair = arg.split("=")
+            name = pair[0]
+            value = pair[1].strip("'")
+            kargs[name] = value
+            
+        return kargs
+ 
+    def get_search_fields(self):
+        return {'changeset' : 'cast(rev as text)',
+                'ticket'    : 'cast(a.id as text)'}
+                     
+    

trac/dbms/db_sqlite.py

@@ +1 @@
+# -*- coding: iso8859-1 -*-
+#
+# Copyright (C) 2005 Edgewall Software
+# Copyright (C) 2005 Christopher Lenz <cmlenz@gmx.de>
+#
+# Trac is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation; either version 2 of the
+# License, or (at your option) any later version.
+#
+# Trac is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+# General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+#
+# Author: Christopher Lenz <cmlenz@gmx.de>
+
+import os
+import sqlite
+import string
+import traceback
+
+from trac import db_default
+
+from trac.db import ConnectionWrapper
+
+from trac.util import TracError
+
+class SQLiteConnection(ConnectionWrapper):
+    """
+    Connection wrapper for SQLite.
+    """
+
+    __slots__ = ['cnx']
+
+    def __init__(self, dbpath, env, create=0, timeout=10000):
+        
+        if create:
+            self.create_db(env.path, dbpath)
+        
+        if dbpath != ':memory:':
+            if not os.access(dbpath, os.F_OK):
+                raise TracError, traceback.print_exc()
+                raise TracError, 'Database "%s" not found.' % dbpath
+            
+            dbdir = os.path.dirname(dbpath)
+            if not os.access(dbpath, os.R_OK + os.W_OK) or \
+                   not os.access(dbdir, os.R_OK + os.W_OK):
+                raise TracError, 'The web server user requires read _and_ ' \
+                                 'write permission to the database %s and ' \
+                                 'the directory this file is located in.' \
+                                 % dbpath
+
+        import sqlite
+        cnx = sqlite.connect(dbpath, timeout=timeout)
+        ConnectionWrapper.__init__(self, cnx)
+
+    def get_last_id(self, table, field):
+        return self.cnx.db.sqlite_last_insert_rowid()
+        
+    def create_db(self, path, dbpath):
+        
+        # make the directory to hold the database
+        os.mkdir(os.path.join(path, 'db'))
+        
+        # create the database by getting a connection
+        cnx = sqlite.connect(dbpath, timeout=100)
+        
+        # populate the default schema
+        schema = db_default.schema
+        sql = ""
+        
+        for tables in schema:
+            table = tables[0]
+            fields = tables[1]
+            sql_fields = []
+            unique = []
+            sql += "CREATE TABLE %s (\n" % table
+            
+            for field in fields:
+                name = field[0]
+                type = field[1]
+                size = field[2]
+                pk   = field[3]
+                
+                if type == 'auto':
+                    type = 'INTEGER PRIMARY KEY'
+                    
+                if pk == 'u':
+                    unique.append(name)
+                
+                sql_fields.append(" %s %s" % (name, type))
+                
+            if len(unique):
+                sql_fields.append(" UNIQUE(%s)" % ",".join(unique))
+            sql += ",\n".join(sql_fields)
+            sql += "\n);\n\n"
+            
+        cursor = cnx.cursor()
+        cursor.execute(sql)
+        cnx.commit()
+        
+        cursor.close()
+        cnx.close()
+
+    def get_search_fields(self):
+        return {'changeset' : 'rev',
+                'ticket'    : 'a.id'}
+        

trac/Search.py

@@ -135 +135 @@
-                                'Search Error')
-
-        cursor = self.db.cursor ()
-
+
+
-        q = []
-        if changeset:
-            q.append("SELECT 1 as type, message AS title, message, author, "
-rev
+%s
-                     "FROM revision WHERE %s OR %s" %
-
+
+
-                      self.query_to_sql(query, 'author')))
-        if tickets:
-            q.append("SELECT DISTINCT 2 as type, a.summary AS title, "
-                     "a.description AS message, a.reporter AS author, "
-a.id
+%s
-                     "FROM ticket a LEFT JOIN ticket_change b ON a.id = b.ticket "
-                     "WHERE (b.field='comment' AND %s ) OR "
-                     "%s OR %s OR %s OR %s OR %s" %
-
+
+
-                      self.query_to_sql(query, 'summary'),
-                      self.query_to_sql(query, 'keywords'),
-                      self.query_to_sql(query, 'description'),

trac/Query.py

@@ -154 +154 @@
-        sql.append("\nFROM ticket")
-        for k in [k for k in cols if k in custom_fields]:
-           sql.append("\n  LEFT OUTER JOIN ticket_custom AS %s ON " \
-
+ tc
-
-        for col in [c for c in ['status', 'resolution', 'priority', 'severity']
-                    if c == self.order or c == self.group]:
-            sql.append("\n  LEFT OUTER JOIN (SELECT name AS %s_name, " \
-                                            "value AS %s_value " \
-
+ e
-                       " ON %s_name=%s" % (col, col, col, col, col))
-        for col in [c for c in ['milestone', 'version']
-                    if c == self.order or c == self.group]:
-            time_col = col == 'milestone' and 'due' or 'time'
-            sql.append("\n  LEFT OUTER JOIN (SELECT name AS %s_name, " \
-
+ a
-                       " ON %s_name=%s" % (col, time_col, col, col, col, col))
-
-        def get_constraint_sql(name, value, mode, neg):

trac/Report.py

@@ -112 +112 @@
-        cursor = self.db.cursor()
-        cursor.execute("INSERT INTO report (title,sql,description) "
-                       "VALUES (%s,%s,%s)", (title, sql, description))
-
+"report", "id"
-        self.db.commit()
-        req.redirect(self.env.href.report(id))
-

trac/tests/tracadmin-tests.txt

@@ -1 +1 @@
-===== test_help_ok =====
-%(version)s
+0.8
-
-Usage: trac-admin </path/to/projenv> [command [subcommand] [option ...]]
-
-Invoking trac-admin without command starts interactive mode.
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-
-Visit the Trac Project at http://trac.edgewall.com/
-

trac/tests/environment.py

@@ -39 +39 @@
-
-    def test_config(self):
-        """Testing env.get/set_config"""
-
+
+
-        self.env.set_config('foo', 'bar', 'baz')
-        self.env.save_config()
-        assert self.env.get_config('foo', 'bar') == 'baz'

trac/tests/query.py

@@ -45 +45 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,priority,milestone,component,time,changetime
-FROM ticket
-
+e 
-ORDER BY COALESCE(priority,'')='',priority_value,id""")
-
-    def test_all_ordered_by_priority_desc(self):
@@ -54 +54 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,priority,milestone,component,time,changetime
-FROM ticket
-
+e 
-ORDER BY COALESCE(priority,'')='' DESC,priority_value DESC,id""")
-
-    def test_all_ordered_by_version(self):
@@ -63 +63 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,priority,milestone,version,time,changetime
-FROM ticket
-
+a 
-ORDER BY COALESCE(version,'')='',COALESCE(version_time,0)=0,version_time,version,id""")
-
-    def test_all_ordered_by_version_desc(self):
@@ -72 +72 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,priority,milestone,version,time,changetime
-FROM ticket
-
+a 
-ORDER BY COALESCE(version,'')='' DESC,COALESCE(version_time,0)=0 DESC,version_time DESC,version DESC,id""")
-
-    def test_constrained_by_milestone(self):
@@ -91 +91 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,priority,component,version,milestone,time,changetime
-FROM ticket
-
+a 
-ORDER BY COALESCE(milestone,'')='',COALESCE(milestone_time,0)=0,milestone_time,milestone,COALESCE(id,0)=0,id""")
-
-    def test_all_grouped_by_milestone_desc(self):
@@ -100 +100 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,priority,component,version,milestone,time,changetime
-FROM ticket
-
+a 
-ORDER BY COALESCE(milestone,'')='' DESC,COALESCE(milestone_time,0)=0 DESC,milestone_time DESC,milestone DESC,COALESCE(id,0)=0,id""")
-
-    def test_grouped_by_priority(self):
@@ -109 +109 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,milestone,component,version,priority,time,changetime
-FROM ticket
-
+e 
-ORDER BY COALESCE(priority,'')='',priority_value,id""")
-
-    def test_constrained_by_milestone_not(self):
@@ -180 +180 @@
-        self.assertEqual(sql,
-"""SELECT id,summary,status,owner,priority,milestone,component,time,changetime, foo.value AS foo
-FROM ticket
-
+ tc
-WHERE COALESCE(foo,'')='something'
-ORDER BY COALESCE(id,0)=0,id""")
-

trac/tests/db.py

@@ +1 @@
+
+import unittest
+from trac.Environment import Environment # TODO: take out trac. when being run
+                                         #       from normal trac/test.py
+
+class DBTestBase:
+    def setUp(self):
+        pass
+
+    def tearDown(self):
+        pass
+
+class DBTestCase(DBTestBase, unittest.TestCase):
+    def test_connect(self):
+        """Testing db.connect"""
+        env = Environment('/var/trac/test')  # TODO: make more generic
+                                             # for now, it's my local pgsql env
+        cnx = env.get_db_cnx()
+        cursor = cnx.cursor()
+        cursor.execute("select * from system")
+        row = cursor.fetchone()
+        print row
+        cnx.close()
+
+    def test_parse_args(self):
+        """Testing db.parse_args"""
+        
+def suite():
+    return unittest.makeSuite(DBTestCase,'test')
+    
+# TODO: take this out when we put this in with the other tests
+# Q: will we be able to do these tests for different dbms's in the test suite?
+if __name__ == '__main__':
+    unittest.main(defaultTest='suite')
+        

trac/db.py

@@ -81 +81 @@
-        return IterableCursor(cursor)
-
-
-class SQLiteConnection(ConnectionWrapper):
-    """
-    Connection wrapper for SQLite.
-    """
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
-    db_str = env.get_config('trac', 'database', 'sqlite:db/trac.db')
-    scheme, rest = db_str.split(':', 1)
-
-    if scheme == 'sqlite':
-        if not rest.startswith('/'):
-            rest = os.path.join(env.path, rest)
-
+
+
-
+    if scheme == 'pypgsql':
+        import trac.dbms.db_pypgsql
+        return trac.dbms.db_pypgsql.PyPgSQLConnection(rest, env, create)
+
-    raise TracError, 'Unsupported database type "%s"' % scheme
-
-

trac/Environment.py

@@ -19 +19 @@
-#
-# Author: Jonas Borgström <jonas@edgewall.com>
-#
-# Todo: Move backup and upgrade from db.py
-#
-
-from trac import db, db_default, Logging, Mimeview, util
+from types import *
-
-import ConfigParser
-import os
@@ -41 +40 @@
-    A Trac environment consists of a directory structure containing
-    among other things:
-     * a configuration file.
-sqlite 
+
-     * Project specific templates and wiki macros.
-     * wiki and ticket attachments.
-    """
-
+, db_str='sqlite:db/trac.db'
-        self.path = path
-        if create:
-
+db_str
-        self.verify()
-        self.load_config()
-        try: # Use binary I/O on Windows
@@ -69 +68 @@
-    def get_db_cnx(self):
-        return db.get_cnx(self)
-
-
+, db_str
-        def _create_file(fname, data=None):
-            fd = open(fname, 'w')
-            if data: fd.write(data)
@@ -108 +107 @@
-# Site CSS - Place custom CSS, including overriding styles here.
-?>
-""")
-
-
-
-
-
-
-
+
+
+
+
+
+
-
+        # Create default database by attempting to get connection
+        cnx = db.get_cnx(self, create=1)
+        cnx.close
+
-    def insert_default_data(self):
-        def prep_value(v):
-            if v == None:
-                return 'NULL'
-            else:
-
+
+
+
+
-        cnx = self.get_db_cnx()
-        cursor = cnx.cursor()
-        
@@ -131 +136 @@
-            for row in db_default.data[t][2]:
-                values = ','.join(map(prep_value, row))
-                sql = "INSERT INTO %s (%s) VALUES(%s);" % (table, cols, values)
-
+
+
+
+
-        for s,n,v in db_default.default_config:
-            if not self.cfg.has_section(s):
-                self.cfg.add_section(s)
-            self.cfg.set(s, n, v)
-        self.save_config()
-        cnx.commit()
-
-    def get_version(self):
-        cnx = self.get_db_cnx()
@@ -295 +302 @@
-        """Simple SQLite-specific backup. Copy the database file."""
-        db_str = self.get_config('trac', 'database', 'sqlite:db/trac.db')
-        if db_str[:7] != 'sqlite:':
-
+
+
+
+
+
-        db_name = os.path.join(self.path, db_str[7:])
-        if not dest:
-            dest = '%s.%i.bak' % (db_name, self.get_version())

trac/Ticket.py

@@ -107 +107 @@
-                       % (','.join(std_fields),
-                          ','.join(['%s'] * len(std_fields))),
-                       map(lambda n, self=self: self[n], std_fields))
-
+"ticket", "id"
-
-        custom_fields = filter(lambda n: n[:7] == 'custom_', self.keys())
-        for name in custom_fields:
@@ -151 +151 @@
-                cursor.execute("SELECT * FROM ticket_custom " 
-                               "WHERE ticket=%s and name=%s", (id, fname))
-                if cursor.fetchone():
+                    # TODO: is this correct below?  no where clause?
+                    raise TracError, ("is this correct? - UPDATE ticket_custom SET value=%s", (self[name],))
-                    cursor.execute("UPDATE ticket_custom SET value=%s",
-                                   (self[name],))
-                else:

contrib/tracdb-sqlite2pg.py

@@ +1 @@
+#!/usr/bin/env python
+
+"""
+Import a Trac database from sqlite to postgresql.
+
+Requires:  Trac from http://trac.edgewall.com/
+           Python 2.3 from http://www.python.org/
+           PostgreSQL from http://www.postgresql.org/
+           PySQLITE ?
+           others?
+           
+Author: Brad Anderson <brad@dsource.org>
+
+
+"""
+
+import sys, os
+import sqlite
+import pgdb
+from trac.util import sql_escape
+
+tables=['revision','node_change', 'auth_cookie', 'enum', 'system', 'lock', 
+        'ticket', 'ticket_change', 'ticket_custom', 'report', 'permission',
+        'component', 'milestone', 'version', 'attachment', 'session', 'wiki',
+        'forums', 'topics', 'posts']
+
+# for debugging, one table at a time
+# tables=['wiki']
+
+
+## simple field translation mapping.  if string not in
+## mapping, just return string, otherwise return value
+#class FieldTranslator(dict):
+#    def __getitem__(self, item):
+#        if not dict.has_key(self, item):
+#            return item
+#            
+#        return dict.__getitem__(self, item)
+
+
+
+def convert(_env, _host, _db, _user, _password, _clean):
+    
+    
+    # init PostgreSQL environment
+    print "PostgreSQL('%s':'%s':'%s'): connecting..." \
+                      % (_host, _db, _user)
+    pg_con = pgdb.connect(host=_host, database=_db, 
+                          user=_user, password=_password)
+    pg_cur = pg_con.cursor()
+
+    # init Trac environment
+    print "Trac SQLite('%s'): connecting..." % (_env)
+    trac_con = sqlite.connect(os.path.join(_env, "db/trac.db"),
+                              timeout=10000)
+    trac_cur = trac_con.cursor()
+    print
+
+#    fieldtypes = FieldTranslator()
+#    fieldtypes['0'] = 'int'
+#    fieldtypes['6'] = 'text'
+    
+    # loop thru tables, converting them
+    for t in tables:
+        print
+        print "Converting Table: '%s'" % t
+
+        # clear out existing table
+#        if _clean == 1:
+#            pg_cur.execute("truncate table %s;" % t)
+        pg_cur.execute("truncate table %s;" % t)
+        
+        # get table info
+        sql = "SELECT * FROM %s" % t
+        trac_cur.execute(sql)
+        cols = trac_cur.rs.col_defs
+        counter = 0
+        
+        # loop thru rows, moving data from sqlite to pgsql
+        while 1:
+            row = trac_cur.fetchone()
+            counter += 1
+            if not row:
+                break
+            flds=""
+            vals=""
+            for c in cols:
+                field = c[0]
+                type = c[1]
+                value = row[field]
+                if value != None:
+                    flds += ", %s" % field
+                    if type in [6]:
+                        value = "'%s'" % sql_escape(value)
+                    vals += ", %s" % value
+            flds = flds[2:]
+            vals = vals[2:]
+            sql  = "INSERT INTO %s " % t
+            sql += "(%s) " % flds
+            sql += "VALUES (%s);" % vals
+
+            pg_cur.execute(sql)
+            pg_con.commit()
+            
+            if counter % 100 == 0:
+                print "%s rows" % counter
+        
+    trac_cur.close()
+    trac_con.close()
+    pg_cur.close()
+    pg_con.close()
+    print
+        
+    
+def usage():
+    print "tracdb-sqlite2pg - Converts a Trac database from sqlite to " \
+          "postgresql."
+    print
+    print "This script is designed to transfer an existing Trac SQLite "
+    print "database to an existing Trac PosgreSQL database.  The two " 
+    print "databases should be of the same version.  Check in the 'system' "
+    print "table in each database to verify.  This script runs on version 8 "
+    print "and above.  This is for Trac pre 0.9 (revision 1169 and higher)."
+    print
+    print "Note: you can make a Trac PosgreSQL database with trac-admin "
+    print "(the version that accompanies this script)"
+    print
+    print "THIS WILL DESTROY ALL DATA IN THE POSTGRESQL DATABASE !! "
+    print
+    print
+    print "Usage: tracdb-sqlite2pg.py [options]"
+    print
+    print "Available Options:"
+    print "  -t  | --tracenv /path/to/trac/env - full path to Trac environment"
+    print "  -h  | --host <pgsql hostname>     - PostgreSQL hostname"
+    print "  -db | --database <pgsql database> - PostgreSQL database name"
+    print "  -u  | --user <pgsql username>     - PostgreSQL username"
+    print "  -p  | --passwd <pgsql password>   - PostgreSQL password"
+#    print "  -c  | --clean                     - clean out PG database " \
+#          "before converting. (recommended)"
+    print "  --help | help                     - this help info"
+    print
+    print "Additional configuration options can be defined directly in the " \
+          "script."
+    print
+    sys.exit(0)
+
+def main():
+    global TRAC_ENV, PG_HOST, PG_DB, PG_USER, PG_PASSWORD, PG_CLEAN
+    if len (sys.argv) > 1:
+        if sys.argv[1] in ['--help','help'] or len(sys.argv) < 4:
+            usage()
+        iter = 1
+        while iter < len(sys.argv):
+            if sys.argv[iter] in ['-t', '--tracenv'] and iter+1 < len(sys.argv):
+                TRAC_ENV = sys.argv[iter+1]
+                iter = iter + 1
+            elif sys.argv[iter] in ['-h', '--host'] and iter+1 < len(sys.argv):
+                PG_HOST = sys.argv[iter+1]
+                iter = iter + 1
+            elif sys.argv[iter] in ['-db', '--database'] and \
+                    iter+1 < len(sys.argv):
+                PG_DB = sys.argv[iter+1]
+                iter = iter + 1
+            elif sys.argv[iter] in ['-u', '--user'] and iter+1 < len(sys.argv):
+                PG_USER = sys.argv[iter+1]
+                iter = iter + 1
+            elif sys.argv[iter] in ['-p', '--passwd'] and \
+                    iter+1 < len(sys.argv):
+                PG_PASSWORD = sys.argv[iter+1]
+                iter = iter + 1
+#            elif sys.argv[iter] in ['-c', '--clean']:
+#                PG_CLEAN = 1
+            else:
+                print "Error: unknown parameter: " + sys.argv[iter]
+                sys.exit(0)
+            iter = iter + 1
+    else:
+        usage()
+        
+    convert(TRAC_ENV, PG_HOST, PG_DB, PG_USER, PG_PASSWORD, PG_CLEAN)
+
+
+if __name__ == '__main__':
+    main()