From 4d9d8d22e3666076988c30afb4e7c6fe365c19aa Mon Sep 17 00:00:00 2001
From: chenluhua1980 <Chenluhua@qq.com>
Date: 星期二, 06 一月 2026 18:53:04 +0800
Subject: [PATCH] 1.修复一个回复错误的问题;

---
 SourceCode/Bond/Servo/HsmsPassive.cpp | 1121 +++++++++++++++++++++++++++++++++++++++++++++++++++++-----
 1 files changed, 1,023 insertions(+), 98 deletions(-)

diff --git a/SourceCode/Bond/Servo/HsmsPassive.cpp b/SourceCode/Bond/Servo/HsmsPassive.cpp
index e96e8a8..da6e106 100644
--- a/SourceCode/Bond/Servo/HsmsPassive.cpp
+++ b/SourceCode/Bond/Servo/HsmsPassive.cpp
@@ -8,6 +8,8 @@
 #include <time.h>  
 #include <stdlib.h>  
 #include <string.h>  
+#include <algorithm>
+#include <set>
 #include <regex>
 
 
@@ -151,23 +153,47 @@
 
 void CHsmsPassive::unlinkEventReport(unsigned int CEID)
 {
+	LOGI("<CHsmsPassive>unlinkEventReport enter");
 	SERVO::CCollectionEvent* pEvent = getEvent(CEID);
 	if (pEvent != nullptr) {
 		pEvent->setReport(nullptr);
+		LOGI("<CHsmsPassive>unlink Event Report.CEID=%d", CEID);
 	}
+}
+
+bool CHsmsPassive::shouldSpool(uint8_t streamId, uint8_t functionId) const
+{
+	// Comment: stream 1 is not spooled
+	if (streamId == 1) return false;
+
+	// Comment: m=0 turns off all streams and fns
+	if (!m_spoolingEnabled) return false;
+
+	// Blacklist semantics: in map => do NOT spool/cache.
+	// Not in map => allow spooling by default.
+	auto it = m_spoolBlacklistByStream.find(streamId);
+	if (it == m_spoolBlacklistByStream.end()) return true;
+
+	// Empty set => all functions in this stream
+	if (it->second.empty()) return true;
+
+	return it->second.find(functionId) == it->second.end();
 }
 
 SERVO::CReport* CHsmsPassive::defineReport(unsigned int RPTID, std::vector<unsigned int>& vids)
 {
+	LOGI("<CHsmsPassive>defineReport enter");
 	// 娣诲姞瀹氫箟report
 	SERVO::CReport* pReport = new SERVO::CReport(RPTID, vids);
 	for (auto vid : vids) {
 		SERVO::CVariable* pVariable = getVariable(vid);
 		if (pVariable != nullptr) {
 			pReport->addVariable(pVariable);
+			LOGI("<CHsmsPassive>defineReport RPTID=%d", RPTID);
 		}
 	}
 	m_reports.push_back(pReport);
+	writeReportsToFile(m_strReportFilepath);
 
 	return pReport;
 }
