[Zope] ZRDB

[Tino Wildenhain]

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