--- deliantra/server/lib/cf.pm	2006/03/17 03:22:00	1.9
+++ deliantra/server/lib/cf.pm	2006/06/12 13:25:27	1.16
@@ -67,25 +67,25 @@
 my @exts;
 my @hook;
 my %command;
+my %extcmd;
 
 sub inject_event {
-   my ($data) = @_;
+   my $extension = shift;
+   my $event_code = shift;
 
-   my $cb = $hook[$data->{event_code}]{$data->{extension}}
+   my $cb = $hook[$event_code]{$extension}
       or return;
 
-   $cb->($data)
+   &$cb
 }
 
 sub inject_global_event {
-   my ($data) = @_;
+   my $event = shift;
 
-   my $cb = $hook[$data->{event_code}]
+   my $cb = $hook[$event]
       or return;
 
-   $_->($data) for values %$cb;
-
-   ()
+   List::Util::max map &$_, values %$cb
 }
 
 sub inject_command {
@@ -102,19 +102,27 @@
    my ($name, $time, $cb) = @_;
 
    my $caller = caller;
-
-   warn "registering command '$name/$time' to '$caller'";
+   #warn "registering command '$name/$time' to '$caller'";
 
    push @{ $command{$name} }, [$time, $cb, $caller];
    $COMMAND{"$name\000"} = List::Util::max map $_->[0], @{ $command{$name} };
 }
 
+sub register_extcmd {
+   my ($name, $cb) = @_;
+
+   my $caller = caller;
+   #warn "registering extcmd '$name' to '$caller'";
+
+   $extcmd{$name} = [$cb, $caller];
+}
+
 sub register {
    my ($base, $pkg) = @_;
 
    for my $idx (0 .. $#EVENT) {
       if (my $ref = $pkg->can ("on_$EVENT[$idx]")) {
-         warn "registering $EVENT[$idx] hook to '$pkg'\n";
+         #warn "registering $EVENT[$idx] hook to '$pkg'\n";
          $hook[$idx]{$base} = $ref;
       }
    }
@@ -175,6 +183,11 @@
       }
    }
 
+   # remove extcmds
+   for my $name (grep $extcmd{$_}[1] eq $pkg, keys %extcmd) {
+      delete $extcmd{$name};
+   }
+
    Symbol::delete_package $pkg;
 }
 
@@ -217,10 +230,23 @@
 };
 
 #############################################################################
+# extcmd framework, basically convert ext <id> <pkg> arg1 args
+# into pkg::->on_extcmd_arg1 (...) while shortcutting a few
+
+sub on_extcmd {
+   my ($pl, $buf) = @_;
+
+   my ($type) = $buf =~ s/^(\S+) // ? $1 : "";
+
+   $extcmd{$type}[0]->($pl, $buf)
+      if $extcmd{$type};
+}
+
+#############################################################################
 # load/save/clean perl data associated with a map
 
 *on_mapclean = sub {
-   my $map = shift->{map};
+   my ($map) = @_;
 
    my $path = $map->tmpname;
    defined $path or return;
@@ -230,7 +256,7 @@
 
 *on_mapin =
 *on_mapload = sub {
-   my $map = shift->{map};
+   my ($map) = @_;
 
    my $path = $map->tmpname;
    $path = $map->path unless defined $path;
@@ -247,7 +273,7 @@
 };
 
 *on_mapout = sub {
-   my $map = shift->{map};
+   my ($map) = @_;
 
    my $path = $map->tmpname;
    $path = $map->path unless defined $path;
@@ -277,43 +303,37 @@
 # load/save perl data associated with player->ob objects
 
 *on_player_load = sub {
-   my ($event) = @_;
-   my $path = $event->{message};
-   my $ob = $event->{who};
+   my ($ob, $path) = @_;
 
-   open my $fh, "<:raw", "$path.cfperl"
-      or return; # no perl data
+   if (open my $fh, "<:raw", "$path.cfperl") {
 
-   my $data = Storable::thaw do { local $/; <$fh> };
+      #d##TODO#remove
 
-   $data->{version} <= 1
-      or return; # too new
+      my $data = Storable::thaw do { local $/; <$fh> };
 
-   %$ob = %{$data->{ob}};
-};
+      $data->{version} <= 1
+         or return; # too new
 
-*on_player_save = sub {
-   my ($event) = @_;
-   my $path = $event->{message};
-   my $ob = $event->{who};
+      %$ob = %{$data->{ob}};
+      return;
+   }
 
-   if (keys %$ob) {
-      open my $fh, ">:raw", "$path.cfperl"
-         or die "$path.cfperl: $!";
+   for my $o ($ob, $ob->inv) {
+      if (my $value = $o->get_ob_key_value ("_perl_data")) {
+         $o->set_ob_key_value ("_perl_data");
 
-      stat $path;
+         %$o = %{ Storable::thaw pack "H*", $value };
+      }
+   }
+};
 
-      print $fh Storable::nfreeze {
-         size    => (stat _)[7],
-         time    => (stat _)[9],
-         version => 1,
-         ob      => $ob,
-      };
+*on_player_save = sub {
+   my ($ob, $path) = @_;
 
-      chmod SAVE_MODE, "$path.cfperl"; # very racy, but cf-compatible *g*
-   } else {
-      unlink "$path.cfperl";
-   }
+   $_->set_ob_key_value (_perl_data => unpack "H*", Storable::nfreeze $_)
+      for grep %$_, $ob, $ob->inv;
+
+   unlink "$path.cfperl";#d##TODO#remove
 };
 
 register "<global>", __PACKAGE__;