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

Villagers are sometimes named "Forename the" #19

Closed
Fourmisain opened this issue Mar 13, 2021 · 3 comments
Closed

Villagers are sometimes named "Forename the" #19

Fourmisain opened this issue Mar 13, 2021 · 3 comments

Comments

@Fourmisain
Copy link

professionNames, childNames and surNames were all set to true.

I think I found how to reproduce the issue.

Start with 2 villagers with professions like such:
2021-03-13
Breed them.
Their child won't have a surname (which is already an issue I believe?) and will also have the profession text of their parent - which I'm not sure if intended. Is it?
2021-03-13
When such a child grows up, it seems "the" is used as the surname
2021-03-13
Though this apparently doesn't happen all the time?
2021-03-13


Ahhh!
I wrote a whole paragraph here about how I'm trying to rewrite the naming functions, maybe the way names are stored etc.

I just compiled the code of my prototype wanted to do some testing to get a (draft) pull request up - but I noticed that the code here on github is outdated, it's still on version 1.5.2, please fix!

@Fourmisain
Copy link
Author

Fourmisain commented Jun 1, 2021

I just realized 1.6.0 is now pushed!

It's been so long since I wrote the rewrite that getting the PR done might take a while, if you want you can have a look at the (very unfinished) code.

Initially I wanted to add separate fields for fore- and surnames, but I quickly gave up on that idea (commented it out) since it's not hard using the simple "Forename <Surname> <the Jobtitle>" scheme, where you first split at the potential " the " and then at the potential space.

I remember thinking of having names with spaces but also thinking about config options which could cause issues, like the one that switches fore- and surnames:
If you generate villagers one way, switch the setting, restart and generate new villagers, it becomes impossible to distinguish between fore- and surnames, which would (especially) be an issue when they could have spaces.
That would be something separate fields could solve, but then you also need to think about custom names from nametags - another field? "realNameTagName"?
Seemed like too much effort and too error-prone due to the introduced redundancy, so I kept the basics and simplified it as much as I could in the brief time till I realized I worked with an out-of-date version.

@MeeniMc
Copy link

MeeniMc commented Jun 13, 2021

Issue is still present in 2.0 for mc 1.17

@OverlordsIII
Copy link
Owner

Should be patched in latest version

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

No branches or pull requests

3 participants