The 8.2 patch that added support for an alias on the target table of

UPDATE/DELETE forgot to teach ruleutils.c to display the alias.
Per bug #4141 from Mathias Seiler.

diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c
index 26bda928fa606dbefe36784232548f7800bad85c..5426d86230cd828b5fb8e0458385021f9a1f3c68 100644 (file)
--- a/src/backend/utils/adt/ruleutils.c
+++ b/src/backend/utils/adt/ruleutils.c
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.272 2008/03/28 00:21:56 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.273 2008/05/03 23:19:20 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2443,9 +2443,13 @@ get_update_query_def(Query *query, deparse_context *context)
        appendStringInfoChar(buf, ' ');
        context->indentLevel += PRETTYINDENT_STD;
    }
-   appendStringInfo(buf, "UPDATE %s%s SET ",
+   appendStringInfo(buf, "UPDATE %s%s",
                     only_marker(rte),
                     generate_relation_name(rte->relid));
+   if (rte->alias != NULL)
+       appendStringInfo(buf, " %s",
+                        quote_identifier(rte->alias->aliasname));
+   appendStringInfoString(buf, " SET ");
 
    /* Add the comma separated list of 'attname = value' */
    sep = "";
@@ -2517,12 +2521,15 @@ get_delete_query_def(Query *query, deparse_context *context)
    Assert(rte->rtekind == RTE_RELATION);
    if (PRETTY_INDENT(context))
    {
-       context->indentLevel += PRETTYINDENT_STD;
        appendStringInfoChar(buf, ' ');
+       context->indentLevel += PRETTYINDENT_STD;
    }
    appendStringInfo(buf, "DELETE FROM %s%s",
                     only_marker(rte),
                     generate_relation_name(rte->relid));
+   if (rte->alias != NULL)
+       appendStringInfo(buf, " %s",
+                        quote_identifier(rte->alias->aliasname));
 
    /* Add the USING clause if given */
    get_from_clause(query, " USING ", context);