Spring Boot JPA, Repository don't delete record

Question

I make method ( acceptUseroffermapping ) in a REST-controller ( UserOfferController ) in which I want to delete record in the DB ( UserOfferMapping table). But the problem is that record not deleted and relation also saved after I run this method.

I have also UserOfferMapping class which maps to User class. In UserOfferController I manipulate with UserOfferMapping : creating, selecting records from DB and also trying to delete records but have fail.

UserOfferController.java:

/*...*/
    @POST
    @RequestMapping("/acceptUserOfferMapping")
    public void acceptUseroffermapping(@RequestBody Map<String,
            String> body) throws ParseException {

        String userId = body.get("userId");
        String offerId = body.get("offerId");

        Optional<User> user = userRepository.findById(Integer.parseInt(userId));

        UserOfferMapping mapping = userOfferMappingRepository.getById(Integer.parseInt(userId));
        user.get().getUserOfferMapping().remove(mapping);
        userRepository.save(user.get());
        userOfferMappingRepository.deleteById(Integer.parseInt(offerId));
    }
/*...*/

User.java:

/*some imports*/

@Entity
@Table(name = "User")
@JsonIgnoreProperties({"hibernateLazyInitializer", "handler"})
public class User {
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @Column(name = "id")
    private int id;
/* ...
 * a lot of fields
 * ...
 */
    // Important section which describes all Role Project and Skill mapping
    @OneToMany(mappedBy = "user", cascade = CascadeType.ALL, orphanRemoval = true, fetch = FetchType.LAZY)
    private Set<UserUserrolemapping> userrolemapings = new HashSet<>();

    @OneToMany(mappedBy = "user", cascade = CascadeType.ALL, orphanRemoval = true, fetch = FetchType.LAZY)
    private Set<Userprojectmapping> userprojectmappings = new HashSet<>();

    @OneToMany(mappedBy = "user", cascade = CascadeType.ALL, orphanRemoval = true, fetch = FetchType.LAZY)
    private Set<UserOfferMapping> userOfferMapping = new HashSet<>();

    @OneToMany(mappedBy = "user", cascade = CascadeType.ALL, orphanRemoval = true, fetch = FetchType.LAZY)
    @OrderBy
    private Set<Userskillmapping> userskillmappings = new HashSet<>();

 /* ...
  *  a lot of fields too
  * ...
  */

  /* getter and setters */
}

UserOfferMappingRepository.java:

public interface UserOfferMappingRepository extends JpaRepository<UserOfferMapping, Integer> {
    public List<UserOfferMapping> getAllByUser(Optional<User> user);

    public UserOfferMapping getUserOfferMappingByUserAndProjectAndUserRole(User user, Userproject userproject, Userrole userrole);

    public UserOfferMapping getById(int id);

    public void deleteById(int id);
}

UserOfferMapping.java:

@Entity
public class UserOfferMapping {
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private int id;
    @ManyToOne
    @JoinColumn(name = "userid")
    @JsonBackReference
    private User user;

    @ManyToOne
    @JoinColumn(name = "roleid")
    private Userrole userRole;

    @ManyToOne
    @JoinColumn(name = "projectid")
    private Userproject project;
    @Column(name = "fromdate", nullable = true)
    private Date fromdate;
    @Column(name = "todate", nullable = true)
    private Date todate;

    @Column(name = "chance")
    private int chance;
    @Column(name = "percent")
    private int percent;

    public int getId() {
        return id;
    }

    public User getUser() {
        return user;
    }

    public Userrole getUserRole() {
        return userRole;
    }

    public Userproject getProject() {
        return project;
    }

    public Date getFromdate() {
        return fromdate;
    }

    public int getChance() {
        return chance;
    }

    public int getPercent() {
        return percent;
    }

    public void setId(int id) {
        this.id = id;
    }

    public void setUser(User user) {
        this.user = user;
    }

    public void setUserRole(Userrole userRole) {
        this.userRole = userRole;
    }

    public void setProject(Userproject project) {
        this.project = project;
    }

    public void setFromdate(Date fromdate) {
        this.fromdate = fromdate;
    }

    public void setChance(int chance) {
        this.chance = chance;
    }

    public void setPercent(int percent) {
        this.percent = percent;
    }

    public void setTodate(Date todate) {
        this.todate = todate;
    }

    public Date getTodate() {
        return todate;
    }
}

Answer 1

Can you try to use this

  public interface UserOfferMappingRepository extends JpaRepository<UserOfferMapping, Integer> {
    public List<UserOfferMapping> getAllByUser(Optional<User> user);

    public UserOfferMapping getUserOfferMappingByUserAndProjectAndUserRole(User user, Userproject userproject, Userrole userrole);

    public UserOfferMapping getById(int id);

   // public void deleteById(int id);

   @Modifying(clearAutomatically = true)
  @Query(value = "Delete from UserOfferMapping c WHERE c.id=:id")
  public void deleteById(@Param("id") int id);
}

Answer 2

So, you have bidirectional entity association. Try to add mapping.setUser(null); before userRepository.save .

Answer 3

Persisting and deleting objects requires a transaction in JPA so that is why you have to define @Transactional annotation before the method in Repository for example `

@Transactional
 public void deleteById(@Param("id") int id);