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

Change Achievements to Advancements / Fix 1.15 support #68

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

Conversation

jacob1
Copy link

@jacob1 jacob1 commented May 24, 2017

I know this plugin is dead but ... I still use it so I updated it to relay 1.12 advancements

merge if you want, otherwise maybe someone will find it useful.

Edit: Updated to remove achievement support entirely. 1.15 Bukkit removed that api finally.
If anyone needs a new version for 1.15, I compiled it here: https://starcatcher.us/Moo/CraftIRC.jar
This is perhaps the most important plugin on my server, since I'm always on IRC and want to talk to the members, and know when I should get online.

@dequis
Copy link
Member

dequis commented May 24, 2017

Wow, was that really the whole API break? Some pointless renames and .getKey().getKey() instead of .name()? Impressive.

I'd suggest not renaming every instance of achievement to advancement. Event type, formats, attributes, hold type, all that stuff doesn't need renaming.

A way to keep backwards compat would be nice.

@jacob1
Copy link
Author

jacob1 commented May 24, 2017

The Advancements api seems to support more stuff (since advancements themselves support more things than achievements). But it is pretty much the same to get the name. The names are different now, before it would be "BREW_POTION", it is now "nether/brew_potion" (might be nice to prettify the names into the in game versions...). Achievements didn't seem to get reported through the old api anymore which is why I did this.

I guess I can check if backwards compatibility is possible. I'm not too experienced in this though.I renamed everything because I wasn't supporting old achievements anyway. The config file key probably shouldn't have been renamed.

@jacob1
Copy link
Author

jacob1 commented May 25, 2017

Backwards compatibility added, I tested 1.12-pre5 and 1.11.2, it relayed them in both cases

@jacob1
Copy link
Author

jacob1 commented May 25, 2017

Apparently it notifies using the advancements API on every recipe that is unlocked, or something. I will have to fix that, it's extremely spammy.

@jacob1
Copy link
Author

jacob1 commented May 26, 2017

massive spam removed. I think it's fine and ready to merge now.

@Chinimala
Copy link

Will you give the new .jar with these changes ?

@jacob1
Copy link
Author

jacob1 commented Jun 11, 2017

@Chinimala
Copy link

Thank you very much

@jacob1
Copy link
Author

jacob1 commented Jun 12, 2017

I removed the deprecation warning on server startup, and tested 1.12 / 1.11.2 again, it all works fine.

@jacob1
Copy link
Author

jacob1 commented Dec 14, 2019

Backwards compat turned out to be a bad idea. The old achievements api was removed in 1.15. I changed it back to only supporting advancements, this should handle anyone in 1.12+. Anyone for some reason still on an older version can just use an old version of the plugin from 3 years ago.

If anyone wants this plugin for 1.15 on their server, compile this PR, or use the jar file I compiled here: https://starcatcher.us/Moo/CraftIRC.jar

@jacob1 jacob1 changed the title Change Achievements to Advancements Change Achievements to Advancements / Fix 1.15 support Dec 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.

3 participants