[drupal-devel] [bug] Forum module's use of if in Postgresql

[patrickslee]

Issue status update for http://drupal.org/node/23499

 Project:      Drupal
 Version:      4.6.0
 Component:    forum.module
 Category:     bug reports
 Priority:     normal
 Assigned to:  patrickslee
 Reported by:  cchamilt
 Updated by:   patrickslee
 Status:       patch

I found the definition in database.pgsql. It is like this:


CREATE FUNCTION "if"(boolean, anyelement, anyelement) RETURNS
anyelement AS '
  SELECT CASE WHEN $1 THEN $2 ELSE $3 END;
' LANGUAGE 'sql';


So it is actually wanting boolean.




patrickslee



Previous comments:
------------------------------------------------------------------------

May 24, 2005 - 16:32 : cchamilt

Postgresql function "if" is defined to return a boolean value, yet forum
requests an integer - last_comment_uid.


user error:
query: SELECT n.nid, l.last_comment_timestamp, IF(l.last_comment_uid,
cu.name, l.last_comment_name) as last_comment_name, l.last_comment_uid
FROM node n, node_comment_statistics l /*! USE INDEX
(node_comment_timestamp) */, users cu, term_node r WHERE  n.nid = r.nid
AND r.tid = 1 AND n.status = 1 AND n.type = 'forum' AND
l.last_comment_uid = cu.uid AND n.nid = l.nid ORDER BY
l.last_comment_timestamp DESC LIMIT 1 OFFSET 0 in
/var/www/hosts/www.ambigc.com/htdocs/includes/database.pgsql.inc on
line 62.


warning: pg_query() [function.pg-query]: Query failed: ERROR:  function
if(integer, character varying, character varying) does not exist
HINT:  No function matches the given name and argument types. You may
need to add explicit type casts. in
/var/www/hosts/www.ambigc.com/htdocs/includes/database.pgsql.inc on
line 45.




------------------------------------------------------------------------

May 25, 2005 - 10:07 : patrickslee

Attachment: http://drupal.org/files/issues/forum.module_3.patch (2.69 KB)

I have also encountered this bug and made a possible patch by replacing
IF statements to CASE.
A patch file is attached.




------------------------------------------------------------------------

May 25, 2005 - 10:28 : adrian

There is an IF function defined in the database.pgsql function. On what
page is this happening, as what might be happening is that the type
isn't being picked up correctly. 


It's defined as IF(anyelement, anyelement, text) returns anyelement.
IIRC. What I have seen happen is that it wants the text field to return
integer, or vice versa.