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

[fixed] localhost building #2116

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Gingerbeard5773
Copy link
Contributor

Status

  • READY

Description

  • Fixes a bug related to building two structures at the same time on localhost.
  • Improves mod support by removing hack code.

Steps to Test or Reproduce

On localhost, build a building.
If using the old build: you will notice that the building is made twice and one of the buildings will be destroyed.
If using the fixed build: no duplicate building is produced.

Netcode was altered so make sure to test on dedicated server.

It should be noted that because important builder files were modified, this PR may break some mods- though it should be easy to update those mods to fix any issue.

* Fixes an issue where players would build two buildings at once on localhost.
* Fixes asu's construction sound hack.
* Stronger, leaner, more simple netcode 💪
this attached check causes problems with mods, so i am moving it to quarters.as where it should have been originally anyways.
@Vam-Jam Vam-Jam self-assigned this Sep 4, 2024
@Vam-Jam Vam-Jam added the fix Fixes a bug label Sep 4, 2024
@Vam-Jam Vam-Jam added this to the Next Patch milestone Sep 4, 2024
Vam-Jam
Vam-Jam previously approved these changes Sep 4, 2024
Copy link
Member

@Vam-Jam Vam-Jam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worked locally, and does not seem to change anything in dedi + client CTF.

I have not tested the audio changed however since linux is being a pain.

Entities/Characters/Builder/BuilderCommon.as Show resolved Hide resolved
@Vam-Jam Vam-Jam added the ready A pull request that functions correctly and is ready for testing label Sep 4, 2024
@Vam-Jam Vam-Jam linked an issue Sep 4, 2024 that may be closed by this pull request
crazy how this didnt cause issues.
Copy link
Member

@Vam-Jam Vam-Jam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still works

TerminalHash added a commit to TerminalHash/Gruhsha-CTF that referenced this pull request Sep 5, 2024
Copy link
Contributor

@TerminalHash TerminalHash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works like a clock, tested on Gruhsha CTF (dedi and localhost).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes a bug ready A pull request that functions correctly and is ready for testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug when building a building in offline (Build 4762)
3 participants