Slimefun / Slimefun4

Slimefun 4 - A unique Spigot/Paper plugin that looks and feels like a modpack. We've been giving you backpacks, jetpacks, reactors and much more since 2013.
GNU General Public License v3.0
958 stars 547 forks source link

Books and player heads metadata copy/dupe #4020

Open CoolFido opened 9 months ago

CoolFido commented 9 months ago

โ— Checklist

๐Ÿ“ Description

If you connect two chests via Cargo Nodes, you can copy metadata of signed books and player heads from one to another, resulting in, for example, duplication of "cheque" books, or other valuable books or heads.

๐Ÿ“‘ Reproduction Steps

  1. Have two signed books (A and B).
  2. Place these blocks (listed from left to right, but placement order does not matter, appaerently):
    • Chest,
    • Cargo Node (Input),
    • Cargo Manager
    • Cargo Node (Output),
    • Chest.
      1. Set the Cargo Node (Input) type to Blacklist.
      2. Insert the book to be copied (A) into the right (output) chest.
      3. Insert the other book (B) into the left (input) chest.
      4. In the right chest you will have the copied book (A) two times.

The same applies to player heads.

Also it works for higher quantities of items: for example, if you were to fill the right chest with the A books, and then proceed to insert some B books into the left chest, the item stacks in the right chest will all get eventually filled up.

On the other hand, if you would only have 1 A book in the right chest and you were to put 17 B books into the left chest, you would end up with 16 A books and 2 B books in the right chest. So the issue is clearly linked to incomplete item stacks.

๐Ÿ’ก Expected Behavior

I would expect the B book to transfer to the right (output) chest, instead of getting another A book.

๐Ÿ“ท Screenshots / Videos

See the bug reproduction process screencap: https://ctrlv.tv/ZLMh

๐Ÿ“œ Server Log

No response

๐Ÿ“‚ /error-reports/ folder

No response

๐Ÿ’ป Server Software

Paper

๐ŸŽฎ Minecraft Version

1.17.x

โญ Slimefun version

Tested on:

This Server uses the following setup of Slimefun:
Paper git-Paper-411 (MC: 1.17.1)
Slimefun RC - 35 (git 5e901972)
Metrics-Module #29
Java 17
Installed Addons: (2)
  CMILib v1.2.5.3
  CMI v9.2.1.1

,

This Server uses the following setup of Slimefun:
Paper git-Paper-411 (MC: 1.17.1)
Slimefun DEV - 1104 (git c977454e)
Metrics-Module #29
Java 17
Installed Addons: (2)
  CMILib v1.2.5.3
  CMI v9.2.1.1

and:

This Server uses the following setup of Slimefun:
Paper git-Paper-81 (MC: 1.19)
Slimefun RC - 35 (git 5e901972)
Metrics-Module #29
Java 17
Installed Addons: (14)
  DyedBackpacks vDEV - 17 (git 8b329e96)
  Residence v5.0.2.0
  EcoPower vDEV - 5 (git 5c1ecd68)
  HotbarPets vDEV - 38 (git fa6c871d)
  ExoticGarden vDEV - 78 (git 7f9a5f63)
  InfinityExpansion vDEV - 144 (git 9da37202)
  ElectricSpawners vDEV - 23 (git 6ec96773)
  LiteXpansion vDEV - 192 (git 661862ed)
  SoulJars vDEV - 24 (git 78d1c6fa)
  SlimyTreeTaps vDEV - 20 (git 963f0383)
  PrivateStorage vDEV - 18 (git 5832db21)
  FluffyMachines vDEV - 132 (git ccf029ba)
  CMI v9.6.0.7
  Core v1.0

๐Ÿงญ Other plugins

No response

Boomer-1 commented 9 months ago

before anything can be done you must update to the latest dev version. once you do, i believe this issue will be resolved. I'll keep this open for a couple days to give you time to update and confirm

CoolFido commented 9 months ago

@Boomer-1

before anything can be done you must update to the latest dev version. once you do, i believe this issue will be resolved. I'll keep this open for a couple days to give you time to update and confirm

As of right now, accroding to the download page the latest dev version is: 09 Oct 2023 (18:54:31) #c9774.

As you can see in my issue, the versions that feature the described bug do include the version Slimefun DEV - 1104 (git c977454e), which also is the version used in the attached screencapture.

Boomer-1 commented 9 months ago

the screenshots you provided show you are using RC35 which is nearly a year old. you must update to 1104

CoolFido commented 9 months ago

the screenshots you provided show you are using RC35 which is nearly a year old. you must update to 1104

?

https://ctrlv.tv/ZLMh @ 0:07: obrazek

Boomer-1 commented 9 months ago

and you've been able to replicate this on 1104?

CoolFido commented 9 months ago

https://github.com/Slimefun/Slimefun4/issues/4020#issuecomment-1804844208

As you can see in my issue, THE VERSIONS THAT FEATURE THE DESCRIBED BUG D O I N C L U D E
THE VERSION Slimefun DEV - 1104
(git c977454e), which also is the version used in the attached screencapture.

Boomer-1 commented 9 months ago

It's ok I was able to replicate it.

PriestOfTerror commented 4 months ago

Please correct this error. has persisted for a long time and affects a large portion of slimefun, rendering it unusable.

MineHubPruvodce commented 3 months ago

Please correct this error. has persisted for a long time and affects a large portion of slimefun, rendering it unusable.

Boomer-1 commented 3 months ago

the devs are aware but are working on much more significant issues atm.

croudom10 commented 1 week ago

Haven't been able to replicate with player heads on 1149. Books are still copied however.