Login
User Name:

Password:



Register
Forgot your password?
Vote for Us!
parse description bug
Dec 15, 2017, 10:08 pm
By Remcon
Couple bugs
Dec 12, 2017, 5:42 pm
By Remcon
Bug in disarm( )
Nov 12, 2017, 6:54 pm
By GatewaySysop
Bug in will_fall( )
Oct 23, 2017, 1:35 am
By GatewaySysop
Bug in do_zap( ), do_brandish( )
Oct 18, 2017, 1:52 pm
By GatewaySysop
LOP 1.45
Author: Remcon
Submitted by: Remcon
LOP Heroes Edition
Author: Vladaar
Submitted by: Vladaar
Heroes sound extras
Author: Vladaar
Submitted by: Vladaar
6Dragons 4.3
Author: Vladaar
Submitted by: Vladaar
Memwatch
Author: Johan Lindh
Submitted by: Vladaar
Users Online
CommonCrawl, Sogou, Google

Members: 0
Guests: 1
Stats
Files
Topics
Posts
Members
Newest Member
477
3,706
19,240
608
LAntorcha
Today's Birthdays
There are no member birthdays today.
Related Links
» SmaugMuds.org » Codebases » LoP Codebase » LoP1.39 Questions/Issues/Anom...
Forum Rules | Mark all | Recent Posts

LoP1.39 Questions/Issues/Anomalies + Suggestions?
< Newer Topic :: Older Topic >

Pages:<< prev ... 2, 3, 4, 5, 6 ... next >>
Post is unread #61 May 15, 2011, 6:10 pm
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

Currently no, it shouldn't be to hard to make it show them though.
       
Post is unread #62 May 15, 2011, 6:19 pm
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

Thanks I will work on that, I have an idea how to go about it. But the deleting thing I have no clue how to do.
       
Post is unread #63 May 15, 2011, 6:21 pm
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

the deleting thing?
       
Post is unread #64 May 15, 2011, 6:26 pm
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

I wanna make it where if the owner of a bank account deletes their pfile, the accounts will delete as well.
       
Post is unread #65 May 15, 2011, 6:35 pm
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

oh I had missed the one you posted before mine about deleting, thought I already added that in guess not though (just got done looking), give me a bit ill see lol.
       
Post is unread #66 May 15, 2011, 6:41 pm
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

K thanks :) I was just able to port this over, and I am doing some fine touches to get it to where I want it for my mud.
       
Post is unread #67 May 15, 2011, 6:45 pm
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

Sorry to be bugging you like this, just trying to get a feel for this.

Is there anyway I can make it where the account name has to be enter inorder for the bank account to be deleted?
Because currently just typing bank delete will, delete the first account in the list created by you.
       
Post is unread #68 May 15, 2011, 6:56 pm   Last edited May 15, 2011, 7:01 pm by Remcon
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

Look for
   /* Just put arg as account */
   if( !str_cmp( account, "create" ) || !str_cmp( account, "balance" )
   || !str_cmp( account, "deposit" ) || !str_cmp( account, "withdraw" )
   || !str_cmp( account, "transfer" ) || !str_cmp( account, "delete" )
   || !str_cmp( account, "share" ) )
   {
      snprintf( arg, sizeof( arg ), "%s", account );
      if( !( nbank = get_first_bank( ch ) ) )
         snprintf( account, sizeof( account ), "%s", ch->name );
      else
         snprintf( account, sizeof( account ), "%s", nbank );
   }
   else
      argument = one_argument( argument, arg );

Id take the || !str_cmp( account, "delete" ) out and make a spot before that ifcheck to make sure they entered a valid account. If they had used delete as the argument that is.
       
Post is unread #69 May 15, 2011, 8:11 pm
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

Also I noticed that if a player deleted their name should be removed off of the share list for bank accounts. This would keep people from making a new character with this name from having access to old accounts.
       
Post is unread #70 May 27, 2011, 10:28 pm
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

I noticed while inputting the channels code into my own codebase that anything using phistory doesn't actually save or write. I doubled checked this on a fresh copy of LOP1.40. So I know this time I didn't miss anything :)
       
Post is unread #71 May 28, 2011, 5:52 am
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

Well it use to haha. Seen what you ment though :)

Looks like it doesn't like trying to do it like that in linking and unlinking, know it use to work great like that haha. Just have to do it the longer way for link and unlink in add_phistory.
       
Post is unread #72 May 28, 2011, 7:06 am
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

What is the longer way?
       
Post is unread #73 May 28, 2011, 7:36 am
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

