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

feat: special magic level #3843

Merged
merged 3 commits into from
Dec 14, 2021
Merged

feat: special magic level #3843

merged 3 commits into from
Dec 14, 2021

Conversation

ArturKnopik
Copy link
Contributor

@ArturKnopik ArturKnopik commented Dec 12, 2021

16:06 You see a rainbow shield (Def:30, energy magic level +20, fire magic level +3, healing magic level +10).
It can only be wielded properly by knights of level 100 or higher.
It weighs 69.00 oz.
Strange elemental magic flows over this shield.
specialSkillMlvl
{"magiclevelenergy", ITEM_PARSE_MAGICLEVELENERGY},
{"magiclevelfire", ITEM_PARSE_MAGICLEVELFIRE},
{"magiclevelpoison", ITEM_PARSE_MAGICLEVELPOISON},
{"magiclevelearth", ITEM_PARSE_MAGICLEVELPOISON},
{"magiclevelice", ITEM_PARSE_MAGICLEVELICE},
{"magiclevelholy", ITEM_PARSE_MAGICLEVELHOLY},
{"magicleveldeath", ITEM_PARSE_MAGICLEVELDEATH},
{"magiclevellifedrain", ITEM_PARSE_MAGICLEVELLIFEDRAIN},
{"magiclevelmanadrain", ITEM_PARSE_MAGICLEVELMANADRAIN},
{"magicleveldrown", ITEM_PARSE_MAGICLEVELDROWN},
{"magiclevelphysical", ITEM_PARSE_MAGICLEVELPHYSICAL},
{"magiclevelhealing", ITEM_PARSE_MAGICLEVELHEALING},
{"magiclevelundefined", ITEM_PARSE_MAGICLEVELUNDEFINED},

Pull Request Prelude

  • I have followed [proper The Forgotten Server code styling][code].
  • I have read and understood the [contribution guidelines][cont] before making this PR.
  • I am aware that this PR may be closed if the above-mentioned criteria are not fulfilled.

@Zbizu
Copy link
Contributor

Zbizu commented Dec 12, 2021

I believe that they should also be pushed to lua (in itemType:getAbilities())

also, the variable names could be shortened to just MAGICLEVELELEMENT rather than SPECIALMAGICLEVELELEMENT

src/items.h Outdated Show resolved Hide resolved
src/player.h Outdated Show resolved Hide resolved
src/item.cpp Outdated Show resolved Hide resolved
@EPuncker EPuncker added the feature New feature or functionality label Dec 13, 2021
src/protocolgame.cpp Outdated Show resolved Hide resolved
src/item.cpp Outdated Show resolved Hide resolved
src/item.cpp Outdated Show resolved Hide resolved
src/item.cpp Outdated Show resolved Hide resolved
src/item.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@EPuncker EPuncker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested and working :)

@EPuncker EPuncker merged commit a1775d2 into otland:master Dec 14, 2021
ArturKnopik added a commit to ArturKnopik/forgottenserver that referenced this pull request Dec 18, 2021
> Specialised Magic Levels: items will be able to receive a magic level modifier for specific elements, e.g. "fire magic level +2", which means only fire damage spells are boosted. Possible elements include Fire, Earth, Energy, Ice, Holy, Death and Healing.

https://tibia.fandom.com/wiki/Updates/12.70.10953
ArturKnopik added a commit to ArturKnopik/forgottenserver that referenced this pull request Dec 18, 2021
@ArturKnopik ArturKnopik deleted the specialSkills branch December 18, 2021 18:57
@omarcopires
Copy link
Contributor

This feature ends a task from issue #3758
New types of items modifiers were be implemented: Specialised Magic Levels, Cleave, Magic Shield Capacity, Perfect Shot (aka Range Bonus), Damage Reflection [12.70.10953]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants