Better Intergration about chickensmod HOT 14 CLOSED

Hi. Thank you for you suggestions, I really appreciate that. Let's continue with the first think in this thread and that's the chicken registration. I have created #41 for the mod version issue.

from chickensmod.

BTW, what do you think about MC 1.9.4, do you think that it make any sense to keep it updated? I would like to know your opinion.

from chickensmod.

To be honest, Most ppl just updated from 1.9.4 to 1.10.2. I don't see a reason to keep the 1.9.4 updated. And more than likely the 1.10.2 version will still work on 1.9.4 unless you happen to call a new method. I will be busy today, but we can discuss more on the chicken registry later.

from chickensmod.

The chickens registry, it was designed in a way that it won't allow to have 'children' without both 'parents'. So as you said, parents need to be in the registry first. I haven't realised that it may be to strict and that it could cause hard times for addon developers. I think I can change it to be flexible and e.g. do a validation in the post load event and possibly removing invalid combinations (chicken which any of its parents are not in the registry). What do you think?

from chickensmod.

I'm guessing you will store the parents String ID during configuration load than in post actually find the parents in the chicken registry. or something along those lines. Either way that will make sure every chicken is loaded and it should find the parent. if it exist.

Side note I am realizing as I go back though your code.. I could have done some things differently.. mainly I did some things I didn't need too. So I am going to rewrite parts of my code before posting it up.

I will try to keep up to date so More chickens wont be broken for the new version.

from chickensmod.

@setycz I may have over thought this, I'm thinking I can handle the parents on my side. I started to look at the code again, and realized I can just handle mine on my side believe. I think I can simply call a new method to go though and set all the parent's after the registering. I realized the fisrt time I wrote this I sort of rushed its. Missed a few things I could have done better. I will get back with ya.

from chickensmod.

Ok sure. Let me know what you'll find out and based on that I can increase the priority if this issues slows your development down.

from chickensmod.

https://github.com/GenDeathrow/MoreChickens I think were good on the issue with parents now. I had to change some things over, but it works now. Im still cleaning up the code, just because I did a lot of dumb things the first time. and I will be condensing the code down more. Some things are redundant

from chickensmod.

Hey I noticed you deleted a method getAllItems() im getting a crash report with the newest update cause I had been calling that method. Not a big deal but could you @depreciate any method you wont be using and leave them in for an update or two.Just So I have time to fix it?

from chickensmod.

GenDeathrow/MoreChickens#2

from chickensmod.

I do like how you separated the enabled/disabled chickens though.

from chickensmod.

@GenDeathrow I have created a hotfix for the missing getAllItems(), it's great that your mod is now on GitHub so I can check what methods do you use.

from chickensmod.

I appreciate.. I did just release a fix for the new stuff anyways. probably not available yet. Hatchery also got an update to display your dna stats in the eggs. and drops inside the nesting pens with jei.

And I'm working around your mod for the most part, if you remove something that's ok in my book. I have your mod inside my dev environment. So if you make changes hopefully I will see them and can change as needed.

from chickensmod.

Integration problem is handled in More Chickens mod, further improvements can be done later in a new issue.

from chickensmod.