Skip to content
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

Spell overhaul - move item enchantments to new itemench class. Lower reagent cost/focus time for weapon enchants. Vamp drain imp rate increased from 5-10%. Apparition cost lowered. #504

Merged
merged 8 commits into from
Jul 30, 2014

Conversation

skittles1
Copy link

Part of overall spell overhauls. Split off spells that primarily affect/are cast on items to a new subclass of Spell.

Spell changes:
Artifice is changed to only affect weapons/armor, enchantment time lowered (see #493).
Enchant weapon focus time lowered to 15 seconds, reagents lowered to 2 elders/1 tooth.
Glow reagents lowered to 1 mushroom/1 elder.
Holy weapon focus time lowered to 15 seconds, reagents lowered to 2 fairy wings/1 tooth.
Shroud focus time lowered to 8 seconds, reagents lowered to 2 mushrooms/2 elders.
Unholy weapon focus time lowered to 15 seconds, reagents lowered to 2 fairy wings/1 tooth.

Spell descriptions updated with reagent cost.

@skittles1 skittles1 changed the title Spell overhaul - move item enchantments to new itemench class. Spell overhaul - move item enchantments to new itemench class. Lower reagent cost/focus time for weapon enchants. Jul 6, 2014
@skittles1 skittles1 added this to the 1.0.1.3 milestone Jul 6, 2014
@GarOfMeridian
Copy link

Funny, I was just thinking about this.

Feature request: can we make kraanan, qor, and shal enchantments permanent, but overwritable the same way buffs are? As in, I cast a Qor enchantment, and the previous enchantment (of any of the three types) is removed and replaced.

@skittles1
Copy link
Author

Weapon enchantments are now overwriteable. This affects all weapon enchants (Kran, Shal, Qor, fire, cold, shock, acid) but doesn't allow enchanting inherently enchanted weapons (such as mystic sword).

I can make them permanent, but it'll have to wait due to needing to modify files we've already modified. However, a guildmate has pointed out to me that one might want the enchantment gone from the weapon for whatever reason, so it might be a good idea to let them expire. We can lower the casting time again if need be, to make it less tedious to keep rededicating weapons.

Also: glow will recast all the time, but shroud at this stage will not.

@skittles1 skittles1 changed the title Spell overhaul - move item enchantments to new itemench class. Lower reagent cost/focus time for weapon enchants. Spell overhaul - move item enchantments to new itemench class. Lower reagent cost/focus time for weapon enchants. Vamp drain imp rate increased from 5-10%. Jul 8, 2014
@skittles1
Copy link
Author

Increased vampiric drain imp rate from 5 to 10.

@skittles1 skittles1 changed the title Spell overhaul - move item enchantments to new itemench class. Lower reagent cost/focus time for weapon enchants. Vamp drain imp rate increased from 5-10%. Spell overhaul - move item enchantments to new itemench class. Lower reagent cost/focus time for weapon enchants. Vamp drain imp rate increased from 5-10%. Apparition cost lowered. Jul 11, 2014
@skittles1
Copy link
Author

Lowered Apparition mana cost from 30 to 14, vigor cost from 20 to 12, cast time from 4 seconds to 2 seconds and reagents from 1 web moss and 2 ferns to 1 moss 1 fern. Not a very useful spell, but it'll be much less painful to cast.

@skittles1 skittles1 modified the milestone: 1.0.1.3 Jul 22, 2014
@amenius
Copy link

amenius commented Jul 26, 2014

+1 to all above changes.

We could add an item or spell to remove the permanent enchantments. Maybe even as a PVP component.

@Daenks Daenks merged commit 2dbf87f into OpenMeridian:master Jul 30, 2014
@skittles1 skittles1 deleted the itemspells branch August 1, 2014 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants