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

Fix FramezAPI usage #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

OvermindDL1
Copy link

Use the Framez for MC 1.7.10 API.

Fixes ReikaKalseki/Reika_Mods_Issues#1956

Use the Framez for MC 1.7.10 API.

Fixes ReikaKalseki/Reika_Mods_Issues#1956
@OvermindDL1
Copy link
Author

There may be other changes required for the Framez interface to compile (can't really test it obviously...), but at the very least this PR is necessary.

Fixed naming to match the FramezAPI
@OvermindDL1
Copy link
Author

Decided to find the FramezAPI and noticed a lot more was wrong, so this would never actually successfully initialize before, it looks like everything is implemented properly now, though without a buildsystem for dragonAPI it's still a bit difficult to actually test that assertion.

@Override
public boolean canHandle(World world, int x, int y, int z) {
// TODO Auto-generated method stub
return false;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should obviously be changed with whatever it needs to be, probably the isBlacklisted list check.

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.

1 participant