← Back to team overview

mudlet-makers team mailing list archive

[Bug 1238339] [NEW] changing room area doeas not remove it from previous area rooms list

 

Public bug reported:

version: Mudlet 3.0-rc1 compiled from repo
OS: Arch Linux, newest.

I think script and output will tell everything:

lua script
=====
local newroomid = createRoomID();
addRoom(newroomid);

echo("Before adding new room to area\n");
echo("Area 7 rooms:\n");
display(getAreaRooms(7));
echo("Area 8 rooms:\n");
display(getAreaRooms(8));


setRoomArea(newroomid,7);
echo("\n\nAfter adding new room to area 7\n");
echo("Area 7 rooms:\n");
display(getAreaRooms(7));
echo("Area 8 rooms:\n");
display(getAreaRooms(8));

setRoomArea(newroomid,8);
echo("\n\nAfter adding new room to area 8\n");
echo("Area 7 rooms:\n");
display(getAreaRooms(7));
echo("Area 8 rooms:\n");
display(getAreaRooms(8));
echo("^^^^^ I think this is bug, area 7 shoudn't have room "..newroomid.."\n");

echo("\n\nDeleting room "..newroomid.."\n");
deleteRoom(newroomid);
echo("Area 7 rooms now:\n");
display(getAreaRooms(7));
echo("Area 8 rooms now:\n");
display(getAreaRooms(8));
echo("^^^^^ this is even more serious bug, room is removed only from the last area, non existing room still exist in previous area");
=====


OUTPUT:
Before adding new room to area
Area 7 rooms:
{}
Area 8 rooms:
{}


After adding new room to area 7
Area 7 rooms:
{
  [0] = 43
}
Area 8 rooms:
{}


After adding new room to area 8
Area 7 rooms:
{
  [0] = 43
}
Area 8 rooms:
{
  [0] = 43
}
^^^^^ I think this is bug, area 7 shoudn't have room 43


Deleting room 43
Area 7 rooms now:
{
  [0] = 43
}
Area 8 rooms now:
{}
^^^^^ this is even more serious bug, room is removed only from the last area, non existing room still exist in previous area

** Affects: mudlet
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Mudlet
Makers, which is subscribed to Mudlet.
https://bugs.launchpad.net/bugs/1238339

Title:
  changing room area doeas not remove it from previous area rooms list

Status in Mudlet the MUD client:
  New

Bug description:
  version: Mudlet 3.0-rc1 compiled from repo
  OS: Arch Linux, newest.

  I think script and output will tell everything:

  lua script
  =====
  local newroomid = createRoomID();
  addRoom(newroomid);

  echo("Before adding new room to area\n");
  echo("Area 7 rooms:\n");
  display(getAreaRooms(7));
  echo("Area 8 rooms:\n");
  display(getAreaRooms(8));

  
  setRoomArea(newroomid,7);
  echo("\n\nAfter adding new room to area 7\n");
  echo("Area 7 rooms:\n");
  display(getAreaRooms(7));
  echo("Area 8 rooms:\n");
  display(getAreaRooms(8));

  setRoomArea(newroomid,8);
  echo("\n\nAfter adding new room to area 8\n");
  echo("Area 7 rooms:\n");
  display(getAreaRooms(7));
  echo("Area 8 rooms:\n");
  display(getAreaRooms(8));
  echo("^^^^^ I think this is bug, area 7 shoudn't have room "..newroomid.."\n");

  echo("\n\nDeleting room "..newroomid.."\n");
  deleteRoom(newroomid);
  echo("Area 7 rooms now:\n");
  display(getAreaRooms(7));
  echo("Area 8 rooms now:\n");
  display(getAreaRooms(8));
  echo("^^^^^ this is even more serious bug, room is removed only from the last area, non existing room still exist in previous area");
  =====

  
  OUTPUT:
  Before adding new room to area
  Area 7 rooms:
  {}
  Area 8 rooms:
  {}

  
  After adding new room to area 7
  Area 7 rooms:
  {
    [0] = 43
  }
  Area 8 rooms:
  {}

  
  After adding new room to area 8
  Area 7 rooms:
  {
    [0] = 43
  }
  Area 8 rooms:
  {
    [0] = 43
  }
  ^^^^^ I think this is bug, area 7 shoudn't have room 43

  
  Deleting room 43
  Area 7 rooms now:
  {
    [0] = 43
  }
  Area 8 rooms now:
  {}
  ^^^^^ this is even more serious bug, room is removed only from the last area, non existing room still exist in previous area

To manage notifications about this bug go to:
https://bugs.launchpad.net/mudlet/+bug/1238339/+subscriptions


Follow ups

References