e797aecf619d60f6b0da91bec8c4cdc63e8a02c4
angie
  Wed Dec 7 10:21:24 2011 -0800
Feature #6172 (ISCA track tweaks): refinements suggested by b0b;also spotted the opportunity to link to the Human Phenotype Ontology
browser, helpful when the Phenotype is something like "Craniosynostosis".

diff --git src/hg/hgc/gvfClick.c src/hg/hgc/gvfClick.c
index 7593575..2e5ad74 100644
--- src/hg/hgc/gvfClick.c
+++ src/hg/hgc/gvfClick.c
@@ -75,25 +75,39 @@
 	    else
 		printf("<B>End range</B>: inner end %s, outer end %s<BR>\n",
 		       rangeStart, rangeEnd);
 	    }
 	else
 	    // not formatted as expected, just print as-is:
 	    printf("<B>%s</B>: %s<BR>\n", tag, htmlEncode(ba->attrVals[i]));
 	}
     // Parent sounds like mom or dad (as in var_origin)... tweak it too:
     else if (sameString(tag, "Parent"))
 	{
 	printf("<B>Variant region:</B> "
 	       "<A HREF=\"http://www.ncbi.nlm.nih.gov/dbvar/variants/%s/\" "
 	       "TARGET=_BLANK>%s</A><BR>\n", ba->attrVals[i], htmlEncode(ba->attrVals[i]));
 	}
+    else if (sameString(tag, "Name"))
+	{
+	char *url = trackDbSetting(tdb, "url");
+	// Show the Name only if it hasn't already appeared in the URL:
+	if (url == NULL || !stringIn("$$", url))
+	    printf("<B>%s</B>: %s<BR>\n", tag, htmlEncode(ba->attrVals[i]));
+	}
+    else if (sameWord(tag, "Phenotype_id") && startsWith("HPO:HP:", ba->attrVals[i]))
+	{
+	subChar(tag, '_', ' ');
+	printf("<B>%s</B>: <A HREF=\"http://www.berkeleybop.org/obo/%s\" "
+	       "TARGET=_BLANK>%s</A><BR>\n", tag, ba->attrVals[i]+strlen("HPO:"),
+	       htmlEncode(ba->attrVals[i]));
+	}
     else
 	{
 	subChar(tag, '_', ' ');
 	printf("<B>%s</B>: %s<BR>\n", tag, htmlEncode(ba->attrVals[i]));
 	}
     }
 sqlFreeResult(&sr);
 hFreeConn(&conn);
 /* printTrackHtml is done in genericClickHandlerPlus. */
 }