{"action":"created","issue":{"url":"https://api.github.com/repos/Sembiance/mtgjson/issues/64","labels_url":"https://api.github.com/repos/Sembiance/mtgjson/issues/64/labels{/name}","comments_url":"https://api.github.com/repos/Sembiance/mtgjson/issues/64/comments","events_url":"https://api.github.com/repos/Sembiance/mtgjson/issues/64/events","html_url":"https://github.com/Sembiance/mtgjson/issues/64","id":98734126,"number":64,"title":"Remove 'variations' field as it is redundant","user":{"login":"Sembiance","id":81436,"avatar_url":"https://avatars.githubusercontent.com/u/81436?v=3","gravatar_id":"","url":"https://api.github.com/users/Sembiance","html_url":"https://github.com/Sembiance","followers_url":"https://api.github.com/users/Sembiance/followers","following_url":"https://api.github.com/users/Sembiance/following{/other_user}","gists_url":"https://api.github.com/users/Sembiance/gists{/gist_id}","starred_url":"https://api.github.com/users/Sembiance/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sembiance/subscriptions","organizations_url":"https://api.github.com/users/Sembiance/orgs","repos_url":"https://api.github.com/users/Sembiance/repos","events_url":"https://api.github.com/users/Sembiance/events{/privacy}","received_events_url":"https://api.github.com/users/Sembiance/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Sembiance/mtgjson/labels/discuss","name":"discuss","color":"006b75"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-03T12:34:18Z","updated_at":"2015-08-20T09:12:12Z","closed_at":"2015-08-18T17:50:05Z","body":"I am thinking about removing the 'variations' field. Essentially all it is doing is pointing to other cards in the same set that have the same name. That's it. Seems pretty easy for anyone consuming the MTGJSON data that if multiple cards in the same set have the same name, clearly they are variations of each other.\r\n\r\nAlso, right now the 'variations' field uses multiverseid's which many sets do not have, thus the field is already absent on a good number of cards and sets. So removing it would actually bring some consistency to the data."},"comment":{"url":"https://api.github.com/repos/Sembiance/mtgjson/issues/comments/132947112","html_url":"https://github.com/Sembiance/mtgjson/issues/64#issuecomment-132947112","issue_url":"https://api.github.com/repos/Sembiance/mtgjson/issues/64","id":132947112,"user":{"login":"esb","id":28695,"avatar_url":"https://avatars.githubusercontent.com/u/28695?v=3","gravatar_id":"","url":"https://api.github.com/users/esb","html_url":"https://github.com/esb","followers_url":"https://api.github.com/users/esb/followers","following_url":"https://api.github.com/users/esb/following{/other_user}","gists_url":"https://api.github.com/users/esb/gists{/gist_id}","starred_url":"https://api.github.com/users/esb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/esb/subscriptions","organizations_url":"https://api.github.com/users/esb/orgs","repos_url":"https://api.github.com/users/esb/repos","events_url":"https://api.github.com/users/esb/events{/privacy}","received_events_url":"https://api.github.com/users/esb/received_events","type":"User","site_admin":false},"created_at":"2015-08-20T09:12:12Z","updated_at":"2015-08-20T09:12:12Z","body":"This change is troublesome for me. I need an array of alternatives to make it easy to match card versions. Not all systems use the same version numbering so often I have to visually verify different card versions to match things up. I need the alternatives to keep track of this process.\r\n\r\nIt was useful to know that a card had different versions as I loaded the data, so that I could build the required matching data. With that gone now, it looks like I will have to code up a second pass of the data to try to reconstruct the version data.\r\n\r\nIt would be really useful to reinstate this data as it makes my processing task a little bit easier. If people don't need the data, then it can just be ignored."}}