/* Types: 0 = tell, 1 = say, 2 = yell, 3 = whisper, 4 = fchat */
void add_phistory( int type, CHAR_DATA *ch, char *argument )
{
   PER_HISTORY *phistory, *phistory_next, *phistory_remove = NULL, *history_start = NULL;
   int y = 0;

   if( !ch || is_npc( ch ) || !ch->pcdata || !argument || argument[0] == '\0' )
      return;
   if( type < 0 || type > 4 )
      return;

   if( type == 0 )
      history_start = ch->pcdata->first_tell;
   else if( type == 1 )
      history_start = ch->pcdata->first_say;
   else if( type == 2 )
      history_start = ch->pcdata->first_yell;
   else if( type == 3 )
      history_start = ch->pcdata->first_whisper;
   else if( type == 4 )
      history_start = ch->pcdata->first_fchat;

   for( phistory = history_start; phistory; phistory = phistory_next )
   {
      phistory_next = phistory->next;
      if( !phistory_remove )
         phistory_remove = phistory;
      if( ++y >= 20 )
      {
         if( type == 0 )
            UNLINK( phistory_remove, ch->pcdata->first_tell, ch->pcdata->last_tell, next, prev );
         else if( type == 1 )
            UNLINK( phistory_remove, ch->pcdata->first_say, ch->pcdata->last_say, next, prev );
         else if( type == 2 )
            UNLINK( phistory_remove, ch->pcdata->first_yell, ch->pcdata->last_yell, next, prev );
         else if( type == 3 )
            UNLINK( phistory_remove, ch->pcdata->first_whisper, ch->pcdata->last_whisper, next, prev );
         else if( type == 4 )
            UNLINK( phistory_remove, ch->pcdata->first_fchat, ch->pcdata->last_fchat, next, prev );
         free_phistory( phistory_remove );
         phistory_remove = NULL;
         y--;
      }
   }
   phistory = NULL;
   CREATE( phistory, PER_HISTORY, 1 );
   if( !phistory )
   {
      bug( "%s: couldn't create a phistory.\r\n", __FUNCTION__ );
      return;
   }
   smash_tilde( argument );
   phistory->text = STRALLOC( argument );
   phistory->chtime = current_time;
   if( type == 0 )
      LINK( phistory, ch->pcdata->first_tell, ch->pcdata->last_tell, next, prev );
   else if( type == 1 )
      LINK( phistory, ch->pcdata->first_say, ch->pcdata->last_say, next, prev );
   else if( type == 2 )
      LINK( phistory, ch->pcdata->first_yell, ch->pcdata->last_yell, next, prev );
   else if( type == 3 )
      LINK( phistory, ch->pcdata->first_whisper, ch->pcdata->last_whisper, next, prev );
   else if( type == 4 )
      LINK( phistory, ch->pcdata->first_fchat, ch->pcdata->last_fchat, next, prev );
   else
      free_phistory( phistory );
}

Seems to work good like that :)
       
Post is unread #74 May 28, 2011, 7:47 am   Last edited May 28, 2011, 7:49 am by dbna2
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

Thats LOL luckly I double checked this against a fresh copy of your codebase. I spent days wondering why it wasn't working

Also ealier in this thread u posted u had a fix for fread_phistory because of a memory leak? Can you post what the fix was?
       
Post is unread #75 May 28, 2011, 7:58 am
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

in fread_phistory find
   CREATE( phistory, PER_HISTORY, 1 );
   if( !phistory )
   {
      bug( "%s: couldn't create a phistory.\r\n", __FUNCTION__ );
      fread_time( fp );
      fread_string( fp );
      return;
   }

Change the fread_string to fread_flagstring since it doesn't need to be added into memory.
       
Post is unread #76 May 28, 2011, 8:07 am
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

Thanks :) Don't need memory leaks roaming around lol.
       
Post is unread #77 May 28, 2011, 8:17 am
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

No doubt and thanks for all the issues you have pointed out lately :)
       
Post is unread #78 May 28, 2011, 8:45 am
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

Not a problem, thanks for making such handy features :)
       
Post is unread #79 May 28, 2011, 5:04 pm
Go to the top of the page
Go to the bottom of the page

dbna2
Sorcerer
GroupMembers
Posts600
JoinedDec 2, 2008

Suggestion for the friend System: Have it send a reminder every now and then if there are any names waitting to be approved/deleted.

I already made this, but I thought I would share the idea with you.
       
Post is unread #80 May 29, 2011, 12:22 pm
Go to the top of the page
Go to the bottom of the page

Remcon
Geomancer
GroupAdministrators
Posts1,866
JoinedJul 26, 2005

K, did add that and decided that instead of putting the pending one on the one that sent the request to put it on the one that needs to accept/deny the request and made the changes needed to make the rest of them work with it. In all not to bad. Thanks for the idea :)
       
Pages:<< prev ... 2, 3, 4, 5, 6 ... next >>