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

Item functions #587

Open
wants to merge 20 commits into
base: dev
Choose a base branch
from
Open

Item functions #587

wants to merge 20 commits into from

Conversation

mastercoms
Copy link
Member

This PR will remove the item and block hierarchy, which has many problems with conflicting item and block behaviors and creates a bad system for modding support, which we may want to pursue in the future.

It uses functions to provide default item and block functionality, and provides a simple system mods and plugins can use to extend this functionality and add their own items/blocks with unique functionality in the future.

@Pr0methean
Copy link
Contributor

I'll review this tonight or tomorrow.

@Pr0methean
Copy link
Contributor

This looks to me a little too much like Aspect-Oriented Programming. How much use are stack traces going to be under this new regime?

@mastercoms
Copy link
Member Author

Do you think this could be better implemented in Java 9's Flow API?

Also, I feel like it would make more sense to migrate to 1.13 first before doing such a massive change to our item/block type system.

@mastercoms
Copy link
Member Author

Closed because we're waiting for 1.13 and this branch will probably be rewritten by then

@mastercoms
Copy link
Member Author

Reopening this branch since it has become relevant again.

@mastercoms mastercoms reopened this Aug 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants