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.

Sign up for GitHub

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

Jump to bottom

Removes remnants of my enthusiasm #11192

Merged
merged 1 commit into from
Dec 9, 2023
Merged

Removes remnants of my enthusiasm #11192

merged 1 commit into from
Dec 9, 2023

Conversation

TobyThorne
Copy link
Member

@TobyThorne TobyThorne commented Dec 9, 2023 •
edited
Loading

  • собираем mob/Login() и mob/Destroy() со всего билда в одну дымящуюся кучку, в противном случае всё это дело косоёбится в непредсказуемом порядке, модульность может попить говна
  • больше никогда не дёргаем ..() в mob/Login(), см. дискус на тг
  • modules/organs -> modules/mob/organs ибо я заебался их искать каждый раз
  • фиксим ещё какие-то рандомные дестрои

в результате...
нихуя не добились мобы всё ещё не удаляются всё это говно лагает из-за хардделов ебись оно всё в сраку мне уже похуй

  • Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
  • Я внимательно прочитал все свои изменения и багов в них не нашел.
  • Я запускал сервер со своими изменениями локально и все протестировал.
  • Я ознакомился c Guide to Contribute.

@TobyThorne TobyThorne added 🔨 рефакторинг Организация и улучшение текущего кода 🔧 фикс Фикс пофикшенного фикса 📜 CL не требуется Тссс labels Dec 9, 2023
@TobyThorne TobyThorne requested a review from a team as a code owner December 9, 2023 07:51
@TobyThorne TobyThorne merged commit c98a882 into ChaoticOnyx:dev Dec 9, 2023
11 checks passed
rufuszero pushed a commit to ZeroHubProjects/ZeroOnyx that referenced this pull request Sep 8, 2024
@TobyThorne @rufuszero
fix: unify and fix mob login/destroy behavior
040927c 
This is an experimental cherry-pick from a year into the future.
Original Author: TobyThorne (https://github.com/TobyThorne)
Original PR: ChaoticOnyx/OnyxBay#11192
Original Commit: ChaoticOnyx/OnyxBay@c98a882

Unify mob/Login() and mob/Destroy() functions
Move all organs under code/modules/mob/organs path and split
to internal/external
Fix some additional code checks and Destroys() that were causing
runtimes
Apply an optimisation by AffectedArc07 from
ParadiseSS13/Paradise#20270 to mob/Login();
While it might be not relevant now because we're not doing the same
mob/Move() restrictions, it's best to avoid this hard to debug issue if
we ever do apply those restrictions in the future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Doster-d Doster-d Doster-d approved these changes

@igorsaux igorsaux igorsaux approved these changes

Assignees
No one assigned
Labels
📜 CL не требуется Тссс 🔧 фикс Фикс пофикшенного фикса 🔨 рефакторинг Организация и улучшение текущего кода
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TobyThorne @igorsaux @Doster-d