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

add E2 to :hasInstance() #1837

Merged
merged 2 commits into from
Aug 25, 2024
Merged

Conversation

ax255
Copy link
Contributor

@ax255 ax255 commented Aug 23, 2024

No description provided.

Copy link
Owner

@thegrb93 thegrb93 left a comment

Choose a reason for hiding this comment

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

Need to return false if entity is neither starfall or e2

@thegrb93 thegrb93 merged commit d1bb96d into thegrb93:master Aug 25, 2024
1 check passed
github-actions bot added a commit that referenced this pull request Aug 25, 2024
* add E2 to :hasInstance()

* return false if neither sf or e2 d1bb96d
thegrb93 pushed a commit that referenced this pull request Aug 28, 2024
* add E2 to :hasInstance()

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