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

Copy frozen prop param fix to other spawn functions #1839

Merged
merged 2 commits into from
Aug 25, 2024
Merged

Conversation

thegrb93
Copy link
Owner

No description provided.

Copy link
Contributor

@SuperTemich2005 SuperTemich2005 left a comment

Choose a reason for hiding this comment

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

oh wow, i started an entire trend :D

nothing looks like it would automatically crash all existing starfalls so yeah...

@thegrb93 thegrb93 merged commit c18f4a2 into master Aug 25, 2024
1 check passed
@thegrb93 thegrb93 deleted the prop-frozen branch August 25, 2024 17:09
github-actions bot added a commit that referenced this pull request Aug 25, 2024
* Copy frozen prop param fix to other spawn functions

* Missed one c18f4a2
thegrb93 added a commit that referenced this pull request Aug 28, 2024
* Copy frozen prop param fix to other spawn functions

* Missed one
thegrb93 added a commit that referenced this pull request Aug 28, 2024
* Start refactor

* progress

* move

* Fixes serverside errors (#1829)

* Progress

* More progress

* Big refactor

* Remove code param

* Progress

* Forgot this

* Fixes

* more progress

* Fix trace not working for entity subtypes (#1832)

* Additional isvalid checks (#1831)

* Make starfall error hook use 'world' for server errors (#1833)

* Make starfall error hook use 'world' for server errors

* Use world instead of false for server errors

* Add Entity:getRenderBounds() (#1834)

* Add Entity:getRenderBounds()

* Edit Entity:getRenderBounds()

* Fixes

* Add file and line number to preprocessor errors

* small fix

* fix

* add E2 to :hasInstance() (#1837)

* add E2 to :hasInstance()

* return false if neither sf or e2

* For prop.createSent(): check if frozen is a boolean (#1838)

* For prop.createSent(): check if frozen is a boolean

I forgot to specify `frozen` in one of my SF projects and my `data` ended up in its place, yet I received no error and it wasn't until I spent 1 second of my life to double-check docs when I fixed my problem. I highly doubt anyone is going to get any problems with me adding one extra check that 99% of good starfalls won't crash from.

* Update lua/starfall/libs_sv/prop.lua

Co-authored-by: thegrb93 <[email protected]>

* Update prop.lua

* Update prop.lua

* Copy frozen prop param fix to other spawn functions (#1839)

* Copy frozen prop param fix to other spawn functions

* Missed one

* Fix

* Final fixes

---------

Co-authored-by: ъ знака <[email protected]>
Co-authored-by: ax255 <[email protected]>
Co-authored-by: SuperTemich2005 <[email protected]>
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