Skip to content

Commit 86bc41e

Browse files
committed
Small code cleanup
1 parent eb5408c commit 86bc41e

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

LDK/api-src/org/labkey/api/ldk/table/ContainerScopedTable.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,12 +124,12 @@ protected class UpdateService extends SimpleQueryUpdateService
124124
{
125125
private KeyManager _keyManager = new KeyManager();
126126

127-
public UpdateService(SimpleUserSchema.SimpleTable ti)
127+
public UpdateService(SimpleUserSchema.SimpleTable<?> ti)
128128
{
129129
super(ti, ti.getRealTable());
130130
}
131131

132-
public UpdateService(SimpleUserSchema.SimpleTable simpleTable, TableInfo table, DefaultQueryUpdateService.DomainUpdateHelper helper)
132+
public UpdateService(SimpleUserSchema.SimpleTable<?> simpleTable, TableInfo table, DefaultQueryUpdateService.DomainUpdateHelper helper)
133133
{
134134
super(simpleTable, table, helper);
135135
}

LDK/src/org/labkey/ldk/query/BuiltInColumnsCustomizer.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
package org.labkey.ldk.query;
22

3-
import org.apache.logging.log4j.LogManager;
43
import org.apache.logging.log4j.Logger;
54
import org.labkey.api.data.ColumnInfo;
65
import org.labkey.api.data.MutableColumnInfo;
76
import org.labkey.api.data.TableCustomizer;
87
import org.labkey.api.data.TableInfo;
98
import org.labkey.api.gwt.client.FacetingBehaviorType;
9+
import org.labkey.api.util.logging.LogHelper;
1010

1111
import java.sql.Timestamp;
1212

@@ -18,7 +18,7 @@
1818
*/
1919
public class BuiltInColumnsCustomizer implements TableCustomizer
2020
{
21-
private static final Logger _log = LogManager.getLogger(TableCustomizer.class);
21+
private static final Logger _log = LogHelper.getLogger(TableCustomizer.class, "Logs information from BuiltInColumnsCustomizer");
2222
private boolean _disableFacetingForNumericCols = true;
2323

2424
public BuiltInColumnsCustomizer()
@@ -136,6 +136,12 @@ private static void setNonEditable(MutableColumnInfo col)
136136

137137
public static void processColumn(MutableColumnInfo col)
138138
{
139+
if (col.isLocked())
140+
{
141+
_log.debug("BuiltInColumnsCustomizer was called on a locked column: " + col.getName(), new Exception());
142+
return;
143+
}
144+
139145
for (COL_ENUM colEnum : COL_ENUM.values())
140146
{
141147
if (colEnum.name().equalsIgnoreCase(col.getName()))

0 commit comments

Comments
 (0)