AkashicTome icon indicating copy to clipboard operation
AkashicTome copied to clipboard

Cannot add multiple books from the same mod

Open WenXin20 opened this issue 4 years ago • 14 comments

Minecraft v1.15.2 Forge v31.2.31 Akashic Tome v1.3.13

I tried to add multiple books from the same mod to the tome, but it does not seem to work.

WenXin20 avatar Aug 03 '20 19:08 WenXin20

This worked in 1.12.2, or at least I was able to add multiple blueprints to the Akashic Tome when I configured it to do so. Has something changed since then?

DoomRater avatar Nov 16 '20 16:11 DoomRater

Yes indeed. There was special handling in the 1.12.2 version of the code that is not present in the 1.15.2 update. Looks like a copy and paste oversight from the Morph-o-Tool. https://github.com/Vazkii/AkashicTome/blob/647da9c9033600a89f09bb726b7a0791e8950784/src/main/java/vazkii/akashictome/AttachementRecipe.java#L68-L75

DoomRater avatar Nov 16 '20 20:11 DoomRater

Possible this could be remedied? In 1.16.5 this issue is still present - can only add one book for each mod. Several popular mods these days have multiple books (Tinker's Construct and Cooking For Blockheads for example.)

nanonestor avatar May 01 '21 21:05 nanonestor

please we need this fix, also same issues with morphing tool

benbenlaw avatar May 02 '21 11:05 benbenlaw

The Morph-o-tool never had support for more than one tool at a time, and the way it functions would make that functionality problematic. Suppose you look at a block- which of the two tools should a Morph-o-tool become, if multiple tools are allowed?

DoomRater avatar May 02 '21 13:05 DoomRater

i mean you cant put in 2 things from the same mod into the morphing tool

benbenlaw avatar May 02 '21 13:05 benbenlaw

  1. This is not the issues page for Morph-o-tool.
  2. I just got through explaining why that behavior would be a probolem.

DoomRater avatar May 02 '21 13:05 DoomRater

i know just saying, so is this the same will akashic tome never be able to support 2 different books from one mod?

benbenlaw avatar May 02 '21 13:05 benbenlaw

This is a pity. I mainly got Akeshick Tome to store my Tinker's Manuals (5 books in total)

ffuentesm avatar May 22 '21 15:05 ffuentesm

As much as I want to say more people commenting on the state of things will make things move faster, no one has even commented on the code difference I pointed out. I have been unable to compile on 1.12.2 to even check there, I don't have an environment set up to check current code and whether reverting code will make it function as it did before, and many people simply don't use the issues tracker as a discussion board.

DoomRater avatar May 22 '21 16:05 DoomRater

Popping in to say this is still an issue.

Versions
Minecraft v1.16.5
Forge v36.1.16
Akashic Tome v1.4-16

LabGoblin avatar May 28 '21 01:05 LabGoblin

Still an issue for 1.18. It applies to patchouli as well so even books from different mods can't be added if their manuals are through patchouli.

AndrewSatterfield avatar Mar 27 '22 13:03 AndrewSatterfield

@Vazkii @Alwinfy @williewillus Apologies for the pings, but is anyone aware of this issue? It's been around a long time, there was a PR for a proposed fix, and many new issues mention this problem.

copygirl avatar Apr 01 '22 18:04 copygirl

Sorry, there hasn't really been resources to devote to AT as we're all busy with our individual mods and real life. I'll take a look soon (tm).

williewillus avatar Jun 26 '22 19:06 williewillus