Skip to content
This repository was archived by the owner on Feb 27, 2024. It is now read-only.

Sponge duplication bug #3008

Closed
focamacho opened this issue Oct 30, 2019 · 2 comments
Closed

Sponge duplication bug #3008

focamacho opened this issue Oct 30, 2019 · 2 comments
Labels
type: bug Something isn't working version: 1.12 API 7

Comments

@focamacho
Copy link

I am currently running

  • SpongeForge version: 7.1.7-RC3928
  • Forge version: 14.23.5.2847
  • Java version: 8
  • Operating System: Windows 10
  • Plugins/Mods:
    Mods (9): Minecraft, Minecraft Coder Pack, Forge Mod Loader, Minecraft Forge, SpongeAPI, Sponge, Chameleon, SpongeForge, Storage Drawers
    Plugins (4): Minecraft, Minecraft Coder Pack, SpongeAPI, Sponge

Issue Description

  1. Place a empty drawer on the ground.
  2. Type "/" ou any command in the chat.
  3. Right click the drawer with any item in your hand.
  4. Infinite items.

Video: https://youtu.be/vDk3xRsKKI0

Works with Mekanism bins & Thermal Expansion cache too.

@ImMorpheus ImMorpheus added type: bug Something isn't working version: 1.12 API 7 labels Nov 2, 2019
@LXGaming
Copy link
Contributor

LXGaming commented Nov 4, 2019

Debugging this I've discovered that this issue is very similar to this one:
SpongePowered/Sponge#2187 (comment)

This line throws a CommandException:
https://github.com/SpongePowered/SpongeCommon/blob/257331d774c78faf8557a4e8f27dd46ab0bf04af/src/main/java/org/spongepowered/common/command/SpongeCommandManager.java#L340

Which causes it to skip this very important line:
https://github.com/SpongePowered/SpongeCommon/blob/257331d774c78faf8557a4e8f27dd46ab0bf04af/src/main/java/org/spongepowered/common/command/SpongeCommandManager.java#L342

This causes the players inventory to be in a capture state for future events when it shouldn't be.

@dualspiral
Copy link
Contributor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug Something isn't working version: 1.12 API 7
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants