Roots Classic

Roots Classic

10M Downloads

Wrong standing stones requirements (1.19)

bloche1871 opened this issue ยท 5 comments

commented

It seems that many of the rituals are set to require the wrong standing stone pattern. For example the living tools require a full setup with attuned stones, when they should require no standing stones.

commented

I am basing this on what i could find online on wikis and videos about the mod, and also on the original version of the mod back when it was forked from the original Roots.
You can see the original code for the rituals here, and many of the rituals have different standing stone requirements (unless there is something I misunderstand):
https://github.com/Lothrazar/RootsClassic/blob/60826cb8660dd555713ff6295f4622d263cc2640/src/main/java/elucent/rootsclassic/ritual/RitualManager.java

commented

It has always required a full setup

commented

I based the requirements on the 1.12 branch as that was the only version I've ever touched before helping out on the mod

commented

So I was unaware that in 1.10 they had different requirements

commented

OK I see, it looks like the change was introduced with this commit "1.1.2 zenscript for add recipe" which introduced a different format for registering rituals:
bf5754b

I believe it's safe to assume this is a mistake. It looks like the new ritual registry code was copy-pasted for each ritual, and the author forgot to adjust the "level" requirement.
In addition, the rituals are laid out logically in the tablet if you go by the original standing stones requirement, but the order no longer makes sense with the new requirements.