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

🐛排序 按汇总排序 结果错乱 #2487

Closed
4 tasks
shuangxunian opened this issue Dec 20, 2023 · 6 comments · Fixed by #2488
Closed
4 tasks

🐛排序 按汇总排序 结果错乱 #2487

shuangxunian opened this issue Dec 20, 2023 · 6 comments · Fixed by #2488
Assignees
Labels

Comments

@shuangxunian
Copy link
Contributor

🏷 Version

Package Version
@antv/s2 1.54.1
@antv/s2-react
@antv/s2-vue

Sheet Type

  • [ x ] PivotSheet
  • TableSheet
  • GridAnalysisSheet
  • StrategySheet
  • EditableSheet

🖋 Description

1.54.1版本的按值排序出来的结果是乱的,当降到1.49.0时正常

⌨️ Code Snapshots

官网链接:
https://s2.antv.antgroup.com/zh/examples/analysis/sort/#custom-totals

代码链接:
https://pastebin.ubuntu.com/p/NHw3BFmcSw/

仅是s2DataConfig不同,其它相同。

🔗 Reproduce Link

🤔 Steps to Reproduce

将代码执行,可发现 sortParams[0] 下的 sortMethod 为 desc,即应为降序,但是对应的排在第一个的 count_1[&]origin 的值为1,5在1的后面。

😊 Expected Behavior

正确的结果:desc时5应该为第一个(1.49.0即这样),

😅 Current Behavior

💻 System information

Environment Info
System macOS 12.3
Browser chrome: v120.0
@wjgogogo wjgogogo added the 👀 need more info Issue/pr 描述太少, 没有可用信息 label Dec 20, 2023
Copy link
Contributor

你好 @shuangxunian,你所提供的信息不足于我们排查问题, 请按照 issue 模板填写相关信息, 提供 gif, 截图, 代码片段, 配置信息, 版本号, 可复现链接等方式, 详细说明复现步骤, 感谢配合, 谢谢! 15 天内未回复issue自动关闭。

Hello, @shuangxunian, the information you provided is not enough for us to troubleshoot the problem. Please complete the issue description, provide gifs, screenshots, config, version. And explain the reproduction steps in detail. Thanks so much for your cooperation! The issue will be closed without any replay within 15 days.

@wjgogogo
Copy link
Contributor

[&]是内部使用的保留字段,不建议使用,可能会导致和内部的分隔符冲突,以前之前在排序场景下是ok的,在其他某些场景下也有问题,count_1[&]origin可以改为count_1-origin等形式

@shuangxunian
Copy link
Contributor Author

[&]是内部使用的保留字段,不建议使用,可能会导致和内部的分隔符冲突,以前之前在排序场景下是ok的,在其他某些场景下也有问题,count_1[&]origin可以改为count_1-origin等形式

-这个不太行,会和用户的数据冲突,那要是[xbb]这个呢?可以吗,我在代码里面没有搜到对应的关键字

@github-actions github-actions bot removed the 👀 need more info Issue/pr 描述太少, 没有可用信息 label Dec 21, 2023
@wjgogogo
Copy link
Contributor

wjgogogo commented Dec 21, 2023

[&]是内部使用的保留字段,不建议使用,可能会导致和内部的分隔符冲突,以前之前在排序场景下是ok的,在其他某些场景下也有问题,count_1[&]origin可以改为count_1-origin等形式

-这个不太行,会和用户的数据冲突,那要是[xbb]这个呢?可以吗,我在代码里面没有搜到对应的关键字

可以的,也可以等 #2488 修复

@shuangxunian
Copy link
Contributor Author

[&]是内部使用的保留字段,不建议使用,可能会导致和内部的分隔符冲突,以前之前在排序场景下是ok的,在其他某些场景下也有问题,count_1[&]origin可以改为count_1-origin等形式

-这个不太行,会和用户的数据冲突,那要是[xbb]这个呢?可以吗,我在代码里面没有搜到对应的关键字

可以的,也可以等 #2488 修复

万分感谢 也就是说我还是可以继续使用[&]这个字段吗,刚刚前后端共同查了一下,发现改动[&] => [xbb]的话代价有点大,也有风险。

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-v1.54.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants