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
[3.3.5] Core/Spells: Fix Glyph of Arcane Shot #16622
Conversation
By analyzing the blame information on this pull request, we identified @joschiwald, @Kandera and @Vincent-Michael to be potential reviewers |
|
||
DELETE FROM `spell_script_names` WHERE `ScriptName` = 'spell_hun_glyph_of_arcane_shot'; | ||
INSERT INTO `spell_script_names` (`spell_id`, `ScriptName`) VALUES | ||
(56841,'spell_hun_glyph_of_arcane_shot'); |
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.
fix end of line
|
||
int32 mana = procSpell->CalcPowerCost(GetTarget(), procSpell->GetSchoolMask()); | ||
int32 basepoint = CalculatePct(mana, aurEff->GetAmount()); | ||
GetTarget()->CastCustomSpell(GetTarget(), SPELL_HUNTER_GLYPH_OF_ARCANE_SHOT, &basepoint, NULL, NULL, true, NULL, aurEff, GetCasterGUID()); |
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.
Some devs often complain about NULL
it is preferable to use nullptr
402b706
to
67873d0
Compare
@@ -0,0 +1,7 @@ | |||
-- | |||
DELETE FROM `spell_proc_event` WHERE `entry`= 56841; | |||
INSERT INTO `spell_proc_event` VALUES (56841, 0, 9, 2048, 2048, 2048, 256, 0, 0, 0, 0); |
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.
hex looks better, and always add column names to insert querys
58962d6
to
6eaf66c
Compare
All suggestions have been applied, ready for merge. 👍 |
there are some suggestions pending, you know |
who says? |
Ready for merge |
[3.3.5] Core/Spells: Fix Glyph of Arcane Shot
Takeover of #16531 with recommended changes.
Closes #16500 and #16531
(Thanks to https://github.com/Eliminationzx)