search

MatrexsVigil / harvestcraft

All rights reserved - None of my artwork is available to be used for FREE in your game EVER. If you want to HIRE ME, please contact me. Thank you. This code is here for people to learn from and to help find bugs ONLY.
https://www.patreon.com/pamsmods
89 stars 86 forks source link

[Question 1.10.2] Mustard Seeds and Sesame Seeds #8

Closed TokkenGmr closed 6 years ago

TokkenGmr commented 7 years ago

I was updating some Agricraft JSON's when someone posted thisSo I was wondering if you intended for the output of the Mustard Seed and the Sesame Seed to be replanted. As it is right no you have the seed for each that can be planted and the output for each which are seeds and they can eaten or planted. Just wanted to know if it is intended, that way I know the best way to move forward. Thanks in advance

MatrexsVigil commented 7 years ago

All crop items in HarvestCraft are meant to be used for planting (like vanilla carrots or potatoes).

winnetrie83 commented 7 years ago

So we don't even need seeds to plant stuff? Then i don't understand why you added seeds in the first place, if they have no real purpose! For us it looks like a bug that we can plant the seed and the food item. If this isn't a bug, then there is no purpose for all those seeds. I never tried before to plant the crop item because there were seeds for a reason i though. Turns out crops can be planted.....I don't wanne be rude (not at all), but what is the purpose for the seeds?

Edit: Maybe you can add a config option to enable/disable this. I would appreciate it!

richardhendricks commented 7 years ago

winnetrie83 - Dial back the exclamation marks, it makes you look like a whiner. Seriously.

The seeds were used originally, but Pam changed it recently so that there are no longer separate seeds, because honestly they're just a pain in the ass to keep around and deal with. It's just smarter overall to not use seeds because it reduces the mod size and the amount of stuff that has to be created when a new crop is made.

winnetrie83 commented 7 years ago

@richardhendricks

winnetrie83 - Dial back the exclamation marks, it makes you look like a whiner. Seriously.

Wow what an attitude! Courtesy pls! I guess that's something strange to you. That was not the intention at all. It makes you look stupid to think so!

Btw, i disagree with you. It isn't smarter. I don't see the point why it is "smarter". For me it makes more sense to plant seeds and not the crops. Nevertheless a config option to enable disable this won't hurt after all. So people can decide for themself to use seeds to farm or just use the crops.

richardhendricks commented 7 years ago

intention is exactly what is difficult to convey with text. Excessive exclamation marks without any other emotional indicators project a feeling of exasperation and/or anger. If that is not the feeling you want to convey, I suggest adding emoticons to let people know. Especially since your comment is asking about a feature change, the tone is automatically confrontational without additional context.

It's smarter because it:

The only place it really would offer some gameplay is if Harvestcraft is used together with an agricultural breeding mod like Agricraft. Otherwise, seeds don't add a single gameplay option. It doesn't change or offer any sort of additional choices for the player.

winnetrie83 commented 7 years ago

The only place it really would offer some gameplay is if Harvestcraft is used together with an agricultural breeding mod like Agricraft. Otherwise, seeds don't add a single gameplay option. It doesn't change or offer any sort of additional choices for the player.

You're right about that, therefore i request a config option to enable/disable this. I use this mod with agricraft, but you can still plant the crops with vanilla farming disabled. Not sure who has to make it compatible (harvestcraft or agricraft), but a config option would be really helpfull.

Btw, i'm 34 years old i never intend to shout/scream/yell/cry or whatever. Why would i? I'm not good with emoticons either. I'll try from now on, just to be sure people do not understand me wrong :-p

MatrexsVigil commented 6 years ago

Not accepting non 1.12.2 bug reports at this time.