viewer#1619 Fix outfits search pulling up results sometimes

akleshchev · akleshchev · commit bd8438f70836 · 2024-06-07T12:07:40.000+03:00
Search forces a refresh - can search items that don't exist yet
Always notify parent of rect changes despite visibility (alterantively -
rearrange on open)
diff --git a/indra/newview/llinventoryitemslist.cpp b/indra/newview/llinventoryitemslist.cpp
@@ -115,7 +115,7 @@ void LLInventoryItemsList::doIdle()
 {
 	if (mRefreshState == REFRESH_COMPLETE) return;
 
-	if (isInVisibleChain() || mForceRefresh )
+	if (isInVisibleChain() || mForceRefresh || !getFilterSubString().empty())
 	{
 		refresh();
 
diff --git a/indra/newview/lloutfitslist.cpp b/indra/newview/lloutfitslist.cpp
@@ -567,7 +567,7 @@ void LLOutfitsList::onFilterSubStringChanged(const std::string& new_string, cons
 		LLWearableItemsList* list = dynamic_cast<LLWearableItemsList*>(tab->getAccordionView());
 		if (list)
 		{
-			list->setFilterSubString(new_string, tab->getDisplayChildren());
+			list->setFilterSubString(new_string, true);
 		}
 
 		if (old_string.empty())

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ void LLInventoryItemsList::doIdle()`
`115`	`115`	`{`
`116`	`116`	`if (mRefreshState == REFRESH_COMPLETE) return;`
`117`	`117`
`118`		`- if (isInVisibleChain() \|\| mForceRefresh )`
	`118`	`+ if (isInVisibleChain() \|\| mForceRefresh \|\| !getFilterSubString().empty())`
`119`	`119`	`{`
`120`	`120`	`refresh();`
`121`	`121`
Original file line number	Diff line number	Diff line change
`@@ -567,7 +567,7 @@ void LLOutfitsList::onFilterSubStringChanged(const std::string& new_string, cons`
`567`	`567`	`LLWearableItemsList* list = dynamic_cast<LLWearableItemsList*>(tab->getAccordionView());`
`568`	`568`	`if (list)`
`569`	`569`	`{`
`570`		`- list->setFilterSubString(new_string, tab->getDisplayChildren());`
	`570`	`+ list->setFilterSubString(new_string, true);`
`571`	`571`	`}`
`572`	`572`
`573`	`573`	`if (old_string.empty())`