--- deliantra/server/ext/map-tags.ext	2007/09/11 15:40:01	1.4
+++ deliantra/server/ext/map-tags.ext	2007/12/05 11:08:34	1.13
@@ -50,7 +50,7 @@
    BDB::db_get $db_mapinfo, $txn, $key, my $data;
 
    unless ($!) {
-      $data = from_json $data;
+      $data = decode_json $data;
       return if $data->{hash} eq $hash;
       $old_tags = $data->{tags};
    }
@@ -61,7 +61,7 @@
       or return;
 
    my @tags = sort $f->extract_tags;
-   $data = to_json { hash => $hash, tags => \@tags };
+   $data = encode_json { hash => $hash, tags => \@tags };
 
    BDB::db_put $db_mapinfo, $txn, $key, $data;
 
@@ -79,9 +79,10 @@
 
    # we don't actually care if it succeeds or not, as we
    # will just retry an hour later
-   BDB::db_txn_commit $txn;
+   BDB::db_txn_finish $txn;
 
-#   warn "tag-updated $file (= $key) $hash\n";#d#
+   warn "tag-updated $file (= $key) <@tags>\n"
+      if @tags;
 }
 
 sub scan_static {
@@ -102,23 +103,47 @@
       for @$dirs;
 }
 
-cf::async_ext {
-   $Coro::current->prio (Coro::PRIO_MIN);
+sub reload {
+   my $guard = cf::lock_acquire "map-tags::reload";
+
+   my $start = EV::time;
 
-   while () {
-      my $start = Event::time;
+   # 1. check for maps no longer existing
+   {
+      my @delkeys;
+
+      my $cursor = $db_mapinfo->cursor;
+      for (;;) {
+         BDB::db_c_get $cursor, my $key, my $data, BDB::NEXT;
+         last if $!;
+
+         my $data = JSON::XS::decode_json $data;
+         my ($ver, undef, undef, $path) = split /,/, $data->{hash}, 4;
+         push @delkeys, [$key, $data->{tags}]
+            if $ver != 1 || Coro::AIO::aio_stat $path;
+      }
+      BDB::db_c_close $cursor;
 
-      # 1. check for maps no longer existing
+      for (@delkeys) {
+         my ($key, $tags) = @$_;
+         my $txn = $cf::DB_ENV->txn_begin;
+         BDB::db_del $db_mapinfo, $txn, $key;
+         for my $tag (@{ $tags || [] }) {
+            remove_tag_target $txn, $tag, $key;
+         }
+         BDB::db_txn_finish $txn;
+      }
+   }
 
-      # 2. scan all static maps
-      scan_static $cf::MAPDIR, "/";
+   # 2. scan all static maps
+   scan_static $cf::MAPDIR, "/";
 
-      # 3. scan all dynamic maps
-      for my $path (@{ cf::map::tmp_maps or [] }, @{ cf::map::random_maps or [] }) {
+   # 3. scan all dynamic maps
+   for my $path (@{ cf::map::tmp_maps or [] }, @{ cf::map::random_maps or [] }) {
 #         my $map = cf::map::find $path;
 #         extract_map_tags "t/$map", $path;
-      }
-      
+   }
+   
       # now hunt for all per-player maps
 #      scan_dir $cf::PLAYERDIR
 #      for my $login (@{ cf::player::list_logins or [] }) {
@@ -146,11 +171,18 @@
 #      }
 #   }
 
-      warn sprintf "map-tag scan (%fs)", Event::time - $start;
-      Coro::Timer::sleep $SCHEDULE_INTERVAL;
-   }
+   warn sprintf "map-tag scan (%fs)", EV::time - $start;
+}
+
+our $RELOAD_SCHEDULER = cf::periodic $SCHEDULE_INTERVAL, Coro::unblock_sub {
+   $Coro::current->prio (Coro::PRIO_MIN);
+   $Coro::current->desc ("map-tag scanner");
+   reload;
 };
 
+cf::async { reload }; # force at startup
+
+# find all objects with the given tag, or at least try to
 sub find($) {
    my ($tag) = @_;