Skip to content

Primary key value unexpectedly updated when database adds row #11018

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.

Sign up for GitHub

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

Jump to bottom
Closed
TCOTC opened this issue Apr 13, 2024 · 1 comment
Closed

Primary key value unexpectedly updated when database adds row #11018

TCOTC opened this issue Apr 13, 2024 · 1 comment
Assignees
@88250 @Vanessa219
Labels
Bug
Milestone
3.0.10

Comments

@TCOTC
Copy link
Contributor

TCOTC commented Apr 13, 2024 •
edited by 88250
Loading

主键升序排序时，空值应该置底

In what scenarios do you need this feature?

主键为空时，升序未置底：

image

Describe the optimal solution

跟其他类型的属性一样，升序和降序都置底：

image

Describe the candidate solution

No response

Other information

No response
@88250
Copy link
Member

88250 commented Apr 13, 2024

测试了下，应该是因为某种情况下（比如添加行时）虽然是新建，但是会意外修改主键值（还是空字符串）导致误识别为已经编辑过所以参与了排序。

后续会改进更新值的接口，如果新旧值对比一样则不更新值。

88250 added a commit that referenced this issue Apr 13, 2024
@88250
🐛 #11018

Verified

This commit was signed with the committer’s verified signature.
88250 D
GPG key ID: 86211BA83DF03017
Verified
Learn about vigilant mode
4e63228
@88250 88250 changed the title 主键升序排序时，空值应该置底 Avoid accidentally updating primary key values Apr 13, 2024
@88250 88250 added this to the 3.0.10 milestone Apr 13, 2024
@88250 88250 changed the title Avoid accidentally updating primary key values Primary key value unexpectedly updated when database adds row Apr 13, 2024
@88250 88250 added Bug and removed Enhancement labels Apr 13, 2024
88250 added a commit that referenced this issue Apr 13, 2024
@88250
🐛 #11018

Verified

This commit was signed with the committer’s verified signature.
88250 D
GPG key ID: 86211BA83DF03017
Verified
Learn about vigilant mode
5c77101
88250 added a commit that referenced this issue Apr 14, 2024
@88250
🐛 Primary key value unexpectedly updated when database adds row #11018

Verified

This commit was signed with the committer’s verified signature.
88250 D
GPG key ID: 86211BA83DF03017
Verified
Learn about vigilant mode
bc8b295
@88250 88250 closed this as completed Apr 14, 2024
88250 added a commit that referenced this issue Apr 14, 2024
@silent-tan @Vanessa219 @88250
Add database lineNumber field type (#11008)
8a354da 
* 🎨 add database lineNumber type

* 🎨 fix #10896

* 🎨 Improve mobile app appearance language #11009

* 🎨 Improve database template field calc #11011

* ⬆️ Upgrade kernel deps

* ⬆️ Upgrade kernel deps

* 🎨 fix #10896

* 🎨 fix #10896

* 🐛 #11015

* 🐛 #11018

* 🐛 #11018

* 🎨 刚创建时无 id，更新需和 oldValue 保持一致

* 🐛 Database date field between filter calculation error Fix #10979

* 🐛 Primary key value unexpectedly updated when database adds row #11018

* 🎨 #11013

* 🎨 搜索可汇总字段时排除行号类型字段 #11008

* 🎨 lineNumber no need to join the calc op and remove useless todo tag

---------

Co-authored-by: Vanessa <lly219@gmail.com>
Co-authored-by: Daniel <845765@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@88250 88250

@Vanessa219 Vanessa219

Labels
Bug
Projects
None yet
Milestone
3.0.10
Development

No branches or pull requests
3 participants
@88250 @Vanessa219 @TCOTC