Hi, browsing thru Shared.DC.ZRDB.Results i found this construct: -------- line 117 ------------ if not name: raise ValueError, 'Empty column name, %s' % name ------------------------------ This looks a bit silly, because if name is empty, %s will be replaced with nothing... Furthermore, would it not be better to provide a generic name for the empty column? Sure, it is difficult to provide such a name who does not collide with others in namespace, but it would be very helpful, if the underlying database-adaptor does not provide full names under certain circumstances, such as ODBC with CenturaSQLBase always returns empty column names to Zope with the count() Operator. Even if used "as" to rename. How about something like: if not name: name='column%d' % i or such? Next there is always some hassle with those 'table.column' columns returning from databases who do not support column renaming. Would it be a way to rename them automatically to something like table_column? Yes, I know there could be a column already beeing called so. But if one knows she can use table.table_column in such circumstances. May be as alias? Regards Tino Wildenhain
participants (1)
-
Tino Wildenhain