fix(useAntdTable): reset pageSize (#2085)

[Ali-ovo]

[中文版模板 / Chinese template]

🤔 This is a ...

• New feature
• Bug fix
• Site / documentation update
• Demo update
• TypeScript definition update
• Bundle size optimization
• Performance optimization
• Enhancement feature
• Internationalization
• Refactoring
• Code style optimization
• Test Case
• Branch merge
• Other (about what?)

🔗 Related issue link

fix: #2085

💡 Background and solution

📝 Changelog

Language	Changelog
🇺🇸 English	fix not reset pageSize
🇨🇳 Chinese	修复 reset 的时候没有重置当前 table 的 pageSize

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

• Doc is updated/provided or not needed
• Demo is updated/provided or not needed
• TypeScript definition is updated/provided or not needed
• Changelog is provided or not needed

[Ali-ovo]

修复了这个 issues

[Ali-ovo]

ready for review @crazylxr

[miracles1919]

@Ali-ovo 补个测试用例

[xiaomoziyi]

这个版本修复修出bug了。

如果reset要充值页码，应该用reset defaultPageSize, 而不是defaultParams，defaultParams是作为初次传参使用的。不然的话，refreshDeps的说明是refreshDeps 变化，会重置 current 到第一页，并重新发起请求。现在这个版本，会把current重置到defaultParams的current.
希望尽快修复！！！！ @crazylxr @Ali-ovo

[Ali-ovo]

这个版本修复修出bug了。

如果reset要充值页码，应该用reset defaultPageSize, 而不是defaultParams，defaultParams是作为初次传参使用的。不然的话，refreshDeps的说明是refreshDeps 变化，会重置 current 到第一页，并重新发起请求。现在这个版本，会把current重置到defaultParams的current. 希望尽快修复！！！！ @crazylxr @Ali-ovo

hooks/packages/hooks/src/useAntdTable/index.tsx

Lines 158 to 163 in b92ca0e

	_submit(
	defaultParams?.[0] || {
	pageSize: options.defaultPageSize || 10,
	current: 1,
	},
	);

貌似 defaultParams 和 defaultPageSize 会有一个同时存在的情况 所有导致 reset 有些歧义
目前 reset 的逻辑是优先判断的 defaultParams

[xiaomoziyi]

收到。

[Ali-ovo]

收到。

个人感觉 defaultParams 中 pageSize 的权重会比 defaultPageSize 的权重高， 如有 defaultParams 你应当这么使用

defaultParans:{
    pageSize: xxx
}

而不是重复定义 defaultParams 和 defaultPageSize

[xiaomoziyi]

 defaultParams应该作为初始默认项，而不是重置默认项使用，current作为页码，在重置就应该回到第一页，而不是去读取defaultParams的current。如果想要改变defaultParams的定义，那是不是应该加参数或者配置作为初始值，初始值只是作为页面刷新或者跳转保留历史项使用 发自我的iPhone

…

------------------ Original ------------------ From: ZhuoYang Wu(阿离) ***@***.***> Date: Mon,Apr 24,2023 5:28 PM To: alibaba/hooks ***@***.***> Cc: Fairy ***@***.***>, Comment ***@***.***> Subject: Re: [alibaba/hooks] fix(useAntdTable): reset pageSize (#2085) (PR#2090) 这个版本修复修出bug了。 如果reset要充值页码，应该用reset defaultPageSize, 而不是defaultParams，defaultParams是作为初次传参使用的。不然的话，refreshDeps的说明是refreshDeps 变化，会重置 current 到第一页，并重新发起请求。现在这个版本，会把current重置到defaultParams的current. 希望尽快修复！！！！ @crazylxr @Ali-ovo https://github.com/alibaba/hooks/blob/b92ca0e4a5265d00e50f1b7d5e073ffaa0b20019/packages/hooks/src/useAntdTable/index.tsx#L158-L163 貌似 defaultParams 和 defaultPageSize 会有一个同时存在的情况 所有导致 reset 有些歧义 目前 reset 的逻辑是优先判断的 defaultParams — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: ***@***.***>

[hchlq]

@Ali-ovo 帮忙看下，不行先回滚回去哈