Fix overly enthusiastic Assert introduced in 8.1: it's expecting a

CaseTestExpr, but forgot that the optimizer is sometimes able to replace
CaseTestExpr by Const.

diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c
index a88f5ef93c6b41b5bca191608a0e318c7f5f992e..821cc99e9f268afe7f974487bb295dd7443239dc 100644 (file)
--- a/src/backend/utils/adt/ruleutils.c
+++ b/src/backend/utils/adt/ruleutils.c
@@ -2,7 +2,7 @@
  * ruleutils.c - Functions to convert stored expressions/querytrees
  *             back to source text
  *
- *   $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.232 2006/08/21 00:57:25 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.233 2006/10/01 17:23:38 tgl Exp $
  **********************************************************************/
 
 #include "postgres.h"
@@ -3584,14 +3584,18 @@ get_rule_expr(Node *node, deparse_context *context,
                         * the optimizer's simplify_boolean_equality() may
                         * have reduced this to just "CaseTestExpr" or
                         * "NOT CaseTestExpr", for which we have to show
-                        * "TRUE" or "FALSE".
+                        * "TRUE" or "FALSE".  Also, depending on context
+                        * the original CaseTestExpr might have been reduced
+                        * to a Const (but we won't see "WHEN Const").
                         */
                        if (IsA(w, OpExpr))
                        {
                            Node       *rhs;
 
                            Assert(IsA(linitial(((OpExpr *) w)->args),
-                                      CaseTestExpr));
+                                      CaseTestExpr) ||
+                                  IsA(linitial(((OpExpr *) w)->args),
+                                      Const));
                            rhs = (Node *) lsecond(((OpExpr *) w)->args);
                            get_rule_expr(rhs, context, false);
                        }