GAE text,blob disallowed in orderby (FIXED)

[alfonsodg]

From robi...@gmail.com on June 19, 2008 12:16:29

Currently, text (and probably blob) are allowed in orderby...

The datastore supports two value types for storing text: short text strings shorter than 500 bytes, and long text strings longer than 500 bytes. Short strings are indexed and can be used in query filter conditions and sort orders. Long strings are not indexed and cannot be used in filter conditions or sort orders.

The datastore also supports a type for non-text byte strings, or "blobs." As with long text strings, blobs are not indexed, and cannot be used in query filters or sort orders.

Original issue: http://code.google.com/p/web2py/issues/detail?id=15

[alfonsodg]

From massimod...@gmail.com on June 19, 2008 12:47:22

want to send me a patch?

[alfonsodg]

From robi...@gmail.com on June 19, 2008 13:02:50

Yes, could you tell me how to run the unit tests for the gql.py?

[alfonsodg]

From robi...@gmail.com on June 19, 2008 16:26:01

This patch does 3 things:

'True' -> 'TRUE', 'False' -> 'FALSE'

propagates the type info through '~' and '|'

raises an exception if orderby type in ['text','blob']

How do I run the tests for gql.py?

Attachment: order_by_appengine.patch

[alfonsodg]

From massimod...@gmail.com on June 19, 2008 19:43:23

Sure about this?

• if object and not str(object)[0].upper()=='F': return "True"
• else: return "False"
• if object and not str(object)[0].upper()=='F': return "TRUE"
• else: return "FALSE" I thought you had me change it the other way?

[alfonsodg]

From robi...@gmail.com on June 19, 2008 20:25:03

Yes, GQL is case-insensitive and all CAPS looks better. Sorry for changing the case.

[alfonsodg]

From massimod...@gmail.com on June 20, 2008 09:29:50

Summary: GAE text,blob disallowed in orderby (FIXED)
Status: Fixed