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

Calling a language specific sheet from a non-language sheet does not work #93

Closed
Critical-Impact opened this issue Oct 26, 2024 · 3 comments

Comments

@Critical-Impact
Copy link
Contributor

Just adding these issues so people are aware and we can track them as they'll need to be fixed before the next major dalamud release.

Easiest way to replicate:

GameData.GetExcelSheet<GatheringPointBase>()!.GetRow(30);
var name = gpb.GatheringType.ValueNullable?.Name ?? "test";

It appears to be using the GatheringPointBase's page's language which is None but GatheringType requires a language to be passed in.

At a guess we'd need to grab the module's default language but I think we should also be able to request a row of a specific language

@Haselnussbomber
Copy link
Contributor

I ran into this today and reported it to Asriel in #dalamud-dev. Apparently she already has it fixed locally and will PR it soon. 🙂

@Haselnussbomber
Copy link
Contributor

This is fixed in 5.2.0 (#95).

@Critical-Impact
Copy link
Contributor Author

Critical-Impact commented Nov 3, 2024

Awesome, closing :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants