Euphy

Script commands for offline characters

18 posts in this topic

If it isn't too much work, it'd be useful to have a few more character commands that work regardless of online/offline status. It's both annoying and inefficient (script-wise) to keep checking whether to attachrid or do an SQL query, since attachrid doesn't work for offline characters (obviously) and SQL queries don't update immediately for online characters. And when setting character/account variables, there's the additional check whether to do an SQL update or insertion. Altogether it'd just be a lot more convenient to have these commands, or something similar:

  • Like setd, set a character or account variable for a specified character/account ID. This would work without attachrid(), and regardless of online/offline status. The variable prefix should automatically determine whether the supplied ID is character or account.
    * setc <character ID>,"<variable name>",<value>;
    * setc <account ID>,"<variable name>",<value>;


  • Like getd, and same as above.
    * getc(<character ID>,"<variable name>")
    * getc(<account ID>,"<variable name>")


  • Also, extend rid2name() and getcharid() to work for offline characters.

Opinions?

7

Share this post


Link to post
Share on other sites

I think these would be very useful!

But would it require more char-server code too? because character saving happens at the char-server level.

0

Share this post


Link to post
Share on other sites

getcharid is some of the ways to know if a character is online or not, it would make problems with some scripts.

0

Share this post


Link to post
Share on other sites

@Omnipotent: That's what we have this command for.

*isloggedin(<account id>{,<char id>})

(I'm aware a few people used getcharid() to check online status, but that shouldn't be the reason offline implementation is held back.)

1

Share this post


Link to post
Share on other sites

Since we dropped TXT, I think it is not too hard to make it possible to modify offline player's variables, or read them.

In fact, you can already do that with query_sql, however built in support would be much more elegant of course.

0

Share this post


Link to post
Share on other sites

@xazax: The problem with doing SQL queries is that variables aren't updated right away. There's always some delay, and from what I've tested it ranges anywhere from a few seconds to minutes.

I didn't realize we had an "@set" command, though. Wouldn't at least half the code for these commands be identical to the atcommand?

0

Share this post


Link to post
Share on other sites

getcharid is some of the ways to know if a character is online or not, it would make problems with some scripts.

They should use playerattached(). In memory getcharid() throw an error if no player attached to the script

No opinion about an offline mode.

0

Share this post


Link to post
Share on other sites

I'm supporting this too, I just wanted to mark out that detail.

@KeyWorld:

But I mean getting the status, and if he's connected the id of a character who is not attached at that moment to the script.

>if(getcharid(0,"john")) he's online, so attachrid and stuff.

@Euphy:

I'm agree on this, only was a detail for some old scripts, but the change required is minimal.

Edited by Omnipotent
0

Share this post


Link to post
Share on other sites

I like the idea.

I want to start working on it after work today.

I'm probably going to make it so if the command is used and the offline character is actually online, it will switch to the regular set routines. It'll be like a safer version of set.

Also, I don't think character ID and account ID as an parameter in the same command would be a feasible idea. A sufficiently large enough character database could have IDs that are the same numbers as the account IDs, and that would mess up the logic. I would probably have to separate those into separate commands.

Edited by sketchyphoenix
2

Share this post


Link to post
Share on other sites

You're amazing. Thank you~

Couldn't you only use one command, though? Because variable types are defined by prefixes, can't you read the variable type first before processing the ID? I really don't know C at all, so correct me if I'm wrong.

0

Share this post


Link to post
Share on other sites

The problem is that only the parser in the script engine recognizes variable affixes and once it leaves the txt file and goes into memory, those get stripped from the variable name before being stored..if i remember correctly. If not, then oh well. That's no impact on anything, I still have to set aside several commands to differentiate between char ID and account ID - and if the server is assuming these players are offline, it can't assume that the calling NPC has a player attached, so it needs that information beforehand.

So I'll need 4 commands. One that alters an entry for a character variable.. One that alters an entry for an account variable. And 2 more to retrieve those. If the character or account is logged in, then the command will simply act as set would, instead of altering the database directly.

1

Share this post


Link to post
Share on other sites

Alright, that makes sense. At least it'll prevent any confusion. Thanks a lot. c:

0

Share this post


Link to post
Share on other sites

The problem is that only the parser in the script engine recognizes variable affixes and once it leaves the txt file and goes into memory, those get stripped from the variable name before being stored..if i remember correctly.

you should be able to identify both smoothly without the need of another 2 commands imo, you can take a look on script.c::set_reg() for reference

0

Share this post


Link to post
Share on other sites

This can slow down the script VM(mainly for getc), the script execution will need to be suspended and wait a response from char-server, so this command should be used wisely.

0

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now