fix: clarify usage of ruleMacro in Firewall class, noting it is for UI convenience only

This commit is contained in:
2025-12-04 20:36:03 -05:00
parent 0830440d83
commit 83823e2103

View File

@@ -628,9 +628,8 @@ class Firewall extends Component
'enable' => $this->ruleEnable ? 1 : 0,
];
if (!empty(trim($this->ruleMacro))) {
$payload['macro'] = trim($this->ruleMacro);
}
// Note: ruleMacro is only used for UI convenience to auto-populate fields
// It is NOT sent to Proxmox API - only the actual proto/dport values are sent
if (!empty(trim($this->ruleIface))) {
$payload['iface'] = trim($this->ruleIface);