Hi Christian,

Thanks, that did solve my problem ... but it made me wonder how other
backend drivers, namely Oracle,
handled this SQL without "= 1" ?

--
Mohammed

On Oct 18, 3:31 pm, Christian Hammond <chip...@chipx86.com> wrote:
> Hi Mohammed,
>
> This query is generated in both reviews/managers.py and
> reviews/datagrids.py. In these cases, it's raw SQL, so you should be able to
> easily modify that.
>
> Preferably, we'd move away from raw SQL now that Django has support for F()
> expressions (things like field_name = field_name + 1 and stuff, which it
> couldn't do before). But a quick fix would be to just do the = 1 on those
> queries.
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board -http://www.reviewboard.org
> VMware, Inc. -http://www.vmware.com
>
> On Mon, Oct 18, 2010 at 8:01 AM, Mohammed Abouzour [Sybase] <
>
> abouz...@gmail.com> wrote:
> > I am trying to upgrade our RB from 1.0rc3 to 1.5. We are running on
> > top of a SQLAnywhere database server. I got the database upgraded
> > successfully, but when I try to access the site it gives me errors
> > regarding badly generated SQL queries. The problem is with the
> > reviews_review.public field. This field is a Boolean field. The query
> > that is generated by RB looks as follows:
>
> > SELECT COUNT(*) FROM reviews_review, ... WHERE reviews_review.public
> > AND ...
>
> > In the SQLAnywhere Django driver, Boolean fields are mapped to BIT
> > fields on the database side. As such, the correct query above should
> > have been
>
> > SELECT COUNT(*) FROM reviews_review, ... WHERE reviews_review.public =
> > 1 AND ...
>
> > I am inheriting this system so I am still learning both Django and RB.
> > What component is responsible for
> > generating the correct syntax for Boolean fields?
>
> > --
> > Mohammed
>
> > --
> > Want to help the Review Board project? Donate today at
> >http://www.reviewboard.org/donate/
> > Happy user? Let us know athttp://www.reviewboard.org/users/
> > -~----------~----~----~----~------~----~------~--~---
> > To unsubscribe from this group, send email to
> > reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> > For more options, visit this group at
> >http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Reply via email to