Carrots and potatoes #39

New issue

Closed

opened 2018-09-12 15:22:34 +00:00 by LadyVulcan · 0 comments

LadyVulcan commented

2018-09-12 15:22:34 +00:00

(Migrated from github.com)

Describe the bug
Carrots and potatoes do not give rewards for planting.

To Reproduce
Steps to reproduce the behavior:

(Optional) Harvest mature carrot or potato. Receive benefits as expected.
Wait for Farmer GUI to dissipate.
Plant carrot or potato.
Verify that no rewards are received, (and Farmer GUI does not chastise to wait longer).

Expected behavior
Planting carrots and potatoes should either give rewards, or display a message concerning cool-down.

Fix
On line 1349 and 1353 in the jobConfig.yml file, the ID for carrots and potatoes needs to be 141:0 and 142:0, which represent the block of crops being placed. The IDs currently in the config are the hand-held items, which never trigger the "Place" action, and thus never give rewards.
Note: Perhaps we can use "CARROT-0" and "POTATO-0", parallel to lines 1288 and 1292 where these crops are broken. I can't find in the config where these are accurate names, but they seem to work for breaking, so they'd probably work for placement as well.

**Describe the bug** Carrots and potatoes do not give rewards for planting. **To Reproduce** Steps to reproduce the behavior: 1. (Optional) Harvest mature carrot or potato. Receive benefits as expected. 2. Wait for Farmer GUI to dissipate. 3. Plant carrot or potato. 4. Verify that no rewards are received, (and Farmer GUI does not chastise to wait longer). **Expected behavior** Planting carrots and potatoes should either give rewards, or display a message concerning cool-down. **Fix** On line 1349 and 1353 in the jobConfig.yml file, the ID for carrots and potatoes needs to be 141:0 and 142:0, which represent the block of crops being placed. The IDs currently in the config are the hand-held items, which never trigger the "Place" action, and thus never give rewards. Note: Perhaps we can use "CARROT-0" and "POTATO-0", parallel to lines 1288 and 1292 where these crops are broken. I can't find in the config where these are accurate names, but they seem to work for breaking, so they'd probably work for placement as well.