PenUltima Online Forum Index Official Core: 096.7
Official Core: 097 2008-02-26
Donate towards the POL web hosting bill!
 POL Home   FAQ   Search    Memberlist   Usergroups    Register    Profile   Log in to check your private messages   Log in
Guild package, please read!
Goto page Previous  1, 2
 
Post new topic   Reply to topic    PenUltima Online Forum Index -> Development Discussion
Display posts from previous:   

Author Message
MontuZ
Distro Developer


Joined: 10 Feb 2006
Posts: 293
Location: Myrtle Beach, South Carolina

PostPosted: Tue Jun 26, 2007 6:27 am    Post subject: Reply with quote

OldnGrey wrote:
Must admit, I am afraid of folk who are used to coding the old old gumps system and write the same way in the new GF system. What do I mean? GFGumps allows you to set variables to GFAddButton presses. It really freaks me to still see tests for numbers in the gump return Smile


OldnGrey - Can you show me an example in code? I'm not sure exactly what you're talking about Razz AND I fixed a possible bug problem, changed mobile.connected to mobile.ip, mobile.connected still isn't working properly(At least as far as I know with the newest core.).

FYI; I believe Austin added logging, I just found out it existed! lol. When I add in Admin/Dev controls I'll be sure to put logging in to that, but I'm not sure what else I could log for players, other than error reports.

Suggestions on the logging for players? AND AND AND .... Surprised If anyone has any suggestions on how I could make my coding more efficient, please do tell.

Author Message
OldnGrey



Joined: 04 Feb 2006
Posts: 517

PostPosted: Tue Jun 26, 2007 7:41 am    Post subject: Reply with quote

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 ">>".

Author Message
MontuZ
Distro Developer


Joined: 10 Feb 2006
Posts: 293
Location: Myrtle Beach, South Carolina

PostPosted: Tue Jun 26, 2007 1:37 pm    Post subject: Reply with quote

I might try to use gumps that way here in a little.

*Fixed the << >> thingy, missed that one. I'll commit changes when I do more work on it.

And I'll change the way the packet's handled, I knew it would soon grow into an issue.

Author Message
OldnGrey



Joined: 04 Feb 2006
Posts: 517

PostPosted: Mon Jul 02, 2007 9:52 am    Post subject: Reply with quote

Is it normal for a change of faction to remove ALL allies and enemies or leaving enemies if it's still a different faction? I am writing this bit now but don't know the convention.

What sort of voting for new leader system are you suggesting? Would you have it activate at "member" logon or use Austin's mail system? I've been putting lots of things through the mail system lately (and a 30 day mail prune too keeps it tidy for all those prople who never bother to read them).

It's a lot of fun doing guilds! A pity my code isn't up to distro standard. However, I am happy to send you the changes I've done in case it helps you at all.


Last edited by OldnGrey on Wed Jul 04, 2007 10:43 pm; edited 1 time in total

Author Message
OldnGrey



Joined: 04 Feb 2006
Posts: 517

PostPosted: Wed Jul 04, 2007 10:20 pm    Post subject: Reply with quote

Some assistance from anyone who knows the conventions please.....

What is the definition of an opposing faction?

The way the distro is written:
If you are order, an opposing faction is chaos but not neutral
If you are chaos, an opposing faction is order but not neutral
However if you are neutral, an opposing faction is chaos and order.

That means an order can request alliance with neutral, but a neutral cannot request alliance with order.

Since I have no clue about the convention, I'd only be guessing.


* I have added an email notification when you remove alliances or war or when changing factions breaks relationship, or when you demote, promote or kick a member.
* Fixed a bug where you could promote or demote members of different guild.
* Fixed a bug where you could promote a guild member to Guild Master.

Author Message
MontuZ
Distro Developer


Joined: 10 Feb 2006
Posts: 293
Location: Myrtle Beach, South Carolina

PostPosted: Thu Jul 05, 2007 7:35 am    Post subject: Reply with quote

