119

I am using Spring 5 in my project. Until today there was available method CrudRepository#findOne.

But after downloading latest snapshot it suddenly disappeared! Is there any reference that the method is not available now?

My dependency list:

apply plugin: 'java'
apply plugin: 'org.springframework.boot'
apply plugin: 'io.spring.dependency-management'


repositories {
    mavenCentral()
    maven { url "https://repo.spring.io/snapshot" }
    maven { url "https://repo.spring.io/milestone" }
}    

dependencies {
    compile 'org.springframework.boot:spring-boot-starter-data-jpa'

    runtime 'com.h2database:h2:1.4.194'
}

UPDATE:

Seems that this method has been replaced with CrudRepository#findById

5 Answers 5

179

Please see DATACMNS-944 which is associated to this commit which has the following renames

╔═════════════════════╦═══════════════════════╗
║      Old name       ║       New name        ║
╠═════════════════════╬═══════════════════════╣
║ findOne(…)          ║ findById(…)           ║
╠═════════════════════╬═══════════════════════╣
║ save(Iterable)      ║ saveAll(Iterable)     ║
╠═════════════════════╬═══════════════════════╣
║ findAll(Iterable)   ║ findAllById(…)        ║
╠═════════════════════╬═══════════════════════╣
║ delete(ID)          ║ deleteById(ID)        ║
╠═════════════════════╬═══════════════════════╣
║ delete(Iterable)    ║ deleteAll(Iterable)   ║
╠═════════════════════╬═══════════════════════╣
║ exists()            ║ existsById(…)         ║
╚═════════════════════╩═══════════════════════╝
3
115

Note that findById is not an exact replacement for findOne, it returns an Optional instead of null.

Being not very familiar with new java things it took me a little while to figure out, but this turns the findById behavior into the findOne one:

return rep.findById(id).orElse(null);
3
  • 1
    Not the best idea: your code will continue working but you're not using the API as you should. Optional was added to clean the code from all the null checks. Just change the return type of you method and use Optional like a good boy-scout should
    – wi2ard
    Jun 5, 2018 at 16:02
  • 6
    @GabiM This would be great if you have control of all the methods downstream. Even if you do have control of everything downstream and your project is not a dependency for some other third party project, if the methods downstream code for null (as in, create if not exists, or do some logic if missing), then you have to fix them as well. Jun 19, 2018 at 18:13
  • 1
    In reference to the link from @GabiM, I just wanted to point out that even that link says "It is important to note that the intention of the Optional class is not to replace every single null reference" Feb 20, 2020 at 14:47
45

We had many hundreds of usages of the old findOne() method. Rather than embark on a mammoth refactor, we ended up creating the following intermediary interface and had our repositories extend it instead of extending JpaRepository directly

@NoRepositoryBean
public interface BaseJpaRepository<T, ID> extends JpaRepository<T, ID> { 
    default T findOne(ID id) { 
        return (T) findById(id).orElse(null); 
    } 
} 
2
  • 1
    Top solution for me. No need for casting. return findById(id).orElse(null); suffices
    – Ken007
    Jan 3, 2020 at 7:05
  • Totally agree. Saved me hundreds of lines of changes. May 5, 2020 at 22:29
8

A pragmatic transform

Old way:

Entity aThing = repository.findOne(1L);

New way:

Optional<Entity> aThing = repository.findById(1L);
1

An other way more. Add a @Query. Even if I would prefer to consume the Optional:

@Repository
public interface AccountRepository extends JpaRepository<AccountEntity, Long> {

    @Query("SELECT a FROM AccountEntity a WHERE a.id =? 1")
    public AccountEntity findOne(Long id);

}

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.