Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix I.R. Mod Sender. Now, if not find a router with receiver in range, #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bamsero
Copy link

@bamsero bamsero commented Jan 10, 2021

This FIX bug with I.R. Mod Sender don't move items to
vanilla inventory, adjacent to router.
I.R. Module Sender fail to insert to vanilla inventory if no router with receiver found in correct direction

git-Paper-392 (MC: 1.16.4) (Implementing API version 1.16.4-R0.1-SNAPSHOT), SensibleToolbox - DEV 37 (git 2a746)

Description

Code was missing one check if router with receiver was not found.

Changes

Add code to insert items to adjacent vanilla inventory.

Related Issues

Checklist

  • [x ] I have fully tested the proposed changes and promise that they will not break everything into chaos.
  • [x ] I followed the existing code standards and didn't mess up the formatting.
  • [x ] I did my best to add documentation to any public classes or methods I added.
  • [ x] I have added Nonnull and Nullable annotations to my methods to indicate their behaviour for null values

but adjacent is a vanilla inventory, router will try to move item to
vanilla inventory. This FIX bug with I.R. Mod Sender don't move items to
vanilla inventory, adjacent to router.
@bamsero bamsero requested a review from a team as a code owner January 10, 2021 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants