{"action":"created","issue":{"url":"https://api.github.com/repos/dalaranwow/dalaran-wow/issues/3314","repository_url":"https://api.github.com/repos/dalaranwow/dalaran-wow","labels_url":"https://api.github.com/repos/dalaranwow/dalaran-wow/issues/3314/labels{/name}","comments_url":"https://api.github.com/repos/dalaranwow/dalaran-wow/issues/3314/comments","events_url":"https://api.github.com/repos/dalaranwow/dalaran-wow/issues/3314/events","html_url":"https://github.com/dalaranwow/dalaran-wow/issues/3314","id":156083399,"number":3314,"title":"[Priest][Spell]Mind Control","user":{"login":"PSAlias","id":18348509,"avatar_url":"https://avatars.githubusercontent.com/u/18348509?v=3","gravatar_id":"","url":"https://api.github.com/users/PSAlias","html_url":"https://github.com/PSAlias","followers_url":"https://api.github.com/users/PSAlias/followers","following_url":"https://api.github.com/users/PSAlias/following{/other_user}","gists_url":"https://api.github.com/users/PSAlias/gists{/gist_id}","starred_url":"https://api.github.com/users/PSAlias/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PSAlias/subscriptions","organizations_url":"https://api.github.com/users/PSAlias/orgs","repos_url":"https://api.github.com/users/PSAlias/repos","events_url":"https://api.github.com/users/PSAlias/events{/privacy}","received_events_url":"https://api.github.com/users/PSAlias/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dalaranwow/dalaran-wow/labels/Class%23Priest","name":"Class#Priest","color":"FFFFFF"},{"url":"https://api.github.com/repos/dalaranwow/dalaran-wow/labels/Spells/Auras","name":"Spells/Auras","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-05-21T04:36:21Z","updated_at":"2016-05-27T19:32:01Z","closed_at":null,"body":"# Spell: Mind Control - _605_\r\n\r\n## Issue Present:\r\n\r\n**Mind Control** (_ID: 605_) is subject to a number of bugs, some of which allow for the possibility of abuse.\r\n\r\n## How Does It Work Currently:\r\n\r\n- Using this spell on an NPC while outside of the range that it would normally aggro on the player does not always cause it to engage in combat with the player.\r\n\r\n- Enemy players are inconsistently subject to the effects of this spell. At times, attempting to use the spell on one gives the priest the message \"Target is immune to charm\" when no effect is in place that should make this the case.\r\n\r\n- Some NPC abilities automatically end the charm effect when they are used by a controlling priest. Example: NPC **Wrathtail Sea Witch** (_ID: 3715_) using spell **Lightning Blast** (_ID: 8598_).\r\n\r\n- The controlling priest does not channel the spell for the duration of the charm effect, and the buff on their hud expires. This happens roughly five seconds after successfully charming a target, and leaves the priest unable to cancel the effect without moving the target out of range.\r\n\r\n- The tooltip on the spell indicates that the maximum level of a controlled target is 32 regardless of the priest's level, when this should increase as the player levels (actual effect of the spell appears to be correct, making this only a visual error on the tooltip, probably a result of following the example [here](http://wotlk.openwow.com/spell=605))\r\n\r\n- **AND THE BIG ONE** Charming a target with this spell completely eliminates its evade leash, allowing a priest to kite a mob indefinitely. No, really. Look at this:\r\n\r\n![mcbug1](https://cloud.githubusercontent.com/assets/18348509/15446336/9e729e42-1eda-11e6-8504-71dd1a143708.jpg)\r\n![mcbug2](https://cloud.githubusercontent.com/assets/18348509/15446338/be4cd50c-1eda-11e6-9ea0-cd6ccdb0d891.jpg)\r\n![mcbug3](https://cloud.githubusercontent.com/assets/18348509/15446340/c95ad08e-1eda-11e6-96c7-4c49faa81ca2.jpg)\r\n\r\nYes, that is **Gamon** (_ID: 6466_) attending the AQ opening. I was bored.\r\n\r\n## How It Should Work:\r\n\r\n- Use of the spell should always aggro and generate a significant amount of threat upon use on an NPC, but should not alter the distance at which it will evade. \r\n\r\n- Players should be subject to the effects of the spell at the same level as humanoid NPCs. \r\n\r\n- Some, but not all NPC abilities should be available for use by a controlling player. These should not inherently break the charm effect.\r\n\r\n- The controlling priest should be channeling and should have the ability to right-click away the MC buff icon for the duration of the spell effect, so as to be able to cancel it at will.\r\n\r\n- The spell tooltip should automatically update itself to reflect the maximum level that it can affect as the priest levels up.\r\n\r\nSee http://wow.gamepedia.com/index.php?title=Dominate_Mind&oldid=2562522 (note the dynamically changing level on the tooltip)\r\nand http://wow.gamepedia.com/index.php?title=List_of_mind_controllable_mobs&oldid=2097103 (this list is obviously not exhaustive, but should provide helpful examples)"},"comment":{"url":"https://api.github.com/repos/dalaranwow/dalaran-wow/issues/comments/222234560","html_url":"https://github.com/dalaranwow/dalaran-wow/issues/3314#issuecomment-222234560","issue_url":"https://api.github.com/repos/dalaranwow/dalaran-wow/issues/3314","id":222234560,"user":{"login":"adeutscher","id":2055500,"avatar_url":"https://avatars.githubusercontent.com/u/2055500?v=3","gravatar_id":"","url":"https://api.github.com/users/adeutscher","html_url":"https://github.com/adeutscher","followers_url":"https://api.github.com/users/adeutscher/followers","following_url":"https://api.github.com/users/adeutscher/following{/other_user}","gists_url":"https://api.github.com/users/adeutscher/gists{/gist_id}","starred_url":"https://api.github.com/users/adeutscher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adeutscher/subscriptions","organizations_url":"https://api.github.com/users/adeutscher/orgs","repos_url":"https://api.github.com/users/adeutscher/repos","events_url":"https://api.github.com/users/adeutscher/events{/privacy}","received_events_url":"https://api.github.com/users/adeutscher/received_events","type":"User","site_admin":false},"created_at":"2016-05-27T19:32:01Z","updated_at":"2016-05-27T19:32:01Z","body":"Additional Mind Control feature to add to the list:\r\n\r\n* When a caster who is not flagged for PvP mind controls a target that is flagged for PvP, the caster remains unflagged.\r\n\r\nIf Dalaran is running an off-shoot of TrinityCore, then the reason for this is that `Spell::DoAllEffectOnTarget` is deciding whether or not to flag the caster after the victim has already become an unflagged minion.\r\n\r\nI fixed this in TrinityCore by moving the flagging decision to before spell effects are applied (though the actual flagging action still happens at the same place). See TrinityCore/TrinityCore#17145"}}
{"actor":{"display_login":"adeutscher"}}