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

修改 PageInterceptor 的注册方式 #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

修改 PageInterceptor 的注册方式 #28

wants to merge 1 commit into from

Conversation

drtrang
Copy link
Contributor

@drtrang drtrang commented Oct 30, 2017

主要将 PageInterceptor 的注册方式由从 SqlSessionFactory 改为 mybatis-spring-boot 提供的 ConfigurationCustomizer 方式,便于再添加自定义的拦截器并且指定拦截器的顺序。

  • 升级 mybatis 到 3.4.5
  • 升级 springboot 到 1.5.8.RELEASE
  • 修复 pagehelper-spring-boot-samples 版本号不正确的问题
  • 修改 PageInterceptor 的注册方式为 ConfigurationCustomizer

* 升级 springboot 到 1.5.8.RELEASE
* 修复 pagehelper-spring-boot-samples 版本号不正确的问题
* 修改 PageInterceptor 的注册方式为 ConfigurationCustomizer
@drtrang
Copy link
Contributor Author

drtrang commented Oct 30, 2017

这种方式也没有解决插件顺序的问题,我再研究一下。

@abel533
Copy link
Collaborator

abel533 commented Oct 30, 2017

这种方式在配合官方starter时确实更好。

如果方便,建议在此基础上再做改动,用条件注解支持使用官方starter和不使用的情况。

或者合并后我添加。

@drtrang
Copy link
Contributor Author

drtrang commented Oct 31, 2017

还是大佬亲自动手吧,效果更理想 : )

有个问题大佬看看有办法解决吗,我想自定义拦截器的顺序,比如指定顺序为 PageInterceptor、SqlFormatterInterceptor,PageInterceptor 是分页插件自带的,如果要在它之后增加一个拦截器应该怎么做?

目前 SqlSessionFactory 的注册方式没想到好办法。
新版 ConfigurationCustomizer 的注册方式也指定不了顺序,我尝试了 @Order@DependsOn 注解都没用。具体方式是我声明了多个 ConfigurationCustomizer 的 Spring Bean 并用 @Order 注解修饰,但是构造 MybatisAutoConfiguration 对象时的 interceptors 参数并不是按照 @Order 顺序来的。

@fengcbo
Copy link

fengcbo commented Mar 6, 2018

这个PR什么时候能合并?

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.

3 participants