[jboss-cvs] jboss-seam/examples/spring/src/org/jboss/seam/example/spring ...

[Michael Youngstrom]

  User: myoungstrom
  Date: 07/07/03 16:37:38

  Modified:    examples/spring/src/org/jboss/seam/example/spring  
                        BookingService.java UserService.java
  Log:
  Fixes for:
  JBSEAM-1575
  JBSEAM-1583
  
  Revision  Changes    Path
  1.9       +9 -2      jboss-seam/examples/spring/src/org/jboss/seam/example/spring/BookingService.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: BookingService.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/examples/spring/src/org/jboss/seam/example/spring/BookingService.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -b -r1.8 -r1.9
  --- BookingService.java	28 Mar 2007 22:46:50 -0000	1.8
  +++ BookingService.java	3 Jul 2007 20:37:38 -0000	1.9
  @@ -8,6 +8,8 @@
   import javax.persistence.EntityManager;
   import javax.persistence.PersistenceException;
   
  +import org.jboss.seam.annotations.Logger;
  +import org.jboss.seam.log.Log;
   import org.springframework.orm.jpa.JpaCallback;
   import org.springframework.orm.jpa.support.JpaDaoSupport;
   import org.springframework.transaction.annotation.Transactional;
  @@ -20,9 +22,12 @@
   public class BookingService 
       extends JpaDaoSupport 
   {
  +	@Logger
  +	private static Log logger;
       @SuppressWarnings("unchecked")
       @Transactional
       public List<Hotel> findHotels(final String searchPattern, final int firstResult, final int maxResults) {
  +    	logger.debug("Looking for a Hotel.");
           return getJpaTemplate().executeFind(new JpaCallback() {
           	public Object doInJpa(EntityManager em) throws PersistenceException {
                   return em.createQuery("select h from Hotel h where lower(h.name) like :search or lower(h.city) like :search or lower(h.zip) like :search or lower(h.address) like :search")
  @@ -39,6 +44,7 @@
       @SuppressWarnings("unchecked")
       @Transactional
       public List<Booking> findBookingsByUsername(String username) {
  +    	logger.debug("Finding Bookings for user {0}", username);
           return getJpaTemplate().findByNamedParams("select b from Booking b where b.user.username = :username order by b.checkinDate",
                                                     Collections.singletonMap("username", username));
   
  @@ -46,6 +52,7 @@
   
       @Transactional
       public void cancelBooking(Long bookingId) {
  +    	logger.debug("Cancelling booking id: {0}", bookingId);
           if (bookingId == null) {
               throw new IllegalArgumentException("BookingId cannot be null");
           }
  @@ -85,10 +92,10 @@
       public void testNonWebRequest() {
       	List<Hotel> hotels = findHotels("%", 0, 1);
       	if(!hotels.isEmpty()) {
  -    		System.out.println("Asynchronously found hotel: "+hotels.get(0).getName());
  +        	logger.info("Asynchronously found hotel: {0}", hotels.get(0).getName());
       		return;
       	}
  -    	System.out.println("No Hotels Found.");
  +    	logger.info("No Hotels Found.");
       }
   
       @Transactional
  
  
  
  1.8       +7 -1      jboss-seam/examples/spring/src/org/jboss/seam/example/spring/UserService.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: UserService.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/examples/spring/src/org/jboss/seam/example/spring/UserService.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -b -r1.7 -r1.8
  --- UserService.java	25 Jun 2007 23:13:52 -0000	1.7
  +++ UserService.java	3 Jul 2007 20:37:38 -0000	1.8
  @@ -4,7 +4,9 @@
   import javax.persistence.PersistenceContext;
   import javax.persistence.PersistenceException;
   
  +import org.jboss.seam.annotations.Logger;
   import org.jboss.seam.annotations.async.Asynchronous;
  +import org.jboss.seam.log.Log;
   import org.springframework.aop.framework.AopContext;
   import org.springframework.aop.support.AopUtils;
   import org.springframework.transaction.annotation.Transactional;
  @@ -14,6 +16,9 @@
    *
    */
   public class UserService {
  +	@Logger
  +	private static Log logger;
  +
   	public static ThreadLocal<Boolean> currentThread = new ThreadLocal<Boolean>();
   
   	@PersistenceContext
  @@ -68,6 +73,7 @@
               throw new ValidationException("Username "+user.getUsername()+" already exists");
           }
           entityManager.persist(user);
  +        logger.info("Created a new User: {0}", user.getName());
       }
   	
   	@Asynchronous
  @@ -75,6 +81,6 @@
   		if(currentThread.get() != null) {
   			throw new RuntimeException("Not really happening asyncrohously");
   		}
  -		System.out.println("pretending to send email asyncronously");
  +		logger.info("pretending to send email asyncronously");
   	}
   }