humanmade / WordPress-Importer

In-development rewrite of the WordPress (WXR) Importer
Other
358 stars 63 forks source link

xml reader: don't block attempt to read from stdin using `wxr-importer import php://stdin` #147

Open drzraf opened 7 years ago

rmccue commented 7 years ago

To fit better with how CLI apps generally work, you should pass - instead, and we should handle that if we don't already.

drzraf commented 7 years ago

addressed

drzraf commented 7 years ago

side note: If this can be merged : https://github.com/wp-cli/export-command/issues/9 ... then it'd be possible to test transformation identity (admitting an --override flag would exist) by checking that such an operation... $ wp export --stdout | wp wxr-importer import --override - ... implied no change on the source database.