#3757 Allow subfolders in "My Outfits"

1 files changed, 45 insertions, 2 deletions

diff --git a/indra/newview/lloutfitslist.cpp b/indra/newview/lloutfitslist.cpp
index 6e666b8a4b..cfbfe8f8e0 100644
--- a/indra/newview/lloutfitslist.cpp
+++ b/indra/newview/lloutfitslist.cpp
@@ -819,6 +819,49 @@ void LLOutfitListBase::observerCallback(const LLUUID& category_id)
     refreshList(category_id);
 }
 
+class LLIsOutfitListFolder : public LLInventoryCollectFunctor
+{
+public:
+    LLIsOutfitListFolder()
+    {
+        mOutfitsId = gInventory.findCategoryUUIDForType(LLFolderType::FT_MY_OUTFITS);
+    }
+    virtual ~LLIsOutfitListFolder() {}
+
+    bool operator()(LLInventoryCategory* cat, LLInventoryItem* item) override
+    {
+        if (cat)
+        {
+            if (cat->getPreferredType() == LLFolderType::FT_OUTFIT)
+            {
+                return true;
+            }
+            if (cat->getPreferredType() == LLFolderType::FT_NONE
+                && cat->getParentUUID() == mOutfitsId)
+            {
+                LLViewerInventoryCategory* inv_cat = dynamic_cast<LLViewerInventoryCategory*>(cat);
+                if (inv_cat && inv_cat->getDescendentCount() > 3)
+                {
+                    LLInventoryModel::cat_array_t* cats;
+                    LLInventoryModel::item_array_t* items;
+                    gInventory.getDirectDescendentsOf(inv_cat->getUUID(), cats, items);
+                    if (items->size() > 3) // eyes, skin, hair and shape are required
+                    {
+                        // For now assume this to be an old style outfit, not a subfolder
+                        // but ideally no such 'outfits' should be left in My Outfits
+                        // Todo: stop counting FT_NONE as outfits,
+                        // convert obvious outfits into FT_OUTFIT
+                        return true;
+                    }
+                }
+            }
+        }
+        return false;
+    }
+protected:
+    LLUUID mOutfitsId;
+};
+
 void LLOutfitListBase::refreshList(const LLUUID& category_id)
 {
     bool wasNull = mRefreshListState.CategoryUUID.isNull();
@@ -828,13 +871,13 @@ void LLOutfitListBase::refreshList(const LLUUID& category_id)
     LLInventoryModel::item_array_t item_array;
 
     // Collect all sub-categories of a given category.
-    LLIsType is_category(LLAssetType::AT_CATEGORY);
+    LLIsOutfitListFolder is_outfit;
     gInventory.collectDescendentsIf(
         category_id,
         cat_array,
         item_array,
         LLInventoryModel::EXCLUDE_TRASH,
-        is_category);
+        is_outfit);
 
     // Memorize item names for each UUID
     std::map<LLUUID, std::string> names;