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

Added wraping restrictions #2902

Merged
merged 10 commits into from Apr 23, 2020
Merged

Added wraping restrictions #2902

merged 10 commits into from Apr 23, 2020

Conversation

EPuncker
Copy link
Contributor

@EPuncker EPuncker commented Apr 6, 2020

Added:

  • Can't move wrap items outside house tile
  • Can't wrap/unwrap if you are not house owner
  • Can't move carpet items into door tiles, to prevent this:

2020-04-06 23_34_26-Tibia - Perflex
(the item become unreachable/unwrapable)

follow up to 3cccb45

and yes they are on real tibia and I tested the changes on my repo, but hope someone can test it on clean TFS as well

credits:
change on onMoveItem to oen432
change on onWrapItem to Zothion
sarah wesker to moving code to carpet script instead

Added:
- Can't move wrap items outside house tile
- Can't wrap/unwrap if you are not house owner

follow up to otland@3cccb45

and yes they are on real tibia and I tested the changes

credit: onMoveItem to oen432
onWrapItem to Zothion

https://otland.net/threads/tfs-1-3-help-with-script-logic.269582/#post-2600402
https://otland.net/threads/event-antitrash-antitheft-system-for-houses-tfs-1-x.237075/page-2#post-2416471
@DSpeichert DSpeichert added the needs-testing Needs testing with screenshots label Apr 18, 2020
@DSpeichert DSpeichert requested a review from a team April 18, 2020 05:47
@omarcopires
Copy link
Contributor

Tested, the code works perfectly.

omarcopires
omarcopires previously approved these changes Apr 19, 2020
@DSpeichert DSpeichert merged commit cc7a30a into otland:master Apr 23, 2020
@EPuncker EPuncker deleted the unwrap branch April 23, 2020 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-testing Needs testing with screenshots
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants