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

Refactor ppdata #1827

Merged
merged 26 commits into from
Aug 28, 2024
Merged

Refactor ppdata #1827

merged 26 commits into from
Aug 28, 2024

Conversation

thegrb93
Copy link
Owner

No description provided.

thegrb93 and others added 25 commits August 18, 2024 05:22
* Make starfall error hook use 'world' for server errors

* Use world instead of false for server errors
* Add Entity:getRenderBounds()

* Edit Entity:getRenderBounds()
* add E2 to :hasInstance()

* return false if neither sf or e2
* 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

* Missed one
@thegrb93
Copy link
Owner Author

Seems to be working now. Just need to test all the require and include functions.

@thegrb93 thegrb93 marked this pull request as ready for review August 28, 2024 09:03
@thegrb93 thegrb93 merged commit 865be4d into master Aug 28, 2024
1 check failed
@thegrb93 thegrb93 deleted the refactor-ppdata branch August 28, 2024 09:04
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.

4 participants