#3820: DebugCursorWrapper not working on Oracle
-----------------------------------------+----------------------------------
Reporter:  Ben Khoo <[EMAIL PROTECTED]>  |       Owner:  jacob                  
           
  Status:  new                           |   Component:  Uncategorized          
           
 Version:  other branch                  |    Keywords:  Oracle 
DebugCursorWrapper fetchall
   Stage:  Unreviewed                    |   Has_patch:  1                      
           
-----------------------------------------+----------------------------------
 Hello,
 
 I believe there may be an issue with the latest update to the boulder-
 oracle-sprint branch. A change was made on the 23rd of March so that a
 {{{DebugCursorWrapper}}} is returned if {{{DEBUG == True}}}. (Refer to the
 SVN log of /django/branches/boulder-oracle-sprint/django/db/oracle/base.py
 for the message).
 
 The exhibits itself in /django/branches/boulder-oracle-
 sprint/django/db/oracle/base.py line 288 which reads:
 {{{
 for unresolved_row in cursor:
 }}}
 If {{{DEBUG == True}}} the cursor is a {{{DebugCursorWrapper}}} and is not
 iterable. Instead I believe that it should read something like the
 following:
 {{{
 for unresolved_row in cursor.fetchall():
 }}}
 
 Similarly in the file /django/branches/boulder-oracle-
 sprint/django/db/oracle/introspection.py on line 9, it reads:
 {{{
 return [row[0].upper() for row in cursor]
 }}}
 when I believe it should read
 {{{
 return [row[0].upper() for row in cursor.fetchall()]
 }}}
 
 Regards

-- 
Ticket URL: <http://code.djangoproject.com/ticket/3820>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to