{"action":"created","issue":{"url":"https://api.github.com/repos/lodici/magarena-cardbuilder/issues/8","labels_url":"https://api.github.com/repos/lodici/magarena-cardbuilder/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/lodici/magarena-cardbuilder/issues/8/comments","events_url":"https://api.github.com/repos/lodici/magarena-cardbuilder/issues/8/events","html_url":"https://github.com/lodici/magarena-cardbuilder/issues/8","id":70278158,"number":8,"title":"Enable in-text image placement for activated abilities","user":{"login":"ShawnieBoy","id":2363180,"avatar_url":"https://avatars.githubusercontent.com/u/2363180?v=3","gravatar_id":"","url":"https://api.github.com/users/ShawnieBoy","html_url":"https://github.com/ShawnieBoy","followers_url":"https://api.github.com/users/ShawnieBoy/followers","following_url":"https://api.github.com/users/ShawnieBoy/following{/other_user}","gists_url":"https://api.github.com/users/ShawnieBoy/gists{/gist_id}","starred_url":"https://api.github.com/users/ShawnieBoy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ShawnieBoy/subscriptions","organizations_url":"https://api.github.com/users/ShawnieBoy/orgs","repos_url":"https://api.github.com/users/ShawnieBoy/repos","events_url":"https://api.github.com/users/ShawnieBoy/events{/privacy}","received_events_url":"https://api.github.com/users/ShawnieBoy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-04-23T01:38:41Z","updated_at":"2015-04-23T01:38:43Z","closed_at":null,"body":"_From @ShawnieBoy on April 21, 2015 18:37_\n\nMana costs, tap/untap symbol, etc\n\n_Copied from original issue: magarena/magarenaCardBuilder#8_"},"comment":{"url":"https://api.github.com/repos/lodici/magarena-cardbuilder/issues/comments/95384956","html_url":"https://github.com/lodici/magarena-cardbuilder/issues/8#issuecomment-95384956","issue_url":"https://api.github.com/repos/lodici/magarena-cardbuilder/issues/8","id":95384956,"user":{"login":"ShawnieBoy","id":2363180,"avatar_url":"https://avatars.githubusercontent.com/u/2363180?v=3","gravatar_id":"","url":"https://api.github.com/users/ShawnieBoy","html_url":"https://github.com/ShawnieBoy","followers_url":"https://api.github.com/users/ShawnieBoy/followers","following_url":"https://api.github.com/users/ShawnieBoy/following{/other_user}","gists_url":"https://api.github.com/users/ShawnieBoy/gists{/gist_id}","starred_url":"https://api.github.com/users/ShawnieBoy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ShawnieBoy/subscriptions","organizations_url":"https://api.github.com/users/ShawnieBoy/orgs","repos_url":"https://api.github.com/users/ShawnieBoy/repos","events_url":"https://api.github.com/users/ShawnieBoy/events{/privacy}","received_events_url":"https://api.github.com/users/ShawnieBoy/received_events","type":"User","site_admin":false},"created_at":"2015-04-23T01:38:43Z","updated_at":"2015-04-23T01:38:43Z","body":"_From @lodici on April 22, 2015 23:18_\n\nJust to give an example of a refactor using [``getFrameType()``](https://github.com/lodici/magarena-cardbuilder/blob/master/src/magic/cardbuilder/CardReader.java#L61).\r\n\r\nIn getFrameType() you are using ``MagicCard`` but if you dig down into the methods called they are delegated to ``MagicCardDefinition`` methods. ``MagicColor.isMulti()`` is a bit more problematic since it requires a ``MagicCard`` but again if you follow the code to ``numColors()``, what is it checking...``MagicCardDefinition``! This says to me that ``MagicColor.isMulti()`` should either be accepting a ``MagicCardDefinition`` directly or it should be overloaded to accept both a ``MagicCard`` or ``MagicCardDefinition`` object."}}