{"action":"closed","issue":{"url":"https://api.github.com/repos/LukeWoodward/SplitsBrowser/issues/33","labels_url":"https://api.github.com/repos/LukeWoodward/SplitsBrowser/issues/33/labels{/name}","comments_url":"https://api.github.com/repos/LukeWoodward/SplitsBrowser/issues/33/comments","events_url":"https://api.github.com/repos/LukeWoodward/SplitsBrowser/issues/33/events","html_url":"https://github.com/LukeWoodward/SplitsBrowser/issues/33","id":86924095,"number":33,"title":"Issues with IOF files from Winsplits Pro","user":{"login":"dfgeorge","id":11885900,"avatar_url":"https://avatars.githubusercontent.com/u/11885900?v=3","gravatar_id":"","url":"https://api.github.com/users/dfgeorge","html_url":"https://github.com/dfgeorge","followers_url":"https://api.github.com/users/dfgeorge/followers","following_url":"https://api.github.com/users/dfgeorge/following{/other_user}","gists_url":"https://api.github.com/users/dfgeorge/gists{/gist_id}","starred_url":"https://api.github.com/users/dfgeorge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dfgeorge/subscriptions","organizations_url":"https://api.github.com/users/dfgeorge/orgs","repos_url":"https://api.github.com/users/dfgeorge/repos","events_url":"https://api.github.com/users/dfgeorge/events{/privacy}","received_events_url":"https://api.github.com/users/dfgeorge/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-06-10T10:43:55Z","updated_at":"2015-07-05T13:35:30Z","closed_at":"2015-07-05T13:35:30Z","body":"Hi,\r\n\r\nIm using Winsplits Pro to generate IOF 2.0.3 files to display using splitsbrowser (http://www.users.on.net/~dsg/splitsbrowserFile/). I have found a couple of issues:\r\n\r\n1. Missed points dont have a time stamp (this seemt to conform to the DTD). This causes Version2Reader.readSplitTime to throw an exception. I have fixed by returning a null for time when the timestr is null, e.g. return {code: code, time: null}; inside the if test,\r\n\r\n2. Winsplits Pro wraps the runners time in a clock tag, e.g. \r\n    <Time>\r\n    <Clock clockFormat=\"HH:MM:SS\">00:34:54</Clock>\r\n    </Time>\r\n\r\nThis does not seem to follow the DTD :-(. This results in some new line chars being included in the time string, which breaks SplitsBrowser.parseTime. I have fixed by removing the new line chars in SplitsBrowser.parseTime, not elegant but it works and should not have any bad effects. My changes:\r\n\r\n    SplitsBrowser.parseTime = function (timeIn) {\r\n\tvar time = timeIn.replace(/(\\r\\n|\\n|\\r)/gm,\"\")"}}