Move client id check in Scene.Inventory.cs:UpdateInventoryItemAsset so that it doesn't trigger an exception if the item hasn't been found.

In this situation we will now put out a slightly more meaningful log error message instead.
0.7.2-post-fixes
Justin Clark-Casey (justincc) 2011-12-09 22:36:51 +00:00
parent 5b0d0ef1e9
commit f8b0427086
1 changed files with 6 additions and 4 deletions

View File

@ -337,11 +337,12 @@ namespace OpenSim.Region.Framework.Scenes
// Passing something to another avatar or a an object will already // Passing something to another avatar or a an object will already
InventoryItemBase item = new InventoryItemBase(itemID, remoteClient.AgentId); InventoryItemBase item = new InventoryItemBase(itemID, remoteClient.AgentId);
item = InventoryService.GetItem(item); item = InventoryService.GetItem(item);
if (item.Owner != remoteClient.AgentId)
return;
if (item != null) if (item != null)
{ {
if (item.Owner != remoteClient.AgentId)
return;
if (UUID.Zero == transactionID) if (UUID.Zero == transactionID)
{ {
item.Name = itemUpd.Name; item.Name = itemUpd.Name;
@ -388,8 +389,9 @@ namespace OpenSim.Region.Framework.Scenes
} }
else else
{ {
m_log.Error( m_log.ErrorFormat(
"[AGENTINVENTORY]: Item ID " + itemID + " not found for an inventory item update."); "[AGENTINVENTORY]: Item id {0} not found for an inventory item update for {1}.",
itemID, remoteClient.Name);
} }
} }