From 4c96583701a79070ed3a8cdd5b59dbb0ddb46804 Mon Sep 17 00:00:00 2001
From: "weicw1996@qq.com" <weicw1996@qq.com>
Date: Mon, 21 Sep 2026 05:34:59 +0800
Subject: [PATCH] =?UTF-8?q?:bug:=20=E4=BF=AE=E5=A4=8D=E7=A0=81=E5=A4=B4?=
 =?UTF-8?q?=E5=AF=BC=E5=85=A5=E7=88=B6=E6=B8=AF=E5=8F=A3=E5=8F=96=E5=88=97?=
 =?UTF-8?q?=E4=B8=8D=E4=B8=80=E8=87=B4=E5=AF=BC=E8=87=B4=E7=9A=84=E9=9D=99?=
 =?UTF-8?q?=E9=BB=98=E4=B8=A2=E6=95=B0=E6=8D=AE?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

importTerminals 中 setParentCode 取「港口编码」列，而抑制连带报错时
却用「上级港口编码」判断父行是否已声明。用户若把完整编码填入
「码头编码」并让「港口编码」留空，父港口解析失败的异常会被误判为
「父行已声明」而吞掉，码头被静默丢弃且接口返回成功。

- splitByCategory 增加码头行「港口编码」非空校验，明确报错
- 抑制逻辑改为与 setParentCode 取同一列
---
 .../system/service/impl/PortTerminalServiceImpl.java  | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/blade-service/blade-system/src/main/java/org/springblade/system/service/impl/PortTerminalServiceImpl.java b/blade-service/blade-system/src/main/java/org/springblade/system/service/impl/PortTerminalServiceImpl.java
index e572dff..892f4d6 100644
--- a/blade-service/blade-system/src/main/java/org/springblade/system/service/impl/PortTerminalServiceImpl.java
+++ b/blade-service/blade-system/src/main/java/org/springblade/system/service/impl/PortTerminalServiceImpl.java
@@ -186,6 +186,13 @@ public class PortTerminalServiceImpl extends BaseServiceImpl<PortTerminalMapper,
 				failureList.add(buildFailure(data, index, "类型只能为港口或码头"));
 				continue;
 			}
+			// 码头行的「港口编码」既是编号前缀，也是"所属港口"的来源（见 importTerminals）。
+			// 缺失时下游解析不到父港口，而该异常又会被"父行已声明"的抑制逻辑吞掉，
+			// 导致码头被静默丢弃、接口却返回成功——所以必须在这里明确拦下。
+			if (CATEGORY_TERMINAL.equals(category) && Func.isEmpty(trimToEmpty(excel.getPortCode()))) {
+				failureList.add(buildFailure(data, index, "码头必须填写所属港口编码"));
+				continue;
+			}
 			// 只要类型是港口就登记为"本文件已声明"，即使它的编码格式不合法：
 			// 这样引用它的码头行不会被连带报错，用户只需修正这一个港口行。
 			if (CATEGORY_PORT.equals(category)) {
@@ -263,7 +270,9 @@ public class PortTerminalServiceImpl extends BaseServiceImpl<PortTerminalMapper,
 			} catch (ParentPortNotFoundException exception) {
 				// 上级港口就声明在本文件里，只是那一行自己校验失败（已被标红）。
 				// 此时码头行本身没有问题，不再连带报错，避免用户看到"两行都错"的假象。
-				if (!declaredPortCodeSet.contains(trimToEmpty(excel.getParentCode()).toUpperCase(Locale.ROOT))) {
+				// 注意这里必须与上面 setParentCode 取同一列（港口编码）；
+				// 若改用「上级港口编码」，父港口取不到时会被误判为"已声明"而吞掉异常。
+				if (!declaredPortCodeSet.contains(trimToEmpty(excel.getPortCode()).toUpperCase(Locale.ROOT))) {
 					failureList.add(buildFailure(data, index, resolveMessage(exception)));
 				}
 			} catch (Exception exception) {