Skip to content

Commit e0c5f37

Browse files
committed
Reduce logging
1 parent 7098baf commit e0c5f37

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

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

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ private void setDetailsUrl(AbstractTableInfo ti)
128128
List<String> keyFields = ti.getPkColumnNames();
129129
if (keyFields.isEmpty())
130130
{
131-
_log.error("No key fields found for the table: " + ti.getPublicSchemaName() + "." + ti.getPublicName());
131+
_log.debug("No key fields found for the table, cannot set details URL: " + ti.getPublicSchemaName() + "." + ti.getPublicName());
132132
return;
133133
}
134134

@@ -185,7 +185,13 @@ else if (_settings.isSetEditLinkOverrides())
185185
List<String> keyFields = ti.getPkColumnNames();
186186
if (keyFields.isEmpty())
187187
{
188-
_log.error("No key fields found for the table: " + ti.getPublicSchemaName() + "." + ti.getPublicName());
188+
// There does not seem to be a more direct test for 'is editable'
189+
if (ti.getUpdateService() == null)
190+
{
191+
return;
192+
}
193+
194+
_log.debug("No key fields found for the table, cannot customize edit UI: " + ti.getPublicSchemaName() + "." + ti.getPublicName());
189195
return;
190196
}
191197

0 commit comments

Comments
 (0)