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

@PersonalSpace trigger malfunctioning #1250

Open
GladieUO opened this issue Jun 15, 2024 · 8 comments
Open

@PersonalSpace trigger malfunctioning #1250

GladieUO opened this issue Jun 15, 2024 · 8 comments
Milestone

Comments

@GladieUO
Copy link
Contributor

GladieUO commented Jun 15, 2024

https://wiki.spherecommunity.net/index.php?title=@PersonalSpace
Basically it will allow stepper step over you only once with full stam, than he get blocked, untill you set ARGN1 to 0 and create your own stamina drain.

ARGN2 - doesnt do anything so setting it to 1 or 0 does non

This is basically how I had to do it to allow players step over NPCs and drain some stamina

ON=@personalspace
IF (<SRC.ISPLAYER>)
IF (<SRC.STAM> > 10)
SRC.STAM -=10
ARGN1=0
ENDIF
ENDIF

@Jhobean
Copy link
Contributor

Jhobean commented Jun 15, 2024

What version of sphere you used?
This was touched on last commit cad5459#diff-e1ae3608ed3c89cae2f5c81d003b6a1eb1b38b9313e6ccae97fd96b106c27e6e

@GladieUO
Copy link
Contributor Author

What version of sphere you used? This was touched on last commit cad5459#diff-e1ae3608ed3c89cae2f5c81d003b6a1eb1b38b9313e6ccae97fd96b106c27e6e

Compiled at Jun 14 2024 (21:17:53) [branch master / build 3752 / GIT hash 8e44484]

@Tolokio
Copy link
Contributor

Tolokio commented Jun 25, 2024

image

Right now, the lane marked in red causes any attempt to fail if stam is lesser than maxstam.
This means any custom script will need to do what u did. Set argn=0 to always allow step and then use your own system to allow or deny.
If argn is not set 0, only chars at maxstam will step.

Can be changed, but I see no real drama to let it be as it is cos u can set ur own custom system. But I can agree it is confusing.

@Jhobean
Copy link
Contributor

Jhobean commented Jul 2, 2024

Could you try to set this to 1 and test again?
image

@GladieUO
Copy link
Contributor Author

GladieUO commented Jul 2, 2024

Could you try to set this to 1 and test again?

Im using this setting already

@Tolokio
Copy link
Contributor

Tolokio commented Jul 15, 2024

we should add a local.AllowifStamIsLowerThanMaxStam or another sphere.ini config for it. Whatever u decide I can try.

@GladieUO
Copy link
Contributor Author

we should add a local.AllowifStamIsLowerThanMaxStam or another sphere.ini config for it. Whatever u decide I can try.

I cant make that decision, thats up to devs.
I only know that we can make workaround, but the trigger is useless as it is now. Without argn1=0 you cant do anything, and still argn2 doesnt do anything either way. 😶

@Tolokio
Copy link
Contributor

Tolokio commented Jul 16, 2024

the trigger is not useless but I agree with you that there should be a way to bypass that behaviour. I think a local.bypassStam should do the job.
Whatever U (the community) decide I can try.

MAybe we could remove the check I marked in red, and include that check on the sphere-x scriptpack instead source.

@cbnolok cbnolok added this to the X1.1 milestone Sep 5, 2024
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

No branches or pull requests

4 participants