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

Pigs/Skulls etc. stay #189

Closed
Matthww opened this issue Dec 14, 2018 · 3 comments
Closed

Pigs/Skulls etc. stay #189

Matthww opened this issue Dec 14, 2018 · 3 comments
Labels

Comments

@Matthww
Copy link

@Matthww Matthww commented Dec 14, 2018

Please make sure your issue complies with these guidelines:

    • Server Software must be the official PMMP w/o vanilla patching plugins (i.e. TeaSpoon)
    • Issue must have not been reported previously
    • Make sure you are on the latest version of PMMP & PiggyCustomEnchants
    • Have a detailed title, like "CustomEnchants::getName() must be..."
    • If possible, provide crashdumps/errors related to the issue.

General

As you can see in the screenshot Pigs (Porkified) witherskull etc stay.

image

Version:

  • PiggyCustomEnchants: 1.4.2
  • PMMP: 3.5.1

Steps to Reproduce the Issue

  1. Shoot with the Porkified or Witherskull bow

Extra Information

@Aericio Aericio added bug wontfix client-side and removed bug labels Dec 18, 2018
@Haxley

This comment has been minimized.

Copy link
Contributor

@Haxley Haxley commented Aug 25, 2019

How do we work around this? Persisting pigs lag everything. Combine porkified with volley and gg.

@Aericio

This comment has been minimized.

Copy link
Collaborator

@Aericio Aericio commented Aug 25, 2019

I suppose I'll just comment out these enchantments.

@Haxley

This comment has been minimized.

Copy link
Contributor

@Haxley Haxley commented Aug 28, 2019

I suggest making porkified unable to be enchanted if volley is enchanted already, vice versa. Do it at the point of enchanting rather than universal ban imo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.