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 Force TP to realleash #1069

Merged
merged 1 commit into from
Aug 3, 2024

Conversation

Medea-Destiny
Copy link
Collaborator

This fix takes advantage of the EXC_REFRESH function to add ForceTP to strict leash, so that when strict leash is active, a leasher can send a TP lure to leashee and force tp them regardless of whether they normally have Force TP ability. When the strict leash restrictions end (after leash is dropped, or 60 seconds after leash holder goes out of range), the ForceTP along with other strict leash restrictions will be dropped, but existing ForceTP set through exceptions will not be impacted.

Copy link
Collaborator

@Pingout Pingout left a comment

Choose a reason for hiding this comment

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

Tested with Trinkitz. We checked the use of this change for many different configuration of Access, everything from Trusted, Captured, non-Owner, Unowned, and no access. Each scenario worked perfectly.

Copy link
Contributor

@SilkieSabra SilkieSabra left a comment

Choose a reason for hiding this comment

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

Trinkitz and I tested this. It works as expected. It does add some power to public access since.. if someone leashes the wearer on public access, and then an owner teleports them, while the leash is still active the public access user can teleport them back. This might be fun but also might surprise the owner.

@SilkieSabra SilkieSabra merged commit 9b4277e into 8.3_Features-branch Aug 3, 2024
2 checks passed
@SilkieSabra SilkieSabra deleted the Medea-Destiny-realleash-forcetp branch August 3, 2024 23:41
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.

3 participants