Remove redundant addClass() calls

labkey-adam · labkey-adam · commit 497f4a63c330 · 2025-04-14T08:26:12.000-07:00
diff --git a/mGAP/src/org/labkey/mgap/query/PhenotypeVariantLinkDisplayColumnFactory.java b/mGAP/src/org/labkey/mgap/query/PhenotypeVariantLinkDisplayColumnFactory.java
@@ -9,7 +9,6 @@
 import org.labkey.api.query.DetailsURL;
 import org.labkey.api.query.FieldKey;
 import org.labkey.api.util.LinkBuilder;
-import org.labkey.api.util.PageFlowUtil;
 import org.labkey.api.writer.HtmlWriter;
 
 import java.util.List;
@@ -53,7 +52,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)
                 if (releaseId != null && omim != null)
                 {
                     DetailsURL url = DetailsURL.fromString("/mgap/variantList.view?release=" + releaseId + "&query.omim_phenotype~contains=" + omim, ContainerManager.getForId(containerId));
-                    out.write(LinkBuilder.labkeyLink("View Variants", url.getActionURL()).addClass("labkey-text-link"));
+                    out.write(LinkBuilder.labkeyLink("View Variants", url.getActionURL()));
                 }
             }
         };
diff --git a/mGAP/src/org/labkey/mgap/query/TracksPerReleaseGenomeBrowserDisplayColumnFactory.java b/mGAP/src/org/labkey/mgap/query/TracksPerReleaseGenomeBrowserDisplayColumnFactory.java
@@ -29,7 +29,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)
                 if (jbrowseId != null && trackName != null)
                 {
                     DetailsURL url = DetailsURL.fromString("/mgap/genomeBrowser.view?database=" + jbrowseId + "&activeTracks=" + trackName, ContainerManager.getForId(containerId));
-                    out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()).addClass("labkey-text-link"));
+                    out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()));
                 }
             }
         };
diff --git a/mGAP/src/org/labkey/mgap/query/VariantListJBrowseDisplayColumnFactory.java b/mGAP/src/org/labkey/mgap/query/VariantListJBrowseDisplayColumnFactory.java
@@ -13,7 +13,6 @@
 import org.labkey.api.query.FieldKey;
 import org.labkey.api.util.HtmlString;
 import org.labkey.api.util.LinkBuilder;
-import org.labkey.api.util.PageFlowUtil;
 import org.labkey.api.writer.HtmlWriter;
 
 import java.util.List;
@@ -69,15 +68,15 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)
                 if (jbrowseId != null)
                 {
                     DetailsURL url = DetailsURL.fromString("/jbrowse/browser.view?database=" + jbrowseId + "&location=" + contig + ":" + start + ".." + stop + "&highlight=" + contig + ":" + position + ".." + (position + length - 1), ContainerManager.getForId(containerId));
-                    out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()).addClass("labkey-text-link"));
+                    out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()));
                     delim = HtmlString.BR;
                 }
 
                 if (primaryTrack != null)
                 {
                     out.write(delim);
                     DetailsURL url = DetailsURL.fromString("/jbrowse/genotypeTable.view?trackId=" + primaryTrack + "&chr=" + contig + "&start=" + position + "&stop=" + position, ContainerManager.getForId(containerId));
-                    out.write(LinkBuilder.labkeyLink("View Genotypes At Position", url.getActionURL()).addClass("labkey-text-link"));
+                    out.write(LinkBuilder.labkeyLink("View Genotypes At Position", url.getActionURL()));
                     delim = HtmlString.BR;
                 }
 
diff --git a/mcc/src/org/labkey/mcc/query/RequestReviewActionsDisplayColumnFactory.java b/mcc/src/org/labkey/mcc/query/RequestReviewActionsDisplayColumnFactory.java
@@ -12,7 +12,6 @@
 import org.labkey.api.security.User;
 import org.labkey.api.security.UserManager;
 import org.labkey.api.util.LinkBuilder;
-import org.labkey.api.util.PageFlowUtil;
 import org.labkey.api.writer.HtmlWriter;
 import org.labkey.mcc.MccManager;
 
@@ -45,7 +44,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)
                 String requestId = ctx.get(getBoundKey("requestId"), String.class);
                 Container requestContainer = MccManager.get().getMCCRequestContainer(ctx.getContainer());
                 DetailsURL url = DetailsURL.fromString("/mcc/requestReview.view?requestId=" + requestId + "&mode=rabReview", requestContainer);