Thanks for bringing that up OldnGrey. I'll fix the opposing factions thing when I get a chance too.

I meant to have Neutral an opposing faction to Chaos/Order and Chaos/Order shouldn't be able to request an alliance with them. I don't know what I was thinking when I wrote it, lol.

Author Message
OldnGrey



Joined: 04 Feb 2006
Posts: 517

PostPosted: Thu Jul 05, 2007 8:24 am    Post subject: Reply with quote

Sorry, my fault. I've been playing with factions too much. Cross that off your list.

As far as actual bugs go, only 2 so far.
* you can promote/demote/remove members from a different guild
* you can promote someone to be another GuildMaster.
Both easily fixed.

I seem to have gotten factions, ondelete and monthly cleanups written without much drama and mailouts to effected guilds with the SendEmail function.
Also I've moved the hook to its own package under packethooks because it's more generic than just guilds.

Now for the one I've been dreading: the "In Vote" bit at logon. Smile

By the way, events.inc seems to be redundant.

What has been of most interest in this package is the fact that your style is very easy to follow even if you do use Austin's settings functions.

All my comments here are not so much for you to use what I've written, it's more to get your interest fired up again Smile

Author Message
OldnGrey



Joined: 04 Feb 2006
Posts: 517

PostPosted: Fri Jul 13, 2007 9:58 pm    Post subject: Reply with quote

I've finished the player guilds package, including autopruning of members and guilds, voting, factions, changing titles etc. It makes extensive use of logging and the Distro's email package.

But I can't post it in the scripts forum because it's not allowing any attachments.

If you are interested, let me know. Feel free to get it and tear it apart - it might help you finish the real distro package a bit faster.

Mine is not a drop in for the distro because I use my own logging (096 distro actually) and use the title_guild for other things too, but it's pretty much compatible.

Author Message
ncrsn



Joined: 10 Feb 2006
Posts: 168

PostPosted: Fri Jul 13, 2007 10:17 pm    Post subject: Reply with quote

Can't wait.

We are currently using somewhat tuned guild package from D95 and it's quite... Outdated. I have started scripting a new version several times but never really got it done, so I'm really looking forward for this.

Author Message
OldnGrey



Joined: 04 Feb 2006
Posts: 517

PostPosted: Fri Jul 13, 2007 10:38 pm    Post subject: Reply with quote

If you mean Bishop's guildstone package, then I have been using that too.

Part of the autocleanup is to look for guilds using the guildstone system and remove members and guilds. (My itemdesc.cfg contains blanks for the old guildstone and guilddeed so you can disable the old package entirely.)

For a long time I was toying with the Guildstone package to update it to 097, but it had 2 hurdles - it was written in the first version of GFGumps, plus I had already decided to use as much of the 097 distro as possible. So my challenge was to take MontuZ's guild system and finish it. I hope MontuZ does finish it and I'd be honoured if my version helped in any way.

Author Message
ncrsn



Joined: 10 Feb 2006
Posts: 168

PostPosted: Fri Jul 13, 2007 10:55 pm    Post subject: Reply with quote

OldnGrey wrote:
If you mean Bishop's guildstone package, then I have been using that too.

Yap, that's it.

OldnGrey wrote:
Part of the autocleanup is to look for guilds using the guildstone system and remove members and guilds. (My itemdesc.cfg contains blanks for the old guildstone and guilddeed so you can disable the old package entirely.)

Do you mean old guilds will be lost? If so, quess it wouldn't be too hard to write a script that saves present guilds with members into a variable or datafile and after removing those from POL's memory adds them again using new package. Hard to say for sure, but may be worth a try.

Can I get this finished package somewhere or will you wait for MontuZ to bless your work (and/or, like you said, finish guilds himself)?

Post new topic   Reply to topic    PenUltima Online Forum Index -> Development Discussion All times are GMT - 4 Hours
Goto page Previous  1, 2
Page 2 of 2

 




Powered by phpBB © 2001, 2005 phpBB Group :: Theme & Graphics by GHS & Scott E. Royalty