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

floorchange unpass objects #3680

Closed
2 tasks done
ghost opened this issue Sep 22, 2021 · 15 comments · Fixed by #3681
Closed
2 tasks done

floorchange unpass objects #3680

ghost opened this issue Sep 22, 2021 · 15 comments · Fixed by #3681
Labels
enhancement Increase or improvement in quality, value, or extent

Comments

@ghost
Copy link

ghost commented Sep 22, 2021

Before creating an issue, please ensure:

  • This is a bug in the software that resides in this repository, and not a
    support matter (use https://otland.net/forums/support.16/ for support)
  • This issue is reproducible without changes to the C++ code in this repository

Steps to reproduce (include any configuration/script required to reproduce)

  1. place 3 parcels and try to go up a floor onto an "unpass" object like a statue or trough
  2. try to stack 2 unpassable objects aswell by pushing them into floorchange, you will get same result "not enough room"

Expected behaviour

tested on rl-tibia and it should be possible
bug

stack

Actual behaviour

"there is not enough room."

Environment

Reproduction environment

@ghost
Copy link

ghost commented Sep 22, 2021

maybe it was possible on 10.98 version, we will have to wait for version 12.x to try it again since we can't test on official 10.98 anymore, unless we find a video

@Zbizu
Copy link
Contributor

Zbizu commented Sep 22, 2021

if (tmpTile == nullptr || (tmpTile->getGround() == nullptr && !tmpTile->hasFlag(TILESTATE_BLOCKSOLID))) {

if (tmpTile == nullptr || (tmpTile->getGround() == nullptr && !tmpTile->hasFlag(TILESTATE_BLOCKSOLID))) {

my guess is that we have to use something other than TILESTATE_BLOCKSOLID

we could try TILESTATE_IMMOVABLEBLOCKSOLID, but I'm not sure if it's working properly

@ghost
Copy link
Author

ghost commented Sep 22, 2021

added another example with stacking items by floorchange

@ghost
Copy link
Author

ghost commented Sep 22, 2021

maybe it was possible on 10.98 version, we will have to wait for version 12.x to try it again since we can't test on official 10.98 anymore, unless we find a video

version 7.7 works the same as tibia 12.71, so doubt any difference on 10.98
123123

@Zbizu
Copy link
Contributor

Zbizu commented Sep 22, 2021

Yeah it was like that since feature introduction probably. Please test if my pull request fixes the issue.

@ghost
Copy link
Author

ghost commented Sep 22, 2021

Yeah it was like that since feature introduction probably. Please test if my pull request fixes the issue.

yes it is working, only thing left is stacking those items by adding into floorchange like on the second picture

@Zbizu
Copy link
Contributor

Zbizu commented Sep 22, 2021

hey, do you play on pvp world? Please test if you can levitate/parcelwalk towards magic walled tiles

@ghost
Copy link
Author

ghost commented Sep 22, 2021

hey, do you play on pvp world? Please test if you can levitate/parcelwalk towards magic walled tiles

you cannot, also teleport effect on levitate and magic rope should be sent on previous position

@Zbizu
Copy link
Contributor

Zbizu commented Sep 22, 2021

I'll look into magic rope later. I've added a commit that changes how magic effect is launched for levitate. Check if the spell is 100% accurate now.

@ghost
Copy link
Author

ghost commented Sep 22, 2021

I'll look into magic rope later. I've added a commit that changes how magic effect is launched for levitate. Check if the spell is 100% accurate now.

ye levitate seems to be alright now, i only noticed that it seems to be case sensitive for the parameter

@Zbizu
Copy link
Contributor

Zbizu commented Sep 22, 2021

added a commit to ignore case sensitivity and spaces in param

@ghost
Copy link
Author

ghost commented Oct 31, 2021

@DSpeichert you could atleast read before randomly closing, that PR didnt fix all

@Zbizu
Copy link
Contributor

Zbizu commented Oct 31, 2021

what else is here to do?

@ghost
Copy link
Author

ghost commented Oct 31, 2021

what else is here to do?

  • read number 2 in the reproduce steps, check second image if you don't get it
  • we talked about magic rope

@DSpeichert DSpeichert reopened this Nov 1, 2021
@DSpeichert DSpeichert added the needs-triage Needs testing with screenshot/video confirmation label Nov 1, 2021
@ghost
Copy link
Author

ghost commented Nov 1, 2021

33

@DSpeichert DSpeichert added enhancement Increase or improvement in quality, value, or extent and removed needs-triage Needs testing with screenshot/video confirmation labels Feb 6, 2022
@ghost ghost closed this as completed Jul 21, 2024
@ghost ghost closed this as not planned Won't fix, can't repro, duplicate, stale Jul 21, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Increase or improvement in quality, value, or extent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants