View Issue Details

IDProjectCategoryView StatusLast Update
0001796Industrial-Craft²Agriculture / crops / Boozeceptionpublic2016-03-02 20:31
ReporterTimeslice Assigned Toestebes  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Summary0001796: Crop Harvester harvesting potatoes too soon
DescriptionCurrently (in 806), it is not possible to have poisonous potatoes automatically harvested. Whether or not the cropnalyzer is installed in the harvester it will always harvest potatoes (assuming they weren't already poisonous when the harvester was placed).

Expected behavior: The harvester should harvest poisonous potatoes WITHOUT a cropnalyzer, and should harvest normal potatoes WITH a cropnalyzer.

The reverse would also be acceptable.
TagsNo tags attached.
Minecraft Version

Activities

estebes

2016-01-09 14:58

developer   ~0004353

i dont think this is a bug or that harvesting poisonous potatoes is the expected behavior. The crop harvester has 2 modes - without the cropnalyzer it will harvest as soon as the crop canBeHarvested returns true - with the cropnalyzer it will harvest the crop once crop size == optimal harvest size - so it will never let potatoes get to the poisonous stage because both of those conditions happen before that.

Timeslice

2016-01-09 16:05

reporter   ~0004354

Last edited: 2016-01-09 16:05

Oh, it's a bug, but it's a design bug, not a code bug. Doing the reverse is probably the most intuitive. Regular harvester will harvest potatoes as expected, adding a cropnalyzer will get the poisonous potatoes. Usually the user flow is "Is the harvester giving me what I want? No? Then add the cropnalyzer." In this case that doesn't help any.

The bottom line is the harvester should support all IC2 crops, including poisonous potatoes and venomillia.

estebes

2016-03-02 20:31

developer   ~0004393

Crops will have changes in the future and this issue will be resolved.

Issue History

Date Modified Username Field Change
2016-01-09 11:50 Timeslice New Issue
2016-01-09 14:58 estebes Note Added: 0004353
2016-01-09 16:05 Timeslice Note Added: 0004354
2016-01-09 16:05 Timeslice Note Edited: 0004354
2016-03-02 20:31 estebes Note Added: 0004393
2016-03-02 20:31 estebes Status new => closed
2016-03-02 20:31 estebes Assigned To => estebes
2016-03-02 20:31 estebes Resolution open => fixed