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

Не работает отображение названий на старых книгах #1925

Open
VanicGame opened this issue Jan 9, 2025 · 9 comments
Labels
bug Для задач: ошибка, которую необходимо исправить confirmed

Comments

@VanicGame
Copy link
Contributor

VanicGame commented Jan 9, 2025

Описание ошибки:

Релизная запись: И книжникам ныне радость: название книги при наведении на слот с книгой в инвентаре будет обновляться при каждом открытии книги (работает и со старыми книгами).

Реальные факты:
изображение

Как воспроизвести:

Шаги воспроизведения на [боевом]:

  1. Смотрим название старых книг, созданных до релиза 2024.12

Ожидаемое поведение:

Отображение названия книги

Дополнительная информация:

@VanicGame VanicGame added bug Для задач: ошибка, которую необходимо исправить unconfirmed labels Jan 9, 2025
@blanka-kato
Copy link

blanka-kato commented Jan 14, 2025

Игрок Swed уточнил баг

  • Изначально старые книги игрока не отображают свой заголовок.
  • Если в старую книгу внести изменения, то название будет отображаться корректно, как задумано.
  • Чужие старые книги не показывают свой заголовок.

Т.е. проблема сейчас в тех книгах, которые игроки не могут изменить самостоятельно или не являются их владельцами.
Покупные книги, к примеру.

Я сам не могу проверить этот баг.
Доверюсь словам Swed

@alek13
Copy link
Member

alek13 commented Jan 14, 2025

правка задумывалась следующая:

  • при использовании книги обновлять tooltip (вне зависимости кто автор)

были првки: 4bf0c76
потом фикс: 843e571

@blanka-kato
Copy link

  • вне зависимости кто автор

Я понимаю, что лезу не в своё дело. И не знаком ни с Lua, ни с разботкой модов для Luanti.
Прошу камнями не кидать.
Но глянул эти ссылки с правкой и фиксами и зацепился вот за этот момент
Возможно, тут проверка на то, что игрок является владельцем книги и обновление названия книги к этой проверке привязано.

@alek13
Copy link
Member

alek13 commented Jan 15, 2025

@lord-papirus , глянешь что там происходит ?

@lord-papirus
Copy link
Contributor

Да, посмотрю.

@lord-papirus
Copy link
Contributor

@alek13 Не могу воспроизвести на своем локальном, у меня нет старых книг.
Насколько старые книги должны быть? Если откатить реп до того старого сотояния, запустить, сделать книгу, закрыть, накатить новый реп и протестить. получится так?

@Petus228666
Copy link

1 такая есть у меня на боевом от Ваника. Кому надо,советую забрать как можно скорее

@lord-papirus
Copy link
Contributor

На боевом у меня тоже есть, только в мой локальный мир ее не перенести.
Правку и тестирование кода хочется сначала делать на локальном, так быстрее и безопаснее.

Попробовал откат/книга/накат (до 15.11.2024), заголовок на месте. Попробую еще на год откатить.

@lord-papirus
Copy link
Contributor

Да, откат до 2023.08 позволил сделать такую книгу.
Норм, теперь воспроизводится.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Для задач: ошибка, которую необходимо исправить confirmed
Projects
None yet
Development

No branches or pull requests

5 participants