Re: [Apache Bloodhound] #548: Simple ticket report fails

2013-06-17 Thread Apache Bloodhound
#548: Simple ticket report fails
-+-
  Reporter:  olemis  |  Owner:  astaric
  Type:  defect  | Status:  review
  Priority:  blocker |  Milestone:  Release 6
 Component:  |Version:
  multiproduct   |   Keywords:  sql reports, multiproduct db
Resolution:  |  translator, trac util
-+-
Changes (by astaric):

 * owner:  rjollos = astaric


Comment:

 I see no reason that lower case token should not be supported. I'll look
 into this.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/548#comment:9
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker


Re: [Apache Bloodhound] #548: Simple ticket report fails

2013-06-17 Thread Apache Bloodhound
#548: Simple ticket report fails
-+-
  Reporter:  olemis  |  Owner:  astaric
  Type:  defect  | Status:  closed
  Priority:  blocker |  Milestone:  Release 6
 Component:  |Version:
  multiproduct   |   Keywords:  sql reports, multiproduct db
Resolution:  fixed   |  translator, trac util
-+-
Changes (by astaric):

 * status:  review = closed
 * resolution:   = fixed


Comment:

 r1493739 fixes sql translator to support queries with lowercase tokens
 (tests included).

 As the rest of the issues seems to be fixed already, I am closing the
 ticket.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/548#comment:10
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker


Re: [Apache Bloodhound] #548: Simple ticket report fails

2013-06-13 Thread Apache Bloodhound
#548: Simple ticket report fails
-+-
  Reporter:  olemis  |  Owner:  olemis
  Type:  defect  | Status:  review
  Priority:  blocker |  Milestone:  Release 6
 Component:  |Version:
  multiproduct   |   Keywords:  sql reports, multiproduct db
Resolution:  |  translator, trac util
-+-
Changes (by rjollos):

 * owner:  rjollos = olemis
 * status:  accepted = review


Comment:

 (In [1492534])

 Refs #548: Account for possibility that template timestamp argument could
 be a unicode string.

 ---

 Reassigning to Olemis for review and final confirmation of whether the
 issue is fixed.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/548#comment:4
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker


Re: [Apache Bloodhound] #548: Simple ticket report fails

2013-06-13 Thread Apache Bloodhound
#548: Simple ticket report fails
-+-
  Reporter:  olemis  |  Owner:  olemis
  Type:  defect  | Status:  review
  Priority:  blocker |  Milestone:  Release 6
 Component:  |Version:
  multiproduct   |   Keywords:  sql reports, multiproduct db
Resolution:  |  translator, trac util
-+-

Comment (by olemis):

 Replying to [comment:4 rjollos]:
  (In [1492534])
 
  Refs #548: Account for possibility that template timestamp argument
 could be a unicode string.
 
  ---
 
  Reassigning to Olemis for review and final confirmation of whether the
 issue is fixed.

 After [1492534] the last query (i.e. `SELECT * FROM ticket`) will work.
 Nevertheless the former (i.e. `select * from ticket`) still fails . If
 this expected behavior then feel free to close this ticket as `fixed`.

 thanks !

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/548#comment:5
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker


[Apache Bloodhound] #548: Simple ticket report fails

2013-06-06 Thread Apache Bloodhound
#548: Simple ticket report fails
-+---
  Reporter:  olemis  |Owner:
  Type:  defect  |   Status:  new
  Priority:  major   |  Version:
Resolution:  |
-+---
 While running the functional test suite it was noticed that the SQL
 statement `select * from ticket` in a report leads to the following error

 {{{#!html

 div class=alertReport execution failed: preException: Failed to
 translate SQL 'select * from ticket ', exception 'select'/pre hrpre
 style=white-space: preselect * from ticket /pre/div
 }}}

 On the other hand the SQL statement `SELECT * FROM ticket` seems to be
 translated correctly but also fails with the following error

 {{{
 UnboundLocalError: local variable 'dt' referenced before assignment
 }}}

 Traceback

 {{{#!py

 File
 
/path/to/bloodhound/bloodhound_dashboard/bhdashboard/widgets/templates/widget_grid.html,
 line 107, in Expression ucell.value != '' and format_time(value) or
 '--'
   td class=date py:attrs=td_attrs${cell.value != '' and
 format_time(value) or '--'}
 File /path/to/bloodhound/trac/trac/util/datefmt.py, line 704, in
 user_time
   return func(*args, **kwargs)
 File /path/to/bloodhound/trac/trac/util/datefmt.py, line 244, in
 format_time
   return _format_datetime(t, format, tzinfo, locale, 'time')
 File /path/to/bloodhound/trac/trac/util/datefmt.py, line 197, in
 _format_datetime
   t = to_datetime(t, tzinfo or localtz)
 File /path/to/bloodhound/trac/trac/util/datefmt.py, line 90, in
 to_datetime
   if dt:
 }}}

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/548
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker


Re: [Apache Bloodhound] #548: Simple ticket report fails

2013-06-06 Thread Apache Bloodhound
#548: Simple ticket report fails
-+-
  Reporter:  olemis  |  Owner:  nobody
  Type:  defect  | Status:  new
  Priority:  blocker |  Milestone:  Release 6
 Component:  |Version:
  multiproduct   |   Keywords:  sql reports, multiproduct db
Resolution:  |  translator, trac util
-+-
Changes (by olemis):

 * keywords:  sql reports, multiproduct db translator = sql reports,
 multiproduct db translator, trac util


-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/548#comment:2
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound issue tracker