Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: FTB Quests Reward Tables items do not save name after exiting Reward Tables interface. #1442

Open
Miqote-Petter opened this issue Jan 1, 2025 · 3 comments
Assignees
Labels
1: Bug Problem with the usage of a mod. FTB Quests

Comments

@Miqote-Petter
Copy link

Miqote-Petter commented Jan 1, 2025

Mod

FTB Quests

Mod version

FTB Quests (Neoforge): 2101.1.1

NeoForge / Forge / Fabric version

Neoforge 1.21.1 21.1.90

What issue are you having?

When making a reward table with game stages as a reward, the "title" option resets upon leaving the reward table interface. This results in the stages reward title reverting to "stages: xxx."

Steps to reproduce

  1. Open Edit Rewards Tables
  2. Add a reward table, name it as you see fit.
  3. Add a game stage reward. Fill it out, including title. At this point, it will properly show the title of the Stage reward.
  4. Exit the Reward Tables interface.
  5. Either: open Rewards Tables and edit again or add as a reward to quest.
  6. Reward table items have reverted back to "stages: xxx" instead of a proper title.
@Miqote-Petter Miqote-Petter added the 1: Bug Problem with the usage of a mod. label Jan 1, 2025
@Miqote-Petter Miqote-Petter changed the title [Bug]: [Bug]: FTB Quests Reward Tables items do not save name after exiting Reward Tables interface. Jan 1, 2025
@desht
Copy link
Contributor

desht commented Jan 2, 2025

Confirmed.

@desht desht self-assigned this Jan 2, 2025
@desht
Copy link
Contributor

desht commented Jan 2, 2025

This is unfortunately going to be a bit more difficult to resolve than I hoped. Technical detail: the rewards embedded in a reward table are the same Java class as the rewards in a quest (class Reward). However, rewards in a reward table don't have a unique quest object ID (because they haven't historically needed them), whereas rewards in a quest do have their own ID.

Now in 1.21, FTB Quests has a new translation system whereby all translatable strings are moved out of the quest files and into a lang/<lang-code>.snbt file. Entries in this file are keyed by the... quest object ID. And since reward table rewards have no ID (or more precisely a fake ID of 0), their title isn't working.

The fix for this will be to ensure these rewards all get their own unique ID's, which is possible to do, but a bigger change than I hoped, and one which will need a fair bit of testing. So won't be in the next FTB Quests release, but the following one.

@Miqote-Petter
Copy link
Author

Miqote-Petter commented Jan 2, 2025 via email

desht added a commit to FTBTeam/FTB-Quests that referenced this issue Jan 9, 2025
Necessary for translations to work for them in the new 1.21
translations system.

Also a few other small fixes & improvements to the reward table
editor in general.

FTBTeam/FTB-Mods-Issues#1442
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1: Bug Problem with the usage of a mod. FTB Quests
Projects
None yet
Development

No branches or pull requests

2 participants