--- deliantra/server/lib/cf.pm	2007/01/05 21:51:42	1.142
+++ deliantra/server/lib/cf.pm	2007/01/07 02:39:14	1.143
@@ -49,10 +49,11 @@
 our $UPTIME; $UPTIME ||= time;
 our $RUNTIME;
 
-our %MAP; # all maps
+our %PLAYER; # all users
+our %MAP;    # all maps
 our $LINK_MAP; # the special {link} map
 our $RANDOM_MAPS = cf::localdir . "/random";
-our %EXT_CORO;
+our %EXT_CORO; # coroutines bound to extensions
 
 binmode STDOUT;
 binmode STDERR;
@@ -1073,19 +1074,76 @@
 
 Functions and methods that extend core crossfire objects.
 
+=cut
+
+package cf::player;
+
 =head3 cf::player
 
 =over 4
 
-=item cf::player::exists $login
+=item cf::player::find $login
 
-Returns true when the given account exists.
+Returns the given player object, loading it if necessary (might block).
 
 =cut
 
-sub cf::player::exists($) {
-   cf::player::find $_[0]
-      or -f sprintf "%s/%s/%s/%s.pl", cf::localdir, cf::playerdir, ($_[0]) x 2;
+sub path($) {
+   sprintf "%s/%s/%s/%s.pl",
+           cf::localdir, cf::playerdir,
+           (ref $_[0] ? $_[0]->ob->name : $_[0]) x 2
+}
+
+sub find_active($) {
+   $cf::PLAYER{$_[0]}
+      and $cf::PLAYER{$_[0]}->active
+      and $cf::PLAYER{$_[0]}
+}
+
+sub exists($) {
+   my ($login) = @_;
+
+   $cf::PLAYER{$login}
+      or cf::sync_job { !aio_stat $login }
+}
+
+sub find($) {
+   return $cf::PLAYER{$_[0]} || do {
+      my $login = $_[0];
+
+      my $guard = cf::lock_acquire "user_find:$login";
+
+      $cf::PLAYER{$login} ||= (load_pl path $login or return);
+   };
+}
+
+sub save($) {
+   my ($pl) = @_;
+
+   return if $pl->{deny_save};
+
+   my $path = path $pl;
+   my $guard = cf::lock_acquire "user_save:$path";
+
+   return if $pl->{deny_save};
+   $pl->{last_save} = $cf::RUNTIME;
+
+   Coro::cede;
+   $pl->save_pl ($path);
+   Coro::cede;
+}
+
+sub new($) {
+   my ($login) = @_;
+
+   my $self = create;
+
+   $self->ob->name ($login);
+   $self->{deny_save} = 1;
+
+   $cf::PLAYER{$login} = $self;
+
+   $self
 }
 
 =item $player->ext_reply ($msgid, $msgtype, %msg)
@@ -1094,14 +1152,16 @@
 
 =cut
 
-sub cf::player::ext_reply($$$%) {
+sub ext_reply($$$%) {
    my ($self, $id, %msg) = @_;
 
    $msg{msgid} = $id;
 
-   $self->send ("ext " . to_json \%msg);
+   $self->send ("ext " . cf::to_json \%msg);
 }
 
+package cf;
+
 =back
 
 
@@ -1306,6 +1366,10 @@
 
    local $self->{last_access} = $self->last_access;#d#
 
+   cf::async {
+      $_->contr->save for $self->players;
+   };
+
    if ($uniq) {
       $self->save_objects ($save, cf::IO_HEADER | cf::IO_OBJECTS);
       $self->save_objects ($uniq, cf::IO_UNIQUES);
@@ -1528,6 +1592,7 @@
 
    $map->load;
 
+   return unless $self->contr->active;
    $self->activate_recursive;
    $self->enter_map ($map, $x, $y);
 }
@@ -1549,7 +1614,6 @@
       # try to abort aborted map switching on player login :)
       # should happen only on crashes
       if ($pl->ob->{_link_pos}) {
-
          $pl->ob->enter_link;
          (async {
             # we need this sleep as the login has a concurrent enter_exit running