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

Disable interaction with players while Dragon is alive. #93

Closed
wants to merge 5 commits into from

Conversation

ShiroSR
Copy link

@ShiroSR ShiroSR commented Sep 2, 2019

Opened an issue asking for this a few hours ago, ended up doing it myself with my non-existent knowledge of Java and coding in general.
Didn't hide the players because it's counterproductive.

@biscuut
Copy link
Collaborator

biscuut commented Sep 3, 2019

Hi thanks for this, but maybe this is too specific? Can I change it maybe to disallow clicking players when you are holding out a bow?

@@ -49,7 +49,7 @@ private void removeEntities(List<Entity> list) {
if (!SkyblockAddons.getInstance().getConfigValues().isDisabled(Feature.DISABLE_PROFILE_VIEW_IF_DRAGON_ALIVE)) {
Minecraft mc = Minecraft.getMinecraft();
for (Entity elist : mc.theWorld.loadedEntityList) {
if (elist instanceof EntityEnderman) {
if (elist instanceof EntityDragon) {
double coreX = -670;
Copy link

@ViralTaco ViralTaco Sep 3, 2019

Choose a reason for hiding this comment

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

Shouldn't this be final?
Edit: I mean the doubles… Also it probably should be -670.0 but that's a minor detail.

Copy link
Author

Choose a reason for hiding this comment

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

I was testing with Endermans (Zealots) and forgot to change it back to Ender Dragon, the doubles should be fine though.

@ShiroSR
Copy link
Author

ShiroSR commented Sep 4, 2019

Hi thanks for this, but maybe this is too specific? Can I change it maybe to disallow clicking players when you are holding out a bow?

Way better idea than mine, didn't think about it.

@biscuut biscuut closed this Sep 14, 2019
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.

None yet

3 participants