Fix dataclass attachment on update

Author: XingY

experiment/src/org/labkey/experiment/api/ExpDataClassDataTableImpl.java

@@ -134,6 +134,7 @@
 
 import java.io.IOException;
 import java.io.UncheckedIOException;
+import java.nio.file.Path;
 import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -1429,6 +1430,17 @@ protected Map<String, Object> updateRow(User user, Container container, Map<Stri
 
         }
 
+        // DataClassDataUpdateService needs to skip Attachment column convert before _update
+        // TODO: move override when implementing consolidating dataclass update methods
+        @Override
+        protected Object convertColumnValue(ColumnInfo col, Object value, User user, Container c, @Nullable Path fileLinkDirPath) throws ValidationException
+        {
+            if (PropertyType.ATTACHMENT == col.getPropertyType())
+                return value;
+
+            return super.convertColumnValue(col, value, user, c, fileLinkDirPath);
+        }
+
         @Override
         protected Map<String, Object> _update(User user, Container c, Map<String, Object> row, Map<String, Object> oldRow, Object[] keys) throws SQLException, ValidationException
         {