[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