Skip to content
This repository has been archived by the owner on Aug 21, 2019. It is now read-only.

Update to latest nukkitx #102

Merged
merged 2 commits into from
Feb 7, 2018
Merged

Conversation

boy0001
Copy link
Contributor

@boy0001 boy0001 commented Feb 4, 2018

No description provided.

@@ -1039,7 +1037,7 @@ public void sendTitle(String title, String subtitle, int fadeIn, int stay, int f

@Override
public void setAllowFlight(boolean value) {
nukkit.getAdventureSettings().setCanFly(value);
nukkit.getAdventureSettings().set(AdventureSettings.Type.ALLOW_FLIGHT, true);
Copy link
Owner

Choose a reason for hiding this comment

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

This should be set to value I think, not to true

@@ -1100,7 +1098,7 @@ public void setFireTicks(int fireTicks) {

@Override
public void setFlying(boolean flying) {
nukkit.getAdventureSettings().setFlying(flying);
nukkit.getAdventureSettings().set(AdventureSettings.Type.FLYING, true);
Copy link
Owner

Choose a reason for hiding this comment

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

Same here.

@rutgerkok
Copy link
Owner

Thank you! I found two places where I think there's a bug, could you take a look at them?

@twixi
Copy link

twixi commented Feb 7, 2018

its work?

@rutgerkok rutgerkok merged commit 9b36e97 into rutgerkok:master Feb 7, 2018
@rutgerkok
Copy link
Owner

Merged!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants