Jump to content

Recommended Posts

Posted

Hi,

it would be very cool, if Forge could at a new event: OnWorldGenerator added and if they could change the worldgenerators from private to public because there is no way to get information out of the GameRegistry class.

It would be very helpful for my mod World Generation Manager.

Posted

Ok, the World Generation Manager allows you to regenerate a specific mod but therefore i need the worldGenerators of the mods. So i edited the base class (GameRegistry). So it would be very cool if there is an event for this and that the worldGenerators are public. What i mean is that it's currently private static. So you have no access to it. If you could change it to public static, other ways that modders would have to make their on list (with the event when a worldgenerator is added).

I hope you understand it :-)

Posted

No, my mod is working perfectly, but someone ask me if i could ask if this could be a standard feature of forge:

http://www.minecraftforum.net/topic/1879772-

 

You could actually add this in as a PR (Pull-Request, allows you to change Forge classes and submit the changes to go into the official source) for Forge, make it a bit easier for mods to do this. Would surely change Forge a lot, I'm looking for adding Retrogen ores (ores that generate after the world loads, aka ores that can generate in non-generated chunks) in my mod, so this would help :).

Posted

You could actually add this in as a PR

 

Yeah... that's kinda how you do it when you are suggesting additions to an open source project...

Create the hook/event you need, submit the PR and if it's a reasonable and well implemented then the forge team will consider it and may end up adding it.

 

If you guys dont get it.. then well ya.. try harder...

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.