persist data

[Arian Zand]

Hi,
A few weeks ago I posted my problem I could not persist my records.
I still have this problem but I fixed some errors.

I have a Form with a few textboxes to make a reservation for a hotel.
It's need your name,firstame, arrival etc...

When I press on my save button it doesnt persist and i dont know why

I put some system.outprintlns to check my function does receive the parameters and it does ....

I worked on this way:

My Reservationcontroller

package eu.test.test4.controller;

import javax.validation.Valid;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Controller;
import org.springframework.ui.ModelMap;
import org.springframework.validation.BindingResult;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;

import be.hub.web4.dao.ReservationDAO;
import be.hub.web4.model.Reservation;

@Controller
public class ReservationController {
	@Autowired
	private ReservationDAO reservationdao;
	
	@RequestMapping(value={"/newReservation"},method=RequestMethod.GET)
    public String ReservationFormulier(ModelMap model){
        Reservation reservation = new Reservation();
        model.addAttribute("dereservatie", reservation);
        return "/newReservation";
    }
	
	@RequestMapping(value={"/newReservation"},method=RequestMethod.POST)
    public String ReservationMaken(@ModelAttribute("dereservatie") @Valid Reservation reservation,
                                                        BindingResult result, ModelMap model){
		System.out.println("Controller wordt opgeroepen");
	
	
      reservationdao.saveReservation(reservation.getRoomId(),reservation.getFirstname(),
       reservation.getLastname(),reservation.getArrival(),reservation.getDeparture(),reservation.getNumberOfNights(),reservation.getNumberOfPersons(),reservation.getPrice());
       
      System.out.println("Controllertest second time : " + reservation.getFirstname() );
      
       return "home";
    }
	
	
	
}

My ReservationDao

package eu.test.test4.dao;
import eu.test.test4.Reservation;


public interface ReservationDAO 
{
	public Reservation saveReservation(int roomId, String firstname, String lastname, String arrival, String departure,String numberOfNights,String numberOfPersons,double price);
	
}

My controllerImplementation

package eu.test.test4.dao;

import javax.persistence.EntityManager;
import javax.persistence.PersistenceContext;


import org.springframework.stereotype.Repository;
import org.springframework.transaction.annotation.Transactional;

import be.hub.web4.model.Reservation;

@Repository
@Transactional
public class ReservationDAOcoll implements ReservationDAO{
	
	@PersistenceContext
	private EntityManager em;
	
	//@Transactional
	public Reservation saveReservation(int roomId, String firstname,
			String lastname, String arrival, String departure,
			String numberOfNights, String numberOfPersons, double price) {

		Reservation reservation = new Reservation();
		
		reservation.setRoomId(roomId);
		reservation.setFirstname(firstname);
		
		System.out.println("The firstname: " + reservation.getFirstname());
		reservation.setLastname(lastname);
		reservation.setArrival(arrival);
		reservation.setDeparture(departure);
		reservation.setNumberOfNights(numberOfNights);
		reservation.setNumberOfPersons(numberOfPersons);
		reservation.setPrice(price);
		System.out.println("ReservationDaoImpl saveReservation is called" + firstname + " " + lastname + " successfully booked:" + arrival + " till: " + departure + " #nights: " + numberOfNights + " #persons:" + numberOfPersons + " Price: " + price);
		//em.persist(reservation);
		em.merge(reservation);
		return reservation;
	}
}

My domain

package eu.test.test4.model;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
//import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.Table;
//import javax.validation.constraints.Digits;
import org.hibernate.validator.constraints.NotEmpty;



//import org.hibernate.validator.constraints.*;


@Entity
@Table(name = "tblreservations")
public class Reservation {
	@Id@GeneratedValue(strategy = GenerationType.AUTO)
	@Column(name = "reservationId")
	private int reservationId;
	
	@Column(name = "roomId")
	private int roomId;
	