@@ -191,7 +217,7 @@
 
 int CHsmsPassive::onRecvMsg(IMessage* pMessage)
 {
-	LOGI("onRecvMsg:%s", pMessage->toString());
+	// LOGI("onRecvMsg:%s", pMessage->toString());
 	Lock();
 	if (m_pActiveAction != nullptr &&
 		(m_pActiveAction->getSendMessage()->getHeader()->systemBytes == pMessage->getHeader()->systemBytes)) {
@@ -228,40 +254,133 @@
 
 int CHsmsPassive::loadVarialbles(const char* pszFilepath)
 {
-	CStdioFile file;
-	if (!file.Open(pszFilepath, CFile::modeRead)) {
+	m_strVariableFilepath = pszFilepath;
+	m_bVariableUtf8 = false;
+	m_bVariableUtf8Bom = false;
+	// 鍏堣鍘熷瀛楄妭锛屽悗缁啀鎸� UTF-8/BOM 鎴栨湰鍦扮紪鐮佽浆鎹�
+	CFile file;
+	if (!file.Open(pszFilepath, CFile::modeRead | CFile::shareDenyNone)) {
 		return -1;
 	}
 
-	std::regex pattern("^\\d+,.*");  // 鍖归厤浠ユ暟瀛�+閫楀彿寮�澶寸殑瀛楃涓�
+	const ULONGLONG nLen = file.GetLength();
+	if (nLen == 0) {
+		return -1;
+	}
+
+	std::string buffer;
+	buffer.resize(static_cast<size_t>(nLen));
+	file.Read(buffer.data(), static_cast<UINT>(nLen));
+	file.Close();
+
+	const unsigned char* bytes = reinterpret_cast<const unsigned char*>(buffer.data());
+	size_t offset = 0;
+	CStringW content;
+
+	// UTF-8 BOM
+	if (nLen >= 3 && bytes[0] == 0xEF && bytes[1] == 0xBB && bytes[2] == 0xBF) {
+		offset = 3;
+		m_bVariableUtf8 = true;
+		m_bVariableUtf8Bom = true;
+	}
+
+	// UTF-16 LE BOM
+	if (nLen >= 2 && bytes[0] == 0xFF && bytes[1] == 0xFE) {
+		const wchar_t* wdata = reinterpret_cast<const wchar_t*>(buffer.data() + 2);
+		const size_t wlen = (nLen - 2) / sizeof(wchar_t);
+		content.SetString(wdata, static_cast<int>(wlen));
+	}
+	// UTF-16 BE BOM
+	else if (nLen >= 2 && bytes[0] == 0xFE && bytes[1] == 0xFF) {
+		const size_t wlen = (nLen - 2) / sizeof(wchar_t);
+		std::wstring temp;
+		temp.reserve(wlen);
+		for (size_t i = 0; i < wlen; ++i) {
+			wchar_t ch = static_cast<wchar_t>(bytes[2 + i * 2] << 8 | bytes[2 + i * 2 + 1]);
+			temp.push_back(ch);
+		}
+		content = temp.c_str();
+	}
+	// 灏濊瘯 UTF-8锛堝惈鏃� BOM锛�
+	else {
+		auto tryUtf8 = [&](size_t off) -> bool {
+			int need = MultiByteToWideChar(CP_UTF8, MB_ERR_INVALID_CHARS, buffer.data() + off,
+				static_cast<int>(buffer.size() - off), nullptr, 0);
+			if (need <= 0) return false;
+			std::wstring temp;
+			temp.resize(need);
+			MultiByteToWideChar(CP_UTF8, 0, buffer.data() + off,
+				static_cast<int>(buffer.size() - off), temp.data(), need);
+			content = temp.c_str();
+			m_bVariableUtf8 = true;
+			return true;
+		};
+
+		if (!tryUtf8(offset)) {
+			// 鍥為��鍒版湰鍦颁唬鐮侀〉
+			int need = MultiByteToWideChar(CP_ACP, 0, buffer.data(), static_cast<int>(buffer.size()), nullptr, 0);
+			if (need > 0) {
+				std::wstring temp;
+				temp.resize(need);
+				MultiByteToWideChar(CP_ACP, 0, buffer.data(), static_cast<int>(buffer.size()), temp.data(), need);
+				content = temp.c_str();
+			}
+		}
+	}
+
+	if (content.IsEmpty()) {
+		return -1;
+	}
+
+	std::wregex pattern(L"^\\d+,.+");  // 鍖归厤浠ユ暟瀛�+閫楀彿寮�澶寸殑瀛楃涓�
 	std::vector<SERVO::CVariable*> variables;
 	int index, last;
-	CString strLine;
-	CString strId, strName, strFormat, strRemark;
-	while (file.ReadString(strLine)) {
-		if (!std::regex_match((LPTSTR)(LPCTSTR)strLine, pattern)) {
+	CStringW strLine;
+	CStringW strId, strName, strFormat, strRemark;
+	std::wstringstream ss(content.GetString());
+	auto narrowFromW = [](const CStringW& s) -> std::string {
+		int need = WideCharToMultiByte(CP_ACP, 0, s, -1, nullptr, 0, nullptr, nullptr);
+		if (need <= 0) return {};
+		std::string out(static_cast<size_t>(need - 1), '\0');
+		WideCharToMultiByte(CP_ACP, 0, s, -1, out.data(), need, nullptr, nullptr);
+		return out;
+	};
+	std::wstring line;
+	while (std::getline(ss, line, L'\n')) {
+		strLine = line.c_str();
+		strLine.Trim();
+		if (strLine.IsEmpty()) continue;
+		if (!std::regex_match(static_cast<LPCWSTR>(strLine), pattern)) {
 			continue;
 		}
 
 		last = 0;
-		index = strLine.Find(",", last);
+		index = strLine.Find(L",", last);
 		if (index < 0) continue;
 		strId = strLine.Left(index);
 		last = index + 1;
 
-		index = strLine.Find(",", last);
+		index = strLine.Find(L",", last);
 		if (index < 0) continue;
 		strName = strLine.Mid(last, index - last);
 		last = index + 1;
 
-		index = strLine.Find(",", last);
+		index = strLine.Find(L",", last);
 		if (index < 0) continue;
 		strFormat = strLine.Mid(last, index - last);
 		strRemark = strLine.Right(strLine.GetLength() - index - 1);
-		strRemark.Replace(_T("\\r\\n"), _T("\r\n"));
+		strRemark.Replace(L"\\r\\n", L"\r\n");
+
+		std::string sId = narrowFromW(strId);
+		std::string sName = narrowFromW(strName);
+		std::string sFormat = narrowFromW(strFormat);
+		std::string sRemark = narrowFromW(strRemark);
 
 		SERVO::CVariable* pVarialble = new SERVO::CVariable(
-			(LPTSTR)(LPCTSTR)strId, (LPTSTR)(LPCTSTR)strName, (LPTSTR)(LPCTSTR)strFormat, (LPTSTR)(LPCTSTR)strRemark);
+			sId.c_str(),
+			sName.c_str(),
+			sFormat.c_str(),
+			sRemark.c_str());
 		variables.push_back(pVarialble);
 	}
 
@@ -272,14 +391,23 @@
 		}
 	}
 
-
-	file.Close();
 	return 0;
 }
 
 std::vector<SERVO::CVariable*>& CHsmsPassive::getVariables()
 {
 	return m_variabels;
+}
+
+unsigned int CHsmsPassive::getMaxVariableId() const
+{
+	unsigned int maxId = 0;
+	for (auto item : m_variabels) {
+		if (item && item->getVarialbleId() > maxId) {
+			maxId = item->getVarialbleId();
+		}
+	}
+	return maxId;
 }
 
 SERVO::CVariable* CHsmsPassive::getVariable(int variableId)
@@ -312,6 +440,59 @@
 	m_variabels.clear();
 }
 
+CStringA WideToUtf8(const CStringW& ws)
+{
+	int need = WideCharToMultiByte(CP_UTF8, 0, ws, -1, nullptr, 0, nullptr, nullptr);
+	if (need <= 0) return "";
+	CStringA out;
+	LPSTR buf = out.GetBufferSetLength(need - 1);
+	WideCharToMultiByte(CP_UTF8, 0, ws, -1, buf, need, nullptr, nullptr);
+	out.ReleaseBuffer();
+	return out;
+}
+
+CStringA WideToAnsi(const CStringW& ws)
+{
+	int need = WideCharToMultiByte(CP_ACP, 0, ws, -1, nullptr, 0, nullptr, nullptr);
+	if (need <= 0) return "";
+	CStringA out;
+	LPSTR buf = out.GetBufferSetLength(need - 1);
+	WideCharToMultiByte(CP_ACP, 0, ws, -1, buf, need, nullptr, nullptr);
+	out.ReleaseBuffer();
+	return out;
+}
+
+static CStringA AnsiToUtf8(const std::string& s)
+{
+	int wlen = MultiByteToWideChar(CP_ACP, 0, s.c_str(), -1, nullptr, 0);
+	if (wlen <= 0) return "";
+	CStringW ws;
+	LPWSTR wbuf = ws.GetBufferSetLength(wlen - 1);
+	MultiByteToWideChar(CP_ACP, 0, s.c_str(), -1, wbuf, wlen);
+	ws.ReleaseBuffer();
+	return WideToUtf8(ws);
+}
+
+int CHsmsPassive::deleteVariable(int variableId)
+{
+	Lock();
+	auto it = std::find_if(m_variabels.begin(), m_variabels.end(), [=](SERVO::CVariable* v) {
+		return v != nullptr && v->getVarialbleId() == variableId;
+		});
+	if (it == m_variabels.end()) {
+		Unlock();
+		return -1;
+	}
+	delete *it;
+	m_variabels.erase(it);
+	auto filepath = m_strVariableFilepath;
+	Unlock();
+
+	if (filepath.empty()) return -2;
+
+	return writeVariablesToFile(filepath);
+}
+
 void CHsmsPassive::setVariableValue(const char* pszName, __int64 value)
 {
 	auto v = getVariable(pszName);
@@ -336,32 +517,221 @@
 	}
 }
 
