Ancient Spellcraft

Ancient Spellcraft

6M Downloads

[Bug] extra Unnamed Spell "Perfect Theory Spell"

KnightCa opened this issue ยท 1 comments

commented

Is there an existing issue for this?

  • I have searched the existing issues

Observed behaviour

Ancient Spellcraft seems to have two almost identical Master Sage Utility spells, both with a mana cost of 120 and almost identical names. Can we have the extraneous one removed (I'm guessing "Perfect Theory Spell")?

ID "Perfect Theory Spell"
Minecraft/Github: "Unnamed Spell"
Asset/Spell: Chargeup 20, Cooldown 100
Github: metadata 194, Non-Continuous
https://github.com/WinDanesz/AncientSpellcraft/wiki/Unnamed-Spell

ID "Perfect Theory"
Minecraft/Github: "Perfect Theory"
Asset/Spell: Chargeup 0, Cooldown 20, Range 5
Github: metadata 193, Continuous
https://github.com/WinDanesz/AncientSpellcraft/wiki/Perfect-Theory

Expected behaviour

I expected all spells found in Ancient Spellcraft to have a name and there to not be duplicates.

Steps to reproduce

Look at the two mentioned spells on github and the asset/spells configs.

Crashlog

No response

Environment (Singleplayer/Server, etc.)

Issue present on both sides

Mod version

1.5.10

Forge version

1.12.2 latest version

Other mods

Electroblob Wizardry latest version for 1.12.2

commented

Discovered the following:

The Unnamed Spell is a special spellbook, created by (and reserved for) the Sage spell "Perfect Theory".

Perfect Theory uses a Theory Point to record the last fluke (effect you get when a spell fails to be discovered). It is recorded into a blank Mystic Spellbook while it's on a Mystic Lectern.

Since the fluke is not a standard spell and can fit a variety of effects, depending on the fluke.. then it's a "Unnamed" spell.