	//@Length(min = 2)
	@Column(name = "firstname", nullable = false)
	@NotEmpty(message="Vul naam in aub")
	private String firstname;
	

	//@Length(min = 4)
	@Column(name = "lastname")
	private String lastname;
	
	@Column(name = "arrival")
	//@NotEmpty
	private String arrival;
	
	@Column(name = "departure")
	//@NotEmpty
	private String departure;
	
	//@Digits(integer = 3, fraction = 0)
	@Column(name = "numNights")
	private String numberOfNights;
	

	//@Digits(integer = 3, fraction = 0)
	@Column(name = "numPers")
	private String numberOfPersons;
	
	@Column(name = "price")
	private double price;
	
public Reservation(){
		
	}
	public Reservation(int roomId, String firstname, String lastname, String arrival,String departure, String numberOfNights, String numberOfPersons, double price){
		this.roomId = roomId;
		this.firstname = firstname;
		this.lastname = lastname;
		this.arrival = arrival;
		this.departure = departure;
		this.numberOfNights = numberOfNights;
		this.numberOfPersons = numberOfPersons;
		this.price = price;
	}

	public int getReservationId() {
		return reservationId;
	}

	public void setReservationId(int reservationId) {
		this.reservationId = reservationId;
	}

	public int getRoomId() {
		return roomId;
	}

	public void setRoomId(int roomId) {
		this.roomId = roomId;
	}

	public String getFirstname() {
		return firstname;
	}

	public void setFirstname(String firstname) {
		this.firstname = firstname;
	}

	public String getLastname() {
		return lastname;
	}

	public void setLastname(String lastname) {
		this.lastname = lastname;
	}

	public String getArrival() {
		return arrival;
	}

	public void setArrival(String arrival) {
		this.arrival = arrival;
	}

	public String getDeparture() {
		return departure;
	}

	public void setDeparture(String departure) {
		this.departure = departure;
	}

	public String getNumberOfNights() {
		System.out.println("Get number of nights is called" + "  " + numberOfNights);
		return numberOfNights;
	}

	public void setNumberOfNights(String numberOfNights) {
		System.out.println("Set number of nights is called");
		this.numberOfNights = numberOfNights;
	}

	public String getNumberOfPersons() {
		return numberOfPersons;
	}

	public void setNumberOfPersons(String numberOfPersons) {
		this.numberOfPersons = numberOfPersons;
	}

	public double getPrice() {
		return price;
	}

	public void setPrice(double price) {
		this.price = price;
	}
}

My view

<%@taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
<%@taglib prefix="form" uri="http://www.springframework.org/tags/form" %>

<html>
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
        
        
    </head>
    
    <body>
        <p>New Reservation</p>
        <c:url var="url" value="/newReservation.html" />
        <form:form action="${url}" commandName="dereservatie">   <%-- Spring form tags --%>
            <fieldset>
                <div>
                    <label>roomId:</label><form:input path="roomId"/>
                    
                </div>
                <div>
                    <label>firstname:</label><form:input path="firstname"/>
                    
                </div>
                <div>
                    <label>lastname:</label><form:input path="lastname"/>
                    
                </div>
                <div>
                    <label>arrival:</label><form:input path="arrival"/>
                    
                </div>
                <div>
                    <label>departure:</label><form:input path="departure"/>
                    
                </div>
                <div>
                    <label>numberOfNights:</label><form:input path="numberOfNights"/>
                    
                </div>
                
                 <div>
                    <label>numberOfPersons:</label><form:input path="numberOfPersons"/>
                    
                </div>
                 <div>
                    <label>price:</label><form:input path="price"/>
                    
                </div>

                <div><input name="submit" type="submit" value="save"/></div>
            </fieldset>
        </form:form>
    </body>
</html>

Thanks!

[Mark Spritzler]

OK, so you are saying your Reservation parameter in your Controller POST method is being populated.

You see your printlns show in the console.