+static bool isValidFormat(const std::string& fmt)
+{
+	static const std::set<std::string> allow = { "U1","U2","I2","A20","A50","L" };
+	return allow.count(fmt) > 0;
+}
+
+int CHsmsPassive::addVariable(const char* pszName, const char* pszFormat, const char* pszRemark, int& outId)
+{
+	if (pszName == nullptr || pszFormat == nullptr) return -1;
+	std::string fmt = pszFormat;
+	std::transform(fmt.begin(), fmt.end(), fmt.begin(), ::toupper);
+	if (!isValidFormat(fmt)) return -2;
+
+	Lock();
+	int maxId = 0;
+	for (auto v : m_variabels) {
+		if (v != nullptr && v->getVarialbleId() > maxId) {
+			maxId = v->getVarialbleId();
+		}
+	}
+	outId = maxId + 1;
+
+	SERVO::CVariable* pNew = new SERVO::CVariable(std::to_string(outId).c_str(), pszName, fmt.c_str(), pszRemark ? pszRemark : "");
+	m_variabels.push_back(pNew);
+	auto filepath = m_strVariableFilepath;
+	Unlock();
+
+	if (filepath.empty()) return -3;
+	return writeVariablesToFile(filepath);
+}
+
+int CHsmsPassive::updateVariable(int variableId, const char* pszName, const char* pszFormat, const char* pszRemark)
+{
+	if (pszName == nullptr || pszFormat == nullptr) return -1;
+	std::string fmt = pszFormat;
+	std::transform(fmt.begin(), fmt.end(), fmt.begin(), ::toupper);
+	if (!isValidFormat(fmt)) return -2;
+
+	Lock();
+	auto it = std::find_if(m_variabels.begin(), m_variabels.end(), [=](SERVO::CVariable* v) {
+		return v != nullptr && v->getVarialbleId() == variableId;
+		});
+	if (it == m_variabels.end()) {
+		Unlock();
+		return -4;
+	}
+	(*it)->setName(pszName);
+	(*it)->setFormat(fmt.c_str());
+	(*it)->setRemark(pszRemark ? pszRemark : "");
+	auto filepath = m_strVariableFilepath;
+	Unlock();
+
+	if (filepath.empty()) return -3;
+	return writeVariablesToFile(filepath);
+}
+
+int CHsmsPassive::writeVariablesToFile(const std::string& filepath)
+{
+	if (filepath.empty()) return -3;
+
+	CFile file;
+	if (!file.Open(filepath.c_str(), CFile::modeCreate | CFile::modeWrite | CFile::shareDenyNone)) {
+		return -3;
+	}
+
+	// header
+	const std::string headerAnsi = "SVID,SV Name,SV Format,SV Remark\r\n";
+	if (m_bVariableUtf8) {
+		if (m_bVariableUtf8Bom) {
+			const BYTE bom[3] = { 0xEF, 0xBB, 0xBF };
+			file.Write(bom, 3);
+		}
+		CStringA header = AnsiToUtf8(headerAnsi);
+		file.Write(header.GetString(), header.GetLength());
+	}
+	else {
+		file.Write(headerAnsi.data(), (UINT)headerAnsi.size());
+	}
+
+	for (auto v : m_variabels) {
+		if (v == nullptr) continue;
+		std::string lineAnsi;
+		lineAnsi.reserve(256);
+		lineAnsi += std::to_string(v->getVarialbleId());
+		lineAnsi.push_back(',');
+		lineAnsi += v->getName();
+		lineAnsi.push_back(',');
+		lineAnsi += SERVO::CVariable::formatToString(v->getFormat());
+		lineAnsi.push_back(',');
+		lineAnsi += v->getRemark();
+		lineAnsi.append("\r\n");
+
+		if (m_bVariableUtf8) {
+			CStringA outLine = AnsiToUtf8(lineAnsi);
+			file.Write(outLine.GetString(), outLine.GetLength());
+		}
+		else {
+			file.Write(lineAnsi.data(), (UINT)lineAnsi.size());
+		}
+	}
+	file.Close();
+
+	return 0;
+}
+
 int CHsmsPassive::loadReports(const char* pszFilepath)
 {
-	CStdioFile file;
-	if (!file.Open(pszFilepath, CFile::modeRead)) {
+	m_strReportFilepath = pszFilepath;
+	m_bReportUtf8 = false;
+	m_bReportUtf8Bom = false;
+	// 鍏煎 UTF-8/BOM 涓庢湰鍦扮紪鐮佽鍙�
+	CFile file;
+	if (!file.Open(pszFilepath, CFile::modeRead | CFile::shareDenyNone)) {
 		return -1;
 	}
 
-	std::regex pattern("^\\d+,\\(\\d+(,\\d+)*\\).*");  // 鍖归厤浠ユ暟瀛�+閫楀彿寮�澶寸殑瀛楃涓�
+	const ULONGLONG nLen = file.GetLength();
+	if (nLen == 0) {
+		return -1;
+	}
+
+	std::string buffer;
+	buffer.resize(static_cast<size_t>(nLen));
+	file.Read(buffer.data(), static_cast<UINT>(nLen));
+	file.Close();
+
+	const unsigned char* bytes = reinterpret_cast<const unsigned char*>(buffer.data());
+	size_t offset = 0;
+	CStringW content;
+
+	// UTF-8 BOM
+	if (nLen >= 3 && bytes[0] == 0xEF && bytes[1] == 0xBB && bytes[2] == 0xBF) {
+		offset = 3;
+		m_bReportUtf8 = true;
+		m_bReportUtf8Bom = true;
+	}
+
+	// UTF-16 LE BOM
+	if (nLen >= 2 && bytes[0] == 0xFF && bytes[1] == 0xFE) {
+		const wchar_t* wdata = reinterpret_cast<const wchar_t*>(buffer.data() + 2);
+		const size_t wlen = (nLen - 2) / sizeof(wchar_t);
+		content.SetString(wdata, static_cast<int>(wlen));
+	}
+	// UTF-16 BE BOM
+	else if (nLen >= 2 && bytes[0] == 0xFE && bytes[1] == 0xFF) {
+		const size_t wlen = (nLen - 2) / sizeof(wchar_t);
+		std::wstring temp;
+		temp.reserve(wlen);
+		for (size_t i = 0; i < wlen; ++i) {
+			wchar_t ch = static_cast<wchar_t>(bytes[2 + i * 2] << 8 | bytes[2 + i * 2 + 1]);
+			temp.push_back(ch);
+		}
+		content = temp.c_str();
+	}
+	else {
+		auto tryUtf8 = [&](size_t off) -> bool {
+			int need = MultiByteToWideChar(CP_UTF8, MB_ERR_INVALID_CHARS, buffer.data() + off,
+				static_cast<int>(buffer.size() - off), nullptr, 0);
+			if (need <= 0) return false;
+			std::wstring temp;
+			temp.resize(need);
+			MultiByteToWideChar(CP_UTF8, 0, buffer.data() + off,
+				static_cast<int>(buffer.size() - off), temp.data(), need);
+			content = temp.c_str();
+			m_bReportUtf8 = true;
+			return true;
+		};
+
+		if (!tryUtf8(offset)) {
+			int need = MultiByteToWideChar(CP_ACP, 0, buffer.data(), static_cast<int>(buffer.size()), nullptr, 0);
+			if (need > 0) {
+				std::wstring temp;
+				temp.resize(need);
+				MultiByteToWideChar(CP_ACP, 0, buffer.data(), static_cast<int>(buffer.size()), temp.data(), need);
+				content = temp.c_str();
+			}
+		}
+	}
+
+	if (content.IsEmpty()) {
+		return -1;
+	}
+
+	std::wregex pattern(L"^\\d+,\\(\\d+(,\\d+)*\\).*");  // 鍖归厤浠ユ暟瀛�+閫楀彿寮�澶寸殑瀛楃涓�
 	std::vector<SERVO::CReport*> reports;
 	int index;
-	CString strLine, strVariable;
-	CString strId;
-	while (file.ReadString(strLine)) {
-		if (!std::regex_match((LPTSTR)(LPCTSTR)strLine, pattern)) {
+	CStringW strLine, strVariable;
+	CStringW strId;
+	std::wstringstream ss(content.GetString());
+	auto narrowFromW = [](const CStringW& s) -> std::string {
+		int need = WideCharToMultiByte(CP_ACP, 0, s, -1, nullptr, 0, nullptr, nullptr);
+		if (need <= 0) return {};
+		std::string out(static_cast<size_t>(need - 1), '\0');
+		WideCharToMultiByte(CP_ACP, 0, s, -1, out.data(), need, nullptr, nullptr);
+		return out;
+	};
+	std::wstring line;
+	while (std::getline(ss, line, L'\n')) {
+		strLine = line.c_str();
+		strLine.Trim();
+		if (strLine.IsEmpty()) continue;
+		if (!std::regex_match(static_cast<LPCWSTR>(strLine), pattern)) {
 			continue;
 		}
 
-		index = strLine.Find(",", 0);
+		index = strLine.Find(L",", 0);
 		if (index < 0) continue;
 		strId = strLine.Left(index);
 		strVariable = strLine.Right(strLine.GetLength() - index - 1);
 		strVariable.Delete(0);
 		strVariable.Delete(strVariable.GetLength() - 1);
-		auto vids = parseVidList(strVariable);
+		CString strVariableA(narrowFromW(strVariable).c_str());
+		auto vids = parseVidList(strVariableA);
 
-		SERVO::CReport* pReport = new SERVO::CReport(atoi((LPTSTR)(LPCTSTR)strId), vids);
+		SERVO::CReport* pReport = new SERVO::CReport(_wtoi(strId), vids);
 		for (auto vid : vids) {
 			SERVO::CVariable* pVariable = getVariable(vid);
 			if (pVariable != nullptr) {
@@ -380,13 +750,23 @@
 	}
 	
 
-	file.Close();
 	return 0;
 }
 
 std::vector<SERVO::CReport*>& CHsmsPassive::getReports()
 {
 	return m_reports;
+}
+
+unsigned int CHsmsPassive::getMaxReportId() const
+{
+	unsigned int maxId = 0;
+	for (auto item : m_reports) {
+		if (item && item->getReportId() > maxId) {
+			maxId = item->getReportId();
+		}
+	}
+	return maxId;
 }
 
 SERVO::CReport* CHsmsPassive::getReport(int rptid)
@@ -413,52 +793,242 @@
 	return false;
 }
 
-void CHsmsPassive::clearAllReport()
+int CHsmsPassive::deleteReport(int rptid)
 {
+	LOGI("<CHsmsPassive>deleteReport enter");
+	if (!removeReport(rptid)) {
+		return -1;
+	}
+	LOGI("<CHsmsPassive>delete Report. rptid=%d", rptid);
+
+	return writeReportsToFile(m_strReportFilepath);
+}
+
+int CHsmsPassive::addReport(int rptid, const std::vector<unsigned int>& vids)
+{
+	if (getReport(rptid) != nullptr) {
+		return -1;
+	}
+	SERVO::CReport* pReport = new SERVO::CReport(rptid, vids);
+	for (auto vid : vids) {
+		SERVO::CVariable* pVariable = getVariable((int)vid);
+		if (pVariable != nullptr) {
+			pReport->addVariable(pVariable);
+		}
+	}
+	m_reports.push_back(pReport);
+	return writeReportsToFile(m_strReportFilepath);
+}
+
+int CHsmsPassive::updateReport(int rptid, const std::vector<unsigned int>& vids)
+{
+	for (auto iter = m_reports.begin(); iter != m_reports.end(); ++iter) {
+		if ((*iter)->getReportId() == rptid) {
+			delete (*iter);
+			SERVO::CReport* pReport = new SERVO::CReport(rptid, vids);
+			for (auto vid : vids) {
+				SERVO::CVariable* pVariable = getVariable((int)vid);
+				if (pVariable != nullptr) {
+					pReport->addVariable(pVariable);
+				}
+			}
+			*iter = pReport;
+			return writeReportsToFile(m_strReportFilepath);
+		}
+	}
+	return -1;
+}
+
+void CHsmsPassive::clearAllReport(BOOL bSave/* = FALSE*/)
+{
+	LOGI("<CHsmsPassive>clearAllReport enter");
 	for (auto item : m_reports) {
 		delete item;
 	}
 	m_reports.clear();
+
+	if(bSave)
+		writeReportsToFile(m_strReportFilepath);
+}
+
+int CHsmsPassive::writeReportsToFile(const std::string& filepath)
+{
+	if (filepath.empty()) return -1;
+
+	CFile file;
+	if (!file.Open(filepath.c_str(), CFile::modeCreate | CFile::modeWrite)) {
+		return -1;
+	}
+
+	if (m_bReportUtf8 && m_bReportUtf8Bom) {
+		const BYTE bom[3] = { 0xEF, 0xBB, 0xBF };
+		file.Write(bom, 3);
+	}
+
+	// header
+	const std::string headerAnsi = "RPTID,(VID1,VID2,...)\r\n";	
+	if (m_bReportUtf8) {
+		CStringA header = AnsiToUtf8(headerAnsi);
+		file.Write(header.GetString(), header.GetLength());
+	}
+	else {
+		file.Write(headerAnsi.data(), (UINT)headerAnsi.size());
+	}
+
+	for (auto rpt : m_reports) {
+		if (rpt == nullptr) continue;
+		std::string line;
+		line.reserve(64);
+		line += std::to_string(rpt->getReportId());
+		line += ",(";
+
+		const auto& vids = rpt->getVids();
+		for (size_t i = 0; i < vids.size(); ++i) {
+			line += std::to_string(vids[i]);
+			if (i + 1 < vids.size()) {
+				line.push_back(',');
+			}
+		}
+		line += ")\r\n";
+
+		if (m_bReportUtf8) {
+			CStringA out = AnsiToUtf8(line);
+			file.Write(out.GetString(), out.GetLength());
+		}
+		else {
+			file.Write(line.data(), (UINT)line.size());
+		}
+	}
+
+	file.Close();
+	return 0;
 }
 
 int CHsmsPassive::loadCollectionEvents(const char* pszFilepath)
 {
-	CStdioFile file;
-	if (!file.Open(pszFilepath, CFile::modeRead)) {
+	m_strCollectionEventFilepath = pszFilepath;
+	m_bCollectionUtf8 = false;
+	m_bCollectionUtf8Bom = false;
+	CFile file;
+	if (!file.Open(pszFilepath, CFile::modeRead | CFile::shareDenyNone)) {
 		return -1;
 	}
 
-	std::regex pattern("^\\d+,[^,]*,[^,]*,\\(\\d+(,\\d+)*\\).*");  // 鍖归厤浠ユ暟瀛�+閫楀彿寮�澶寸殑瀛楃涓�
+	const ULONGLONG nLen = file.GetLength();
+	if (nLen == 0) {
+		return -1;
+	}
+
+	std::string buffer;
+	buffer.resize(static_cast<size_t>(nLen));
+	file.Read(buffer.data(), static_cast<UINT>(nLen));
+	file.Close();
+
+	const unsigned char* bytes = reinterpret_cast<const unsigned char*>(buffer.data());
+	size_t offset = 0;
+	CStringW content;
+
+	// UTF-8 BOM
+	if (nLen >= 3 && bytes[0] == 0xEF && bytes[1] == 0xBB && bytes[2] == 0xBF) {
+		offset = 3;
+		m_bCollectionUtf8 = true;
+		m_bCollectionUtf8Bom = true;
+	}
+
+	// UTF-16 LE BOM
+	if (nLen >= 2 && bytes[0] == 0xFF && bytes[1] == 0xFE) {
+		const wchar_t* wdata = reinterpret_cast<const wchar_t*>(buffer.data() + 2);
+		const size_t wlen = (nLen - 2) / sizeof(wchar_t);
+		content.SetString(wdata, static_cast<int>(wlen));
+	}
+	// UTF-16 BE BOM
+	else if (nLen >= 2 && bytes[0] == 0xFE && bytes[1] == 0xFF) {
+		const size_t wlen = (nLen - 2) / sizeof(wchar_t);
+		std::wstring temp;
+		temp.reserve(wlen);
+		for (size_t i = 0; i < wlen; ++i) {
+			wchar_t ch = static_cast<wchar_t>(bytes[2 + i * 2] << 8 | bytes[2 + i * 2 + 1]);
+			temp.push_back(ch);
+		}
+		content = temp.c_str();
+	}
+	else {
+		auto tryUtf8 = [&](size_t off) -> bool {
+			int need = MultiByteToWideChar(CP_UTF8, MB_ERR_INVALID_CHARS, buffer.data() + off,
+				static_cast<int>(buffer.size() - off), nullptr, 0);
+			if (need <= 0) return false;
+			std::wstring temp;
+			temp.resize(need);
+			MultiByteToWideChar(CP_UTF8, 0, buffer.data() + off,
+				static_cast<int>(buffer.size() - off), temp.data(), need);
+			content = temp.c_str();
+			m_bCollectionUtf8 = true;
+			return true;
+		};
+
+		if (!tryUtf8(offset)) {
+			int need = MultiByteToWideChar(CP_ACP, 0, buffer.data(), static_cast<int>(buffer.size()), nullptr, 0);
+			if (need > 0) {
+				std::wstring temp;
+				temp.resize(need);
+				MultiByteToWideChar(CP_ACP, 0, buffer.data(), static_cast<int>(buffer.size()), temp.data(), need);
+				content = temp.c_str();
+			}
+		}
+	}
+
+	if (content.IsEmpty()) {
+		return -1;
+	}
+
+	// 鍏佽 Attached RPTID 涓虹┖锛�()
+	std::wregex pattern(L"^\\d+,[^,]*,[^,]*,\\(\\d*(,\\d+)*\\).*");  // 鍖归厤浠ユ暟瀛�+閫楀彿寮�澶寸殑瀛楃涓�
 	std::vector<SERVO::CCollectionEvent*> events;
 	int index, last;
-	CString strLine, strRPTIDs;
-	CString strId, strName, strDescription;
-	while (file.ReadString(strLine)) {
-		if (!std::regex_match((LPTSTR)(LPCTSTR)strLine, pattern)) {
+	CStringW strLine, strRPTIDs;
+	CStringW strId, strName, strDescription;
+	std::wstringstream ss(content.GetString());
+	auto narrowFromW = [](const CStringW& s) -> std::string {
+		int need = WideCharToMultiByte(CP_ACP, 0, s, -1, nullptr, 0, nullptr, nullptr);
+		if (need <= 0) return {};
+		std::string out(static_cast<size_t>(need - 1), '\0');
+		WideCharToMultiByte(CP_ACP, 0, s, -1, out.data(), need, nullptr, nullptr);
+		return out;
+	};
+	std::wstring line;
+	while (std::getline(ss, line, L'\n')) {
+		strLine = line.c_str();
+		strLine.Trim();
+		if (strLine.IsEmpty()) continue;
+		if (!std::regex_match(static_cast<LPCWSTR>(strLine), pattern)) {
 			continue;
 		}
 
 		last = 0;
-		index = strLine.Find(",", last);
+		index = strLine.Find(L",", last);
 		if (index < 0) continue;
 		strId = strLine.Left(index);
 		last = index + 1;
 
-		index = strLine.Find(",", last);
+		index = strLine.Find(L",", last);
 		if (index < 0) continue;
 		strName = strLine.Mid(last, index - last);
 		last = index + 1;
 
-		index = strLine.Find(",", last);
+		index = strLine.Find(L",", last);
 		if (index < 0) continue;
 		strDescription = strLine.Mid(last, index - last);
 		strRPTIDs = strLine.Right(strLine.GetLength() - index - 1);
 		strRPTIDs.Delete(0);
 		strRPTIDs.Delete(strRPTIDs.GetLength() - 1);
-		auto prtids = parseVidList(strRPTIDs);
+		CString strRPTIDsA(narrowFromW(strRPTIDs).c_str());
+		auto prtids = parseVidList(strRPTIDsA);
+
+		std::string sName = narrowFromW(strName);
+		std::string sDesc = narrowFromW(strDescription);
 
 		SERVO::CCollectionEvent* pEvent = new SERVO::CCollectionEvent(
-			atoi(strId), (LPTSTR)(LPCTSTR)strName, (LPTSTR)(LPCTSTR)strDescription, prtids);
+			_wtoi(strId), sName.c_str(), sDesc.c_str(), prtids);
 		for (auto rptid : prtids) {
 			SERVO::CReport* pReport = getReport(rptid);
 			if (pReport != nullptr) {
@@ -474,15 +1044,70 @@
 			m_collectionEvents.push_back(item);
 		}
 	}
-
-
-	file.Close();
 	return 0;
 }
 
 std::vector<SERVO::CCollectionEvent*>& CHsmsPassive::getCollectionEvents()
 {
 	return m_collectionEvents;
+}
+
+unsigned int CHsmsPassive::getMaxCollectionEventId() const
+{
+	unsigned int maxId = 0;
+	for (auto item : m_collectionEvents) {
+		if (item && item->getEventId() > maxId) {
+			maxId = item->getEventId();
+		}
+	}
+	return maxId;
+}
+
+int CHsmsPassive::deleteCollectionEvent(unsigned short CEID)
+{
+	for (auto iter = m_collectionEvents.begin(); iter != m_collectionEvents.end(); ++iter) {
+		if ((*iter)->getEventId() == CEID) {
+			delete (*iter);
+			m_collectionEvents.erase(iter);
+			return writeCollectionEventsToFile(m_strCollectionEventFilepath);
+		}
+	}
+	return -1;
+}
+
+int CHsmsPassive::addCollectionEvent(unsigned int CEID, const char* name, const char* desc, const std::vector<unsigned int>& rptids)
+{
+	if (getEvent((unsigned short)CEID) != nullptr) {
+		return -1;
+	}
+	auto* pEvent = new SERVO::CCollectionEvent(CEID, name, desc, const_cast<std::vector<unsigned int>&>(rptids));
+	for (auto rptid : rptids) {
+		SERVO::CReport* pReport = getReport((int)rptid);
+		if (pReport != nullptr) {
+			pEvent->addReport(pReport);
+		}
+	}
+	m_collectionEvents.push_back(pEvent);
+	return writeCollectionEventsToFile(m_strCollectionEventFilepath);
+}
+
+int CHsmsPassive::updateCollectionEvent(unsigned int CEID, const char* name, const char* desc, const std::vector<unsigned int>& rptids)
+{
+	for (auto iter = m_collectionEvents.begin(); iter != m_collectionEvents.end(); ++iter) {
+		if ((*iter)->getEventId() == CEID) {
+			delete (*iter);
+			auto* pEvent = new SERVO::CCollectionEvent(CEID, name, desc, const_cast<std::vector<unsigned int>&>(rptids));
+			for (auto rptid : rptids) {
+				SERVO::CReport* pReport = getReport((int)rptid);
+				if (pReport != nullptr) {
+					pEvent->addReport(pReport);
+				}
+			}
+			*iter = pEvent;
+			return writeCollectionEventsToFile(m_strCollectionEventFilepath);
+		}
+	}
+	return -1;
 }
 
 void CHsmsPassive::clearAllCollectionEvent()
@@ -526,6 +1151,62 @@
 	return result;
 }
 
+int CHsmsPassive::writeCollectionEventsToFile(const std::string& filepath)
+{
+	if (filepath.empty()) return -1;
+
+	CFile file;
+	if (!file.Open(filepath.c_str(), CFile::modeCreate | CFile::modeWrite)) {
+		return -1;
+	}
+
+	if (m_bCollectionUtf8 && m_bCollectionUtf8Bom) {
+		const BYTE bom[3] = { 0xEF, 0xBB, 0xBF };
+		file.Write(bom, 3);
+	}
+
+	const std::string headerAnsi = "CEID,CE Name,Descriptions,Attached RPTID\r\n";
+	if (m_bCollectionUtf8) {
+		CStringA header = AnsiToUtf8(headerAnsi);
+		file.Write(header.GetString(), header.GetLength());
+	}
+	else {
+		file.Write(headerAnsi.data(), (UINT)headerAnsi.size());
+	}
+
+	for (auto ev : m_collectionEvents) {
+		if (ev == nullptr) continue;
+		std::string line;
+		line.reserve(128);
+		line += std::to_string(ev->getEventId());
+		line.push_back(',');
+		line += ev->getName();
+		line.push_back(',');
+		line += ev->getDescription();
+		line.push_back(',');
+		line.push_back('(');
+		auto rptIds = ev->getReportIds();
+		for (size_t i = 0; i < rptIds.size(); ++i) {
+			line += std::to_string(rptIds[i]);
+			if (i + 1 < rptIds.size()) {
+				line.push_back(',');
+			}
+		}
+		line += ")\r\n";
+
+		if (m_bCollectionUtf8) {
+			CStringA out = AnsiToUtf8(line);
+			file.Write(out.GetString(), out.GetLength());
+		}
+		else {
+			file.Write(line.data(), (UINT)line.size());
+		}
+	}
+
+	file.Close();
+	return 0;
+}
+
 int CHsmsPassive::init(CModel* pModel, const char* pszName, unsigned int port)
 {
 	m_pModel = pModel;
@@ -549,7 +1230,7 @@
 		*/
 	};
 	auto onRecvSysMessage = [&](void* pFrom, IMessage* pMessage) -> void {
-		LOGI("<HSMS>onRecvSysMessage:sessionId:%d, sType:%d systemBytes:%d",
+		LOGI("<HSMS>[Received]sessionId:%d, sType:%d systemBytes:%d",
 		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
 		onRecvMsg(pMessage);
 		if (MSG_LINKTEST_REQ == pMessage->getHeader()->sType) {
@@ -576,7 +1257,7 @@
 		HEADER* pHeader = pMessage->getHeader();
 		int nStream = (pHeader->stream & 0x7F);
 
-		LOGI("<HSMS>鏀跺埌娑堟伅 S%dF%d", nStream, pHeader->function);
+		LOGI("<HSMS>[Received]%s", pMessage->toString());
 		if (nStream == 1 && pHeader->function == 1) {
 			// S1F1
 			replyAreYouThere(pMessage);
@@ -721,11 +1402,27 @@
 int CHsmsPassive::serialize(char* pszBuffer, int nBufferSize)
 {
 	int index = 0;
+	const auto calcSpoolCfgSize = [&]() -> int {
+		// magic(4) + ver(2) + enabled(1) + mapSize(4) + entries...
+		int sz = 0;
+		sz += 4; // 'SPOL'
+		sz += 2; // version
+		sz += 1; // enabled
+		sz += 4; // map size
+		for (const auto& kv : m_spoolBlacklistByStream) {
+			sz += 2; // streamId (U16)
+			sz += 4; // fn count (U32)
+			sz += static_cast<int>(kv.second.size()) * 2; // fn ids (U16 each)
+		}
+		return sz;
+	};
 	if (pszBuffer == nullptr) {
 		index += sizeof(int);
 		for (auto item : m_listActionSpooling) {
 			index += item->serialize(pszBuffer, nBufferSize);
 		}
+
+		index += calcSpoolCfgSize();
 
 		return index;
 	}
@@ -740,6 +1437,33 @@
 			nRet = item->serialize(&pszBuffer[index], nBufferSize);
 			if (nRet <= 0) break;
 			index += nRet;
+		}
+
+		// Append spooling config (backward compatible via magic+version)
+		auto writeU32 = [&](uint32_t v) {
+			memcpy(&pszBuffer[index], &v, sizeof(v));
+			index += sizeof(v);
+		};
+		auto writeU16 = [&](uint16_t v) {
+			memcpy(&pszBuffer[index], &v, sizeof(v));
+			index += sizeof(v);
+		};
+		auto writeU8 = [&](uint8_t v) {
+			memcpy(&pszBuffer[index], &v, sizeof(v));
+			index += sizeof(v);
+		};
+
+		const uint32_t magic = 0x4C4F5053; // 'SPOL' little-endian
+		writeU32(magic);
+		writeU16(1); // version
+		writeU8(m_spoolingEnabled ? 1 : 0);
+		writeU32(static_cast<uint32_t>(m_spoolBlacklistByStream.size()));
+		for (const auto& kv : m_spoolBlacklistByStream) {
+			writeU16(static_cast<uint16_t>(kv.first));
+			writeU32(static_cast<uint32_t>(kv.second.size()));
+			for (const auto& fn : kv.second) {
+				writeU16(static_cast<uint16_t>(fn));
+			}
 		}
 
 		return index;
@@ -762,7 +1486,53 @@
 		m_listActionSpooling.push_back(pAction);
 	}
 
-	return index + nRet;
+	// Parse optional spooling config tail (magic+version). If absent, keep defaults.
+	const auto remaining = nBufferSize - index;
+	if (remaining >= 4) {
+		uint32_t magic = 0;
+		memcpy(&magic, &pszBuffer[index], sizeof(magic));
+		if (magic == 0x4C4F5053) { // 'SPOL'
+			index += 4;
+			if (nBufferSize - index >= 2 + 1 + 4) {
+				uint16_t ver = 0;
+				memcpy(&ver, &pszBuffer[index], sizeof(ver));
+				index += 2;
+				if (ver >= 1) {
+					uint8_t enabled = 1;
+					memcpy(&enabled, &pszBuffer[index], sizeof(enabled));
+					index += 1;
+					m_spoolingEnabled = (enabled != 0);
+
+					uint32_t mapSize = 0;
+					memcpy(&mapSize, &pszBuffer[index], sizeof(mapSize));
+					index += 4;
+
+					m_spoolBlacklistByStream.clear();
+					for (uint32_t mi = 0; mi < mapSize; ++mi) {
+						if (nBufferSize - index < 2 + 4) break;
+						uint16_t streamId = 0;
+						memcpy(&streamId, &pszBuffer[index], sizeof(streamId));
+						index += 2;
+						uint32_t fnCount = 0;
+						memcpy(&fnCount, &pszBuffer[index], sizeof(fnCount));
+						index += 4;
+
+						auto& setRef = m_spoolBlacklistByStream[streamId];
+						setRef.clear();
+						for (uint32_t fi = 0; fi < fnCount; ++fi) {
+							if (nBufferSize - index < 2) break;
+							uint16_t fn = 0;
+							memcpy(&fn, &pszBuffer[index], sizeof(fn));
+							index += 2;
+							setRef.insert(fn);
+						}
+					}
+				}
+			}
+		}
+	}
+
+	return index;
 }
 
 unsigned CHsmsPassive::OnCimWork()
@@ -781,15 +1551,34 @@
 		Unlock();
 
 		while (!list.empty()) {
+			CHsmsAction* pAction = nullptr;
 			Lock();
-			CHsmsAction* pAction = list.front();
-			if (m_pPassive == NULL || STATE::SELECTED != m_pPassive->getState()) {
-				m_listActionSpooling.push_back(pAction);
-				Unlock();
-				continue;
-			}
+			pAction = list.front();
 			Unlock();
 			list.pop_front();
+
+			Lock();
+			const bool selected = (m_pPassive != NULL && STATE::SELECTED == m_pPassive->getState());
+			Unlock();
+			if (!selected) {
+				IMessage* pMsg = pAction->getSendMessage();
+				uint8_t streamId = 0;
+				uint8_t functionId = 0;
+				if (pMsg && pMsg->getHeader()) {
+					streamId = static_cast<uint8_t>(pMsg->getHeader()->stream & 0x7F);
+					functionId = static_cast<uint8_t>(pMsg->getHeader()->function & 0xFF);
+				}
+				if (shouldSpool(streamId, functionId)) {
+					Lock();
+					m_listActionSpooling.push_back(pAction);
+					Unlock();
+				}
+				else {
+					LOGI("<HSMS>spooling disabled for S%dF%d, drop action", (int)streamId, (int)functionId);
+					delete pAction;
+				}
+				continue;
+			}
 			TRACE("OnCimWork 004.\n");
 
 			if (pAction->isNeedWaitReply()) {
@@ -848,7 +1637,9 @@
 	ISECS2Item* pItem = pMessage->getBody();
 	pItem->setBinary((const char*)&ack, 1, pszAckName);
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]S%dF%d (SysByte=%u)", s, f, systemBytes);
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 }
 
@@ -889,7 +1680,9 @@
 	pItem->addItem(m_strEquipmentModelType.c_str(), "MDLN");
 	pItem->addItem(m_strSoftRev.c_str(), "SOFTREV");
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]S1F2 (SysByte=%u)", pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 
 	return 0;
@@ -950,7 +1743,9 @@
 	pList->addItem(m_strEquipmentModelType.c_str(), "MDLN");
 	pList->addItem(m_strSoftRev.c_str(), "SOFTREV");
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]%s", pMessage->toString());
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 
 	return 0;
@@ -968,18 +1763,18 @@
 	ASSERT(pMessage);
 
 	unsigned char SVU1 = 0;
-	unsigned int SVID = 0;
+	unsigned short SVID = 0;
 	ISECS2Item* pBody = pRecv->getBody();
 	if (pBody == nullptr || pBody->getType() != SITYPE::L) {
 		pMessage->getBody()->addU1Item(SVU1, "SV");
 		goto MYREPLY;
 	}
-	if (!pBody->getSubItemU4(0, SVID)) {
+	if (!pBody->getSubItemU2(0, SVID)) {
 		pMessage->getBody()->addU1Item(SVU1, "SV");
 		goto MYREPLY;
 	}
 
-	SERVO::CVariable* pVariable = getVariable(SVID);
+	SERVO::CVariable* pVariable = getVariable((int)SVID);
 	if (pVariable == nullptr) {
 		pMessage->getBody()->addU1Item(SVU1, "SV");
 		goto MYREPLY;
@@ -988,7 +1783,9 @@
 
 MYREPLY:
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]%s", pMessage->toString());
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 
 
@@ -1037,7 +1834,9 @@
 	}
 
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]S2F14 (SysByte=%u)", pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 
 	return 0;
@@ -1133,13 +1932,14 @@
 
 	ISECS2Item* pBody = pRecv->getBody();
 	ISECS2Item* defineItem, *rptListItem, * vidListItem;
-	unsigned int dataId, rptid, vid;
+	unsigned short dataId;
+	unsigned int rptid, vid;
 
-	if (!pBody->getSubItemU4(0, dataId)) goto MYREPLY;
+	if (!pBody->getSubItemU2(0, dataId)) goto MYREPLY;
 	rptListItem = pBody->getSubItem(1);
 	if (rptListItem == nullptr) goto MYREPLY;
 	if (rptListItem->getSubItemSize() == 0) {
-		clearAllReport();
+		clearAllReport(TRUE);
 		goto MYREPLY;
 	}
 
@@ -1159,8 +1959,10 @@
 			}
 		}
 
-		removeReport(rptid);
-		if (!vids.empty()) {
+		if (vids.empty()) {
+			deleteReport(rptid);
+		} else {
+			removeReport(rptid);
 			pReport = defineReport(rptid, vids);
 		}
 
@@ -1189,8 +1991,10 @@
 
 	ISECS2Item* pBody = pRecv->getBody();
 	ISECS2Item* linkItem, *ceidListItem, *rptListItem;
-	unsigned int dataId, ceid, rptid;
-	if (!pBody->getSubItemU4(0, dataId)) goto MYREPLY;
+	unsigned short dataId;
+	unsigned int ceid, rptid;
+	bool bChanged = false;
+	if (!pBody->getSubItemU2(0, dataId)) goto MYREPLY;
 	ceidListItem = pBody->getSubItem(1);
 	if (ceidListItem == nullptr) goto MYREPLY;
 	for (int i = 0; i < ceidListItem->getSubItemSize(); i++) {
@@ -1202,21 +2006,31 @@
 			int prtCount = rptListItem->getSubItemSize();
 			if (prtCount == 0) {
 				unlinkEventReport(ceid);
+				bChanged = true;
 			}
 			else {
 				for (int k = 0; k < prtCount; k++) {
 					if (rptListItem->getSubItemU4(k, rptid)) {
 						linkEventReport(ceid, rptid);
+						bChanged = true;
 					}
 				}
 			}
 		}
 	}
 
+	// 鎸佷箙鍖栧埌 CollectionEventList.txt锛堜究浜庝笅娆″惎鍔ㄤ粛淇濇寔 Link/Unlink 缁撴灉锛�
+	if (bChanged && !m_strCollectionEventFilepath.empty()) {
+		writeCollectionEventsToFile(m_strCollectionEventFilepath);
+	}
 
 	// 妫�楠岀粨鏋滄槸鍚︽纭�
 	for (auto item : m_collectionEvents) {
-		LOGE("=== ceid:%d, prtid:%d", item->getEventId(), item->getFirstPortID());
+		unsigned int reportId = item->getFirstReportID();
+		if(reportId != 0)
+			LOGI("=== ceid:%d, prtid:%d", item->getEventId(), reportId);
+		else 
+			LOGI("=== ceid:%d, prtid:--", item->getEventId());
 	}
 	
 MYREPLY:
@@ -1324,9 +2138,12 @@
 
 	// 娓呯┖鎵�鏈�
 	if (pBody->getSubItemSize() == 0) {
-		m_spoolingConfig.clear();
+		LOGI("<CHsmsPassive>turns off all streams and fns");
+		m_spoolBlacklistByStream.clear();
+		m_spoolingEnabled = false;
 		goto MYREPLY;
 	}
+	m_spoolingEnabled = true;
 
 	// 渚濇閰嶇疆Stream
 	for (int i = 0; i < pBody->getSubItemSize(); i++) {
@@ -1335,26 +2152,34 @@
 		unsigned char STRID, FCNID;
 		pStreamItem->getSubItemU1(0, STRID);
 		ISECS2Item* pFcnItemList = pStreamItem->getSubItem(1);
-		if (pFcnItemList->getSubItemSize() == 0) {
-			m_spoolingConfig[STRID].clear();
+		if (pFcnItemList == nullptr || pFcnItemList->getSubItemSize() == 0) {
+			// No functions listed => blacklist the whole stream
+			m_spoolBlacklistByStream[STRID].clear();
 		}
 		else {
+			// Update blacklist for this stream
+			m_spoolBlacklistByStream[STRID].clear();
 			for (int j = 0; j < pFcnItemList->getSubItemSize(); j++) {
 				pFcnItemList->getSubItemU1(j, FCNID);
-				m_spoolingConfig[STRID].insert(FCNID);
+				m_spoolBlacklistByStream[STRID].insert(FCNID);
 			}
 		}
 	}
 
 	// 鎵撳嵃楠岃瘉缁撴灉
-	for (auto s : m_spoolingConfig) {
-		LOGI("====> stream:%d", s.first);
-		for (auto f : s.second) {
-			LOGI("function:%d", f);
+	for (auto s : m_spoolBlacklistByStream) {
+		LOGI("====> spool blacklist stream:%d", s.first);
+		if (s.second.empty()) {
+			LOGI("blacklist all functions");
+		}
+		else {
+			for (auto f : s.second) {
+				LOGI("blacklist function:%d", f);
+			}
 		}
 	}
 MYREPLY:
-	replyAck(2, 42, pRecv->getHeader()->systemBytes, BYTE(0), "ERACK");
+	replyAck(2, 44, pRecv->getHeader()->systemBytes, BYTE(0), "ERACK");
 	return 0;
 }
 
@@ -1379,10 +2204,10 @@
 	ISECS2Item* pBody = pRecv->getBody();
 	if (pBody == nullptr || pBody->getType() != SITYPE::L) ER_PARAM_ERROR;
 
-	unsigned int DATAID;
+	unsigned short DATAID;
 	unsigned char PTN;
 	const char* pszCarrierAction, *pszCarrierId;
-	pBody->getSubItemU4(0, DATAID);
+	pBody->getSubItemU2(0, DATAID);
 	pBody->getSubItemString(1, pszCarrierAction);
 	pBody->getSubItemString(2, pszCarrierId);
 	pBody->getSubItemU1(3, PTN);
@@ -1403,7 +2228,9 @@
 	pErrItem->addU4Item(ERRCODE, "ERRCODE");
 	pErrItem->addItem(strError.c_str(), "ERRTEXT");
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]S3F18 (SysByte=%u)", pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 	
 	return 0;
@@ -1463,6 +2290,7 @@
 
 	// 涓㈠純
 	if (RSDC == 1) {
+		LOGI("<CHsmsPassive>Purge Spooled Data.");
 		Lock();
 		for (auto item : m_listActionSpooling) {
 			delete item;
@@ -1471,6 +2299,7 @@
 		Unlock();
 	}
 	else {
+		LOGI("<CHsmsPassive>Request Spooled Data.");
 		Lock();
 		for (auto item : m_listActionSpooling) {
 			m_listAction.push_back(item);
@@ -1504,7 +2333,9 @@
 	}
 
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]S7F20 (SysByte=%u)", pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 
 	return 0;
@@ -1668,7 +2499,9 @@
 MYREPLY:
 	pReplyItemAck->setU1(bCreateOk ? 0 : 1, "OBJACK");
 	m_pPassive->sendMessage(pReply);
-	LOGI("<HSMS>[SECS Msg SEND]S14F10 (SysByte=%u)", pReply->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pReply->getHeader()->sessionId, pReply->getHeader()->sType, pReply->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pReply->toString());
 	HSMS_Destroy1Message(pReply);
 
 
@@ -1687,7 +2520,7 @@
 
 	// 瑙i噴鏁版嵁锛屽緱鍒癈ProcessJob
 	ISECS2Item* pItemPjs, * pItemPj,* pItemCarriers, * pItemCarrier, *pItemSlots, *pItemRecipes;
-	unsigned int DATAID;
+	unsigned short DATAID;
 	const char* pszPrjobid, *pszMF, *pszCarrierId, *pszRecipeName;
 	std::string strCarrierId;
 	unsigned int len;
@@ -1695,7 +2528,7 @@
 	std::vector<unsigned char> slots;
 	std::vector<SERVO::CProcessJob*> pjs;
 
-	if (!pBody->getSubItemU4(0, DATAID)) return ER_PARAM_ERROR;
+	if (!pBody->getSubItemU2(0, DATAID)) return ER_PARAM_ERROR;
 	pItemPjs = pBody->getSubItem(1);
 	if (pItemPjs == nullptr) return ER_PARAM_ERROR;
 	for (int i = 0; i < pItemPjs->getSubItemSize(); i++) {
@@ -1767,7 +2600,9 @@
 		}
 	}
 	m_pPassive->sendMessage(pMessage);
-	LOGI("<HSMS>[SECS Msg SEND]S16F16 (SysByte=%u)", pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]sessionId:%d, sType:%d systemBytes:%d",
+		pMessage->getHeader()->sessionId, pMessage->getHeader()->sType, pMessage->getHeader()->systemBytes);
+	LOGI("<HSMS>[SEND]%s", pMessage->toString());
 	HSMS_Destroy1Message(pMessage);
 
 
@@ -1798,7 +2633,13 @@
 	pItem->addItem(ALTX, "ALTX");
 	pAction->setSendMessage(pMessage);
 	if (m_pPassive == NULL || STATE::SELECTED != m_pPassive->getState()) {
-		m_listActionSpooling.push_back(pAction);
+		if (shouldSpool(5, 1)) {
+			m_listActionSpooling.push_back(pAction);
+		}
+		else {
+			LOGI("<HSMS>spooling disabled for S5F1, drop alarm report");
+			delete pAction;
+		}
 	}
 	else {
 		m_listAction.push_back(pAction);
@@ -1810,7 +2651,7 @@
 }
 
 // S6F11
-static unsigned int DATAID = 1;
+static unsigned short DATAID = 0;
 int CHsmsPassive::requestEventReportSend(unsigned int CEID)
 {
 	SERVO::CCollectionEvent* pEvent = getEvent(CEID);
@@ -1819,9 +2660,6 @@
 	}
 
 	SERVO::CReport* pReport = pEvent->getFirstReport();
-	if (pReport == nullptr) {
-		return ER_UNLINK_EVENT_REPORT;
-	}
 
 
 	Lock();
@@ -1830,20 +2668,29 @@
 	HSMS_Create1Message(pMessage, m_nSessionId, 6 | REPLY, 11, ++m_nSystemByte);
 	ASSERT(pMessage);
 	ISECS2Item* pItem = pMessage->getBody();
-	pItem->addU4Item(++DATAID, "DATAID");
+	// pItem->addU2Item(++DATAID, "DATAID");		// 鏍规嵁鍒殑鏃ュ織鏄剧ずDATAID鎭掍负0锛屾墍浠ユ垜浠厛鐓т娇鐢�0
+	pItem->addU2Item(0, "DATAID");
 	pItem->addU4Item(CEID, "CEID");
-	ISECS2Item* pItemList1 = pItem->addItem();
-	ISECS2Item* pItemList2 = pItemList1->addItem();
-	pItemList2->addU4Item(pReport->getReportId(), "RPTID");
-	ISECS2Item* pItemList3 = pItemList2->addItem();
+	ISECS2Item* pItemList1 = pItem->addItem(); // L[n] reports
+	if (pReport != nullptr) {
+		ISECS2Item* pItemList2 = pItemList1->addItem();
+		pItemList2->addU4Item(pReport->getReportId(), "RPTID");
+		ISECS2Item* pItemList3 = pItemList2->addItem();
 
-	auto vars = pReport->getVariables();
-	for (auto var : vars) {
-		addVariableValueToItem(pItemList3, var);
+		auto vars = pReport->getVariables();
+		for (auto var : vars) {
+			addVariableValueToItem(pItemList3, var);
+		}
 	}
 	pAction->setSendMessage(pMessage);
 	if (m_pPassive == NULL || STATE::SELECTED != m_pPassive->getState()) {
-		m_listActionSpooling.push_back(pAction);
+		if (shouldSpool(6, 11)) {
+			m_listActionSpooling.push_back(pAction);
+		}
+		else {
+			LOGI("<HSMS>spooling disabled for S6F11, drop event report (CEID=%u)", CEID);
+			delete pAction;
+		}
 	}
 	else {
 		m_listAction.push_back(pAction);
@@ -1874,6 +2721,41 @@
 	return requestEventReportSend("CarrierID_Readed");
 }
 
+int CHsmsPassive::requestEventReportSend_CheckSlotMap()
+{
+	return requestEventReportSend("CheckSlotMap");
+}
+
+int CHsmsPassive::requestEventReportSend_SlotMapVerificationOK()
+{
+	return requestEventReportSend("SlotMapVerificationOK");
+}
+
+int CHsmsPassive::requestEventReportSend_SlotMapVerificationNG()
+{
+	return requestEventReportSend("SlotMapVerificationNG");
+}
+
+int CHsmsPassive::requestEventReportSend_Port_Unload_Ready()
+{
+	return requestEventReportSend("Port_Unload_Ready");
+}
+
+int CHsmsPassive::requestEventReportSend_Port_Load_Ready()
+{
+	return requestEventReportSend("Port_Load_Ready");
+}
+
+int CHsmsPassive::requestEventReportSend_Port_Ready_To_Release()
+{
+	return requestEventReportSend("Port_Ready_To_Release");
+}
+
+int CHsmsPassive::requestEventReportSend_Port_Blocked()
+{
+	return requestEventReportSend("Port_Blocked");
+}
+
 int CHsmsPassive::requestEventReportSend_PJ_Queued()
 {
 	return requestEventReportSend("PJ_Queued");
@@ -1889,6 +2771,49 @@
 	return requestEventReportSend("PJ_End");
 }
 
+int CHsmsPassive::requestEventReportSend_CJ_Start()
+{
+	return requestEventReportSend("CJ_Start");
+}
 
+int CHsmsPassive::requestEventReportSend_CJ_End()
+{
+	return requestEventReportSend("CJ_End");
+}
+
+int CHsmsPassive::requestEventReportSend_Panel_Start()
+{
+	return requestEventReportSend("Panel_Start");
+}
+
+int CHsmsPassive::requestEventReportSend_Panel_End()
+{
+	return requestEventReportSend("Panel_End");
+}
+
+int CHsmsPassive::requestEventReportSend_OCR_PanelID_Read_OK()
+{
+	return requestEventReportSend("OCR_PanelID_Read_OK");
+}
+
+int CHsmsPassive::requestEventReportSend_LoadPortNotAssoc()
+{
+	return requestEventReportSend("LoadPortNotAssoc");
+}
+
+int CHsmsPassive::requestEventReportSend_ProcessDataReport()
+{
+	return requestEventReportSend("ProcessDataReport");
+}
+
+int CHsmsPassive::requestEventReportSend_SubEqpStart()
+{
+	return requestEventReportSend("SubEqpStart");
+}
+
+int CHsmsPassive::requestEventReportSend_SubEqpEnd()
+{
+	return requestEventReportSend("SubEqpEnd");
+}
 
 

--
Gitblit v1.9.3