--- deliantra/server/ext/rent.ext	2007/01/05 01:06:27	1.5
+++ deliantra/server/ext/rent.ext	2008/05/03 11:14:50	1.13
@@ -18,12 +18,10 @@
 sub reject_entry {
    my ($pl) = @_;
 
-   cf::override;
+   my $prev_pos = $pl->ob->{_prev_pos};
+   $pl->ob->goto ($prev_pos ? @$prev_pos : ("/world/world_105_115", 2, 34));
 
-   $pl->goto ("/world/world_105_115", 2, 34)
-      unless
-         $pl->ob->map
-         && !$pl->ob->map->{path}{user_rel};
+   cf::override;
 }
 
 sub update_balance {
@@ -49,6 +47,8 @@
 sub pay_balance {
    my ($pl) = @_;
 
+   cf::cede_to_tick;
+
    update_balance $pl;
 
    return unless $pl->{rent}{balance} > 0;
@@ -61,11 +61,11 @@
       cf::db_put rent => balance => $deduct + cf::db_get rent => "balance";
       $pl->ob->{bank_balance} -= $deduct;
       $pl->{rent}{balance} -= $deduct;
-      $pl->ob->reply (undef, "Something whispers into your ear:\n"
-                           . "Sir, we deducted your apartment rent ($deduct_string) from your bank account.");
+      $pl->ob->reply (undef, "Something whispers into your ear: "
+                           . "Your highness, we deducted your apartment rent ($deduct_string) from your bank account.");
    } else {
-      $pl->ob->reply (undef, "Something whispers into your ear:\n"
-                           . "Sir, we want to deduct the apartment rent ($deduct_string), but the bank informed us that they cannot perform the transaction. "
+      $pl->ob->reply (undef, "Something whispers into your ear: "
+                           . "Your highness, we want to deduct the apartment rent ($deduct_string), but the bank informed us that they cannot perform the transaction. "
                            . "Please even out your balance so we can deduct the fees, otherwise we will be forced to shut down your access to the apartment.");
    }
 }
@@ -94,7 +94,7 @@
       my $type = exists $pl->{rent}{apartment}{$k} ? 1 : 2;
 
       $pl->ob->reply (undef, "model \"$v->[2]\", situated in $v->[1] ("
-                      . (cf::cost_string_from_value $v->[0]) . "/hr)")
+                      . (cf::cost_string_from_value $v->[0]) . "/hr)\n")
          if $type & $types;
    }
 };
@@ -119,7 +119,7 @@
 
    $pl->{rent}{apartment}{$apartment} = undef;
 
-   $pl->ob->reply (undef, "Wonderful decision, sir! "
+   $pl->ob->reply (undef, "Wonderful decision, your highness! "
                         . "We told the proprietor in $apartment{$apartment}[1] to expect you and let you in. "
                         . "We are sure you will be satisfied!");
 };
@@ -147,7 +147,7 @@
       if ($pl->{rent}{last_online_check}) {
          $pl->{rent}{last_online_check} = time
             - List::Util::min 3600,
-                 $pl->ob->get_ob_key_value ("schmorplog_last_save") - $pl->{rent}{last_online_check};
+                 $pl->ob->kv_get ("schmorplog_last_save") - $pl->{rent}{last_online_check};
       } else {
          $pl->{rent}{last_online_check} = time;
       }
@@ -160,37 +160,32 @@
    on_enter => sub {
       my ($map, $pl, $x, $y) = @_;
 
-      # can freely enter homes of other people
-      {
-         my $path = sprintf "%s/%s/%s/",
-            cf::localdir, cf::playerdir, $pl->ob->name;
+      my $pfx = sprintf "~%s/", $pl->ob->name;
 
-         return if $path ne substr $map->path, 0, length $path;
-      }
+      # only do something if entering ones own apartment
+      if ($pfx eq substr $map->path, 0, length $pfx) {
+         for my $path (keys %{ $pl->{rent}{apartment} }) {
+            $path = sprintf "~%s%s", $pl->ob->name, $path;
+
+            if ($map->path eq $path) {
+               if (check_balance $pl) {
+                  $pl->ob->reply (undef, "Welcome to your apartment, your highness!");
+               } else {
+                  $pl->ob->reply (undef, "We are sorry, your highness, you have to pay your rent first.");
+                  reject_entry $pl;
+               }
 
-      for my $path (keys %{ $pl->{rent}{apartment} }) {
-         $path =~ y/\//_/;
-         $path = sprintf "%s/%s/%s/%s",
-            cf::localdir, cf::playerdir, $pl->ob->name, $path;
-
-         if ($map->path eq $path) {
-            if (check_balance $pl) {
-               $pl->ob->reply (undef, "Welcome to your apartment, sir!");
-            } else {
-               $pl->ob->reply (undef, "We are sorry, sir, you have to pay your rent first.");
-               reject_entry $pl;
+               return;
             }
-
-            return;
          }
-      }
 
-      $pl->ob->reply (undef, "Sir, you have to rent this apartment in The Apartment Shop in Scorn first!");
-      reject_entry $pl;
+         $pl->ob->reply (undef, "Your highness, you have to rent this apartment in The Apartment Shop in Scorn first!");
+         reject_entry $pl;
+      }
    },
 ;
 
-our $RENT_TIMER = Event->timer (after => 60, interval => 3600, data => cf::WF_AUTOCANCEL, cb => sub {
+our $RENT_TIMER = cf::periodic 3600, Coro::unblock_sub {
    pay_balance $_ for cf::player::list;
-});
+};