Skip to content

More Forge Events#291

Closed
mrTJO wants to merge 3 commits intoMinecraftForge:masterfrom
mrTJO:master
Closed

More Forge Events#291
mrTJO wants to merge 3 commits intoMinecraftForge:masterfrom
mrTJO:master

Conversation

@mrTJO
Copy link
Copy Markdown

@mrTJO mrTJO commented Dec 3, 2012

Event to Manage CraftingManager Matching Recipes.
Can be used for various things like time and zone based recipes or limit
an item craft to certain users.

Event to Manage CraftingManager Matching Recipes.
Can be used for various things like time and zone based recipes or limit
an item craft to certain users.
@AbrarSyed
Copy link
Copy Markdown
Member

👍
LEX! I need this for ForgeEssentials...

@Chicken-Bones
Copy link
Copy Markdown
Contributor

Fun for ACT's! No worries, this looks awesome, just don't forget you may get 'fake players'

@AbrarSyed
Copy link
Copy Markdown
Member

fake players????

@nevercast
Copy link
Copy Markdown

Fake players are simulated players for doing things like using items, Such as BuildCraft using [BuildCraft] iirc, for some of it's machines, and ComputerCraft using [ComputerCraft] Or Turtle, depending on Config for Turtles mining and using items.

@Chicken-Bones
Copy link
Copy Markdown
Contributor

A player that is simply a random subclass of EntityPlayer. Used in situations like above. Beware of assuming that you are going to get a valid logged in player that's registered to the world.

@AbrarSyed
Copy link
Copy Markdown
Member

perhaps a playername with special characters so we can identify them? and provide different permission check.. or something...

@Chicken-Bones
Copy link
Copy Markdown
Contributor

player.getEntityName() != null will generally suffice

you could go further and use MinecraftServer.getServer().getConfigurationManager().playerEntityList.contains(player)

@AbrarSyed
Copy link
Copy Markdown
Member

I wonder... I could probably just ask auto-crafting machine modders to manually check the permission.. its not very hard... but wtvr,.. thats nice :D

LEX!!! I NEED THIS!!!!

mrTJO added 2 commits December 4, 2012 22:45
And added a test class.
This SMP Event is Fired when the server sends the player info to client,
it can be used to send additional packets regarding the player, ie.
player titles, guild crests, races and even more.
@pokemane
Copy link
Copy Markdown

I would also really like to see this added, as I really don't want to be forced to make a coremod.

@tambry
Copy link
Copy Markdown

tambry commented Dec 12, 2012

@mrTJO Send is irregular form so its 2nd form (Past) is Sent PlayerSpawnSendedEvent would be correct in english by changing its name to: PlayerSpawnSentEvent
Also request this to be added.

EDIT:
Remove the tests and make new Pull Request where all of the commits are squashed into 1tiny one - bigger chance getting pulled.

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.

6 participants