{"action":"created","issue":{"url":"https://api.github.com/repos/humanmade/WordPress-Importer/issues/148","repository_url":"https://api.github.com/repos/humanmade/WordPress-Importer","labels_url":"https://api.github.com/repos/humanmade/WordPress-Importer/issues/148/labels{/name}","comments_url":"https://api.github.com/repos/humanmade/WordPress-Importer/issues/148/comments","events_url":"https://api.github.com/repos/humanmade/WordPress-Importer/issues/148/events","html_url":"https://github.com/humanmade/WordPress-Importer/pull/148","id":251800720,"node_id":"MDExOlB1bGxSZXF1ZXN0MTM2ODY1NzA0","number":148,"title":"Parsing <wp:postmeta> using an attribute-based/less-verbose XML structure","user":{"login":"drzraf","id":1169926,"node_id":"MDQ6VXNlcjExNjk5MjY=","avatar_url":"https://avatars0.githubusercontent.com/u/1169926?v=4","gravatar_id":"","url":"https://api.github.com/users/drzraf","html_url":"https://github.com/drzraf","followers_url":"https://api.github.com/users/drzraf/followers","following_url":"https://api.github.com/users/drzraf/following{/other_user}","gists_url":"https://api.github.com/users/drzraf/gists{/gist_id}","starred_url":"https://api.github.com/users/drzraf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drzraf/subscriptions","organizations_url":"https://api.github.com/users/drzraf/orgs","repos_url":"https://api.github.com/users/drzraf/repos","events_url":"https://api.github.com/users/drzraf/events{/privacy}","received_events_url":"https://api.github.com/users/drzraf/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-08-21T23:53:20Z","updated_at":"2018-08-07T13:05:21Z","closed_at":"2018-08-07T13:05:21Z","author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/humanmade/WordPress-Importer/pulls/148","html_url":"https://github.com/humanmade/WordPress-Importer/pull/148","diff_url":"https://github.com/humanmade/WordPress-Importer/pull/148.diff","patch_url":"https://github.com/humanmade/WordPress-Importer/pull/148.patch"},"body":"`<wp:postmeta>` is usually exported this way:\r\n```\r\n<wp:postmeta>\r\n   <wp:meta_key>_wp_page_template</wp:meta_key>\r\n   <wp:meta_value><![CDATA[some-string.tpl.php]]></wp:meta_value>\r\n</wp:postmeta>\r\n```\r\n(156 bytes)\r\n\r\nCDATA are used for the meta_value, so that HTML entities would not be interpreted by the XML parser.\r\n\r\nThis commit makes the parser aware of a second _short_ format where post's meta\r\n are expressed as attributes:\r\n`<wp:postmeta meta_key=\"_wp_page_template\" meta_value=\"some-string.tpl.php\" />`\r\n(85 bytes)\r\n\r\n\r\n* In most case, none of meta_key and meta_value contain HTML entities.\r\n* An exporter can perfectly define whether short form is ok for a given meta (entity presence, longtext `meta_value` length > X [XML does not set any limit on this btw])\r\n\r\nToDo: explore possible future use of `XMLReader::SUBST_ENTITIES` to completely avoid cluttering XML with CDATA?"},"comment":{"url":"https://api.github.com/repos/humanmade/WordPress-Importer/issues/comments/411048738","html_url":"https://github.com/humanmade/WordPress-Importer/pull/148#issuecomment-411048738","issue_url":"https://api.github.com/repos/humanmade/WordPress-Importer/issues/148","id":411048738,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMTA0ODczOA==","user":{"login":"swissspidy","id":841956,"node_id":"MDQ6VXNlcjg0MTk1Ng==","avatar_url":"https://avatars1.githubusercontent.com/u/841956?v=4","gravatar_id":"","url":"https://api.github.com/users/swissspidy","html_url":"https://github.com/swissspidy","followers_url":"https://api.github.com/users/swissspidy/followers","following_url":"https://api.github.com/users/swissspidy/following{/other_user}","gists_url":"https://api.github.com/users/swissspidy/gists{/gist_id}","starred_url":"https://api.github.com/users/swissspidy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swissspidy/subscriptions","organizations_url":"https://api.github.com/users/swissspidy/orgs","repos_url":"https://api.github.com/users/swissspidy/repos","events_url":"https://api.github.com/users/swissspidy/events{/privacy}","received_events_url":"https://api.github.com/users/swissspidy/received_events","type":"User","site_admin":false},"created_at":"2018-08-07T13:05:21Z","updated_at":"2018-08-07T13:05:21Z","author_association":"COLLABORATOR","body":"Sounds like a wontfix then :-)\r\n\r\nI'm closing this for now."}}
{"actor":{"display_login":"swissspidy"}}