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

增加 eqSql 方法。 #6029

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Conversation

Muyangmin
Copy link
Contributor

该Pull Request关联的Issue

#6028

修改描述

参考 inSql 方法,在 Func、AbstractWrapper和 AbstractChainWrapper 中分别增加了对应的 eqSql 方法定义和实现。

@VampireAchao
Copy link
Contributor

why not com.baomidou.mybatisplus.core.conditions.interfaces.Join#apply(java.lang.String, java.lang.Object...)?

@Muyangmin
Copy link
Contributor Author

Muyangmin commented Mar 29, 2024

You are right, the apply method can do this. But as I mentioned in #6028 ,

  1. Add a special method could reduce string templates and placeholders, which will improve code readability.
  2. There is already inSql, leSql... series method, maybe this eqSql method is better match function semantic consistence?

Just for discussion :)

@nieqiurong nieqiurong merged commit 4162363 into baomidou:3.0 Apr 5, 2024
1 of 2 checks passed
@nieqiurong nieqiurong added this to the v3.5.6 milestone Apr 5, 2024
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