You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openmeetings.apache.org by so...@apache.org on 2013/09/08 09:37:11 UTC

svn commit: r1520843 - /openmeetings/trunk/singlewebapp/src/main/java/org/apache/openmeetings/axis/services/RoomWebService.java

Author: solomax
Date: Sun Sep  8 07:37:11 2013
New Revision: 1520843

URL: http://svn.apache.org/r1520843
Log:
[OPENMEETINGS-784] PropertyUtils is replaces with BeanWrapper to properly set bean property

Modified:
    openmeetings/trunk/singlewebapp/src/main/java/org/apache/openmeetings/axis/services/RoomWebService.java

Modified: openmeetings/trunk/singlewebapp/src/main/java/org/apache/openmeetings/axis/services/RoomWebService.java
URL: http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/main/java/org/apache/openmeetings/axis/services/RoomWebService.java?rev=1520843&r1=1520842&r2=1520843&view=diff
==============================================================================
--- openmeetings/trunk/singlewebapp/src/main/java/org/apache/openmeetings/axis/services/RoomWebService.java (original)
+++ openmeetings/trunk/singlewebapp/src/main/java/org/apache/openmeetings/axis/services/RoomWebService.java Sun Sep  8 07:37:11 2013
@@ -18,6 +18,8 @@
  */
 package org.apache.openmeetings.axis.services;
 
+import static org.apache.openmeetings.OpenmeetingsVariables.webAppRootKey;
+
 import java.util.Calendar;
 import java.util.Date;
 import java.util.HashMap;
@@ -27,8 +29,6 @@ import java.util.List;
 import java.util.Map;
 
 import org.apache.axis2.AxisFault;
-import org.apache.commons.beanutils.PropertyUtils;
-import org.apache.openmeetings.OpenmeetingsVariables;
 import org.apache.openmeetings.data.basic.AuthLevelUtil;
 import org.apache.openmeetings.data.basic.SessiondataDao;
 import org.apache.openmeetings.data.beans.basic.SearchResult;
@@ -52,6 +52,8 @@ import org.apache.openmeetings.utils.Tim
 import org.apache.openmeetings.utils.math.CalendarPatterns;
 import org.red5.logging.Red5LoggerFactory;
 import org.slf4j.Logger;
+import org.springframework.beans.BeanWrapper;
+import org.springframework.beans.BeanWrapperImpl;
 import org.springframework.beans.factory.annotation.Autowired;
 
 /**
@@ -62,9 +64,7 @@ import org.springframework.beans.factory
  * 
  */
 public class RoomWebService {
-
-	private static final Logger log = Red5LoggerFactory.getLogger(
-			RoomWebService.class, OpenmeetingsVariables.webAppRootKey);
+	private static final Logger log = Red5LoggerFactory.getLogger(RoomWebService.class, webAppRootKey);
 
 	@Autowired
 	private AppointmentDao appointmentDao;
@@ -2502,9 +2502,10 @@ public class RoomWebService {
 			Long users_id = sessiondataDao.checkSession(SID);
 			Long user_level = userManager.getUserLevelByID(users_id);
 			if (authLevelUtil.checkWebServiceLevel(user_level)) {
-				log.debug("closeRoom 1 " + room_id);
+				log.debug(String.format("modifyRoomParameter[%s]: %s = %s", room_id, paramName, paramValue));
 				Room r = roomDao.get(room_id);
-				PropertyUtils.setSimpleProperty(r, paramName, paramValue);
+				BeanWrapper rw = new BeanWrapperImpl(r);
+				rw.setPropertyValue(paramName, paramValue);
 				roomDao.update(r, users_id);
 			} else {
 				return -2;