{"action":"opened","issue":{"url":"https://api.github.com/repos/OpenRA/OpenRA/issues/8763","labels_url":"https://api.github.com/repos/OpenRA/OpenRA/issues/8763/labels{/name}","comments_url":"https://api.github.com/repos/OpenRA/OpenRA/issues/8763/comments","events_url":"https://api.github.com/repos/OpenRA/OpenRA/issues/8763/events","html_url":"https://github.com/OpenRA/OpenRA/issues/8763","id":96521675,"number":8763,"title":"AttackMedic and AutoHeal duplicates the logic of AttackFrontal and AutoTarget","user":{"login":"GraionDilach","id":1136302,"avatar_url":"https://avatars.githubusercontent.com/u/1136302?v=3","gravatar_id":"","url":"https://api.github.com/users/GraionDilach","html_url":"https://github.com/GraionDilach","followers_url":"https://api.github.com/users/GraionDilach/followers","following_url":"https://api.github.com/users/GraionDilach/following{/other_user}","gists_url":"https://api.github.com/users/GraionDilach/gists{/gist_id}","starred_url":"https://api.github.com/users/GraionDilach/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GraionDilach/subscriptions","organizations_url":"https://api.github.com/users/GraionDilach/orgs","repos_url":"https://api.github.com/users/GraionDilach/repos","events_url":"https://api.github.com/users/GraionDilach/events{/privacy}","received_events_url":"https://api.github.com/users/GraionDilach/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-07-22T09:35:42Z","updated_at":"2015-07-22T09:35:42Z","closed_at":null,"body":"Currently we have a duplicate logic for healers. This means that if a unit wants to be a healers/active buffer with a damaging weapon as well, the unit can either resort to force-fire to heal/buff (along with forcing the damaging weapons to not work on allies to make sure it won't get fired along with the bufferweapon) via using the right Attack* or just split the functionality of and use purely AttackMedic etc.\r\n\r\nThis also means taht healer/buffers are hardcoded to have \"`AttackFrontal`\".\r\n\r\nWhat I'd propose that instead of the duplication logic, there would be some kind of a `Benefical` tag on the weapon and `AutoTarget` would look at that - if `Benefical=yes`, allied autotargeting is allowed with that weapon, if no, enemy autotargeting is allowed on that weapon.\r\n\r\n#8563 is a case where this becomes a problem."}}