fix: Modify hotspot configuration error

[caixr23]

Modifying hotspot configuration errors resulted in inability to connect issues

Log:
pms: BUG-294499
pms: BUG-286589

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码重复：

   • 在 case "wpa-psk" 和 case "sae" 分支中，proto、group、pairwise 和 psk-flags 的设置是相同的，可以考虑提取成一个函数来减少代码重复。

2. 默认分支处理：

   • 在 switch 语句的 default 分支中，删除了 editDlg.config["802-11-wireless-security"]，这可能会导致在 keyMgmt 不为 wpa-psk 或 sae 时，无线安全配置被删除。如果这是预期行为，应该添加注释说明原因；如果不是，应该保持原有的逻辑。

3. 变量命名：

   • keyMgmt 变量名不够直观，建议使用更具描述性的名称，如 wirelessSecurityType。

4. 错误处理：

   • 在 dccData.exec 调用后没有错误处理机制，如果设置连接信息失败，应该有相应的错误处理逻辑。

5. 代码格式：

   • switch 语句中的 case 和 break 之间应该有一个空行，以提高代码的可读性。

6. 注释：

   • 新增的 WPA3 Personal 选项没有注释说明其用途和配置要求，建议添加注释以便其他开发者理解。

7. 安全性：

   • 如果 editDlg.config 的内容来自用户输入，应该进行适当的验证和清理，以防止潜在的注入攻击。

综合以上意见，代码可以改进如下：

// 提取公共配置设置为一个函数
function setCommonWirelessSecurityConfig() {
    editDlg.config["802-11-wireless-security"]["proto"] = ['wpa', 'rsn']
    editDlg.config["802-11-wireless-security"]["group"] = ['ccmp']
    editDlg.config["802-11-wireless-security"]["pairwise"] = ['ccmp']
    editDlg.config["802-11-wireless-security"]["psk-flags"] = 1
}

// 更新后的代码
switch (wirelessSecurityType) {
    case "wpa-psk":
        setCommonWirelessSecurityConfig()
        editDlg.config["802-11-wireless-security"]["key-mgmt"] = "wpa-psk"
        break
    case "sae":
        setCommonWirelessSecurityConfig()
        editDlg.config["802-11-wireless-security"]["key-mgmt"] = "sae"
        break
    default:
        // 添加注释说明默认行为
        delete editDlg.config["802-11-wireless-security"]
        break
}

// 添加错误处理逻辑
dccData.exec(NetManager.SetConnectInfo, item.id, editDlg.config).then(function(result) {
    if (!result.success) {
        // 处理错误
    }
}).catch(function(error) {
    // 处理错误
})

// 添加注释说明 WPA3 Personal 的配置要求

以上改进建议旨在提高代码的可读性、可维护性和安全性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: caixr23, mhduiy

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment