eb73131071efa47e196ca9b2d736d0157ef221fd kent Mon Dec 6 09:16:47 2021 -0800 Displaying an n/a rather than the empty string on facets with no data. diff --git src/hg/lib/tablesTables.c src/hg/lib/tablesTables.c index 6d3007b..dff300a 100644 --- src/hg/lib/tablesTables.c +++ src/hg/lib/tablesTables.c @@ -463,31 +463,32 @@ { char *op = "remove"; htmlDyStringPrintf(facetBar, "<dd class=\"facet\" style='display: inline-block;'>\n"); htmlDyStringPrintf(facetBar, "<input type=checkbox value=%s class=ttFsCheckBox %s> ", specificallySelected ? "true" : "false", specificallySelected ? "checked" : ""); htmlDyStringPrintf(facetBar, "<a href='%s" "&%s_facet_op=%s|url|" "&%s_facet_fieldName=%s|url|" "&%s_facet_fieldVal=%s|url|" "&%s_page=1'" ">", returnUrl, varPrefix, op, varPrefix, field->fieldName, varPrefix, val->val, varPrefix ); - htmlDyStringPrintf(facetBar, "%s (%d)</a>", val->val, val->selectCount); + htmlDyStringPrintf(facetBar, "%s (%d)</a>", + naForEmpty(val->val), val->selectCount); htmlDyStringPrintf(facetBar, "</dd>\n"); } } slFreeList(&valListCopy); htmlDyStringPrintf(facetBar, "</span></span>\n"); } } if (!isEmpty(where) || gotSelected) { printf("<div>\n"); } @@ -600,31 +601,31 @@ extraAnchorPos= 0; } printf(">\n"); htmlPrintf("<input type=checkbox value=%s class=ttFsCheckBox %s> ", specificallySelected ? "true" : "false", specificallySelected ? "checked" : ""); htmlPrintf("<a href='%s" "&%s_facet_op=%s|none|" "&%s_facet_fieldName=%s|url|" "&%s_facet_fieldVal=%s|url|" "&%s_page=1#%s' " ">", returnUrl, varPrefix, op, varPrefix, field->fieldName, varPrefix, val->val, varPrefix, selfId ); - htmlPrintf("%s (%d)</a>", val->val, val->selectCount); + htmlPrintf("%s (%d)</a>", naForEmpty(val->val), val->selectCount); printf("</dd>\n"); } else if (val->selectCount > 0) { ++valuesNotShown; } } // show "See More" link when facet has lots of values if (valuesNotShown > 0) { char *op = "showAllValues"; htmlPrintf("<dd><a href='%s" "&%s_facet_op=%s|url|" "&%s_facet_fieldName=%s|url|"