Merge "Check for dynamic summaries at display time for search" into oc-dev am: 2046c59b77
am: 74eda7150b
Change-Id: Id496fc3d44f5103c65aa7b4de8ebcfab54f5de7b
This commit is contained in:
@@ -35,6 +35,8 @@ import com.android.settings.overlay.FeatureFactory;
|
||||
*/
|
||||
public abstract class SearchViewHolder extends RecyclerView.ViewHolder {
|
||||
|
||||
private final String DYNAMIC_PLACEHOLDER = "%s";
|
||||
|
||||
private final String mPlaceholderSummary;
|
||||
|
||||
public final TextView titleView;
|
||||
@@ -63,8 +65,10 @@ public abstract class SearchViewHolder extends RecyclerView.ViewHolder {
|
||||
|
||||
public void onBind(SearchFragment fragment, SearchResult result) {
|
||||
titleView.setText(result.title);
|
||||
// TODO (b/36101902) remove check for DYNAMIC_PLACEHOLDER
|
||||
if (TextUtils.isEmpty(result.summary)
|
||||
|| TextUtils.equals(result.summary, mPlaceholderSummary)) {
|
||||
|| TextUtils.equals(result.summary, mPlaceholderSummary)
|
||||
|| TextUtils.equals(result.summary, DYNAMIC_PLACEHOLDER)) {
|
||||
summaryView.setVisibility(View.GONE);
|
||||
} else {
|
||||
summaryView.setText(result.summary);
|
||||
|
@@ -158,6 +158,20 @@ public class IntentSearchViewHolderTest {
|
||||
assertThat(mHolder.summaryView.getVisibility()).isEqualTo(View.GONE);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testBindElements_dynamicSummary_visibilityIsGone() {
|
||||
String dynamicSummary = "%s";
|
||||
SearchResult result = new Builder()
|
||||
.addTitle(TITLE)
|
||||
.addSummary(dynamicSummary)
|
||||
.addPayload(new IntentPayload(null))
|
||||
.build();
|
||||
|
||||
mHolder.onBind(mFragment, result);
|
||||
|
||||
assertThat(mHolder.summaryView.getVisibility()).isEqualTo(View.GONE);
|
||||
}
|
||||
|
||||
private SearchResult getSearchResult(String title, String summary, Drawable icon) {
|
||||
Builder builder = new Builder();
|
||||
builder.setStableId(Objects.hash(title, summary, icon))
|
||||
|
Reference in New Issue
Block a user