--- deliantra/server/ext/map-tags.ext	2007/09/11 15:40:01	1.4
+++ deliantra/server/ext/map-tags.ext	2007/09/13 16:16:36	1.7
@@ -79,7 +79,7 @@
 
    # 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#
 }
@@ -102,23 +102,45 @@
       for @$dirs;
 }
 
-cf::async_ext {
-   $Coro::current->prio (Coro::PRIO_MIN);
+sub reload {
+   my $start = Event::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::from_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 +168,20 @@
 #      }
 #   }
 
-      warn sprintf "map-tag scan (%fs)", Event::time - $start;
-      Coro::Timer::sleep $SCHEDULE_INTERVAL;
-   }
-};
+   warn sprintf "map-tag scan (%fs)", Event::time - $start;
+}
+
+our $RELOAD_SCHEDULER = Event->timer (
+   after    => 0,
+   interval => $SCHEDULE_INTERVAL,
+   data     => cf::WF_AUTOCANCEL,
+   cb       => Coro::unblock_sub {
+      $Coro::current->prio (Coro::PRIO_MIN);
+      reload;
+   },
+);
 
+# find all objects with the given tag, or at least try to
 sub find($) {
    my ($tag) = @_;