Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Spring JPA repository: prevent update on save

My user DB table looks like this:

CREATE TABLE user (
    username VARCHAR(32) PRIMARY KEY,
    first_name VARCHAR(256) NOT NULL,
    last_name VARCHAR(256) NOT NULL,
    password VARCHAR(32) NOT NULL,
    enabled BOOL
) ENGINE = InnoDB;

This is the field definitions of my entity:

@Entity
public class User implements Serializable {
    private static final long serialVersionUID = 1L;

    @Id
    @Column(nullable = false)
    private String username;

    @Column(nullable = false)
    private String firstName;

    @Column(nullable = false)
    private String lastName;

    @Column(nullable = false)
    private String password;

The field username is the key of my table/entity and it's up to me to set its value. When I need to create another user, I do this in my service:

public User insertUserImpl(String username, String firstName, String lastName) {
    Assert.hasText(username);
    Assert.hasText(firstName);
    Assert.hasText(lastName);

    String password = UUID.randomUUID().toString().substring(0, 4); // temp
    User user = new User(username, password);
    user.setFirstName(firstName);
    user.setLastName(lastName);
    user.setEnabled(false);
    this.userRepository.save(user);
    // FIXME - assegnare un ruolo
    return user;
}

Anyway, if the username is already taken, the repository just do an update, because the specified identifier is not null. This is not the behaviour that I want, I need it to throw something like a duplicate entry exception. Is there any way to prevent it? Do I have to do it by myself? E.g.:

User user = this.userRepository.findOne(username);
if(user != null) {
    throw new RuntimeException("Username already taken"); // FIXME - eccezione applicativa
}
like image 392
Antonio Avatar asked Mar 05 '16 17:03

Antonio


Video Answer


2 Answers

When using the default configuration, and using CrudRepository#save() or JpaRepository#save() it will delegate to the EntityManager to use either persists() if it is a new entity, or merge() if it is not.

The strategy followed to detect the entity state, new or not, to use the appropiate method, when using the default configuration is as follows:

  • By default, a Property-ID inspection is performed, if it is null, then it is a new entity, otherwise is not.
  • If the entity implements Persistable the detection will be delegated to the isNew() method implemented by the entity.
  • There is a 3rd option, implementing EntityInformation, but further customizations are needed.

source

So in your case, as you are using the username as ID, and it isn't null, the Repository call ends up delegating to EntityManager.merge() instead of persist(). So there are two possible solutions:

  • use a diferent ID property, set it to null, and use any auto-generation method, or
  • make User implement Persistable and use the isNew() method, to determine if it is a new entity or not.

If for some reason, you don't want to modify your entities, you can also change the behaviour modifying the flush mode configuration. By default, in spring data jpa, hibernate flush mode is set to AUTO. What you want to do is to change it to COMMIT, and the property to change it is org.hibernate.flushMode. You can modify this configuration by overriding a EntityManagerFactoryBean in a @Configuration class.


And if you don't want to mess the configuration of the EntityManager, you can use the JpaRepository#flush() or JpaRepository#saveAndFlush() methods, to commit the pending changes to the database.

like image 113
saljuama Avatar answered Oct 19 '22 22:10

saljuama


One can perhaps use existsById(ID primaryKey) to test it, if userRepository extends CrudRepository:

if(userRepository.existsById(username)){
    //Throw your Exception
} else {
    this.userRepository.save(user);
}

see https://docs.spring.io/spring-data/jpa/docs/current/reference/html/

like image 33
TGU Avatar answered Oct 20 '22 00:10

TGU