Clean up trigger/customizer layer code

Author: bbimber

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

@@ -118,13 +118,19 @@ private void setDetailsUrl(AbstractTableInfo ti)
         }
 
         String schemaName = ti.getUserSchema().getSchemaName();
-        assert schemaName != null;
-
         String queryName = ti.getPublicName();
-        assert queryName != null;
+        if (queryName == null)
+        {
+            _log.error("TableInfo.getPublicName() was null", new Exception());
+            return;
+        }
 
         List<String> keyFields = ti.getPkColumnNames();
-        assert !keyFields.isEmpty() : "No key fields found for the table: " + ti.getPublicSchemaName() + "." + ti.getPublicName();
+        if (keyFields.isEmpty())
+        {
+            _log.error("No key fields found for the table: " + ti.getPublicSchemaName() + "." + ti.getPublicName());
+            return;
+        }
 
         if (_settings.getPrimaryKeyField() != null)
         {
@@ -169,13 +175,19 @@ else if (_settings.isSetEditLinkOverrides())
         {
             //otherwise apply custom urls
             String schemaName = ti.getUserSchema().getSchemaName();
-            assert schemaName != null;
-
             String queryName = ti.getPublicName();
-            assert queryName != null;
+            if (queryName == null)
+            {
+                _log.error("TableInfo.getPublicName() was null", new Exception());
+                return;
+            }
 
             List<String> keyFields = ti.getPkColumnNames();
-            assert !keyFields.isEmpty() : "No key fields found for the table: " + ti.getPublicSchemaName() + "." + ti.getPublicName();
+            if (keyFields.isEmpty())
+            {
+                _log.error("No key fields found for the table: " + ti.getPublicSchemaName() + "." + ti.getPublicName());
+                return;
+            }
 
             if (_settings.getPrimaryKeyField() != null)
             {