search

Lady-Binary / EasyLoU

EasyLoU is the premiere macro tool for Legends of Aria / Legends of Ultima. It is a totally free and open source tool inspired by EasyUO, the popular premiere macroing tool for Ultima Online.
Other
10 stars 12 forks source link

Add X,Y,Z attributes to FINDPERMANENTs. Add ability to use Move() with PERMANENTID. #90

Closed nerdhere closed 4 years ago

nerdhere commented 4 years ago

Is your feature request related to a problem? Please describe. No

Describe the solution you'd like FindPermanent ability to provide x,y,z coordinates for the found objects Example: Use FindPermanent("tree") and somehow pull the tree X,Y,Z coordinates to use with MoveEx

Lady-Binary commented 4 years ago

had a quick look and should be feasible.... should be in the next release! :)

Lady-Binary commented 4 years ago

@nerdhere actually, may I ask you why you need x y z? you want to move over there? it should be feasible by just passing the ID of the permanent object to the Move function ... is that not working ?

moinphyst commented 4 years ago

@Lady-Binary I can confirm that this was not working in 1.0.0.32. Have not tried since r33.

thiagomaf commented 4 years ago

This would be convenient for Mobiles.

Lady-Binary commented 4 years ago

@Lady-Binary I can confirm that this was not working in 1.0.0.32. Have not tried since r33.

@moinphyst are you saying that you tried passing the ID to the Move() function and it didn't work?

@thiagomaf same for mobiles ... Move(MOBILEID) should move to the mobile ... have you tried?

I'll give it a try, maybe Move it's bugged

moinphyst commented 4 years ago

@Lady-Binary Move(id) works just fine for mobiles, it did not work for permanents.

Lady-Binary commented 4 years ago

I'll investigate then, thank you!

just want to confirm with the original poster @nerdhere - can you confirm that you needed x,y,z only because you wanted to move to the rock, and not for another reason that I'm missing?

I'd rather fix Move(ID) so that it works, rather than adding new vars. Or maybe I'll do both :D

thiagomaf commented 4 years ago

It would be convenient to have the mobile's X, Y, Z position so one can move to an offset position - e.g. keep X distance from the mob.

Lady-Binary commented 4 years ago

gotcha, now I understand why you might need it for findmobile.

I don't see why it would be needed for FindPermanent tho.

Lady-Binary commented 4 years ago

created issue #99 for mobiles

Lady-Binary commented 4 years ago

turns out, permanent objects are not regular gameobjects. they do not have a gameobject ID that can be used with Move() and other functions, but they have a PERMANENTID instead.

in order to avoid confusion going forward, i will rename this ID for what it really is, a PERMANENTID.

also, I will add X Y and Z to the FINDPERMANENT results.

Lady-Binary commented 4 years ago

It's now also possible to use Move() with the PERMANENTID :) I think I've fixed it.