From 921a8a55c380fdbe22733a41c41c27532166d54e Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Sun, 11 Feb 2007 22:18:16 +0000 Subject: [PATCH] Add support for optionally escaping periods when converting SQL identifiers to XML names, which will be required for supporting XML export. --- src/backend/parser/parse_expr.c | 6 +++--- src/backend/utils/adt/xml.c | 10 +++++++++- src/include/utils/xml.h | 2 +- 3 files changed, 13 insertions(+), 5 deletions(-) diff --git a/src/backend/parser/parse_expr.c b/src/backend/parser/parse_expr.c index 61b8c0d53d..410999f2c4 100644 --- a/src/backend/parser/parse_expr.c +++ b/src/backend/parser/parse_expr.c @@ -1389,7 +1389,7 @@ transformXmlExpr(ParseState *pstate, XmlExpr *x) newx->op = x->op; if (x->name) - newx->name = map_sql_identifier_to_xml_name(x->name, false); + newx->name = map_sql_identifier_to_xml_name(x->name, false, false); else newx->name = NULL; @@ -1411,10 +1411,10 @@ transformXmlExpr(ParseState *pstate, XmlExpr *x) expr = transformExpr(pstate, r->val); if (r->name) - argname = map_sql_identifier_to_xml_name(r->name, false); + argname = map_sql_identifier_to_xml_name(r->name, false, false); else if (IsA(r->val, ColumnRef)) argname = map_sql_identifier_to_xml_name(FigureColname(r->val), - true); + true, false); else { ereport(ERROR, diff --git a/src/backend/utils/adt/xml.c b/src/backend/utils/adt/xml.c index 5648c7de08..768392c849 100644 --- a/src/backend/utils/adt/xml.c +++ b/src/backend/utils/adt/xml.c @@ -1318,8 +1318,14 @@ is_valid_xml_namechar(pg_wchar c) * Map SQL identifier to XML name; see SQL/XML:2003 section 9.1. */ char * -map_sql_identifier_to_xml_name(char *ident, bool fully_escaped) +map_sql_identifier_to_xml_name(char *ident, bool fully_escaped, bool escape_period) { + /* + * SQL/XML doesn't make use of this case anywhere, so it's + * probably a mistake. + */ + Assert(fully_escaped || !escape_period); + #ifdef USE_LIBXML StringInfoData buf; char *p; @@ -1340,6 +1346,8 @@ map_sql_identifier_to_xml_name(char *ident, bool fully_escaped) else appendStringInfo(&buf, "_x0058_"); } + else if (escape_period && *p == '.') + appendStringInfo(&buf, "_x002E_"); else { pg_wchar u = sqlchar_to_unicode(p); diff --git a/src/include/utils/xml.h b/src/include/utils/xml.h index 6b6aedf540..e673c48b68 100644 --- a/src/include/utils/xml.h +++ b/src/include/utils/xml.h @@ -53,7 +53,7 @@ extern xmltype *xmlroot(xmltype *data, text *version, int standalone); extern bool xml_is_document(xmltype *arg); extern text *xmltotext_with_xmloption(xmltype *data, XmlOptionType xmloption_arg); -extern char *map_sql_identifier_to_xml_name(char *ident, bool fully_escaped); +extern char *map_sql_identifier_to_xml_name(char *ident, bool fully_escaped, bool escape_period); extern char *map_xml_name_to_sql_identifier(char *name); extern char *map_sql_value_to_xml_value(Datum value, Oid type); -- 2.39.5