New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Core/Script: fix bug that prevent double spending reset cooldown #16287
Core/Script: fix bug that prevent double spending reset cooldown #16287
Conversation
By analyzing the blame information on this pull request, we identified @Vincent-Michael, @ShinDarth and @Rushor to be potential reviewers |
@@ -1,5 +1,5 @@ | |||
/* | |||
* Copyright (C) 2008-2016 TrinityCore <http://www.trinitycore.org/> | |||
* Copyright (C) 2008-2015 TrinityCore <http://www.trinitycore.org/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wtf?
that was annoying, Good Job! |
@@ -14,17 +14,18 @@ | |||
* You should have received a copy of the GNU General Public License along | |||
* with this program. If not, see <http://www.gnu.org/licenses/>. | |||
*/ | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
revert this
1219ae8
to
6a2e7d8
Compare
|
||
class DuelResetScript : public PlayerScript | ||
{ | ||
public: | ||
DuelResetScript() : PlayerScript("DuelResetScript") { } | ||
|
||
typedef std::chrono::system_clock Clock; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you should just use SpellHistory::Clock instead of typedefing it here
c3b2e42
to
c0ed819
Compare
Core/Script: fix bug that prevent double spending reset cooldown
With this fix, all spells cast 30 second before duel, it will not be restored