Conversation
fix: 修改报名错误
采用继承的方式重构 Page 分页对象，减少无用代码 1.解决重写了过时方法的问题 2.修复“因没有修改 hashNext()方法”而可能导致的 bug
因为在 application.properties 中指定了 spring.mandatory-file-encoding=UTF-8，FileEncodingApplicationListener 类检测到当前系统编码与 mandatory-file-encoding 不一致的情况下会报错
Page 分页对象重写 offset() 方法。 修复分页指定为 1 时，却返回第 1 页数据的问题 （因起始页为0，正确的应该返回第 2 页的数据）
juzi214032
approved these changes
Jul 16, 2020
colorful3
added a commit
that referenced
this pull request
Jul 17, 2020
* refactor(Page): 重构 Page 分页对象 采用继承的方式重构 Page 分页对象，减少无用代码 1.解决重写了过时方法的问题 2.修复“因没有修改 hashNext()方法”而可能导致的 bug * refactor(UserPermissionVO): 为 UserPermissionVO 的 permissions 属性指定泛型 * refactor(Page): 将 Page 分页对象泛型参数列表补充完整 * chore(pom): 为 Maven 指定 file.encoding 参数，解决运行 mvn test 报错问题 因为在 application.properties 中指定了 spring.mandatory-file-encoding=UTF-8，FileEncodingApplicationListener 类检测到当前系统编码与 mandatory-file-encoding 不一致的情况下会报错 * fix(Page): 修复分页问题 Page 分页对象重写 offset() 方法。 修复分页指定为 1 时，却返回第 1 页数据的问题 （因起始页为0，正确的应该返回第 2 页的数据） * feat: 新增更新用户信息接口对于 group_ids 字段的校验 Co-authored-by: 桔子 <juzi214032@qq.com>
gadfly3173
added a commit
to gadfly3173/chakki-spring
that referenced
this pull request
Jan 29, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.