Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] 先删除模板的话无法批量删除 #526

Open
4 tasks done
Ovler-Young opened this issue May 3, 2024 · 1 comment
Open
4 tasks done

[Bug] 先删除模板的话无法批量删除 #526

Ovler-Young opened this issue May 3, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@Ovler-Young
Copy link
Contributor

Ovler-Young commented May 3, 2024

Verify steps

  • Tracker 我已经在 Issue Tracker 中找过我要提出的问题
  • Latest 我已经使用 最新源码 测试过,问题依旧存在
  • Core 这是 QD 框架存在的问题,并非我所使用的 QD 早期版本(如 20210628及之前版号 等)或模板的特定问题
  • Meaningful 我提交的不是无意义的 催促更新或修复 请求

QD Version

20240210

Bug on OS

Windows

Bug on Platform

Docker/Linux 64位

To Reproduce

  1. 新建一个任务
  2. 将这个任务所用的模板删除
  3. 选择改任务
  4. 使用多任务操作进行删除
  5. 报错

Describe the Bug

报错:
获取信息失败
'NoneType' object is not subscriptable

QD config

No response

QD log

后台无报错。

[I 240503 18:08:33 tornado.access web:2348] 200 GET /task/1/multi?op=delete (89.***) 1.35ms
[I 240503 18:08:33 tornado.access web:2348] 200 POST /task/1/get_tasksinfo (89.***) 24.10ms
[I 240503 18:08:35 tornado.access web:2348] 200 GET /my/ (89.***) 58.95ms

Expected behavior

能正常删除

Screenshots

image

另:复现步骤还未测试

@Ovler-Young Ovler-Young added the bug Something isn't working label May 3, 2024
@Ovler-Young
Copy link
Contributor Author

应该是这里附近

task = await self.db.task.get(taskid, fields=('id', 'note', 'tplid'))
if task:
sitename = (await self.db.tpl.get(task['tplid'], fields=('sitename',)))['sitename']
task['sitename'] = sitename
tasks.append(task)

@a76yyyy a76yyyy self-assigned this Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants