Ancient Spellcraft

Ancient Spellcraft

6M Downloads

[Bug] Sage spells json configurations

KnightCa opened this issue ยท 4 comments

commented

Is there an existing issue for this?

  • I have searched the existing issues

Observed behavior

Ancient Spells asset/spells configs may need some adjusting:

Available in Treasure Chests, by Trade, or by Looting

The following spells are set as Sage spells, but are available by either Treasure, Trade, or Loot. Are these errors (ie they all should be set to only be gained by the experiment/scribe spell)?

Gained by Treasure, Trade, Loot:

  • Experiment
  • Spectral Wall
  • Teleport Object

Gained by Treasure, Loot:

  • Poison Spray

NPC Sages

These Sage spells are allowed for NPC Sages: Appears like they should not be allowed.

  • Experiment
  • Teleport Object

These Sage spells are not allowed for NPC Sages: Appears like they should be allowed.

  • Nether Guard
  • Ray of Enfeeblement
  • Summon Zombie Pigman
  • Transplace

Dispensers

These Sage spells are not allowed by Dispensers: Appears like they should be allowed.

  • Conjure Cake
  • Extension
  • Molten Boulder
  • Ray of Enfeeblement
  • Spectral Floor
  • Ternary Storm

These Sage spells are allowed for Dispensers: Appears like they should not be allowed.

  • Forced Channel
  • Teleport Object
  • Thoughtsteal

Expected behaviour

I expected all Sage spells to be unavailable by Treasure, Trade, or Loot. You would have to use Scribe to discover them.

I expected that Sage spells that summoned minions, attack spells, and allowed options for movement to be allowed for NPC Sages.

I expected that Sage spells that don't seem to be something a NPC would cast to be disallowed.

I thought that spells that require interaction with a users playerdata (such as using your Tome, selected teleportation circle, or replacing a spell slot) would not be found in Dispensers.

I thought that spells that had a clear direct effect not requiring player data would be able to be used in Dispensers.

Steps to reproduce

  1. Open up Ancient Spellcraft json and look at asset/spells for the spells mentioned.

Environment (Singleplayer/Server, etc.)

Issue present on both sides

Mod version

1.5.11-PreRelease-4f214c5

Forge version

1.12.2-forge-14.23.5.2860

Other mods

Electroblob Wizardry 4.3.12

commented

This issue is potentially a duplicate of one of the following issues:

  • [#162] [Bug] extra Unnamed Spell "Perfect Theory Spell" (51.14% match)
commented

This issue is potentially a duplicate of one of the following issues:

  • [#235] [Bug] Runewords and Runesmithing spell json configurations (65.67% match)
commented

This issue is potentially a duplicate of one of the following issues:

  • [#235] [Bug] Runewords and Runesmithing spell json configurations (65.67% match)
commented

This issue is potentially a duplicate of one of the following issues:

  • [#235] [Bug] Runewords and Runesmithing spell json configurations (65.67% match)