 |
 |
 |
 |
|
 |
 |
| Author |
Message |
OldnGrey
Joined: 04 Feb 2006 Posts: 520
|
Posted: Tue Jun 26, 2007 7:41 am Post subject: |
|
|
Dammit man, I'm not criticizing your code! I am just scared of that old gump stuff. And I was thrilled to see the first update in the distro code since May 12!
Oh, by the way, are you dead sure about the uopacket.cfg? Since 0xD7 is a generic packet, I am thinking of putting that initial code into pkg\packethooks.
I got nothing when pressing the Guild button. I can't see how it matches the 0xD7 documentation. Just to prove it all, I tried this which appears to work (for guild button at least):
| Code: |
Packet 0xD7
{
Length variable
SubCommandOffset 8
SubCommandLength 1
}
SubPacket 0xD7
{
SubCommandID 0x28
ReceiveFunction :playerguilds:hook/guildButton:GuildButtonHook
}
SubPacket 0xD7
{
SubCommandID 0x32
ReceiveFunction :playerguilds:hook/guildButton:QuestButtonHook
} |
It's gotta be shared with the combat book and one day the housing package.
Ah, an example of gumpiness then for your amusement!
Original gump style would be to have:
| Code: |
GFAddButton(gump, 125, 113, 2117, 2118, GF_CLOSE_BTN, 8); |
And then in the return you go:
| Code: | case (result[0] )
8: ShowGuilds(character); |
My way to code this stuff (because it makes a bunch more sense to me anyway) would be:
| Code: | | var shguild := GFAddButton(gump, 125, 113, 2117, 2118, GF_CLOSE_BTN); |
And then in the return:
| Code: | case (result[0] )
shguild: ShowGuilds(character); |
Big deal? Then code GFRadioButton this way and notice how nice the results are to extract.
Just to further frustrate you - you have another bug in ranks.inc in the SetGuildRank function. The way it's set now, the creater is not given a Guild_Rank cprop. Look for the "<<" and ">>". |
|
 |
|
|
 |
 |
|