[ViewVC] Diff of: cvs/deliantra/server/common/object.C

Comparing deliantra/server/common/object.C (file contents):
Revision 1.217 by root, Wed Apr 23 07:13:23 2008 UTC vs.
Revision 1.221 by root, Thu Apr 24 12:06:01 2008 UTC

 // find player who can see this object
 object *
 object::visible_to () const
 {
-  if (!flag [FLAG_REMOVED])
+  if (client_visible () && !flag [FLAG_REMOVED])
     {
       // see if we are in a container of sorts
       if (env)
         {
           // the player inventory itself is always visible
       else
         {
           // maybe there is a player standing on the same mapspace
           // this will catch the case where "this" is a player
           if (object *pl = ms ().player ())
+            if (!pl->container || this == pl->container)
-            return pl;
+              return pl;
         }
     }
   return 0;
 }
 object::destroy (bool destroy_inventory)
 {
   if (destroyed ())
     return;
+  if (!is_head () && !head->destroyed ())
+    {
+      LOG (llevError | logBacktrace, "tried to destroy the tail of an object");
+      head->destroy (destroy_inventory);
+    }
   destroy_inv (!destroy_inventory);
   if (is_head ())
     if (sound_destroy)
       play_sound (sound_destroy);
   if (flag [FLAG_REMOVED])
     return;
   INVOKE_OBJECT (REMOVE, this);
-  if (object *pl = visible_to ())
-    esrv_del_item (pl->contr, count);
   flag [FLAG_REMOVED] = true;
   if (more)
     more->remove ();
    * In this case, the object to be removed is in someones
    * inventory.
    */
   if (env)
     {
+      flag [FLAG_REMOVED] = false; // hack around the issue of visible_to checking flag_removed
+      if (object *pl = visible_to ())
+        esrv_del_item (pl->contr, count);
+      flag [FLAG_REMOVED] = true; // hack around the issue of visible_to checking flag_removed
       adjust_weight (env, -total_weight ());
       *(above ? &above->below : &env->inv) = below;
       if (below)
       if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
         otmp->update_stats ();
     }
   else if (map)
     {
-      if (type == PLAYER)
-        {
-          // leaving a spot always closes any open container on the ground
-          if (container && !container->env)
-            // this causes spurious floorbox updates, but it ensures
-            // that the CLOSE event is being sent.
-            close_container ();
-          --map->players;
-          map->touch ();
-        }
       map->dirty = true;
       mapspace &ms = this->ms ();
+      if (object *pl = ms.player ())
+        {
+          if (type == PLAYER) // this == pl(!)
+            {
+              // leaving a spot always closes any open container on the ground
+              if (container && !container->env)
+                // this causes spurious floorbox updates, but it ensures
+                // that the CLOSE event is being sent.
+                close_container ();
+              --map->players;
+              map->touch ();
+            }
+          else if (pl->container == this)
+            {
+              // removing a container should close it
+              close_container ();
+            }
+          esrv_del_item (pl->contr, count);
+        }
       /* link the object above us */
       if (above)
         above->below = below;
       else
              * removed (most likely destroyed), update the player view
              * appropriately.
              */
             pl->close_container ();
+          //TODO: the floorbox prev/next might need updating
-          esrv_del_item (pl, count);
+          esrv_del_item (pl->contr, count);
         }
       for (tmp = ms.bot; tmp; tmp = tmp->above)
         {
           /* No point updating the players look faces if he is the object
 object *
 insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
 {
   assert (!op->flag [FLAG_FREED]);
-  object *top, *floor = NULL;
   op->remove ();
   /* Ideally, the caller figures this out.  However, it complicates a lot
    * of areas of callers (eg, anything that uses find_free_spot would now
    * need extra work
    */
   if (op->nrof && !(flag & INS_NO_MERGE))
     for (object *tmp = ms.bot; tmp; tmp = tmp->above)
       if (object::can_merge (op, tmp))
         {
+          // TODO: we atcually want to update tmp, not op,
+          // but some caller surely breaks when we return tmp
+          // from here :/
           op->nrof += tmp->nrof;
           tmp->destroy (1);
         }
   CLEAR_FLAG (op, FLAG_APPLIED);        /* hack for fixing F_APPLIED in items of dead people */
       /* since *below* originator, no need to update top */
       originator->below = op;
     }
   else
     {
+      object *top, *floor = NULL;
       top = ms.bot;
       /* If there are other objects, then */
       if (top)
         {
     }
   op->map->dirty = true;
   if (object *pl = ms.player ())
+    //TODO: the floorbox prev/next might need updating
     esrv_send_item (pl, op);
   /* If this object glows, it may affect lighting conditions that are
    * visible to others on this map.  But update_all_los is really
    * an inefficient way to do this, as it means los for all players
 object::open_container (object *new_container)
 {
   if (container == new_container)
     return;
-  if (object *old_container = container)
+  object *old_container = container;
+  if (old_container)
     {
       if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))
         return;
 #if 0
       if (object *closer = old_container->inv)
         if (closer->type == CLOSE_CON)
           closer->destroy ();
 #endif
+      // make sure the container is available
+      esrv_send_item (this, old_container);
       old_container->flag [FLAG_APPLIED] = false;
       container = 0;
+      // client needs item update to make it work, client bug requires this to be separate
       esrv_update_item (UPD_FLAGS, this, old_container);
       new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));
       play_sound (sound_find ("chest_close"));
     }
   if (new_container)
         }
 #endif
       new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));
+      // make sure the container is available, client bug requires this to be separate
+      esrv_send_item (this, new_container);
       new_container->flag [FLAG_APPLIED] = true;
       container = new_container;
+      // client needs flag change
       esrv_update_item (UPD_FLAGS, this, new_container);
       esrv_send_inventory (this, new_container);
       play_sound (sound_find ("chest_open"));
     }
+//  else if (!old_container->env && contr && contr->ns)
+//    contr->ns->floorbox_reset ();
 }
 object *
 object::force_find (const shstr name)
 {

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing deliantra/server/common/object.C (file contents): Revision 1.217 by root, Wed Apr 23 07:13:23 2008 UTC vs. Revision 1.221 by root, Thu Apr 24 12:06:01 2008 UTC

Diff Legend

Comparing deliantra/server/common/object.C (file contents):
Revision 1.217 by root, Wed Apr 23 07:13:23 2008 UTC vs.
Revision 1.221 by root, Thu Apr 24 12:06:01 2008 UTC