[Solar-talk] Patch for Solar_Content_Abstract

[Travis Swicegood]

Paul M Jones wrote:
> My only concern there goes something like this:  You have a varchar  
> column, and you want to populate it with a numeric value (which will  
> of course be stored as a string).  If the numeric value is not  
> quoted, will the DB choke on that at insert/update time?
>
> That is, do all the supported DBMSes see these as equally valid?
>
>      UPDATE table SET char_col = 123;
>      UPDATE table SET char_col = '123';
>
> Any ideas here?
>   

Hmm...

Maybe the solution here is to do just what's needed to make this work.  
Since we already know that in this case count($tags) is going to be an 
integer that doesn't really need escaping, just updating that ->having() 
call is the simplest solution with the least ripple effect.  Attached is 
the patch I have in place on my machine which includes the update to 
take into account the lack of a requirement for a column specification 
in the top level count().

-Travis


-------------- next part --------------
A non-text attachment was scrubbed...
Name: Solar_Content_Abstract.patch
Type: text/x-patch
Size: 897 bytes
Desc: not available
Url : http://mail.killersoft.com/pipermail/solar-talk/attachments/20061002/041a9344/attachment.bin