修复菜单从编辑切回「新增顶级菜单」保存会出现错误弹窗

[luckydog-sunny]

修复菜单从编辑切回「新增顶级菜单」时， 保存会出现错误弹窗：此操作已执行过，请点击左侧菜单重新操作。

修复内容：

 if (data.id) {
    form.value.dataType = 'edit'
 }
// 改为
form.value.dataType = data.id ? 'edit' : 'add';

复现操作：菜单管理 -> 新增顶级菜单 -> （点击任意树菜单）-> 新增顶级菜单 -> 填写数据保存 -> 出现错误弹窗

Summary by CodeRabbit

修复

• 优化了权限菜单表单的状态管理逻辑。在添加和编辑操作中，表单模式的设置现在更加明确和可靠，确保在所有使用场景下都能正确初始化和处理表单数据。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c87f31ee-dd81-4188-b613-ff2f32b65a4b

📥 Commits

Reviewing files that changed from the base of the PR and between 86dc71e and 4673cdb.

📒 Files selected for processing (1)

• web/src/modules/base/views/permission/menu/menu-form.vue

---

📝 Walkthrough

演练

更新了 menu-form.vue 中 setData 函数的逻辑，使用三元表达式 data.id ? 'edit' : 'add' 显式设置 form.value.dataType，替代之前仅在 data.id 存在时才切换到 'edit' 的条件块，现在当 data.id 为假值时显式强制设置为 'add'。

变更

内聚组 / 文件	摘要
菜单表单数据类型逻辑 web/src/modules/base/views/permission/menu/menu-form.vue	使用三元表达式将 form.value.dataType 的赋值改为显式条件判断，基于 data.id 的存在性决定为 'edit' 或 'add'，简化了控制流逻辑。

预估代码审查工作量

🎯 2 (简单) | ⏱️ ~8 分钟

相关的可能性 PR

• fix(menu、table): 修复菜单新增可一直点击，优化表结构，修复菜单错误提示未翻译的问题 #374：两个 PR 都修改了同一文件中 setData 中的 form.dataType 赋值逻辑，基于 data.id 的存在性进行判断。
• fix(menu-btn-permission): 修复菜单按钮列表为空时，未清楚的问题 #433：两个 PR 都修改了同一文件中的 setData 函数，其中一个改变了 dataType 赋值，另一个重置并管理 btnPermission。
• fix(menu): 修复编辑类型为M的菜单时，按钮权限列表未回显 #424：两个 PR 都修改了同一文件中的 setData 函数，本 PR 通过三元表达式强制设置 form.value.dataType，而另一个 PR 也简化了 setData 的逻辑及相关渲染。

建议标签

bug, Frontend, size:XS

建议审阅者

• zds-s

诗

🐰 菜单表单焕新颜，
三元若道亮堂堂，
编辑添加显分明，
逻辑简洁代码棒！
小小改动大智慧～

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	拉取请求标题准确反映了变更集的主要内容——修复菜单从编辑模式切回「新增」模式时的表单数据类型错误。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}