-                out.write(LinkBuilder.labkeyLink("Enter Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())).addClass("labkey-text-link"));
+                out.write(LinkBuilder.labkeyLink("Enter Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())));
             }
 
             @Override
diff --git a/mcc/src/org/labkey/mcc/query/RequestScoreActionsDisplayColumnFactory.java b/mcc/src/org/labkey/mcc/query/RequestScoreActionsDisplayColumnFactory.java
@@ -50,7 +50,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)
                         return;
                     }
 
-                    out.write(LinkBuilder.labkeyLink("Contact Investigator", "mailto:" + u.getEmail() + "?subject=MCC Request #" + requestRowId).addClass("labkey-text-link"));
+                    out.write(LinkBuilder.labkeyLink("Contact Investigator", "mailto:" + u.getEmail() + "?subject=MCC Request #" + requestRowId));
                 }
 
                 String status = ctx.get(getBoundKey("requestId", "status"), String.class);
@@ -78,7 +78,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)
                             {
                                 DetailsURL url = DetailsURL.fromString("/mcc/requestReview.view?requestId=" + requestId + "&mode=primaryReview", requestContainer);
                                 out.write(HtmlString.BR);
-                                out.write(LinkBuilder.labkeyLink("Enter MCC Internal Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())).addClass("labkey-text-link"));
+                                out.write(LinkBuilder.labkeyLink("Enter MCC Internal Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())));
                             }
                         }
                         else if (st == MccManager.RequestStatus.RabReview && ctx.get(FieldKey.fromString("pendingRabReviews"), Integer.class) == 0)
@@ -87,7 +87,7 @@ else if (st == MccManager.RequestStatus.RabReview && ctx.get(FieldKey.fromString
                             {
                                 DetailsURL url = DetailsURL.fromString("/mcc/requestReview.view?requestId=" + requestId + "&mode=resourceAvailability", requestContainer);
                                 out.write(HtmlString.BR);
-                                out.write(LinkBuilder.labkeyLink("Enter Resource Availability Assessment", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())).addClass("labkey-text-link"));
+                                out.write(LinkBuilder.labkeyLink("Enter Resource Availability Assessment", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())));
                             }
                         }
                         else if (st == MccManager.RequestStatus.PendingDecision)
@@ -96,7 +96,7 @@ else if (st == MccManager.RequestStatus.PendingDecision)
                             {
                                 DetailsURL url = DetailsURL.fromString("/mcc/requestReview.view?requestId=" + requestId + "&mode=finalReview", requestContainer);
                                 out.write(HtmlString.BR);
-                                out.write(LinkBuilder.labkeyLink("Enter Final Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())).addClass("labkey-text-link"));
+                                out.write(LinkBuilder.labkeyLink("Enter Final Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())));
                             }
                         }
                         else if (st == MccManager.RequestStatus.Approved)

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,6 @@`
`9`	`9`	`import org.labkey.api.query.DetailsURL;`
`10`	`10`	`import org.labkey.api.query.FieldKey;`
`11`	`11`	`import org.labkey.api.util.LinkBuilder;`
`12`		`-import org.labkey.api.util.PageFlowUtil;`
`13`	`12`	`import org.labkey.api.writer.HtmlWriter;`
`14`	`13`
`15`	`14`	`import java.util.List;`
`@@ -53,7 +52,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)`
`53`	`52`	`if (releaseId != null && omim != null)`
`54`	`53`	`{`
`55`	`54`	`DetailsURL url = DetailsURL.fromString("/mgap/variantList.view?release=" + releaseId + "&query.omim_phenotype~contains=" + omim, ContainerManager.getForId(containerId));`
`56`		`- out.write(LinkBuilder.labkeyLink("View Variants", url.getActionURL()).addClass("labkey-text-link"));`
	`55`	`+ out.write(LinkBuilder.labkeyLink("View Variants", url.getActionURL()));`
`57`	`56`	`}`
`58`	`57`	`}`
`59`	`58`	`};`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)`
`29`	`29`	`if (jbrowseId != null && trackName != null)`
`30`	`30`	`{`
`31`	`31`	`DetailsURL url = DetailsURL.fromString("/mgap/genomeBrowser.view?database=" + jbrowseId + "&activeTracks=" + trackName, ContainerManager.getForId(containerId));`
`32`		`- out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()).addClass("labkey-text-link"));`
	`32`	`+ out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()));`
`33`	`33`	`}`
`34`	`34`	`}`
`35`	`35`	`};`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,6 @@`
`13`	`13`	`import org.labkey.api.query.FieldKey;`
`14`	`14`	`import org.labkey.api.util.HtmlString;`
`15`	`15`	`import org.labkey.api.util.LinkBuilder;`
`16`		`-import org.labkey.api.util.PageFlowUtil;`
`17`	`16`	`import org.labkey.api.writer.HtmlWriter;`
`18`	`17`
`19`	`18`	`import java.util.List;`
`@@ -69,15 +68,15 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)`
`69`	`68`	`if (jbrowseId != null)`
`70`	`69`	`{`
`71`	`70`	`DetailsURL url = DetailsURL.fromString("/jbrowse/browser.view?database=" + jbrowseId + "&location=" + contig + ":" + start + ".." + stop + "&highlight=" + contig + ":" + position + ".." + (position + length - 1), ContainerManager.getForId(containerId));`
`72`		`- out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()).addClass("labkey-text-link"));`
	`71`	`+ out.write(LinkBuilder.labkeyLink("View In Genome Browser", url.getActionURL()));`
`73`	`72`	`delim = HtmlString.BR;`
`74`	`73`	`}`
`75`	`74`
`76`	`75`	`if (primaryTrack != null)`
`77`	`76`	`{`
`78`	`77`	`out.write(delim);`
`79`	`78`	`DetailsURL url = DetailsURL.fromString("/jbrowse/genotypeTable.view?trackId=" + primaryTrack + "&chr=" + contig + "&start=" + position + "&stop=" + position, ContainerManager.getForId(containerId));`
`80`		`- out.write(LinkBuilder.labkeyLink("View Genotypes At Position", url.getActionURL()).addClass("labkey-text-link"));`
	`79`	`+ out.write(LinkBuilder.labkeyLink("View Genotypes At Position", url.getActionURL()));`
`81`	`80`	`delim = HtmlString.BR;`
`82`	`81`	`}`
`83`	`82`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)`
`50`	`50`	`return;`
`51`	`51`	`}`
`52`	`52`
`53`		`- out.write(LinkBuilder.labkeyLink("Contact Investigator", "mailto:" + u.getEmail() + "?subject=MCC Request #" + requestRowId).addClass("labkey-text-link"));`
	`53`	`+ out.write(LinkBuilder.labkeyLink("Contact Investigator", "mailto:" + u.getEmail() + "?subject=MCC Request #" + requestRowId));`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`String status = ctx.get(getBoundKey("requestId", "status"), String.class);`
