Hui Liu

> 这个PR是可以review了吗？改一下openapi文件？ 不好意思，这两天在忙别的， 先 review 代码吧， openapi 我今天晚点补上

> 没理解错的话，条目已经在目录里的情况下，`PUT` 方法会报错？ 为啥会报错呢？ 但我发现错误处理没有做干净， put 的时候可能不存在，应该 404

> 啊，是的，说错了，应该是对应条目不存在的时候处理的不太对。 > > 感觉可以直接把对应的条目添加到目录里 两种处理（指加进去和404）都能接受，但个人感觉还是报 404 合适点，put 感觉就是修改资源，post 是新增

嗯，好，我在 api 文档里也备注下

咨询一下， ```sql CREATE TABLE IF NOT EXISTS `chii_index_related` ( `idx_rlt_cat` TINYINT(3) NOT NULL, `idx_rlt_type` SMALLINT(6) UNSIGNED NOT NULL COMMENT '关联条目类型', ) ENGINE = MyISAM DEFAULT CHARSET = utf8 COMMENT =...

> 这个type是条目类型，cat不知道是什么，可能只是我们没用到。 我看了下代码，之前是用 cat 做的分类，没有见到过 type 字段。 昨天还发现一点问题， `ListSubjects` 接口的 `type` 不支持传入 `person` 和 `character` 两个值

我先把目录的补完吧， 期末了要准备 Final 了，时间不是很多，结束如果没有人搞 我可以来整。

是 ctrl 里的一个lint， 我打算再开个pr修来着，毕竟和这次功能没啥关系..Hugh(Hui) LiuKind Regards On Oct 21, 2022, at 18:38, Trim21 ***@***.***> wrote: lint没过 —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this...

> 报错是报在了其他文件，但是是这个PR引起的 发现了.. 我这报错在 `internal/ctrl/count_episode_for_subject.go:30:17:` 但是是因为 `index.go`... 改好了。

> [POST] /indices//subjects 不用返回条目信息了吧 我觉得加上比较合适。不像修改或者删除，信息都是已知或者只是移除。 新增之后如果需要在客户端增加内容或者展示啥东西，就可以省一个找信息的请求（比如 /indices/5/subject 或者 /subject/)