search

Google-Code-Fork / tibiaapi

Automatically exported from code.google.com/p/tibiaapi
MIT License
0 stars 0 forks source link

Some Bugs #60

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
- Trying to see a floor below you is not working (maybe an adress is wrong?)
ex: mapa.ShowFloor(-1, true);

- container.Rename() doesnt change the name

- Using OnLogIn()gives this error when compiling:
 Error  2   No overload for 'OnLogIn' matches delegate
'Tibia.Util.Proxy.ProxyNotification'

-something is wrong with :
client.Proxy.ReceivedProjectilePacket +=
(Proxy.PacketListener)ProjectilePacket;

cos it looks like the proxy never recives a ProjectilePacker but if you
change it to client.Proxy.ReceivedPacketFromServer then it goes to the
function.(check http://www.tpforums.org/forum/showthread.php?t=1665)

- Stack feature should only work with items in the same bp cos it gets a
little crazy if it trys to stack with other bp's.

Original issue reported on code.google.com by marcsal...@gmail.com on 31 Jul 2008 at 12:11

GoogleCodeExporter commented 9 years ago 
i forgot to say that console.Say(text,reciper) is also not working , it makes ur
character move but it doesnt send any message

Original comment by marcsal...@gmail.com on 31 Jul 2008 at 2:28

GoogleCodeExporter commented 9 years ago 
In the Enum of Constants you should update Party enum with the new shared exp:

    public enum Party
    {
        None = 0,
        Inviter = 1,
        Invitee = 2,
        Member = 3,
        Leader = 4,
        MemberSharedExp = 5,
        LeaderSharedExp = 6,
        MemberSharedExpInactive = 7,
        LeaderSharedExpInactive = 8

    }
And then updated the InParty function of Creature with:

        public bool InParty()
        {
            Constants.Party party = Party;
            return (party == Constants.Party.Member || party ==
Constants.Party.Leader || party == Constants.Party.LeaderSharedExp || party ==
Constants.Party.LeaderSharedExpInactive || party == 
Constants.Party.MemberSharedExp
|| party == Constants.Party.MemberSharedExpInactive);
        }

Original comment by marcsal...@gmail.com on 8 Aug 2008 at 3:51

GoogleCodeExporter commented 9 years ago 
Sorry to spam the thread but i cant edit my posts :(
the first bug i reported about the map is NOT a bug it was my mistaked cos i 
updated
tibiaapi by myself and forgot to update it with the oficial release.

my apologise!

Original comment by marcsal...@gmail.com on 8 Aug 2008 at 9:31

GoogleCodeExporter commented 9 years ago 
i can comfirm there are some bugs with the level spy, not sure if it's caused 
by the
map level you are when trying to use it or if its something else but it looks 
quite
random.

Original comment by warma...@gmail.com on 10 Aug 2008 at 2:24

GoogleCodeExporter commented 9 years ago 
Next time please put all of these in different issue reports. We will work on 
them as
soon as possible, but since it seems that you have some coding prowess, we could
always use extra help and submitting file patches (not whole files) is easy and
convenient for us.

Original comment by ian320 on 10 Aug 2008 at 9:24

GoogleCodeExporter commented 9 years ago 
About the last bug i reported (stacking items from diferent bps) i've seen this 
code
line in stack function:

if (first != null && second != null && first.Found && second.Found &&           

!(first.Loc.container == second.Loc.container && second.Count == 100))

So only items from different bps would get stacked. Is this a mistake or you 
wanted
to do this? I think it should only stack items from same BP'S.

This is how i "fixed" it (little modification xd):
if (first != null && second != null && first.Found && second.Found &&
first.Loc.container == second.Loc.container && !(second.Count == 100))

And sorry for posting all this issues in 1 post and also sorry for my bad 
english :)

Original comment by marcsal...@gmail.com on 10 Aug 2008 at 11:24

GoogleCodeExporter commented 9 years ago 
Actually, I designed the stacker to be that way so it could be used like a 
primitive
looter. I think with the addition of a proxy, it would probably be better to 
switch
the functionality of the stack function to work the way you proposed, and let 
the
programmer handle getting the items into one backpack.

Original comment by ian320 on 10 Aug 2008 at 11:50

GoogleCodeExporter commented 9 years ago 
marcsalvat, I have added you as a probationary project member. Feel free to make
changes and bugfixes, your contributes are greatly appreciated.

Original comment by ian320 on 12 Aug 2008 at 2:05

GoogleCodeExporter commented 9 years ago 
-ShowFloor is fixed.

-I tested Container.Rename and it worked fine. If you still have problems, file 
a new
issue report with more details.

-Not sure what you problem with OnLogIn is, but it works fine for me. If you 
still
have problems, file a new issue report with more details.

-ReceivedProjectilePacket still works, but sometimes it comes bundled with 
another
packet, which I think is not being recognized, so it gets skipped.

-You fixed the stack feature.

If you have any issues with these problem still, please make a separate issue 
report
for each one.

Original comment by ian320 on 15 Aug 2008 at 1:27