Skip to content

/ Applied-Energistics-2

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

Jump to bottom

Crash with Name Inscriber Press and Inscriber #3476

Closed
sajhino opened this issue May 2, 2018 · 1 comment
Closed

Crash with Name Inscriber Press and Inscriber #3476

sajhino opened this issue May 2, 2018 · 1 comment
Labels
bug fixed

Comments

@sajhino
Copy link

@sajhino sajhino commented May 2, 2018

Crashlog

Modpack: FTB Revelations 1.8.0
Minecraft Version: 1.12.2
Forge Version: 14.23.2.2627
AE Version: rv5-stable-11

Steps to replicate crash:

  1. Make 2 Name Inscriber Press.
  2. Place them into an Inscriber, one on the top slot and one on the bottom slot.
  3. Place an ME Interface into the middle slot.
  4. Remove the top Name Inscriber Press as the ME Interface is being renamed.
  5. Game crashes.

Can't even load the world anymore after the crash, have to use an older backup.
@Pingger Pingger mentioned this issue May 8, 2018
Closed
@mindforger mindforger added the bug label May 8, 2018
@mindforger
Copy link
Collaborator

@mindforger mindforger commented May 8, 2018

hope you don't mind me slapping the label on it @yueh, this one seems confirmed, a more clean report is over in the linked issue i just closed because of duplication

https://pastebin.com/PnQrSya4

from @bkbarbee for reference
kylev added a commit to kylev/Applied-Energistics-2 that referenced this issue May 16, 2018
@kylev
Make sure there are stacks before calling isEmpty() (Fixes AppliedEne…
b56bab9 
…rgistics#3476)

It was possible to cause a crash by removing something from one of the
slots during a crafting operation.
kylev added a commit to kylev/Applied-Energistics-2 that referenced this issue May 16, 2018
@kylev
Make sure there are item stacks before calling isEmpty() (AppliedEner…
d32cf18 
…gistics#3476)

It was possible to cause a crash by removing something from one of the
slots during a crafting operation.
kylev added a commit to kylev/Applied-Energistics-2 that referenced this issue May 16, 2018
@kylev
Make sure there are item stacks before calling isEmpty() (AppliedEner…
42f75d8 
…gistics#3476)

It was possible to cause a crash by removing something from one of the
slots during a crafting operation.
@kylev kylev mentioned this issue May 29, 2018
Closed
yueh added a commit that referenced this issue May 30, 2018
@yueh
Fixes #3476: Correctly handle an empty top namepress.
Loading status checks…
9690100 
Also some further changes like avoiding to match a recipe without any
input.
yueh added a commit that referenced this issue Jun 2, 2018
@yueh
Fixes #3476: Correctly handle an empty top namepress. (#3521)
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
987ea8d 
Also some further changes like avoiding to match a recipe without any
input.
@yueh yueh added the fixed label Jun 26, 2018
@yueh yueh closed this Jun 26, 2018
@yueh yueh mentioned this issue Sep 22, 2018
Closed
@KingFishQiu KingFishQiu mentioned this issue Sep 29, 2018
Closed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug fixed
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@yueh @mindforger @sajhino