diff --git a/visa/visa-server/src/main/java/cn/axzo/nanopart/visa/server/service/impl/ChangeRecordRelationServiceImpl.java b/visa/visa-server/src/main/java/cn/axzo/nanopart/visa/server/service/impl/ChangeRecordRelationServiceImpl.java
index e7610bf5..0456e0b6 100644
--- a/visa/visa-server/src/main/java/cn/axzo/nanopart/visa/server/service/impl/ChangeRecordRelationServiceImpl.java
+++ b/visa/visa-server/src/main/java/cn/axzo/nanopart/visa/server/service/impl/ChangeRecordRelationServiceImpl.java
@@ -358,11 +358,14 @@ public class ChangeRecordRelationServiceImpl extends ServiceImpl<ChangeRecordRel
         }
         if (CollectionUtils.isNotEmpty(req.getAttach())) {
             List<String> uploadAttachFileNameList = req.getAttach().stream().map(item -> item.getFileName()).collect(Collectors.toList());
-            if (CollectionUtils.isNotEmpty(uploadAttachFileNameList)) {
+            if (CollectionUtils.isNotEmpty(uploadAttachFileNameList)
+                    && Objects.nonNull(req.getOperatorPersonId())
+                    && Objects.nonNull(req.getOperatorOuId())
+                    && Objects.nonNull(req.getWorkspaceId())) {
                 changeRecordLogService.batchAddLog(req.getVisaId(), VisaLogTypeEnum.UPLOAD_FILE
                         , VisaLogParam.builder().personId(req.getOperatorPersonId())
                                 .ouId(req.getOperatorOuId())
-                                .workspaceId(req.getOperatorWorkspaceId())
+                                .workspaceId(req.getWorkspaceId())
                                 .uploadAttach(uploadAttachFileNameList).build());
             }