Mark GetUserInventory as obsolete. It is still used in the default

avatars module and in the REST inventory service, but future inventory
services and/or connector will not include this method anymore, because
it is dangerous on large inventory servers. Marking as obsolete to give
an incentive to rewrite these services to use the recursive fetching
or more targeted retrieval.

OpenSim/Services/Interfaces/IInventoryService.cs

@@ -59,6 +59,7 @@ namespace OpenSim.Services.Interfaces
         /// </summary>
         /// <param name="userID"></param>
         /// <returns></returns>
+        [Obsolete]
         InventoryCollection GetUserInventory(UUID userID);
 
         /// <summary>
@@ -67,6 +68,7 @@ namespace OpenSim.Services.Interfaces
         /// </summary>
         /// <param name="userID"></param>
         /// <param name="callback"></param>
+        [Obsolete]
         void GetUserInventory(UUID userID, InventoryReceiptCallback callback);
 
         /// <summary>