[SQL-CVS] r3886 - in SQLObject/branches/0.9: docs sqlobject/maxdb sqlobject/mssql sqlobject/mysql sqlobject/sqlite sqlobject/sybase

[<sub...@co...>]

Author: phd
Date: 2009-05-18 10:11:06 -0600 (Mon, 18 May 2009)
New Revision: 3886

Modified:
   SQLObject/branches/0.9/docs/News.txt
   SQLObject/branches/0.9/sqlobject/maxdb/maxdbconnection.py
   SQLObject/branches/0.9/sqlobject/mssql/mssqlconnection.py
   SQLObject/branches/0.9/sqlobject/mysql/mysqlconnection.py
   SQLObject/branches/0.9/sqlobject/sqlite/sqliteconnection.py
   SQLObject/branches/0.9/sqlobject/sybase/sybaseconnection.py
Log:
Use sqlmeta.idName instead of 'id' in all connection classes.


Modified: SQLObject/branches/0.9/docs/News.txt
===================================================================
--- SQLObject/branches/0.9/docs/News.txt	2009-05-18 16:00:34 UTC (rev 3885)
+++ SQLObject/branches/0.9/docs/News.txt	2009-05-18 16:11:06 UTC (rev 3886)
@@ -13,6 +13,8 @@
 * Two bug in SQLiteConnection.columnsFromSchema() were fixed: use
   sqlmeta.idName instead of 'id'; convert default 'NULL' to None.
 
+* Use sqlmeta.idName instead of 'id' in all connection classes.
+
 SQLObject 0.9.10
 ================
 

Modified: SQLObject/branches/0.9/sqlobject/maxdb/maxdbconnection.py
===================================================================
--- SQLObject/branches/0.9/sqlobject/maxdb/maxdbconnection.py	2009-05-18 16:00:34 UTC (rev 3885)
+++ SQLObject/branches/0.9/sqlobject/maxdb/maxdbconnection.py	2009-05-18 16:11:06 UTC (rev 3886)
@@ -255,7 +255,7 @@
             # id is defined as primary key --> ok
             # We let sqlobject raise error if the 'id' is used for another column
             field_name = field.lower()
-            if field_name == 'id' and  pkmap[field_name]:
+            if (field_name == soClass.sqlmeta.idName) and pkmap[field_name]:
                 continue
 
             colClass, kw = self.guessClass(data_type,data_len,data_scale)

Modified: SQLObject/branches/0.9/sqlobject/mssql/mssqlconnection.py
===================================================================
--- SQLObject/branches/0.9/sqlobject/mssql/mssqlconnection.py	2009-05-18 16:00:34 UTC (rev 3885)
+++ SQLObject/branches/0.9/sqlobject/mssql/mssqlconnection.py	2009-05-18 16:11:06 UTC (rev 3886)
@@ -216,8 +216,7 @@
                                 % tableName)
         results = []
         for field, size, t, precision, scale, nullAllowed, default, defaultText, is_identity in colData:
-            # Seems strange to skip the pk column?  What if it's not 'id'?
-            if field == 'id':
+            if field == soClass.sqlmeta.idName:
                 continue
             # precision is needed for decimal columns
             colClass, kw = self.guessClass(t, size, precision, scale)

Modified: SQLObject/branches/0.9/sqlobject/mysql/mysqlconnection.py
===================================================================
--- SQLObject/branches/0.9/sqlobject/mysql/mysqlconnection.py	2009-05-18 16:00:34 UTC (rev 3885)
+++ SQLObject/branches/0.9/sqlobject/mysql/mysqlconnection.py	2009-05-18 16:11:06 UTC (rev 3886)
@@ -214,7 +214,7 @@
                                 % tableName)
         results = []
         for field, t, nullAllowed, key, default, extra in colData:
-            if field == 'id':
+            if field == soClass.sqlmeta.idName:
                 continue
             colClass, kw = self.guessClass(t)
             if self.kw.get('use_unicode') and colClass is col.StringCol:

Modified: SQLObject/branches/0.9/sqlobject/sqlite/sqliteconnection.py
===================================================================
--- SQLObject/branches/0.9/sqlobject/sqlite/sqliteconnection.py	2009-05-18 16:00:34 UTC (rev 3885)
+++ SQLObject/branches/0.9/sqlobject/sqlite/sqliteconnection.py	2009-05-18 16:11:06 UTC (rev 3886)
@@ -279,7 +279,7 @@
                         for col in sqlmeta.columnList if col.name != column.name]
         cols = ",\n".join(["    %s" % c for c in cols])
         self.query('CREATE TABLE %s (\n%s\n)' % (sqlmeta.table, cols))
-        all_columns = ', '.join(['id'] + [col.dbName for col in sqlmeta.columnList])
+        all_columns = ', '.join([sqlmeta.idName] + [col.dbName for col in sqlmeta.columnList])
         self.query('INSERT INTO %s (%s) SELECT %s FROM %s' % (
             sqlmeta.table, all_columns, all_columns, new_name))
         self.query('DROP TABLE %s' % new_name)

Modified: SQLObject/branches/0.9/sqlobject/sybase/sybaseconnection.py
===================================================================
--- SQLObject/branches/0.9/sqlobject/sybase/sybaseconnection.py	2009-05-18 16:00:34 UTC (rev 3885)
+++ SQLObject/branches/0.9/sqlobject/sybase/sybaseconnection.py	2009-05-18 16:11:06 UTC (rev 3886)
@@ -138,7 +138,7 @@
                                 % tableName)
         results = []
         for field, t, nullAllowed, default in colData:
-            if field == 'id':
+            if field == soClass.sqlmeta.idName:
                 continue
             colClass, kw = self.guessClass(t)
             kw['name'] = soClass.sqlmeta.style.dbColumnToPythonAttr(field)