Skip to content

fix on_content_loaded + make on_assets_loading#854

Open
Xertis wants to merge 2 commits into
MihailRis:mainfrom
Xertis:main
Open

fix on_content_loaded + make on_assets_loading#854
Xertis wants to merge 2 commits into
MihailRis:mainfrom
Xertis:main

Conversation

@Xertis

@Xertis Xertis commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Раньше on_content_loaded вызывался ПЕРЕД загрузкой ассетов, хотя по документации он должен вызываться после полной загрузки пака. Поэтому я вызов этого ивента перенёс вперёд, чтобы он соответствовал документации

Плюсом добавил новый ивент

function on_assets_loading()

Он вызывается перед загрузкой ассетов, т.е. выполняет ту же роль, что и on_content_loaded до фикса

@Xertis

Xertis commented Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

И че action сломался 🥴

@MihailRis

Copy link
Copy Markdown
Owner

загрузка паков никак не связана с загрузкой ассетов
зачем намеренно ломать совместимость, особенно учитывая, что событие используется и в самом ядре?

@Xertis

Xertis commented Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

загрузка паков никак не связана с загрузкой ассетов зачем намеренно ломать совместимость, особенно учитывая, что событие используется и в самом ядре?

on_content_loaded как и сам скрипт content.lua появились только в 0.32, который всё ещё в разработке, поэтому совместимость не ломается

в плане используется в самом ядре? Я в res/scripts даже файла content.lua не видел

@MihailRis

Copy link
Copy Markdown
Owner

загрузка паков никак не связана с загрузкой ассетов зачем намеренно ломать совместимость, особенно учитывая, что событие используется и в самом ядре?

on_content_loaded как и сам скрипт content.lua появились только в 0.32, который всё ещё в разработке, поэтому совместимость не ломается

в плане используется в самом ядре? Я в res/scripts даже файла content.lua не видел

c post_content спутал
в любом случае - тайминг странный, так как загрузка контента завершается до начала загрузки ассетов, которой может и не происходить, как в том же headless

@MihailRis

Copy link
Copy Markdown
Owner

зачем серверной части, даже без headless, дожидаться загрузки ассетов для логики контента?

@Xertis

Xertis commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

зачем серверной части, даже без headless, дожидаться загрузки ассетов для логики контента?

Ассеты вроде в headless не загружаются, т.е. по идее оно просто скипается и всё

Желание перенести on_content_loaded появилось после того, как я заметил, что загруженные через assets модели в прошлой версии on_content_loaded просто не были доступны, а т.к. описание в документации наталкивает на то, что с паком можно взаимодействовать полноценно после ивента (включая по идее ассеты), то я решил, что будет правильным поставить его вызов после post_content

Но если загрузка контента отвязана от ассетов, то кншн лучше вернуть его назад + отредачить документацию, вопрос в том, уместны ли теперь вообще какие-либо ивенты в content.lua, которые связаны с ассетами

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

Successfully merging this pull request may close these issues.

2 participants