Skip to content

Conversation

@mhduiy
Copy link
Contributor

@mhduiy mhduiy commented Nov 29, 2024

如果model来自C++的QVarientList,则只能通过modelData[control.textRole]去访问数据

pms: TASK-368711
pms: Bug-286969

如果model来自C++的QVarientList,则只能通过modelData[control.textRole]去访问数据

pms: TASK-368711
@mhduiy mhduiy requested a review from 18202781743 November 29, 2024 03:13
deepin-ci-robot added a commit to linuxdeepin/dtk6declarative that referenced this pull request Nov 29, 2024
Synchronize source files from linuxdeepin/dtkdeclarative.

Source-pull-request: linuxdeepin/dtkdeclarative#423
@deepin-ci-robot
Copy link
Contributor

deepin pr auto review

关键摘要:

  • 代码修改中增加了对 model[control.textRole] 是否为 undefined 的检查,以避免潜在的未定义错误。

是否建议立即修改:

  • 否,这次提交的修改是合理的,它增加了对 model[control.textRole] 是否为 undefined 的检查,以防止在 model 中找不到 control.textRole 时出现未定义错误。这是一个改进,不需要立即修改。

@kegechen kegechen merged commit 1f5e1ab into linuxdeepin:master Nov 29, 2024
18 of 20 checks passed
@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, kegechen, mhduiy

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

Details Needs approval from an approver in each of these files:

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

kegechen pushed a commit to linuxdeepin/dtk6declarative that referenced this pull request Nov 29, 2024
Synchronize source files from linuxdeepin/dtkdeclarative.

Source-pull-request: linuxdeepin/dtkdeclarative#423
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants