Skip to content

Commit 400c385

Browse files
committed
Merge remote-tracking branch 'origin/develop' into fb_lineageStatusFilter
2 parents 3b87d4c + d41c8b1 commit 400c385

2 files changed

Lines changed: 5 additions & 2 deletions

File tree

experiment/src/org/labkey/experiment/api/ExpDataImpl.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@
7070
import org.labkey.api.util.NetworkDrive;
7171
import org.labkey.api.util.Pair;
7272
import org.labkey.api.util.Path;
73+
import org.labkey.api.util.PageFlowUtil;
7374
import org.labkey.api.util.StringUtilsLabKey;
7475
import org.labkey.api.util.URLHelper;
7576
import org.labkey.api.util.InputBuilder;
@@ -939,7 +940,8 @@ private void appendParam(StringBuilder sb, @Nullable String dataclass, @Nullable
939940
}
940941
else
941942
{
942-
sb.append(label);
943+
// GitHub Kanban #1946: encode the label (a DataClass name); the if-branch above already encodes via LinkBuilder.simpleLink.
944+
sb.append(PageFlowUtil.filter(label));
943945
}
944946

945947
sb.append("</span> ");

issues/src/org/labkey/issue/actions/ChangeSummary.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,8 @@ static public ChangeSummary createChangeSummary(ViewContext context, IssueListDe
168168
summary += " of " + duplicateOf.getIssueId();
169169
}
170170

171-
sbHTMLChanges.append("<b>").append(summary);
171+
// GitHub Kanban #1946: encode summary, which includes the issue resolution text
172+
sbHTMLChanges.append("<b>").append(PageFlowUtil.filter(summary));
172173
sbHTMLChanges.append("</b><br>\n");
173174
}
174175

0 commit comments

Comments
 (0)