`@@ -78,7 +78,7 @@ public void renderGridCellContents(RenderContext ctx, HtmlWriter out)`
`78`	`78`	`{`
`79`	`79`	`DetailsURL url = DetailsURL.fromString("/mcc/requestReview.view?requestId=" + requestId + "&mode=primaryReview", requestContainer);`
`80`	`80`	`out.write(HtmlString.BR);`
`81`		`- out.write(LinkBuilder.labkeyLink("Enter MCC Internal Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())).addClass("labkey-text-link"));`
	`81`	`+ out.write(LinkBuilder.labkeyLink("Enter MCC Internal Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())));`
`82`	`82`	`}`
`83`	`83`	`}`
`84`	`84`	`else if (st == MccManager.RequestStatus.RabReview && ctx.get(FieldKey.fromString("pendingRabReviews"), Integer.class) == 0)`
`@@ -87,7 +87,7 @@ else if (st == MccManager.RequestStatus.RabReview && ctx.get(FieldKey.fromString`
`87`	`87`	`{`
`88`	`88`	`DetailsURL url = DetailsURL.fromString("/mcc/requestReview.view?requestId=" + requestId + "&mode=resourceAvailability", requestContainer);`
`89`	`89`	`out.write(HtmlString.BR);`
`90`		`- out.write(LinkBuilder.labkeyLink("Enter Resource Availability Assessment", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())).addClass("labkey-text-link"));`
	`90`	`+ out.write(LinkBuilder.labkeyLink("Enter Resource Availability Assessment", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())));`
`91`	`91`	`}`
`92`	`92`	`}`
`93`	`93`	`else if (st == MccManager.RequestStatus.PendingDecision)`
`@@ -96,7 +96,7 @@ else if (st == MccManager.RequestStatus.PendingDecision)`
`96`	`96`	`{`
`97`	`97`	`DetailsURL url = DetailsURL.fromString("/mcc/requestReview.view?requestId=" + requestId + "&mode=finalReview", requestContainer);`
`98`	`98`	`out.write(HtmlString.BR);`
`99`		`- out.write(LinkBuilder.labkeyLink("Enter Final Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())).addClass("labkey-text-link"));`
	`99`	`+ out.write(LinkBuilder.labkeyLink("Enter Final Review", url.getActionURL().addReturnUrl(ctx.getViewContext().getActionURL())));`
`100`	`100`	`}`
`101`	`101`	`}`
`102`	`102`	`else if (st == MccManager.RequestStatus.Approved)`