Merge pull request #2522 from Loskh/fix_item_name

fix: EventItem name for Japanese client.
This commit is contained in:
goat 2025-12-18 15:11:13 +01:00 committed by GitHub
commit 05beea003c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -3,6 +3,7 @@ using System.Runtime.CompilerServices;
using Dalamud.Data;
using Dalamud.Game;
using Dalamud.Game.Text;
using Lumina.Excel.Sheets;
using Lumina.Text;
using Lumina.Text.ReadOnly;
@ -125,10 +126,15 @@ public static class ItemUtil
if (IsEventItem(itemId))
{
// Only English, German, and French have a Name field.
// For other languages, the Name is an empty string, and the Singular field should be used instead.
language ??= dataManager.Language;
var useSingular = language is not (ClientLanguage.English or ClientLanguage.German or ClientLanguage.French);
return dataManager
.GetExcelSheet<EventItem>(language)
.TryGetRow(itemId, out var eventItem)
? eventItem.Name
? (useSingular ? eventItem.Singular : eventItem.Name)
: default;
}