Spring Framework can not save to the database by using EntityManager.merge ()

advertisements

I’m trying to set up a website using Spring. I already managed to read data from a database I've set up, but now encountered a problem when trying to update rows in it. I'm using the merge() function of the EntityManager. I don't get any errors or exceptions, it just wont update the data on the DB.

I'll try to give all relevant code.

The entity class I want to change: I already managed to fill it with data from the DB, so I think it should be alright.

Benutzer.java

@Entity
@Table(name = "BENUTZER")
    @NamedQueries({
    @NamedQuery(name = "Benutzer.findAll", query = "SELECT tt FROM Benutzer tt"),
    @NamedQuery(name = "Benutzer.findByKennung", query = "SELECT tt FROM Benutzer tt WHERE tt.kennung = :kennung"),
public class Benutzer {

@Id
@GeneratedValue
private BigInteger oid;

@Size(min = 2, max = 20)
@Pattern(regexp = "[A-Za-z ]*", message = "Titel darf nur Buchstaben und Leerzeichen beinhalten")
private String titel;

@NotNull
@Size(min = 2, max = 25)
@Pattern(regexp = "[A-Za-z ]*", message = "Anrede darf nur Buchstaben und Leerzeichen beinhalten")
private String anrede;

@NotNull
@Size(min = 2, max = 45)
@Pattern(regexp = "[A-Za-z ]*", message = "Name darf nur Buchstaben und Leerzeichen beinhalten")
private String name;

@NotNull
@Size(min = 2, max = 45)
@Pattern(regexp = "[A-Za-z ]*", message = "Vorname darf nur Buchstaben und Leerzeichen beinhalten")
private String vorname;

@NotNull
@Size(min = 6, max = 16)
@Pattern(regexp = "[0-9]*", message = "Telefonnummer darf nur aus Zahlen bestehen")
private String telefonnummer;

@Size(max = 25, message = "Fax darf maximal 25 Zahlen beinhalten")
@Pattern(regexp = "[0-9]*", message = "Fax darf nur aus Zahlen bestehen")
private String fax;

@NotNull
@Size(min = 2, max = 45)
@Pattern(regexp = "[A-Za-z0-9 ]*", message = "Straße darf nur Buchstaben, Leerzeichen und Hausnummer beinhalten")
private String strasse;

@NotNull
@Size(min = 2, max = 45)
@Pattern(regexp = "[A-Za-z0-9 ]*", message = "Ort darf nur Buchstaben, Leerzeichen und PLZ beinhalten")
private String ort;

@NotNull
@Size(min = 2, max = 45, message = "E-mail muss zwischen 2 und 45 Zeichen beinhalten")
private String email;

private boolean freigeschaltet;

@Temporal(TemporalType.TIMESTAMP)
private Date sperrbeginn;

@NotNull(message = "Kennung darf nicht leer sein")
private String kennung;

@NotNull
private String passwort;

@ManyToMany(fetch = FetchType.EAGER)
@JoinTable(name = "BENUTZER_has_KUNDE", joinColumns = { @JoinColumn(name = "BENUTZER_oid") }, inverseJoinColumns = { @JoinColumn(name = "KUNDE_oid") })
private List<Kunde> kundenListe;

public Benutzer() {

};

+getter and setter
}

}

The DAO : Im using the update(Benutzer benutzer) method in it to try updating the apropriate collumn in the DB.

BenutzerDaoImpl.java

@Transactional
@Repository("benutzerDao")
public class BenutzerDaoImpl implements BenutzerDao{

private EntityManager em;

public EntityManager getEm() {
    return em;
}

@PersistenceContext (type = PersistenceContextType.TRANSACTION)
public void setEm(EntityManager em) {
    this.em = em;
}

public Benutzer findById(BigInteger id) {
    return em.find(Benutzer.class, id);
}

public Benutzer findByKennung(String kennung) {
    Query benutzerByKennung = em.createNamedQuery("Benutzer.findByKennung");
    benutzerByKennung.setParameter("kennung", kennung);
    return (Benutzer) benutzerByKennung.getSingleResult();
}

public List<Benutzer> findAll() {
    CriteriaBuilder cb = em.getCriteriaBuilder();
    CriteriaQuery<Benutzer> criteria = cb.createQuery(Benutzer.class);
    Root<Benutzer> benutzer = criteria.from(Benutzer.class);

    criteria.select(benutzer).orderBy(cb.asc(benutzer.get("name")));
    return em.createQuery(criteria).getResultList();
}

public void register(Benutzer benutzer) {
    em.persist(benutzer);
    return;
}

public void delete(BigInteger id) {
    Benutzer benutzerTemp = findById(id);
    em.remove(benutzerTemp);
}

public void update(Benutzer benutzer) {
    Benutzer benutzerTemp = findById(benutzer.getOid());
    benutzerTemp.setTelefon_vorwahl(benutzer.getTelefon_vorwahl());
    benutzerTemp.setTelefon_nummer(benutzer.getTelefon_nummer());
    benutzerTemp.setFax_vorwahl(benutzer.getFax_vorwahl());
    benutzerTemp.setFax_nummer(benutzer.getFax_nummer());
    benutzerTemp.setStrasse(benutzer.getStrasse());
    benutzerTemp.setStrasse_nummer(benutzer.getStrasse_nummer());
    benutzerTemp.setOrt(benutzer.getOrt());
    benutzerTemp.setEmail(benutzer.getEmail());
    benutzerTemp.setKennung(benutzer.getKennung());
    em.merge(benutzerTemp);
    return;
}

}

And the Controller class that calles the update(Benutzer benutzer) function in saveBenutzerdaten() right at the bottom of the code:

BenutzerdatenController.java

@Controller
@SessionAttributes("activeUser")
public class LoginController {

private BenutzerDao benutzerDao;

private KundeDao kundeDao;

private AnlageDao anlageDao;

@Autowired
public LoginController(BenutzerDao benutzerDao, KundeDao kundeDao, AnlageDao anlageDao){
    this.benutzerDao = benutzerDao;
    this.kundeDao = kundeDao;
    this.anlageDao = anlageDao;
}

@ModelAttribute("activeUser")
   public Benutzer populateActiveUser() {
       return new Benutzer(); // Füllt activeUser beim ersten mall wenn es null ist.
   }

@RequestMapping(value = "/login" ,method = RequestMethod.GET)
public String login(Model model, SessionStatus status) {
    status.setComplete();
    model.addAttribute("benutzerLoginDaten", new Login());
    return "login";
}

@RequestMapping(value = "/login" ,method = RequestMethod.POST)
public String requestLogin(
        @Valid @ModelAttribute("benutzerLoginDaten") Login logindaten,
        BindingResult result, Model model, final RedirectAttributes redirectAttributes) {

    if (!result.hasErrors()) {
        if (logindaten == null) {
            String error = "Fehler beim login";
            model.addAttribute("error", error);
            return "login";
        }
        Benutzer user = null;
        try{
            user = benutzerDao.findByKennung(logindaten.getKennung());
        }catch(Exception e){
            String error = "Unbekannter Benutzer";
            model.addAttribute("error", error);
            return "login";
        }
        if (user == null) {
            String error = "Unbekannter Benutzer";
            model.addAttribute("error", error);
            return "login";
        }
        String pw = user.getPasswort();
        if (pw.equals(logindaten.getPasswort())) {
            redirectAttributes.addFlashAttribute("activeUser", user);
            return "redirect:/home.html";
        } else {
            String error = "Passwort falsch";
            model.addAttribute("error", error);
            return "login";
        }
    }
    return "login";
}

@RequestMapping(value = "/home",method = RequestMethod.GET)
public String home(@ModelAttribute("activeUser") Benutzer activeUser ,Model model, final RedirectAttributes redirectAttributes){
    if(activeUser == null){
        return "redirect:/login.html";
    }
    if(activeUser.getKennung()==null){
        //falls activeUser == null, hat sich kein Benutzer eingelogt
        //und wird damit auf die loginseite weitergeleitet.
        return "redirect:/login.html";
    }

    //erstellen der Anlagenliste mit dazugehörigem Kunden:
    //i und j werden zum mitzählen der schleifendurchläufe verwenden
    //sum zählt die benotigte größe für die Anlagenliste. (summe der anlagen aller kunden zu denen der benutzer zugriff hat)
    int i=0, j = 0, sum = 0;
    List<Kunde> kunden = activeUser.getKundenListe();
    for(i = 0 ; i< kunden.size(); i++){
        List<Anlage> anlagen = kundeDao.getAllAnlagen(kunden.get(i));
        sum += anlagen.size();
    }

    //auffüllen der attribute für select anlagenListe (beschreibung: anlagenbeschreibung, ids: anlagenids):
    String[] beschreibung = new String[sum];
    BigInteger[] ids = new BigInteger[sum];

    Select anlagenListe = new Select();
    sum = 0;
    for(i = 0 ; i < kunden.size(); i++){
        List<Anlage> anlagen = kundeDao.getAllAnlagen(kunden.get(i));
        for(j = 0; j <anlagen.size(); j++){
            //beschreibung format : Anlagenbezeichnung (kunde)
            beschreibung[sum] =
                    ""+anlagen.get(j).getBezeichnung() +
                    " (" +
                            ((kunden.get(i).isFirma())?
                                    kunden.get(i).getFirmenname()
                                    :
                                        kunden.get(i).getPrivatperson_nachname() +
                                    " " +
                                    kunden.get(i).getPrivatperson_vorname()) +
                     ")";
             ids[sum++]= anlagen.get(j).getOid();
         }

     }
    anlagenListe.setSelectOptions(beschreibung);
    anlagenListe.setIds(ids);
    model.addAttribute("activeUser", activeUser);
    model.addAttribute("selection", anlagenListe);
    return "home";
}

@RequestMapping(value = "/home", method = RequestMethod.POST)
public String selectAnlage(@Valid @ModelAttribute("selection") Select selection, BindingResult result, Model model) {

    if(!result.hasErrors()) {
        Anlage anlage = anlageDao.findById(selection.getSelectionOid());
        System.out.println("anlage--  id: " + anlage.getOid() + "  Bezeichnung: " + anlage.getBezeichnung());
        return "home";
    }
    return "home";

}

@RequestMapping(value = "/benutzerdaten", method = RequestMethod.GET)
public String benutzerdaten(@ModelAttribute("activeUser") Benutzer benutzer,Model model){
    return "benutzerdaten";
}

@RequestMapping(value = "/benutzerdaten", method = RequestMethod.POST)
public String saveBenutzerdaten(
        @ModelAttribute("activeUser") Benutzer benutzer,
        Model model) {
    benutzerDao.update(benutzer);
    return "benutzerdaten";
}

}

I've been looking for a solution for quite a long time now, and I've also found quite a lot of people having simular problems, but none of their solutions worked for me. I really hope someone finds the problem. Starting to get quite frustrated. Just ask If you need to see some more code or relevant data.

Also, If you notice anything I should change, please tell me. I'm new to Spring and not sure I'm doing everything right. Especially with the way I handle the logged in User (probably not a good solution to save him in the DAO object?).


I've found a solution. It worked fine after I added a service class and added to my mvc-dispatcher-servlet.xml. (sopro.mvc.swm.service is the package where I have my service classes).


Try following :

In controller try not to user private Benutzer currentlyActive as instant variable. At-least in saveBenutzer method do not use currentlyActive object, instead retrive the object id from the model attribute @ModelAttribute("newBenutzerdaten") Benutzer benutzerdaten. Store id in some hidden variable in the web page so it will be available in the modelAttribute, and pass this modelAttribute directly to the update method.