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

Update src/com/untamedears/PrisonPearl/PrisonPearlManager.java #13

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

Conversation

TinieSnipah
Copy link

Gives enchantment effect, lore and name to PrisonPearls
Reason for edit:
Gives pearl enchantment effect (distinguishable, unstackable)
Gives name of prisoner in inventory

//NOTE: Not sure if this would work, haven't been able to run a test on it. Eclipse finds no issues, just wondering if the 'pearlnum' is needed for the plugin to work

Gives enchantment effect, lore and name to PrisonPearls
Reason for edit:
Gives pearl enchantment effect (distinguishable, unstackable)
Gives name of prisoner in inventory

//NOTE: Not sure if this would work, haven't been able to run a test on it. Eclipse finds no issues, just wondering if the 'pearlnum' is needed for the plugin to work
@ttk2
Copy link

ttk2 commented Jan 5, 2013

Pearlnum is essential. Its how prisonpearl keeps track of pearls.
On Jan 4, 2013 10:21 PM, "TinieSnipah" [email protected] wrote:

Gives enchantment effect, lore and name to PrisonPearls
Reason for edit:
Gives pearl enchantment effect (distinguishable, unstackable)
Gives name of prisoner in inventory

//NOTE: Not sure if this would work, haven't been able to run a test on
it. Eclipse finds no issues, just wondering if the 'pearlnum' is needed for

the plugin to work

You can merge this Pull Request by running:

git pull https://github.com/TinieSnipah/PrisonPearl patch-1

Or view, comment on, or merge it at:

#13
Commit Summary

  • Update src/com/untamedears/PrisonPearl/PrisonPearlManager.java

File Changes

  • M src/com/untamedears/PrisonPearl/PrisonPearlManager.java (19)

Patch Links

@Pheenixm
Copy link

Pheenixm commented Jan 5, 2013

I re-comitted this for you, fixed several errors you made (simple stuff, but you need to get those build paths on your side working). It still says your name on it though.

@TinieSnipah
Copy link
Author

@felzix Yeah I do, haha
Was a bit tired when I wrote it and just derped

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.

4 participants