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

[UPDATE] Modify JPA method name #104

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

[UPDATE] Modify JPA method name #104

wants to merge 1 commit into from

Conversation

LOG-INFO
Copy link

@LOG-INFO LOG-INFO commented Jan 11, 2019

I am studying well by listening to your lecture.

Today I listened to your lecture (Section 11, Lecture 223. Fix Find Owner Like). In this lecture, you fixed

ownerService.findAllByLastNameLike(owner.getLastName());

to

ownerService.findAllByLastNameLike("%"+ owner.getLastName() + "%")

I searched for JPA method keyword, and I found Containing keyword.

So I fixed this code like this, and it works fine!

ownerService.findAllByLastNameContaining(owner.getLastName());

Please check this!

Your lecture is very good for me!
Thank you very much!

@LOG-INFO
Copy link
Author

In additional, This is my first PR to other's repository.
Please let me know if this is the right way to 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.

1 participant