search

Sunekaer / stoneBlock

https://minecraft.curseforge.com/projects/stoneblock
32 stars 40 forks source link

Placeholder Rewards #159

Closed Vaporous closed 5 years ago

Vaporous commented 5 years ago

Bug Report

One of the loot chest rewards seems to have a typo or something in the list of items that it is supposed to give.

Reward gives 9 each of: Mirror, Photovoltaic Cell I, Photovoltaic Cell II, Photovoltaic Cell III, and Item Placeholder. The first 4 items are from Solar Flux Reborn, the last from Better Questing; according to the item placeholder, the original ID it was supposed to give was " solarflux:photovoltaiccell/0 ". I'm going to guess, as there is a Photovoltaic Cell IV in the mod pack, that that was the last item it was supposed to be given.

Client Information

jellonek commented 5 years ago

The correct id for the item should be solarflux:mirror instead of solarflux:photovoltiac_cell_/0.

Sunekaer commented 5 years ago

no it should be solarflux:photovoltaic_cell_4

Sunekaer commented 5 years ago

1.0.17

faminecode commented 5 years ago

Just got this reward on 1.0.17, same issue as OP, placeholder instead of Cell IV.

jellonek commented 5 years ago

Was it new world on 1.0.17 or it was after upgrade to 1.0.17? If later - have you executed /bqs_loot default load after pack upgrade?

faminecode commented 5 years ago

Existing world, after upgrade to 1.0.17. I had run "/bq_admin default load" as instructed by the release notes on first load of the world, but not a loot command.

On Wed, Oct 3, 2018 at 8:20 AM Piotr Skamruk notifications@github.com wrote:

Was it new world on 1.0.17 or it was after upgrade to 1.0.17? If later - have you executed /bqs_loot default load after pack upgrade?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Sunekaer/stoneBlock/issues/159#issuecomment-426617033, or mute the thread https://github.com/notifications/unsubscribe-auth/AKqlEDy1Q35dus78Oa10uJBprvJHsH1wks5uhKuqgaJpZM4W1-o_ .

jellonek commented 5 years ago

@Sunekaer so looks like there is a need to add also an info (or autocmd as a renewable reward) for both bq_admin and bqs_loot default load after pack upgrade.

I can confirm that on plain new world on 1.0.17 this solar panels reward works perfectly ok (as iv'e received it just seconds ago).