Multiverse-NetherPortals

Multiverse-NetherPortals

689k Downloads

nether and end portals related advancement not triggered

benwoo1110 opened this issue · 17 comments

commented

Information

Details

I was able to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.

Description
"We Need to Go Deeper" advancement not triggered when enter Nether Portal as well as "The End?" for end portals, when portals are made on the non-default world. Probably the way mv-np handles nether portals not compatible with vanilla advancement checking. My draft solution is to manually add to player on EntityEnterPortalEvent.

commented

@benwoo1110 should this still be open? Seems like the PR got merged

commented

edit: Should be easy though. I'll open a PR tomorrow, or if you wanna do it go ahead.

Haha didn't see your edit, so when ahead and opened a PR for it already. #218

commented

I think this should be handled by the server, not by us.

commented

Thinking of it yea, maybe we should contact paper/spigot on it.

commented

Yeah, I'm about to take a look and see how easy it is, then gonna report to Spigot.

commented

Looks like it's handled by the client :/

I guess it's up to us.

edit: Should be easy though. I'll open a PR tomorrow, or if you wanna do it go ahead.

commented

No worries, I'll take a look.

commented

It turns out the server CAN trigger these. I'm gonna open a ticket with Spigot.

edit: Yup, I just got it working.

commented

Okay, the PR went through, so this is no longer necessary for us to do in 1.16.5+.

I think it's okay to keep the stuff you added in #218, but we should make it only for versions before 1.16.5.

commented

My memory is really bad... did I end up finishing this?

commented

I did not. Opening a PR now.

commented

Didn’t I fix it here? #218

commented

Yeah, but it's cleaner for the server to handle this, so I submitted a PR to Spigot. If it gets merged, then in MV5 we can remove the check in MVNP.

commented

Ah, it may be worth checking whether other nether related advancement are not triggered as well? There is one called “Subspace Bubble”, not sure if this also will be not detected, since entering nether portal isn’t detected.

“”Use the Nether to travel between 2 points in the Overworld with a minimum horizontal distance of 7000 blocks between each other, 875 blocks in the Nether.“”

commented

Let me see if that's fixed lol.

commented

Without either of our changes, unsurprisingly, it doesn't work.

With only your change, it doesn't work.

With only the server change I made, it works :D

commented

👍