Now what about setting a break point? can you do debugging? The thing we have to check is to see if you have a Transaction. If you can set a break point in the first line of your controller method. Then look at the Controller's Dao reference. If in the debugger it says it is being Proxied, then you have a Proxy.

Basically the code looks fine, although I didn't take a fined tooth comb to it.

The other thing you should post is your configuration of your EntityManagerFactory, and also your TransactionManager configuration as well as your DataSource. Also you have <tx:annotation-driven/> in your config?

Mark

[Arian Zand]

Mark Spritzler wrote:OK, so you are saying your Reservation parameter in your Controller POST method is being populated.

You see your printlns show in the console.

Now what about setting a break point? can you do debugging? The thing we have to check is to see if you have a Transaction. If you can set a break point in the first line of your controller method. Then look at the Controller's Dao reference. If in the debugger it says it is being Proxied, then you have a Proxy.

Basically the code looks fine, although I didn't take a fined tooth comb to it.

The other thing you should post is your configuration of your EntityManagerFactory, and also your TransactionManager configuration as well as your DataSource. Also you have <tx:annotation-driven/> in your config?

Mark

Hi Mark, Thanks for your reply

How can I set break points in java?
Sorry I'm a newbie in Java...

In My servlet-Context I config my Entity manager (I set my annotation driven in comment because otherwise i get an error of Element of attribute dont match I need to set a xmlns but i dont know which one:

<?xml version="1.0" encoding="UTF-8"?>
<beans xmlns="http://www.springframework.org/schema/beans"
	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:mvc="http://www.springframework.org/schema/mvc"
	xmlns:context="http://www.springframework.org/schema/context"
	xsi:schemaLocation="
        http://www.springframework.org/schema/mvc http://www.springframework.org/schema/mvc/spring-mvc-3.0.xsd
        http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans-3.0.xsd
        http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context-3.0.xsd">

	<!-- DispatcherServlet Context: defines this servlet's request-processing 
		infrastructure -->

	<!-- Scans within the base package of the application for @Components to 
		configure as beans -->
	<!-- @Controller, @Service, @Configuration, etc. -->
	
	
	<context:annotation-config />
	<!-- controller is er bijgekomen -->
	<context:component-scan base-package="be.hub.web4.controller" />

	<!-- Enables the Spring MVC @Controller programming model -->
	<mvc:annotation-driven />

	<bean id="dataSource"
		class="org.springframework.jdbc.datasource.DriverManagerDataSource">
		<property name="driverClassName" value="com.mysql.jdbc.Driver" />
		<property name="url" value="jdbc:mysql://localhost:3306/dbhotel" />
		<property name="username" value="root" />
		<property name="password" value="root" />
	</bean>

	<bean id="entityManagerFactory"
		class="org.springframework.orm.jpa.LocalContainerEntityManagerFactoryBean">
		<property name="dataSource" ref="dataSource" />
		<property name="jpaVendorAdapter">
			<bean class="org.springframework.orm.jpa.vendor.HibernateJpaVendorAdapter">
				<property name="showSql" value="false" />
			</bean>
		</property>
		<property name="persistenceUnitName" value="persistenceUnitHUB"></property>
		<property name="jpaDialect">
			<bean class="org.springframework.orm.jpa.vendor.HibernateJpaDialect">
			</bean>
		</property>
	</bean>

	<bean id="transactionManager" class="org.springframework.orm.jpa.JpaTransactionManager">
		<property name="entityManagerFactory" ref="entityManagerFactory" />
	</bean>
	 <!-- <tx: annotation-driven/> -->
</beans>

[Arian Zand]

PROBLEM SOLVED
I forget adding my xlmns schema and location for my transaction manager

[Bill Gorder]

Glad you got that solved. Just a word of caution the following 2 lines probably do not behave the way you think:

em.merge(reservation);  
        return reservation; 

Merge is different then persist in that it returns the managed entity which may not be the one you passed in. What you probably mean to do is:

return em.merge(reservation);