yapi icon indicating copy to clipboard operation
yapi copied to clipboard

fix: read project_id from cat

Open tangcent opened this issue 2 years ago • 9 comments

更新时project_id非必须字段,更新接口作为openapi需对此保持兼容. 故当project_id不存在时,尝试从cat_idproject.


fix #2371

tangcent avatar Oct 20 '21 01:10 tangcent

@ariesly15 PTAL

tangcent avatar Oct 29 '21 02:10 tangcent

是不是原作者不维护了,这个tag的报错最新版本还是存在

MrZbb avatar Jan 10 '22 08:01 MrZbb

是的,又没人维护了

tangcent avatar Jan 19 '22 12:01 tangcent

@hellosean1025 希望尽快处理这个问题

oubeichen avatar Jan 20 '22 03:01 oubeichen

使用最新版本提交带有tag的 保存还是不成功

shcw avatar Feb 21 '22 06:02 shcw

@shencw 你是指合并这个PR还是无法保存吗?

tangcent avatar Feb 21 '22 07:02 tangcent

@shencw 你是指合并这个PR还是无法保存吗?

我的是提示错误,但实际成功了。 使用了最近的master分支也不行。

我这样解决的问题 https://github.com/YMFE/yapi/issues/2371#issuecomment-1046641932

shcw avatar Feb 25 '22 06:02 shcw

@shencw 你是指合并这个PR还是无法保存吗?

我的是提示错误,但实际成功了。 使用了最近的master分支也不行。

我这样解决的问题 #2371 (comment)

这个PR就包含了你说的那个方案,我自行合并后使用一切正常。 就是官方一直没人维护,导致最新版本一直不可用,难办。

oubeichen avatar Feb 25 '22 07:02 oubeichen

1.10.2 存在bug,更新接口如果存在tag标签值,更新成功,提示失败

原因:controllers/interface.js::autoAddTag方法,更新project记录需要project_id,通过up接口前端传参params没有project_id字段,导致projectModel更新报错 image image

解决方法:up方法调用autoAddTag方法前,params赋值project_id, params.project_id = interfaceData.project_id;

image

chentaolian avatar Aug 08 '22 11:08 chentaolian

1.12.0版本保存接口时,没必要自动保存标签,因为添加新标签的时候已经调用标签接口保存过了。 可以将 server.controllers.interface.js#autoAddTag 中的 if (tags && Array.isArray(tags) && tags.length > 0) 直接改成 if (params.project_id && tags && Array.isArray(tags) && tags.length > 0)

yangqiang avatar Mar 02 '23 08:03 yangqiang