Skip to content

Commit 10abf30

Browse files
Improve generics and other warnings
1 parent 59096b7 commit 10abf30

3 files changed

Lines changed: 9 additions & 8 deletions

File tree

flow/src/org/labkey/flow/controllers/FlowController.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ public ModelAndView getView(Object o, BindException errors)
164164
run = schema.getRun();
165165
// script = schema.getScript();
166166

167-
QueryView view = schema.createView(getViewContext(), settings);
167+
QueryView view = schema.createView(getViewContext(), settings, errors);
168168
if (view.getQueryDef() == null)
169169
{
170170
throw new NotFoundException("Query definition '" + settings.getQueryName() + "' in flow schema not found");

flow/src/org/labkey/flow/controllers/editscript/CompensationCalculationPage.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
package org.labkey.flow.controllers.editscript;
1818

1919
import org.apache.commons.lang3.StringUtils;
20+
import org.apache.commons.lang3.Strings;
2021
import org.fhcrc.cpas.flow.script.xml.ChannelDef;
2122
import org.fhcrc.cpas.flow.script.xml.ChannelSubsetDef;
2223
import org.fhcrc.cpas.flow.script.xml.CompensationCalculationDef;
@@ -263,14 +264,14 @@ public SelectBuilder selectKeywordValues(Sign sign, int index)
263264

264265
private boolean subsetNameMatches(String subsetUser, String subsetWorkspace, Sign sign, String channel)
265266
{
266-
if (StringUtils.equals(subsetUser, subsetWorkspace))
267+
if (Strings.CS.equals(subsetUser, subsetWorkspace))
267268
return true;
268-
if (StringUtils.equals(subsetUser, channel + subsetWorkspace))
269+
if (Strings.CS.equals(subsetUser, channel + subsetWorkspace))
269270
return true;
270271
String strSign = sign == Sign.positive ? "+" : "-";
271-
if (StringUtils.equals(subsetUser, channel + strSign + subsetWorkspace))
272+
if (Strings.CS.equals(subsetUser, channel + strSign + subsetWorkspace))
272273
return true;
273-
if (StringUtils.equals(subsetUser, strSign + subsetWorkspace))
274+
if (Strings.CS.equals(subsetUser, strSign + subsetWorkspace))
274275
return true;
275276
return false;
276277
}

microarray/src/org/labkey/microarray/matrix/ExpressionMatrixAssayProvider.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616

1717
package org.labkey.microarray.matrix;
1818

19-
import org.apache.commons.lang3.StringUtils;
19+
import org.apache.commons.lang3.Strings;
2020
import org.fhcrc.cpas.exp.xml.ExperimentRunType;
2121
import org.fhcrc.cpas.exp.xml.SimpleTypeNames;
2222
import org.fhcrc.cpas.exp.xml.SimpleValueType;
@@ -234,7 +234,7 @@ public void beforeXarExportRun(ExpRun run, ExperimentRunType xrun)
234234
}
235235
for (SimpleValueType sv : xrun.getProperties().getSimpleValArray())
236236
{
237-
if (StringUtils.equals("featureSet",sv.getName()))
237+
if (Strings.CS.equals("featureSet",sv.getName()))
238238
{
239239
int featureAnnotationSetRowId = (int)Float.parseFloat(sv.getStringValue());
240240
String featureAnnotationSetName = mapRowIdName.get(featureAnnotationSetRowId);
@@ -267,7 +267,7 @@ public void beforeXarImportRun(ExperimentRunType xrun)
267267
}
268268
for (SimpleValueType sv : xrun.getProperties().getSimpleValArray())
269269
{
270-
if (StringUtils.equals(FEATURE_SET_PROPERTY_NAME,sv.getName()))
270+
if (Strings.CS.equals(FEATURE_SET_PROPERTY_NAME,sv.getName()))
271271
{
272272
String featureAnnotationSetName = sv.getStringValue();
273273
Long featureAnnotationRowId = mapNameRowId.get(featureAnnotationSetName);

0 commit comments

Comments
 (0)