PHP provides two functions to parse XML... xml_parser_create() and xml_parser_create_ns(). We have a wrapper for the first (drupal_xml_parser_create), but not for the second. This patch adds a $namespace parameter to drupal_xml_parser_create() which defaults to false and is backwards compatible.

CommentFileSizeAuthor
xmlparser.patch1.07 KBSteven
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

walkah’s picture

+1 . nice patch, and i can use it :)

Dries’s picture

Status: Needs review » Needs work

How about we always use xml_parser_create_ns() and get rid of the parameter? It seems like good practice to always use support for namespaces. Haven't tested it so we might have to rewrite some bits and pieces -- if that would give us different parsers output.

chx’s picture

Version: 6.x-dev » 7.x-dev
Jaza’s picture

Version: 7.x-dev » 8.x-dev

Bump.

jhedstrom’s picture

Issue summary: View changes
Status: Needs work » Closed (won't fix)

Parsers are now services that can be swapped as needed.