[wp-trac] [WordPress Trac] #22435: Export API
WordPress Trac
noreply at wordpress.org
Thu Dec 18 11:24:29 UTC 2014
#22435: Export API
------------------------------------+-----------------------------
Reporter: nbachiyski | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Future Release
Component: Export | Version:
Severity: normal | Resolution:
Keywords: dev-feedback has-patch | Focuses:
------------------------------------+-----------------------------
Comment (by rmccue):
Replying to [comment:44 danielbachhuber]:
> Specifically, this approach includes a new, one-off library for writing
XML, dubbed `Oxymel`. It uses this new library by chaining, a foreign-to-
WordPress code pattern. These two considerations, plus the other feedback
in this thread, mean committing the patch would not be an inconsequential
decision.
Just poking my head in here to say that another huge benefit of Oyxmel is
that AFAIK it uses a proper XML serializer rather than string
concatenation. XML is one of the things that's super easy to output wrong,
and the spec on parsing means that you pretty much have to be super strict
when parsing. Right now, it's easy to output invalid XML unintentionally,
and have parsers fail.
(And I don't mean this in a theoretical way either, you can easily export
WXR from a site that fails when you try to import it back into the same
version of WP, thanks to PHP's XML parser following the correct rules.)
--
Ticket URL: <https://core.trac.wordpress.org/ticket/22435#comment:48>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list