refactor(omdb): cleanup-fix simplification pass (no behavior change)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: dmealing

server/java/metadata/src/main/java/com/metaobjects/util/DataConverter.java

@@ -53,7 +53,6 @@ public static Object toType( DataTypes dataType, Object val ) {
 
 			// Custom types are opaque to the generic converter; their JDBC binding is
 			// handled by the per-type codec (e.g. TimeCodec for TimeField/LocalTime).
-			// Pass the value through unchanged so the codec read path is not blocked.
 			case CUSTOM: return val;
 
 			default: throw new IllegalStateException( "Unknown type (" + dataType + "), cannot convert object [" + val + "]" );

server/java/metadata/src/test/java/com/metaobjects/util/DataConverterTests.java

@@ -333,8 +333,8 @@ public void testToStringArray() {
 
     /**
      * CUSTOM type is opaque to the generic converter; values must pass through unchanged.
-     * This unblocks TimeField (which uses DataTypes.CUSTOM) from the OMDB read path:
-     * TimeCodec.readInto calls MetaField.setObject → DataConverter.toType(CUSTOM, localTime).
+     * This allows per-type codecs (e.g. TimeCodec for TimeField) to handle their own
+     * JDBC binding without being blocked by the generic converter.
      */
     @Test
     public void testCustomTypePassesThrough() {

server/java/omdb/src/test/java/com/metaobjects/manager/db/validator/MetaClassDBValidatorServiceTest.java

@@ -1,12 +1,5 @@
 /*
  * Copyright (c) 2026 Doug Mealing LLC. All Rights Reserved.
- *
- * Fix B — verifies that MetaClassDBValidatorService.verifyMapping propagates
- * exceptions instead of swallowing them silently.
- *
- * Covers the bug where the empty catch block around validateDefinition() meant
- * that TableDoesNotExistException (thrown when autoCreate=false and the table is
- * absent) was discarded without any diagnostic or propagation.
  */
 package com.metaobjects.manager.db.validator;