Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

Random Portals mod not compatible #2667

Closed
Renari opened this issue Apr 4, 2019 · 9 comments
Closed

Random Portals mod not compatible #2667

Renari opened this issue Apr 4, 2019 · 9 comments

Comments

@Renari
Copy link

Renari commented Apr 4, 2019

I am currently running

  • SpongeForge version: 1.12.2-7.1.5
  • Forge version: 14.23.5.2824
  • Java version: 1.8.0_172
  • Operating System: W10
  • Plugins/Mods: Random Portals

Issue Description
When Random Portals mod is loaded taking a portal (such as the nether portal) by that mod often warps you inside a wall and return portals are not generated. This does not happen without SpongeForge.

@limbo-app limbo-app added the status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage label Apr 4, 2019
@Renari
Copy link
Author

Renari commented Apr 4, 2019

@limbo-app limbo-app removed status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage labels Apr 13, 2019
@TheRandomLabs
Copy link

This probably has something to do with my custom teleporter. Any ideas?

@gabizou
Copy link
Member

gabizou commented Apr 16, 2019

@Zidane could you look into this potentially? I'm still working on verifying some block drop handling between capture types.

@Zidane
Copy link
Member

Zidane commented Apr 16, 2019

@TheRandomLabs Can you link me where you take over the teleportation code to the Nether or if you change the Nether's Teleporter?

@TheRandomLabs
Copy link

Here is the ASM transformer and here is the Teleporter wrapper class.

@phit phit changed the title Random Portals mod not compatible with SpongeForge. Random Portals mod not compatible May 16, 2019
@Zidane
Copy link
Member

Zidane commented Jun 3, 2019

@TheRandomLabs I see you made a code change but I need to figure out where we are inadvertently replacing the World's Teleporter object by seeing where you set this on the WorldServer to begin with. The code change you made should not have been needed.

Can you link me to some code (as those links are dead) and I'll get this fixed up right now.

@TheRandomLabs
Copy link

TheRandomLabs commented Jun 4, 2019

I was using RandomPatches to replace the Teleporter in WorldServer's constructor, but now I'm using reflection to set it to RPOTeleporter during WorldEvent.Load.
Sponge is setting the Teleporter here.
Replacing the Teleporter with my own fixed the issues that were occurring, but I'm not sure if there are any side effects.

@Zidane
Copy link
Member

Zidane commented Jun 4, 2019

@TheRandomLabs In that case, I consider this issue closed. Granted someone should make a PR to Forge to let you set the default teleporter of a World.

We don't actually change the teleporter of the world but more set it to a Vanilla one (or mod one) if a server admin asks us too. Otherwise, it uses whatever Vanilla would give it in this case (Which is just Teleporter of course).

I have no problems with the change you made.

@Zidane Zidane closed this as completed Jun 4, 2019
@TheRandomLabs
Copy link

Alright, thanks.

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

No branches or pull requests

6 participants