{"action":"created","issue":{"url":"https://api.github.com/repos/SemanticMediaWiki/SemanticResultFormats/issues/114","labels_url":"https://api.github.com/repos/SemanticMediaWiki/SemanticResultFormats/issues/114/labels{/name}","comments_url":"https://api.github.com/repos/SemanticMediaWiki/SemanticResultFormats/issues/114/comments","events_url":"https://api.github.com/repos/SemanticMediaWiki/SemanticResultFormats/issues/114/events","html_url":"https://github.com/SemanticMediaWiki/SemanticResultFormats/issues/114","id":96507254,"number":114,"title":"add word format","user":{"login":"WolfgangFahl","id":1336221,"avatar_url":"https://avatars.githubusercontent.com/u/1336221?v=3","gravatar_id":"","url":"https://api.github.com/users/WolfgangFahl","html_url":"https://github.com/WolfgangFahl","followers_url":"https://api.github.com/users/WolfgangFahl/followers","following_url":"https://api.github.com/users/WolfgangFahl/following{/other_user}","gists_url":"https://api.github.com/users/WolfgangFahl/gists{/gist_id}","starred_url":"https://api.github.com/users/WolfgangFahl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WolfgangFahl/subscriptions","organizations_url":"https://api.github.com/users/WolfgangFahl/orgs","repos_url":"https://api.github.com/users/WolfgangFahl/repos","events_url":"https://api.github.com/users/WolfgangFahl/events{/privacy}","received_events_url":"https://api.github.com/users/WolfgangFahl/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/SemanticMediaWiki/SemanticResultFormats/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-22T08:24:44Z","updated_at":"2015-07-30T17:56:48Z","closed_at":null,"body":"As an enhancement I'd like to add a word format. I have started by forking SemanticResultFormats and made some progress see\r\n\r\n https://github.com/BITPlan/SemanticResultFormats/blob/6ac8769ea7ba5b22abfaa504afc015cbe3dc474c/formats/word/SRF_Word.php\r\n\r\nI am having trouble looping over the DataItems and Values - my example has some Date values in it and they are not catched in the field loop yet. \r\n\r\nThere are a few different ways i have seen to do this:\r\n* http://semantic-mediawiki.org/wiki/Writing_result_printers\r\n* https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/master/formats/excel/SRF_Excel.php\r\n* https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/master/formats/math/SRF_Math.php\r\n\r\nand all work slightly different. "},"comment":{"url":"https://api.github.com/repos/SemanticMediaWiki/SemanticResultFormats/issues/comments/126418014","html_url":"https://github.com/SemanticMediaWiki/SemanticResultFormats/issues/114#issuecomment-126418014","issue_url":"https://api.github.com/repos/SemanticMediaWiki/SemanticResultFormats/issues/114","id":126418014,"user":{"login":"JeroenDeDauw","id":146040,"avatar_url":"https://avatars.githubusercontent.com/u/146040?v=3","gravatar_id":"","url":"https://api.github.com/users/JeroenDeDauw","html_url":"https://github.com/JeroenDeDauw","followers_url":"https://api.github.com/users/JeroenDeDauw/followers","following_url":"https://api.github.com/users/JeroenDeDauw/following{/other_user}","gists_url":"https://api.github.com/users/JeroenDeDauw/gists{/gist_id}","starred_url":"https://api.github.com/users/JeroenDeDauw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JeroenDeDauw/subscriptions","organizations_url":"https://api.github.com/users/JeroenDeDauw/orgs","repos_url":"https://api.github.com/users/JeroenDeDauw/repos","events_url":"https://api.github.com/users/JeroenDeDauw/events{/privacy}","received_events_url":"https://api.github.com/users/JeroenDeDauw/received_events","type":"User","site_admin":false},"created_at":"2015-07-30T17:56:48Z","updated_at":"2015-07-30T17:56:48Z","body":"Cool, thanks for the contribution!\r\n\r\nYou know, from this issue description, I thought the format did something special with each \"word\". Realized this was about MS Word only when looking at the php file :)\r\n\r\nAs kgh noted, feel free to submit this as a pull request. Though please do fix that indenting (we use tabs). Other superficial things are wrong @since tags (just delete them, except the one for the class itself), commented out code (please remove) and use protected (please use private where possible). Once the code is a bit more readable and submitted as a PR, it'll be easier to do a more thorough review.\r\n\r\nAlso, please consider writing some tests.\r\n\r\n "}}