Skip to content

feat: Spell Learning System#572

Merged
jprzimba merged 32 commits intozimbadev:mainfrom
Paco161315:feat-spell-learning-system
Mar 19, 2026
Merged

feat: Spell Learning System#572
jprzimba merged 32 commits intozimbadev:mainfrom
Paco161315:feat-spell-learning-system

Conversation

@Paco161315
Copy link
Copy Markdown
Contributor

@Paco161315 Paco161315 commented Jan 20, 2026

1.- Added complete spells to every spell learning NPC in game.
2.- Enabled learning for all game spells.
3.- Added new spells from Isle of Destiny.
4.- Keyword order arranged in every npc to avoid bugs.
5.- Updated real tibia dialogs in some NPCs.

You can enable or disable at config.lua

Paco161315 and others added 2 commits January 20, 2026 05:11
Added spells to NPCs from: Thais, Carlin, Venore, Ab'dendriel, Kazordoon, Edron
Enabled learning for all game spells
Added new spells from Isle of Destiny
I think this cover all game spells but if there is any important missing NPC let me know. I will add the rest of the NPCs from the other cities when I got the time, you can help if you want.
@Paco161315 Paco161315 changed the title Feat: Spell-Learning-System Feat: Spell Learning System Jan 20, 2026
@jprzimba jprzimba changed the title Feat: Spell Learning System feat: Spell Learning System Jan 21, 2026
@jprzimba
Copy link
Copy Markdown
Collaborator

Maybe you can add new config to toggleLearnSpells in config lua so it enable/disable spell:needLearn(true), so we can disable it

Paco161315 and others added 2 commits January 23, 2026 17:12
You may now enable or disable learning spells in config.lua
@Paco161315
Copy link
Copy Markdown
Contributor Author

Maybe you can add new config to toggleLearnSpells in config lua so it enable/disable spell:needLearn(true), so we can disable it

Sent new toggle at config, I just updated the PR

Paco161315 and others added 22 commits January 24, 2026 18:12
This fix some issues when buying spell that use the same keyword combinations, like haste or light, etc.
Some more details fixed for this PR that incliude new beginer spells and a fix in the keyword order of puffels NPC.
Added executeChain target check to avoid console error flood.
Some updateds in spells I was missing on. Sorry, It should be accurate with git now.
Fixed Rune Spell Names and Invisibility, also the keyword order in some more spells.
Fix order in keywords for a proper reaction of the NPC
As the title says, just correcting duplicates with wrong names.
More debug of spells system and a new NPC for party spells at Edron Academy.
I just knew Monks autolearn the virtue spells when promoted so I kinda fixed it here. Also fixed more spell keywords and added Eremo NPC spell learning system.
Enpa Rudra now can promote monks and other players.
Added new basic paladin spell Lesser Ethereal Spear to NPCs and Dawnport Vocation Trial.

Lesser Front Sweep has been added for knights in the Dawnport Vocation Trial aswell.
Fix Fire Wave / Great Fire Wave keyword order for a prooper reaction
Added Magic Patch to Monk in Dawnport Vocation Trial
My Spell was outdated.
All Tibia NPCs added for spell learning system
Paco161315 added a commit to Paco161315/crystalserver that referenced this pull request Mar 13, 2026
Play crystal server with oldschool 7.6 rules. Why not?

1.- Configurable retro features at config.lua
2.- Walk thru players.
3.- Lure creatures.
4.-Block respawn.
5.- Offline Training.
6.- Market.

Note for an extra oldschool feature: Learning Spells please merge: zimbadev#572
@Paco161315 Paco161315 mentioned this pull request Mar 13, 2026
jprzimba added a commit that referenced this pull request Mar 19, 2026
* feat: Retro Bundle

Play crystal server with oldschool 7.6 rules. Why not?

1.- Configurable retro features at config.lua
2.- Walk thru players.
3.- Lure creatures.
4.-Block respawn.
5.- Offline Training.
6.- Market.

Note for an extra oldschool feature: Learning Spells please merge: #572

* Code format - (Clang-format)

---------

Co-authored-by: GitHub Actions <github-actions[bot]@users.noreply.github.com>
Co-authored-by: João Paulo <jprzimba@gmail.com>
@jprzimba jprzimba merged commit ffe6c38 into zimbadev:main Mar 19, 2026
25 checks passed
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.

2 participants