From 1c0cf8959b261e405d3656b6087d2e9dd3e1cbda Mon Sep 17 00:00:00 2001
From: yyy <1083599556@qq.com>
Date: 星期二, 12 五月 2026 21:09:48 +0800
Subject: [PATCH] Merge remote-tracking branch 'origin/1.0' into 1.0

---
 xp-robotaxi-common/xp-common-log/src/main/java/com/xiaopeng/common/log/aspect/AdminOperationLogAspect.java |   16 ++++------------
 1 files changed, 4 insertions(+), 12 deletions(-)

diff --git a/xp-robotaxi-common/xp-common-log/src/main/java/com/xiaopeng/common/log/aspect/AdminOperationLogAspect.java b/xp-robotaxi-common/xp-common-log/src/main/java/com/xiaopeng/common/log/aspect/AdminOperationLogAspect.java
index 2439520..f95d16e 100644
--- a/xp-robotaxi-common/xp-common-log/src/main/java/com/xiaopeng/common/log/aspect/AdminOperationLogAspect.java
+++ b/xp-robotaxi-common/xp-common-log/src/main/java/com/xiaopeng/common/log/aspect/AdminOperationLogAspect.java
@@ -59,16 +59,12 @@
     @Resource
     private AsyncLogService asyncLogService;
 
-    @Around("@annotation(com.xiaopeng.common.log.annotation.AdminOperationLog) || execution(public * com.xiaopeng..controller..*(..))")
+    @Around("@annotation(com.xiaopeng.common.log.annotation.AdminOperationLog)")
     public Object around(ProceedingJoinPoint joinPoint) throws Throwable {
         Method method = getMethod(joinPoint);
         AdminOperationLog annotation = method.getAnnotation(AdminOperationLog.class);
         ApiOperationMeta apiMeta = getApiOperationMeta(method, joinPoint.getTarget().getClass());
         AdminOperationType operationType = resolveOperationType(annotation, apiMeta, method, joinPoint.getArgs());
-        boolean shouldRecord = shouldRecord(annotation, apiMeta, operationType);
-        if (!shouldRecord) {
-            return joinPoint.proceed();
-        }
 
         Object result = null;
         Throwable error = null;
@@ -83,18 +79,14 @@
         }
     }
 
-    private boolean shouldRecord(AdminOperationLog annotation, ApiOperationMeta apiMeta, AdminOperationType operationType) {
-        if (annotation != null) {
-            return true;
-        }
-        return apiMeta.hasAdminTag() && operationType != AdminOperationType.OTHER;
-    }
-
     private void saveLog(ProceedingJoinPoint joinPoint, Method method, AdminOperationLog annotation,
                          ApiOperationMeta apiMeta, AdminOperationType operationType, Object result, Throwable error) {
         try {
             XpAdminOperationLog operationLog = new XpAdminOperationLog();
             AdminBusinessType businessType = resolveBusinessType(annotation, apiMeta);
+            if (businessType == AdminBusinessType.OTHER || operationType == AdminOperationType.OTHER) {
+                return;
+            }
             operationLog.setBusinessType(businessType.getCode());
             operationLog.setBusinessTypeName(businessType.getDesc());
             operationLog.setOperationType(operationType.getCode());

--
Gitblit v1.7.1