Author: solomax
Date: Mon Sep  1 09:08:56 2014
New Revision: 1621711

URL: http://svn.apache.org/r1621711
Log:
Code clean-up

Modified:
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/FlvInterviewConverter.java
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/calendar/management/AppointmentLogic.java
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/record/WhiteboardMapToSVG.java
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/EmoticonsManager.java
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/WhiteboardManager.java
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/documents/LibraryChartLoader.java
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/remote/MainService.java
    
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/servlet/outputhandler/AbstractUploadController.java
    
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/room/RoomDao.java
    
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java
    
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dto/room/RoomSearchResult.java
    
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/backup/BackupImport.java
    
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/Admin.java
    
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/OmHelpFormatter.java
    
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/installation/ImportInitvalues.java
    
openmeetings/trunk/singlewebapp/openmeetings-screenshare/src/main/java/org/apache/openmeetings/screen/webstart/gui/ScreenSharerFrame.java
    
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/CalendarPatterns.java
    
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ConverterProcessResultList.java
    
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ProcessHelper.java
    
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/stringhandlers/ChatString.java
    
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/admin/ldaps/LdapsPanel.java
    
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/app/Application.java
    
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java
    
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java
    
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
    
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/user/record/VideoInfo.java
    
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/util/UserDashboardPersister.java
    
openmeetings/trunk/singlewebapp/openmeetings-webservice/src/main/java/org/apache/openmeetings/webservice/UserWebService.java

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/FlvInterviewConverter.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/FlvInterviewConverter.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/FlvInterviewConverter.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/FlvInterviewConverter.java
 Mon Sep  1 09:08:56 2014
@@ -39,7 +39,7 @@ import org.springframework.beans.factory
 
 public class FlvInterviewConverter extends BaseConverter {
        private static final Logger log = 
Red5LoggerFactory.getLogger(FlvInterviewConverter.class, webAppRootKey);
-       private class ReConverterParams {
+       private static class ReConverterParams {
                private int leftSideLoud = 1;
                private int rightSideLoud = 1;
                private Integer leftSideTime = 0;

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/calendar/management/AppointmentLogic.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/calendar/management/AppointmentLogic.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/calendar/management/AppointmentLogic.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/calendar/management/AppointmentLogic.java
 Mon Sep  1 09:08:56 2014
@@ -199,7 +199,7 @@ public class AppointmentLogic {
                        return;
                }
 
-               long milliseconds = (minutesReminderSend * 60 * 1000);
+               long milliseconds = (minutesReminderSend * 60 * 1000L);
                Calendar start = Calendar.getInstance();
                if (milliseconds < 0) {
                        start.setTimeInMillis(start.getTimeInMillis() + 
milliseconds);

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/record/WhiteboardMapToSVG.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/record/WhiteboardMapToSVG.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/record/WhiteboardMapToSVG.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/record/WhiteboardMapToSVG.java
 Mon Sep  1 09:08:56 2014
@@ -68,13 +68,13 @@ public class WhiteboardMapToSVG extends 
                        
                        List pointsList = (List) graphObject.get(1);
                        
-                       Integer lineWidth = 
Integer.valueOf(graphObject.get(3).toString()).intValue();
-                       Integer col = 
Integer.valueOf(graphObject.get(4).toString()).intValue();
+                       Integer lineWidth = 
Integer.valueOf(graphObject.get(3).toString());
+                       Integer col = 
Integer.valueOf(graphObject.get(4).toString());
                        
-                       Float alpha = 
Float.valueOf(graphObject.get(5).toString()).floatValue();
+                       Float alpha = 
Float.valueOf(graphObject.get(5).toString());
                        
-                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString()).doubleValue();
-                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString()).doubleValue();
+                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString());
+                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString());
                        
                        //log.debug("x,y,width,height: 
"+x+","+y+","+width+","+height);
                        
@@ -95,60 +95,60 @@ public class WhiteboardMapToSVG extends 
                        actionObject[5] = fillDis;
                        actionObject[6] = this.currentrectangleOpacity;*/
                        
-                       Integer lineWidth = 
Integer.valueOf(graphObject.get(2).toString()).intValue();
+                       Integer lineWidth = 
Integer.valueOf(graphObject.get(2).toString());
                        
-                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString()).intValue();
-                       Integer strokeDis= 
Integer.valueOf(graphObject.get(4).toString()).intValue();
+                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString());
+                       Integer strokeDis= 
Integer.valueOf(graphObject.get(4).toString());
                        
                        Color strokeColor = null;
                        if (strokeDis != -1) {
                                strokeColor = new Color(stroke);
                        }
                        
-                       Integer fill = 
Integer.valueOf(graphObject.get(3).toString()).intValue();
-                       Integer fillDis= 
Integer.valueOf(graphObject.get(5).toString()).intValue();
+                       Integer fill = 
Integer.valueOf(graphObject.get(3).toString());
+                       Integer fillDis= 
Integer.valueOf(graphObject.get(5).toString());
                        
                        Color fillColor = null;
                        if (fillDis != -1) {
                                fillColor = new Color(fill);
                        }
                        
-                       Float alpha = 
Float.valueOf(graphObject.get(6).toString()).floatValue();
+                       Float alpha = 
Float.valueOf(graphObject.get(6).toString());
                        
-                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString()).doubleValue();
-                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString()).doubleValue();
-                       Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString()).doubleValue();
-                       Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString()).doubleValue();
+                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString());
+                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString());
+                       Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString());
+                       Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString());
                
                        SVGGraphics2D svgGenerator_temp = new 
SVGGraphics2D(svgGenerator);
                        this.paintRect2D(svgGenerator_temp, x, y, width, 
height, strokeColor, lineWidth, fillColor, alpha);
                        
                } else if (graphType.equals("ellipse")) {
                        
-                       Integer lineWidth = 
Integer.valueOf(graphObject.get(2).toString()).intValue();
+                       Integer lineWidth = 
Integer.valueOf(graphObject.get(2).toString());
                        
-                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString()).intValue();
-                       Integer strokeDis= 
Integer.valueOf(graphObject.get(4).toString()).intValue();
+                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString());
+                       Integer strokeDis= 
Integer.valueOf(graphObject.get(4).toString());
                        
                        Color strokeColor = null;
                        if (strokeDis != -1) {
                                strokeColor = new Color(stroke);
                        }
                        
-                       Integer fill = 
Integer.valueOf(graphObject.get(3).toString()).intValue();
-                       Integer fillDis= 
Integer.valueOf(graphObject.get(5).toString()).intValue();
+                       Integer fill = 
Integer.valueOf(graphObject.get(3).toString());
+                       Integer fillDis= 
Integer.valueOf(graphObject.get(5).toString());
                        
                        Color fillColor = null;
                        if (fillDis != -1) {
                                fillColor = new Color(fill);
                        }
                        
-                       Float alpha = 
Float.valueOf(graphObject.get(6).toString()).floatValue();
+                       Float alpha = 
Float.valueOf(graphObject.get(6).toString());
                        
-                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString()).doubleValue();
-                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString()).doubleValue();
-                       Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString()).doubleValue();
-                       Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString()).doubleValue();
+                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString());
+                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString());
+                       Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString());
+                       Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString());
                
                        SVGGraphics2D svgGenerator_temp = new 
SVGGraphics2D(svgGenerator);
                        this.paintEllipse2D(svgGenerator_temp, x, y, width, 
height, strokeColor, lineWidth, fillColor, alpha);
@@ -156,8 +156,8 @@ public class WhiteboardMapToSVG extends 
                } else if (graphType.equals("letter")) {
                        
                        String text = graphObject.get(1).toString();
-                       Color fontColor = new Color 
(Integer.valueOf(graphObject.get(2).toString()).intValue());
-                       Integer fontSize = 
Integer.valueOf(graphObject.get(3).toString()).intValue();
+                       Color fontColor = new Color 
(Integer.valueOf(graphObject.get(2).toString()));
+                       Integer fontSize = 
Integer.valueOf(graphObject.get(3).toString());
                        
                        String fontStyle = graphObject.get(4).toString();
                        Integer style = null;
@@ -173,46 +173,46 @@ public class WhiteboardMapToSVG extends 
                        
                        log.debug("fontStyle,style "+style+" fs: "+fontStyle);
                        
-                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString()).doubleValue();
-                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString()).doubleValue();
-                       Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString()).doubleValue();
-                       Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString()).doubleValue();
+                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString());
+                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString());
+                       Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString());
+                       Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString());
                        
                        SVGGraphics2D svgGenerator_temp = new 
SVGGraphics2D(svgGenerator);
-                       this.paintTextByWidthHeight(svgGenerator_temp, (int) 
Math.round(x), (int) Math.round(y), (int) Math.round(width), 
+                       paintTextByWidthHeight(svgGenerator_temp, (int) 
Math.round(x), (int) Math.round(y), (int) Math.round(width), 
                                                (int) Math.round(height), text, 
style, fontSize, fontColor);
                        
                } else if (graphType.equals("drawarrow")) {
                        
-                       Integer thickness = 
Integer.valueOf(graphObject.get(2).toString()).intValue();
+                       Integer thickness = 
Integer.valueOf(graphObject.get(2).toString());
                        
-                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString()).intValue();
-                       Integer strokeDis= 
Integer.valueOf(graphObject.get(4).toString()).intValue();
+                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString());
+                       Integer strokeDis= 
Integer.valueOf(graphObject.get(4).toString());
                        
                        Color strokeColor = null;
                        if (strokeDis != -1) {
                                strokeColor = new Color(stroke);
                        }
                        
-                       Integer fill = 
Integer.valueOf(graphObject.get(3).toString()).intValue();
-                       Integer fillDis= 
Integer.valueOf(graphObject.get(5).toString()).intValue();
+                       Integer fill = 
Integer.valueOf(graphObject.get(3).toString());
+                       Integer fillDis= 
Integer.valueOf(graphObject.get(5).toString());
                        
                        Color fillColor = null;
                        if (fillDis != -1) {
                                fillColor = new Color(fill);
                        }
                        
-                       Float alpha = 
Float.valueOf(graphObject.get(6).toString()).floatValue();
+                       Float alpha = 
Float.valueOf(graphObject.get(6).toString());
                        
-                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString()).doubleValue();
-                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString()).doubleValue();
+                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString());
+                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString());
                        //Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString()).doubleValue();
                        //Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString()).doubleValue();
                        
-                       Double x1 = 
Double.valueOf(graphObject.get(7).toString()).doubleValue();
-                       Double y1 = 
Double.valueOf(graphObject.get(8).toString()).doubleValue();
-                       Double x2 = 
Double.valueOf(graphObject.get(9).toString()).doubleValue();
-                       Double y2 = 
Double.valueOf(graphObject.get(10).toString()).doubleValue();
+                       Double x1 = 
Double.valueOf(graphObject.get(7).toString());
+                       Double y1 = 
Double.valueOf(graphObject.get(8).toString());
+                       Double x2 = 
Double.valueOf(graphObject.get(9).toString());
+                       Double y2 = 
Double.valueOf(graphObject.get(10).toString());
                        
                        GeomPoint start = new GeomPoint();
                start.setLocation(x+x1,y+y1);
@@ -244,10 +244,10 @@ public class WhiteboardMapToSVG extends 
                                
                                Image myImage = ImageIO.read(myFile);
                                
-                               int x = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-5).toString()).doubleValue());
-                               int y = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-4).toString()).doubleValue());
-                               int width = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-3).toString()).doubleValue());
-                               int height = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-2).toString()).doubleValue());
+                               int x = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-5).toString()));
+                               int y = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-4).toString()));
+                               int width = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-3).toString()));
+                               int height = (int) 
Math.round(Double.valueOf(graphObject.get(graphObject.size()-2).toString()));
                                
                                SVGGraphics2D svgGenerator_temp = new 
SVGGraphics2D(svgGenerator);
                                svgGenerator_temp.drawImage(myImage, x, y, 
width, height, null);
@@ -273,22 +273,22 @@ public class WhiteboardMapToSVG extends 
 //                actionObject[12] = height;  
 //                actionObject[13] = newName;
                        
-                       Integer lineWidth = 
Integer.valueOf(graphObject.get(2).toString()).intValue();
+                       Integer lineWidth = 
Integer.valueOf(graphObject.get(2).toString());
                        
-                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString()).intValue();
+                       Integer stroke = 
Integer.valueOf(graphObject.get(1).toString());
                        Color strokeColor = new Color(stroke);
                        
-                       Float alpha = 
Float.valueOf(graphObject.get(3).toString()).floatValue();
+                       Float alpha = 
Float.valueOf(graphObject.get(3).toString());
                        
-                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString()).doubleValue();
-                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString()).doubleValue();
+                       Double x = 
Double.valueOf(graphObject.get(graphObject.size()-5).toString());
+                       Double y = 
Double.valueOf(graphObject.get(graphObject.size()-4).toString());
                        //Double width = 
Double.valueOf(graphObject.get(graphObject.size()-3).toString()).doubleValue();
                        //Double height = 
Double.valueOf(graphObject.get(graphObject.size()-2).toString()).doubleValue();
 
-                       Double x1 = 
Double.valueOf(graphObject.get(4).toString()).doubleValue();
-                       Double y1 = 
Double.valueOf(graphObject.get(5).toString()).doubleValue();
-                       Double x2 = 
Double.valueOf(graphObject.get(6).toString()).doubleValue();
-                       Double y2 = 
Double.valueOf(graphObject.get(7).toString()).doubleValue();
+                       Double x1 = 
Double.valueOf(graphObject.get(4).toString());
+                       Double y1 = 
Double.valueOf(graphObject.get(5).toString());
+                       Double x2 = 
Double.valueOf(graphObject.get(6).toString());
+                       Double y2 = 
Double.valueOf(graphObject.get(7).toString());
                        
                        SVGGraphics2D svgGenerator_temp = new 
SVGGraphics2D(svgGenerator);
                        this.drawLine(svgGenerator_temp, x1, y1, x2, y2, 
strokeColor, lineWidth, x, y, alpha);

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/EmoticonsManager.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/EmoticonsManager.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/EmoticonsManager.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/EmoticonsManager.java
 Mon Sep  1 09:08:56 2014
@@ -21,6 +21,7 @@ package org.apache.openmeetings.core.dat
 import java.io.BufferedReader;
 import java.io.File;
 import java.io.FileReader;
+import java.io.IOException;
 import java.util.LinkedList;
 
 import org.apache.openmeetings.util.OmFileHelper;
@@ -47,22 +48,24 @@ public class EmoticonsManager {
        private LinkedList<LinkedList<String>> emotfilesList = new 
LinkedList<LinkedList<String>>();
 
        @SuppressWarnings("unchecked")
-       public void loadEmot(){
+       public void loadEmot() {
+               BufferedReader reader = null;
                try {
                        XStream xStream = new XStream(new XppDriver());
                        xStream.setMode(XStream.NO_REFERENCES);
-                       BufferedReader reader = new BufferedReader(new 
FileReader(new File(OmFileHelper.getPublicEmotionsDir(), "emotes.xml")));
-                   String xmlString = "";
-                   while (reader.ready()) {
-                       xmlString += reader.readLine();
-                   }
-                   reader.close();
-                   emotfilesList = 
ChatString.replaceAllRegExp((LinkedList<LinkedList<String>>) 
xStream.fromXML(xmlString));
+                       reader = new BufferedReader(new FileReader(new 
File(OmFileHelper.getPublicEmotionsDir(), "emotes.xml")));
+                   emotfilesList = 
ChatString.replaceAllRegExp((LinkedList<LinkedList<String>>) 
xStream.fromXML(reader));
                    
                    log.debug("##### loadEmot completed");
                    
                } catch (Exception err) {
                        log.error("[loadEmot]",err);
+               } finally {
+                       if (reader != null) {
+                               try {
+                                       reader.close();
+                               } catch (IOException e) {}
+                       }
                }
        }
        

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/WhiteboardManager.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/WhiteboardManager.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/WhiteboardManager.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/data/whiteboard/WhiteboardManager.java
 Mon Sep  1 09:08:56 2014
@@ -24,22 +24,19 @@ import java.util.List;
 import java.util.Map;
 
 import org.apache.openmeetings.db.dto.room.WhiteboardObject;
-import org.apache.openmeetings.util.OpenmeetingsVariables;
+import static org.apache.openmeetings.util.OpenmeetingsVariables.webAppRootKey;
 import org.red5.logging.Red5LoggerFactory;
 import org.slf4j.Logger;
 import org.springframework.beans.factory.annotation.Autowired;
 
 public class WhiteboardManager {
-
-       private static final Logger log = Red5LoggerFactory.getLogger(
-                       WhiteboardManager.class, 
OpenmeetingsVariables.webAppRootKey);
+       private static final Logger log = 
Red5LoggerFactory.getLogger(WhiteboardManager.class, webAppRootKey);
        
        @Autowired
-       private WhiteBoardObjectListManagerById whiteBoardObjectListManagerById;
+       private WhiteBoardObjectListManagerById wbListManagerById;
 
        @SuppressWarnings({ "rawtypes", "unchecked" })
-       public void addWhiteBoardObjectById(Long room_id, Map whiteboardObj,
-                       Long whiteBoardId) {
+       public void addWhiteBoardObjectById(Long room_id, Map whiteboardObj, 
Long whiteBoardId) {
                try {
                        log.debug("addWhiteBoardObjectById: ", whiteboardObj);
 
@@ -59,20 +56,18 @@ public class WhiteboardManager {
                        List actionObject = (List) whiteboardObj.get(3);
 
                        if (action.equals("moveMap")) {
-                               WhiteboardObject whiteboardObject = 
whiteBoardObjectListManagerById
+                               WhiteboardObject whiteboardObject = 
wbListManagerById
                                                
.getWhiteBoardObjectListByRoomIdAndWhiteboard(room_id,
                                                                whiteBoardId);
 
-                               whiteboardObject.setX(Integer.valueOf(
-                                               
actionObject.get(1).toString()).intValue());
-                               whiteboardObject.setY(Integer.valueOf(
-                                               
actionObject.get(2).toString()).intValue());
+                               
whiteboardObject.setX(Integer.valueOf(actionObject.get(1).toString()));
+                               
whiteboardObject.setY(Integer.valueOf(actionObject.get(2).toString()));
 
-                               whiteBoardObjectListManagerById
+                               wbListManagerById
                                                
.setWhiteBoardObjectListRoomObjAndWhiteboardId(room_id,
                                                                
whiteboardObject, whiteBoardId);
                        } else if (action.equals("draw") || 
action.equals("redo")) {
-                               WhiteboardObject whiteboardObject = 
whiteBoardObjectListManagerById
+                               WhiteboardObject whiteboardObject = 
wbListManagerById
                                                
.getWhiteBoardObjectListByRoomIdAndWhiteboard(room_id,
                                                                whiteBoardId);
 
@@ -86,20 +81,20 @@ public class WhiteboardManager {
                                        log.debug("objectOID: " + objectOID);
                                        whiteboardObject.getRoomItems()
                                                        .put(objectOID, 
actionObject);
-                                       whiteBoardObjectListManagerById
+                                       wbListManagerById
                                                        
.setWhiteBoardObjectListRoomObjAndWhiteboardId(
                                                                        
room_id, whiteboardObject, whiteBoardId);
                                }
                        } else if (action.equals("clear")) {
-                               WhiteboardObject whiteboardObject = 
whiteBoardObjectListManagerById
+                               WhiteboardObject whiteboardObject = 
wbListManagerById
                                                
.getWhiteBoardObjectListByRoomIdAndWhiteboard(room_id,
                                                                whiteBoardId);
                                whiteboardObject.setRoomItems(new 
HashMap<String, List>());
-                               whiteBoardObjectListManagerById
+                               wbListManagerById
                                                
.setWhiteBoardObjectListRoomObjAndWhiteboardId(room_id,
                                                                
whiteboardObject, whiteBoardId);
                        } else if (action.equals("delete") || 
action.equals("undo")) {
-                               WhiteboardObject whiteboardObject = 
whiteBoardObjectListManagerById
+                               WhiteboardObject whiteboardObject = 
wbListManagerById
                                                
.getWhiteBoardObjectListByRoomIdAndWhiteboard(room_id,
                                                                whiteBoardId);
                                String objectOID = 
actionObject.get(actionObject.size() - 1)
@@ -132,9 +127,7 @@ public class WhiteboardManager {
                                                || 
objectType.equals("mindmapnode")
                                                || objectType.equals("flv")) {
 
-                                       Integer zIndex = Integer.valueOf(
-                                                       
actionObject.get(actionObject.size() - 8)
-                                                                       
.toString()).intValue();
+                                       Integer zIndex = 
Integer.valueOf(actionObject.get(actionObject.size() - 8).toString());
 
                                        log.debug("1|zIndex " + zIndex);
                                        log.debug("2|zIndex "
@@ -179,7 +172,7 @@ public class WhiteboardManager {
 
                                
whiteboardObject.getRoomItems().remove(objectOID);
 
-                               whiteBoardObjectListManagerById
+                               wbListManagerById
                                                
.setWhiteBoardObjectListRoomObjAndWhiteboardId(room_id,
                                                                
whiteboardObject, whiteBoardId);
                        } else if (action.equals("size") || 
action.equals("editProp")
@@ -188,7 +181,7 @@ public class WhiteboardManager {
                                        || action.equals("flv")
                                        || action.equals("editTextMindMapColor")
                                        || 
action.equals("editTextMindMapFontColor")) {
-                               WhiteboardObject whiteboardObject = 
whiteBoardObjectListManagerById
+                               WhiteboardObject whiteboardObject = 
wbListManagerById
                                                
.getWhiteBoardObjectListByRoomIdAndWhiteboard(room_id,
                                                                whiteBoardId);
                                String objectOID = 
actionObject.get(actionObject.size() - 1)
@@ -213,8 +206,7 @@ public class WhiteboardManager {
 
                                                        String baseObjectName = 
actionObject.get(
                                                                        
actionObject.size() - 1).toString();
-                                                       Integer slidesNumber = 
Integer.valueOf(
-                                                                       
actionObject.get(8).toString()).intValue();
+                                                       Integer slidesNumber = 
Integer.valueOf(actionObject.get(8).toString());
 
                                                        
log.debug("updateObjectsToSlideNumber :: "
                                                                        + 
baseObjectName + "," + slidesNumber);
@@ -243,40 +235,14 @@ public class WhiteboardManager {
                                                                                
|| actionObjectStored.get(0).equals(
                                                                                
                "letter")) {
 
-                                                                       Map 
swfObj = (Map) actionObjectStored
-                                                                               
        .get(actionObjectStored.size() - 7);
+                                                                       Map 
swfObj = (Map) actionObjectStored.get(actionObjectStored.size() - 7);
                                                                        
log.debug("swfObj :1: " + swfObj);
 
                                                                        if 
(swfObj != null) {
-
-                                                                               
if (swfObj.get("name").equals(
-                                                                               
                baseObjectName)) {
-
-                                                                               
        if (Integer.valueOf(
-                                                                               
                        swfObj.get("slide")
-                                                                               
                                        .toString())
-                                                                               
                        .intValue() == slidesNumber) {
-
-                                                                               
                swfObj.put("isVisible", true);
-
-                                                                               
                actionObjectStored.set(
-                                                                               
                                actionObjectStored
-                                                                               
                                                .size() - 7,
-                                                                               
                                swfObj);
-
-                                                                               
        } else {
-
-                                                                               
                swfObj.put("isVisible", false);
-
-                                                                               
                actionObjectStored.set(
-                                                                               
                                actionObjectStored
-                                                                               
                                                .size() - 7,
-                                                                               
                                swfObj);
-
-                                                                               
        }
-
+                                                                               
if (swfObj.get("name").equals(baseObjectName)) {
+                                                                               
        swfObj.put("isVisible", 
Integer.valueOf(swfObj.get("slide").toString()).intValue() == slidesNumber);
+                                                                               
        actionObjectStored.set(actionObjectStored.size() - 7, swfObj);
                                                                                
}
-
                                                                        }
 
                                                                        
log.debug("swfObj :1: " + swfObj);
@@ -290,12 +256,12 @@ public class WhiteboardManager {
                                        }
                                }
 
-                               whiteBoardObjectListManagerById
+                               wbListManagerById
                                                
.setWhiteBoardObjectListRoomObjAndWhiteboardId(room_id,
                                                                
whiteboardObject, whiteBoardId);
                        } else if (action.equals("clearSlide")) {
 
-                               WhiteboardObject whiteboardObject = 
whiteBoardObjectListManagerById
+                               WhiteboardObject whiteboardObject = 
wbListManagerById
                                                
.getWhiteBoardObjectListByRoomIdAndWhiteboard(room_id,
                                                                whiteBoardId);
 
@@ -307,18 +273,18 @@ public class WhiteboardManager {
 
                                }
 
-                               whiteBoardObjectListManagerById
+                               wbListManagerById
                                                
.setWhiteBoardObjectListRoomObjAndWhiteboardId(room_id,
                                                                
whiteboardObject, whiteBoardId);
                        } else if (action.equals("whiteboardObj")) {
-                               WhiteboardObject whiteboardObject = 
whiteBoardObjectListManagerById
+                               WhiteboardObject whiteboardObject = 
wbListManagerById
                                                
.getWhiteBoardObjectListByRoomIdAndWhiteboard(room_id,
                                                                whiteBoardId);
 
                                whiteboardObject.setFullFit((Boolean) 
actionObject.get(1));
                                whiteboardObject.setZoom((Integer) 
actionObject.get(2));
 
-                               whiteBoardObjectListManagerById
+                               wbListManagerById
                                
.setWhiteBoardObjectListRoomObjAndWhiteboardId(room_id,
                                                whiteboardObject, whiteBoardId);
                        } else {

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/documents/LibraryChartLoader.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/documents/LibraryChartLoader.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/documents/LibraryChartLoader.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/documents/LibraryChartLoader.java
 Mon Sep  1 09:08:56 2014
@@ -21,6 +21,7 @@ package org.apache.openmeetings.core.doc
 import java.io.BufferedReader;
 import java.io.File;
 import java.io.FileReader;
+import java.io.IOException;
 import java.util.ArrayList;
 
 import org.apache.openmeetings.util.Logger;
@@ -48,6 +49,7 @@ public class LibraryChartLoader {
 
     @SuppressWarnings("rawtypes")
        public ArrayList loadChart(File dir, String fileName) {
+       BufferedReader reader = null;
         try {
             File file = new File(dir, fileName + fileExt);
 
@@ -56,22 +58,21 @@ public class LibraryChartLoader {
             XStream xStream = new XStream(new XppDriver());
             xStream.setMode(XStream.NO_REFERENCES);
 
-            BufferedReader reader = new BufferedReader(new FileReader(file));
-            String xmlString = "";
-            while (reader.ready()) {
-                xmlString += reader.readLine();
-            }
-            reader.close();
-            // lMap = (LinkedHashMap) xStream.fromXML(xmlString);
-            ArrayList lMapList = (ArrayList) xStream.fromXML(xmlString);
+            reader = new BufferedReader(new FileReader(file));
+            ArrayList lMapList = (ArrayList) xStream.fromXML(reader);
 
             return lMapList;
         } catch (Exception err) {
             log.error(err);
+        } finally {
+                       if (reader != null) {
+                               try {
+                                       reader.close();
+                               } catch (IOException e) {}
+                       }
         }
 
         return null;
-
     }
 
 }

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/remote/MainService.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/remote/MainService.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/remote/MainService.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/remote/MainService.java
 Mon Sep  1 09:08:56 2014
@@ -119,7 +119,7 @@ public class MainService implements IPen
                Long users_id = sessiondataDao.checkSession(SID);
                Set<Right> rights = userDao.getRights(users_id);
                if (AuthLevelUtil.hasAdminLevel(rights) || 
AuthLevelUtil.hasWebServiceLevel(rights)) {
-                       users = userDao.get(new Long(USER_ID));
+                       users = userDao.get(USER_ID);
                } else {
                        users.setFirstname("No rights to do this");
                }

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/servlet/outputhandler/AbstractUploadController.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/servlet/outputhandler/AbstractUploadController.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/servlet/outputhandler/AbstractUploadController.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-core/src/main/java/org/apache/openmeetings/core/servlet/outputhandler/AbstractUploadController.java
 Mon Sep  1 09:08:56 2014
@@ -45,7 +45,7 @@ public abstract class AbstractUploadCont
        @Autowired
        protected ConfigurationDao configurationDao;
        
-       protected class UploadInfo {
+       protected static class UploadInfo {
                MultipartFile file;
                Long userId;
                String sid;

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/room/RoomDao.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/room/RoomDao.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/room/RoomDao.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/room/RoomDao.java
 Mon Sep  1 09:08:56 2014
@@ -143,8 +143,8 @@ public class RoomDao implements IDataPro
                                .getResultList();
        }
        
-       public Long getRoomsCapacityByIds(List<Long> ids) {
-               return ids == null || ids.isEmpty() ? 0
+       public long getRoomsCapacityByIds(List<Long> ids) {
+               return ids == null || ids.isEmpty() ? 0L
                        : em.createNamedQuery("getRoomsCapacityByIds", 
Long.class).setParameter("ids", ids).getSingleResult();
        }
        
@@ -153,10 +153,10 @@ public class RoomDao implements IDataPro
        }
        
        private String getSipNumber(long roomId) {
-        if (isSipEnabled()) {
-               return cfgDao.getConfValue("red5sip.room_prefix", String.class, 
"400") + roomId;
-        }
-        return null;
+               if (isSipEnabled()) {
+                       return cfgDao.getConfValue("red5sip.room_prefix", 
String.class, "400") + roomId;
+               }
+               return null;
        }
        
        public Room update(Room entity, Long userId) {
@@ -168,7 +168,7 @@ public class RoomDao implements IDataPro
                }
                if (entity.isSipEnabled() && isSipEnabled()) {
                        String sipNumber = getSipNumber(entity.getId());
-                       if (!sipNumber.equals(entity.getConfno())) {
+                       if (sipNumber != null && 
!sipNumber.equals(entity.getConfno())) {
                                entity.setConfno(sipNumber);
                        }
                        sipDao.update(sipNumber, entity.getPin());

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java
 Mon Sep  1 09:08:56 2014
@@ -144,8 +144,8 @@ public class UserDao implements IDataPro
        }
        
        private void setAdditionalParams(TypedQuery<?> q, Map<String, Object> 
params) {
-               for (String key : params.keySet()) {
-                       q.setParameter(key, params.get(key));
+               for (Map.Entry<String, Object> me: params.entrySet()) {
+                       q.setParameter(me.getKey(), me.getValue());
                }
        }
 

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dto/room/RoomSearchResult.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dto/room/RoomSearchResult.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dto/room/RoomSearchResult.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-db/src/main/java/org/apache/openmeetings/db/dto/room/RoomSearchResult.java
 Mon Sep  1 09:08:56 2014
@@ -34,13 +34,15 @@ public class RoomSearchResult {
        }
        
        public RoomSearchResult(SearchResult<Room> copy) {
-               this.objectName = copy.getObjectName();
-               this.records = copy.getRecords();
-               this.result = new ArrayList<RoomDTO>(copy.getResult().size());
-               for (Room r : copy.getResult()) {
-                       result.add(new RoomDTO(r));
+               if (copy != null) {
+                       this.objectName = copy.getObjectName();
+                       this.records = copy.getRecords();
+                       this.result = new 
ArrayList<RoomDTO>(copy.getResult().size());
+                       for (Room r : copy.getResult()) {
+                               result.add(new RoomDTO(r));
+                       }
+                       this.errorId = copy.getErrorId();
                }
-               this.errorId = copy.getErrorId();
        }
 
        public String getObjectName() {

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/backup/BackupImport.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/backup/BackupImport.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/backup/BackupImport.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/backup/BackupImport.java
 Mon Sep  1 09:08:56 2014
@@ -684,11 +684,13 @@ public class BackupImport {
        }
        
        private Node getNode(Node doc, String name) {
-               NodeList nl = doc.getChildNodes();
-               for (int i = 0; i < nl.getLength(); ++i) {
-                       Node node = nl.item(i);
-                       if (node.getNodeType() == Node.ELEMENT_NODE && 
name.equals(node.getNodeName())) {
-                               return node;
+               if (doc != null) {
+                       NodeList nl = doc.getChildNodes();
+                       for (int i = 0; i < nl.getLength(); ++i) {
+                               Node node = nl.item(i);
+                               if (node.getNodeType() == Node.ELEMENT_NODE && 
name.equals(node.getNodeName())) {
+                                       return node;
+                               }
                        }
                }
                return null;
@@ -916,7 +918,6 @@ public class BackupImport {
 
                DocumentBuilder dBuilder = 
DocumentBuilderFactory.newInstance().newDocumentBuilder();
                Document doc = dBuilder.parse(xml);
-               NodeList nl = getNode(getNode(doc, "root"), 
listNodeName).getChildNodes();
                userEmailMap.clear();
                //add existence email from database
                List<User>  users = usersDao.getAllUsers();
@@ -926,26 +927,30 @@ public class BackupImport {
                        }
                        userEmailMap.put(u.getAdresses().getEmail(), -1);
                }
-               // one of the old OM version created 2 nodes "deleted" this 
code block handles this
-               for (int i = 0; i < nl.getLength(); ++i) {
-                       Node user = nl.item(i);
-                       NodeList nl1 = user.getChildNodes();
-                       boolean deletedFound = false;
-                       for (int j = 0; j < nl1.getLength(); ++j) {
-                               Node node = nl1.item(j);
-                               if (node.getNodeType() == Node.ELEMENT_NODE && 
"deleted".equals(node.getNodeName())) {
-                                       if (deletedFound) {
-                                               user.removeChild(node);
-                                               break;
+               Node nList = getNode(getNode(doc, "root"), listNodeName);
+               if (nList != null) {
+                       NodeList nl = nList.getChildNodes();
+                       // one of the old OM version created 2 nodes "deleted" 
this code block handles this
+                       for (int i = 0; i < nl.getLength(); ++i) {
+                               Node user = nl.item(i);
+                               NodeList nl1 = user.getChildNodes();
+                               boolean deletedFound = false;
+                               for (int j = 0; j < nl1.getLength(); ++j) {
+                                       Node node = nl1.item(j);
+                                       if (node.getNodeType() == 
Node.ELEMENT_NODE && "deleted".equals(node.getNodeName())) {
+                                               if (deletedFound) {
+                                                       user.removeChild(node);
+                                                       break;
+                                               }
+                                               deletedFound = true;
                                        }
-                                       deletedFound = true;
                                }
                        }
                }
                
                StringWriter sw = new StringWriter();
                Transformer xformer = 
TransformerFactory.newInstance().newTransformer();
-        xformer.transform(new DOMSource(doc), new StreamResult(sw));
+               xformer.transform(new DOMSource(doc), new StreamResult(sw));
         
                List<User> list = new ArrayList<User>();
                InputNode root = NodeBuilder.read(new 
StringReader(sw.toString()));

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/Admin.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/Admin.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/Admin.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/Admin.java
 Mon Sep  1 09:08:56 2014
@@ -559,8 +559,8 @@ public class Admin {
                if (cfg.ical_timeZone == null) {
                        System.out.println("Please enter timezone, Possible 
timezones are:");
                        
-                       for (String tzIcal : tzMap.keySet()) {
-                               System.out.println(String.format("%1$-25s%2$s", 
"\"" + tzIcal + "\"", tzMap.get(tzIcal)));
+                       for (Map.Entry<String,String> me : tzMap.entrySet()) {
+                               System.out.println(String.format("%1$-25s%2$s", 
"\"" + me.getKey() + "\"", me.getValue()));
                        }
                        System.exit(1);
                }

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/OmHelpFormatter.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/OmHelpFormatter.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/OmHelpFormatter.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/cli/OmHelpFormatter.java
 Mon Sep  1 09:08:56 2014
@@ -26,6 +26,8 @@ import java.util.Collections;
 import java.util.Comparator;
 import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.Map;
+import java.util.Map.Entry;
 
 import org.apache.commons.cli.HelpFormatter;
 import org.apache.commons.cli.Option;
@@ -110,9 +112,9 @@ public class OmHelpFormatter extends Hel
                                map.get(g).add(o);
                        }
                }
-               for (String _key : map.keySet()) {
-                       final String key = _key;
-                       List<OmOption> options = map.get(key);
+               for (Map.Entry<String, List<OmOption>> me : map.entrySet()) {
+                       final String key = me.getKey();
+                       List<OmOption> options = me.getValue();
                        Collections.sort(options, new Comparator<OmOption>(){
                                public int compare(OmOption o1, OmOption o2) {
                                        boolean o1opt = !o1.isOptional(key);
@@ -148,11 +150,11 @@ public class OmHelpFormatter extends Hel
                char[] delimiter = new char[width - 2];
                Arrays.fill(delimiter, '-');
 
-               for (String key : optList.keySet()) {
-                       if (GENERAL_OPTION_GROUP.equals(key)) {
+               for (Entry<String, List<OmOption>> me : optList.entrySet()) {
+                       if (GENERAL_OPTION_GROUP.equals(me.getKey())) {
                                sb.append("General 
options:").append(getNewLine());
                        }
-                       for (OmOption option : optList.get(key)) {
+                       for (OmOption option : me.getValue()) {
                                StringBuilder optBuf = new 
StringBuilder(option.getHelpPrefix());
 
                                if (optBuf.length() < maxPrefixLength) {
@@ -163,7 +165,7 @@ public class OmHelpFormatter extends Hel
 
                                int nextLineTabStop = maxPrefixLength + descPad;
 
-                               if (option.isOptional(key)) {
+                               if (option.isOptional(me.getKey())) {
                                        optBuf.append(optional);
                                }
                                if (option.getDescription() != null) {

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/installation/ImportInitvalues.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/installation/ImportInitvalues.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/installation/ImportInitvalues.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-install/src/main/java/org/apache/openmeetings/installation/ImportInitvalues.java
 Mon Sep  1 09:08:56 2014
@@ -53,6 +53,7 @@ import java.util.Hashtable;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Set;
 
 import org.apache.openmeetings.db.dao.basic.ConfigurationDao;
@@ -561,9 +562,6 @@ public class ImportInitvalues {
 
        public void loadInitUserAndOrganisation(InstallationConfig cfg) throws 
Exception {
                Long default_lang_id = Long.parseLong(cfg.defaultLangId);
-               if (default_lang_id == null) {
-                       default_lang_id = 1L;
-               }
 
                // Add default group
                Organisation org = new Organisation();
@@ -636,8 +634,7 @@ public class ImportInitvalues {
                Iterator<Element> it = root.elementIterator("lang"); 
it.hasNext();) {
                        Element item = it.next();
                        String country = item.getText();
-                       Integer id = 
Integer.valueOf(item.attribute("id").getValue())
-                                       .intValue();
+                       Integer id = 
Integer.valueOf(item.attribute("id").getValue());
 
                        String rtl = item.attribute("rightToLeft").getValue();
                        String code = item.attribute("code").getValue();
@@ -687,10 +684,10 @@ public class ImportInitvalues {
        public void loadLanguagesFile(String langName) throws Exception {
                Map<Integer, Map<String, Object>> listlanguages = 
getLanguageFiles();
                log.debug("Number of languages found: " + listlanguages.size());
-               for (int langId : listlanguages.keySet()) {
-                       Map<String, Object> langMap = listlanguages.get(langId);
+               for (Entry<Integer, Map<String, Object>> me : 
listlanguages.entrySet()) {
+                       Map<String, Object> langMap = me.getValue();
                        if (langName.equals(langMap.get("name"))) {
-                               loadLanguagesFile(listlanguages, new 
Hashtable<Long, Fieldvalues>(3000), langId);
+                               loadLanguagesFile(listlanguages, new 
Hashtable<Long, Fieldvalues>(3000), me.getKey());
                                break;
                        }
                }
@@ -707,15 +704,14 @@ public class ImportInitvalues {
 
                log.debug("loadInitLanguages rtl from xml: " + rtl);
 
-               Boolean langRtl = false;
-
-               if (rtl != null && rtl.equals("true")) {
-                       langRtl = true;
-               }
+               Boolean langRtl = Boolean.TRUE.equals(rtl);
 
                long ticks = System.currentTimeMillis();
                FieldLanguage lang = fieldLanguageDaoImpl.addLanguage(langId, 
langName, langRtl, code);
-
+               if (lang == null) {
+                       log.error("Failed to create language");
+                       throw new Exception("Failed to create language");
+               }
                SAXReader reader = new SAXReader();
                Document document = reader.read(new File(
                                OmFileHelper.getLanguagesDir(), langName + 
".xml"));
@@ -728,7 +724,7 @@ public class ImportInitvalues {
                        Element item = (Element) it.next();
                        // log.error(item.getName());
 
-                       Long id = 
Long.valueOf(item.attributeValue("id")).longValue();
+                       Long id = Long.valueOf(item.attributeValue("id"));
                        String name = item.attributeValue("name");
                        String value = "";
 
@@ -772,17 +768,6 @@ public class ImportInitvalues {
                log.debug("Lang ADDED: " + lang + "; seconds passed: " + 
(System.currentTimeMillis() - ticks) / 1000);
        }
        
-       public static void main(String... args) {
-               int pr = 22;
-               int progressDelta = 50 - pr;
-               
-               double deltaProgressPerLanguageFile = new 
Double(progressDelta)/new Double(34);
-               
-               pr += (2 * deltaProgressPerLanguageFile);
-               
-               System.err.println("deltaProgressPerLanguageFile: 
"+deltaProgressPerLanguageFile +" "+ pr);
-       }
-
        public void loadLanguagesFiles() throws Exception {
                Map<Integer, Map<String, Object>> listlanguages = 
getLanguageFiles();
 

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-screenshare/src/main/java/org/apache/openmeetings/screen/webstart/gui/ScreenSharerFrame.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-screenshare/src/main/java/org/apache/openmeetings/screen/webstart/gui/ScreenSharerFrame.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-screenshare/src/main/java/org/apache/openmeetings/screen/webstart/gui/ScreenSharerFrame.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-screenshare/src/main/java/org/apache/openmeetings/screen/webstart/gui/ScreenSharerFrame.java
 Mon Sep  1 09:08:56 2014
@@ -137,7 +137,7 @@ public class ScreenSharerFrame extends J
                }
        }
        
-       private class KeyValue<T> {
+       private static class KeyValue<T> {
                private String key;
                private T value;
                
@@ -172,7 +172,7 @@ public class ScreenSharerFrame extends J
        }
        
        //this implementation will not allow to Enable Panel in runtime
-       private class DisabledPanel extends JPanel {
+       private static class DisabledPanel extends JPanel {
                private static final long serialVersionUID = 1L;
 
                @Override
@@ -184,7 +184,7 @@ public class ScreenSharerFrame extends J
                }
        }
        
-       private class VerticalSlider extends MouseListenerable {
+       private static class VerticalSlider extends MouseListenerable {
                private static final long serialVersionUID = 1L;
 
                public VerticalSlider() {
@@ -200,7 +200,7 @@ public class ScreenSharerFrame extends J
                }
        }
        
-       private class HorizontalSlider extends MouseListenerable {
+       private static class HorizontalSlider extends MouseListenerable {
                private static final long serialVersionUID = 1L;
 
                public HorizontalSlider() {

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/CalendarPatterns.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/CalendarPatterns.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/CalendarPatterns.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/CalendarPatterns.java
 Mon Sep  1 09:08:56 2014
@@ -18,6 +18,8 @@
  */
 package org.apache.openmeetings.util;
 
+import static org.apache.openmeetings.util.OpenmeetingsVariables.webAppRootKey;
+
 import java.text.SimpleDateFormat;
 import java.util.Date;
 import java.util.TimeZone;
@@ -31,12 +33,23 @@ import org.slf4j.Logger;
  *
  */
 public class CalendarPatterns {
+       private static final Logger log = 
Red5LoggerFactory.getLogger(CalendarPatterns.class, webAppRootKey);
        
-       private static final Logger log = 
Red5LoggerFactory.getLogger(CalendarPatterns.class, "openmeetings");
-       
-       public static SimpleDateFormat dateFormat__ddMMyyyyHHmmss = new 
SimpleDateFormat("dd.MM.yyyy HH:mm:ss");
-       public static SimpleDateFormat dateFormat__ddMMyyyy = new 
SimpleDateFormat("dd.MM.yyyy");
-       public static SimpleDateFormat dateFormat__ddMMyyyyBySeparator = new 
SimpleDateFormat("dd-MM-yyyy");
+       public static ThreadLocal<SimpleDateFormat> dateFormat__ddMMyyyyHHmmss 
= new ThreadLocal<SimpleDateFormat>() {
+               protected SimpleDateFormat initialValue() {
+                       return new SimpleDateFormat("dd.MM.yyyy HH:mm:ss");
+               };
+       };
+       public static ThreadLocal<SimpleDateFormat> dateFormat__ddMMyyyy = new 
ThreadLocal<SimpleDateFormat>() {
+               protected SimpleDateFormat initialValue() {
+                       return new SimpleDateFormat("dd.MM.yyyy");
+               };
+       };
+       public static ThreadLocal<SimpleDateFormat> 
dateFormat__ddMMyyyyBySeparator = new ThreadLocal<SimpleDateFormat>() {
+               protected SimpleDateFormat initialValue() {
+                       return new SimpleDateFormat("dd-MM-yyyy");
+               };
+       };
     
     public static String getDateByMiliSeconds(Date t){
         SimpleDateFormat sdf = new SimpleDateFormat("dd.MM.yyyy");
@@ -96,13 +109,13 @@ public class CalendarPatterns {
                
                Date resultDate = null;
                
-               resultDate = validDate(dateFormat__ddMMyyyyHHmmss, dateString);
+               resultDate = validDate(dateFormat__ddMMyyyyHHmmss.get(), 
dateString);
                
                if ( resultDate != null ) {
                        return resultDate;
                }
                
-               resultDate = validDate(dateFormat__ddMMyyyy, dateString);
+               resultDate = validDate(dateFormat__ddMMyyyy.get(), dateString);
                
                if ( resultDate != null ) {
                        return resultDate;
@@ -185,7 +198,7 @@ public class CalendarPatterns {
     
     public static Date parseDate(String dateString) {
        try {
-               return dateFormat__ddMMyyyy.parse(dateString);
+               return dateFormat__ddMMyyyy.get().parse(dateString);
        } catch (Exception e) {
                log.error("parseDate",e);
        }
@@ -194,7 +207,7 @@ public class CalendarPatterns {
     
     public static Date parseDateBySeparator(String dateString) {
        try {
-               return dateFormat__ddMMyyyyBySeparator.parse(dateString);
+               return dateFormat__ddMMyyyyBySeparator.get().parse(dateString);
        } catch (Exception e) {
                log.error("parseDate",e);
        }
@@ -206,7 +219,7 @@ public class CalendarPatterns {
                if (dateString == null || dateString.length()==0 || 
dateString.equals("null") ) {
                        return null;
                }
-               return dateFormat__ddMMyyyyHHmmss.parse(dateString);
+               return dateFormat__ddMMyyyyHHmmss.get().parse(dateString);
        } catch (Exception e) {
                log.error("parseDate",e);
        }

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ConverterProcessResultList.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ConverterProcessResultList.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ConverterProcessResultList.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ConverterProcessResultList.java
 Mon Sep  1 09:08:56 2014
@@ -22,7 +22,6 @@ import static org.apache.openmeetings.ut
 import static org.red5.logging.Red5LoggerFactory.getLogger;
 
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.Map;
 import java.util.Map.Entry;
 
@@ -104,36 +103,24 @@ public class ConverterProcessResultList 
         * @return
         */
        public FileImportError[] convertToFileImportErrors() {
-               FileImportError[] fileImportErrors = new 
FileImportError[jobslist
-                               .size()];
+               FileImportError[] errors = new FileImportError[jobslist.size()];
 
                int i = 0;
                // Axis need Objects or array of objects, Map won't work
-               for (Iterator<String> iter = jobslist.keySet().iterator(); iter
-                               .hasNext();) {
+               for (Map.Entry<String, ConverterProcessResult> me : 
jobslist.entrySet()) {
 
-                       ConverterProcessResult returnAttribute = 
jobslist.get(iter
-                                       .next());
+                       ConverterProcessResult result = me.getValue();
 
-                       fileImportErrors[i] = new FileImportError();
-                       fileImportErrors[i]
-                                       
.setCommand((returnAttribute.getCommand() != null) ? returnAttribute
-                                                       
.getCommand().toString() : "");
-                       fileImportErrors[i]
-                                       .setError((returnAttribute.getError() 
!= null) ? returnAttribute
-                                                       .getError().toString() 
: "");
-                       fileImportErrors[i]
-                                       
.setExitValue((returnAttribute.getExitValue() != null) ? Integer
-                                                       
.valueOf(returnAttribute.getExitValue().toString())
-                                                       .intValue() : 0);
-                       fileImportErrors[i]
-                                       
.setProcess((returnAttribute.getProcess() != null) ? returnAttribute
-                                                       
.getProcess().toString() : "");
+                       errors[i] = new FileImportError();
+                       errors[i].setCommand((result.getCommand() != null) ? 
result.getCommand().toString() : "");
+                       errors[i].setError((result.getError() != null) ? 
result.getError().toString() : "");
+                       errors[i].setExitValue((result.getExitValue() != null) 
? Integer.valueOf(result.getExitValue().toString()) : 0);
+                       errors[i].setProcess((result.getProcess() != null) ? 
result.getProcess().toString() : "");
 
                        i++;
                }
                
-               return fileImportErrors;
+               return errors;
        }
 
        public int size() {

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ProcessHelper.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ProcessHelper.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ProcessHelper.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/process/ProcessHelper.java
 Mon Sep  1 09:08:56 2014
@@ -92,11 +92,11 @@ public class ProcessHelper {
        }
        
        private static String getCommand(String[] argv) {
-               String tString = "";
+               StringBuffer tString = new StringBuffer();
                for (int i = 0; i < argv.length; i++) {
-                       tString += argv[i] + " ";
+                       tString.append(argv[i]).append(" ");
                }
-               return tString;
+               return tString.toString();
        }
        
        private static void debugCommand(String desc, String[] argv) {

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/stringhandlers/ChatString.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/stringhandlers/ChatString.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/stringhandlers/ChatString.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-util/src/main/java/org/apache/openmeetings/util/stringhandlers/ChatString.java
 Mon Sep  1 09:08:56 2014
@@ -195,8 +195,9 @@ public class ChatString {
 
        public static String link(String input) {
                try {
-
-                       String tReturn = "";
+                       Pattern p = Pattern.compile(
+                                                       "(^|[ 
\t\r\n])((ftp|http|https|gopher|mailto|news|nntp|telnet|wais|file|prospero|aim|webcal):(([A-Za-z0-9$_.+!*(),;/?:@&~=-])|%[A-Fa-f0-9]{2}){2,}(#([a-zA-Z0-9][a-zA-Z0-9$_.+!*(),;/?:@&~=%-]*))?([A-Za-z0-9$_+!*();/?:~-]))");
+                       StringBuffer tReturn = new StringBuffer();
 
                        String parts[] = input.split(" ");
 
@@ -206,10 +207,7 @@ public class ChatString {
 
                                // System.out.println("Part 1 "+text);
 
-                               Matcher matcher = Pattern
-                                               .compile(
-                                                               "(^|[ 
\t\r\n])((ftp|http|https|gopher|mailto|news|nntp|telnet|wais|file|prospero|aim|webcal):(([A-Za-z0-9$_.+!*(),;/?:@&~=-])|%[A-Fa-f0-9]{2}){2,}(#([a-zA-Z0-9][a-zA-Z0-9$_.+!*(),;/?:@&~=%-]*))?([A-Za-z0-9$_+!*();/?:~-]))")
-                                               .matcher(text);
+                               Matcher matcher = p.matcher(text);
 
                                if (matcher.find()) {
                                        text = matcher
@@ -221,14 +219,14 @@ public class ChatString {
                                // System.out.println("Part 2 "+text);
 
                                if (t != 0) {
-                                       tReturn += " ";
+                                       tReturn.append(" ");
                                }
 
-                               tReturn += text;
+                               tReturn.append(text);
 
                        }
 
-                       return tReturn;
+                       return tReturn.toString();
 
                } catch (Exception e) {
                        log.error("[link]", e);

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/admin/ldaps/LdapsPanel.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/admin/ldaps/LdapsPanel.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/admin/ldaps/LdapsPanel.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/admin/ldaps/LdapsPanel.java
 Mon Sep  1 09:08:56 2014
@@ -73,7 +73,7 @@ public class LdapsPanel extends AdminPan
                                        }
                                });
                                item.add(AttributeModifier.replace("class", 
"clickable ui-widget-content"
-                                               + (lc.getId() == 
form.getModelObject().getId() ? " ui-state-active" : "")));
+                                               + 
(lc.getId().equals(form.getModelObject().getId()) ? " ui-state-active" : "")));
                        }
                };
                add(listContainer.add(dataView).setOutputMarkupId(true));

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/app/Application.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/app/Application.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/app/Application.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/app/Application.java
 Mon Sep  1 09:08:56 2014
@@ -28,6 +28,7 @@ import static org.springframework.web.co
 
 import java.util.HashSet;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Set;
 import java.util.UUID;
 import java.util.concurrent.ConcurrentHashMap;
@@ -250,10 +251,10 @@ public class Application extends Authent
        }
        
        public static long getRoom(Client c) {
-               for (long roomId : ROOMS.keySet()) {
-                       Set<Client> clients = ROOMS.get(roomId);
+               for (Entry<Long, Set<Client>> me : ROOMS.entrySet()) {
+                       Set<Client> clients = me.getValue();
                        if (clients.contains(c)) {
-                               return roomId;
+                               return me.getKey();
                        }
                }
                return -1;

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java
 Mon Sep  1 09:08:56 2014
@@ -330,10 +330,10 @@ public class SignInPage extends BaseInit
                // generate random password
                byte[] rawPass = new byte[16];
                Random rnd = new Random();
-               for (int i = 0; i < 16; i++) {
-                       rawPass[i] = (byte) (97 + rnd.nextInt(25));
+               for (int i = 0; i < 25; i++) {
+                       rawPass[i] = (byte) ('!' + rnd.nextInt(93));
                }
-               String pass = new String(rawPass);
+               String pass = new String(rawPass, "UTF-8");
                // check if the user already exists and register new one if 
it's needed
                if (user == null) {
                        Integer defaultlangId = 
Integer.valueOf(configurationDao.getConfValue("default_lang_id", String.class, 
"1"));
@@ -354,7 +354,7 @@ public class SignInPage extends BaseInit
                                return;
                        }
                        user.setLastlogin(new Date());
-                       user = userDao.update(user, pass, -1);
+                       userDao.update(user, pass, -1);
                }
                
                if (WebSession.get().signIn(login, pass, Type.oauth, serverId)) 
{

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java
 Mon Sep  1 09:08:56 2014
@@ -499,17 +499,17 @@ public class InstallWizard extends Wizar
                        
                        List<SelectOption> list = new ArrayList<SelectOption>();
                        
-                       for (Integer key : allLanguagesAll.keySet()) {
-                               String langName = (String) 
allLanguagesAll.get(key).get("name");
-                               String langCode = (String) 
allLanguagesAll.get(key).get("code");
-                               SelectOption op = new 
SelectOption(key.toString(), langName);
+                       for (Map.Entry<Integer,Map<String,Object>> me : 
allLanguagesAll.entrySet()) {
+                               String langName = (String) 
me.getValue().get("name");
+                               String langCode = (String) 
me.getValue().get("code");
+                               SelectOption op = new 
SelectOption(me.getKey().toString(), langName);
                                if (langCode != null) {
                                        if 
(getSession().getLocale().toString().startsWith(langCode)) {
                                                option = op;
                                        }
                                        list.add(op);
                                }
-                               if (option == null && 
key.toString().equals(cfg.defaultLangId)) {
+                               if (option == null && 
me.getKey().toString().equals(cfg.defaultLangId)) {
                                        option = op;
                                }
                        }

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
 Mon Sep  1 09:08:56 2014
@@ -34,6 +34,8 @@ import static org.apache.openmeetings.we
 import static org.apache.openmeetings.web.app.WebSession.getUserId;
 import static org.apache.openmeetings.web.util.OmUrlFragment.ROOMS_PUBLIC;
 
+import java.io.IOException;
+import java.io.InputStream;
 import java.net.URL;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -230,9 +232,11 @@ public class RoomPanel extends BasePanel
                        @Override
                        protected void onEvent(AjaxRequestTarget target) {
                                String app = "";
+                               InputStream jnlp = null;
                                try {
                                        ConfigurationDao cfgDao = 
getBean(ConfigurationDao.class);
-                                       app = 
IOUtils.toString(getClass().getClassLoader().getResourceAsStream("APPLICATION.jnlp"),
 "UTF-8");
+                                       jnlp = 
getClass().getClassLoader().getResourceAsStream("APPLICATION.jnlp");
+                                       app = IOUtils.toString(jnlp, "UTF-8");
                                        String baseUrl = cfgDao.getBaseUrl();
                                        URL url = new URL(baseUrl);
                                        String path = url.getPath();
@@ -263,6 +267,12 @@ public class RoomPanel extends BasePanel
                                                        ;
                                } catch (Exception e) {
                                        log.error("Unexpected error while 
creating jnlp file", e);
+                               } finally {
+                                       if (jnlp != null) {
+                                               try {
+                                                       jnlp.close();
+                                               } catch (IOException e) {}
+                                       }
                                }
                                download.setResourceStream(new 
StringResourceStream(app, "application/x-java-jnlp-file"));
                                download.initiate(target);

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/user/record/VideoInfo.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/user/record/VideoInfo.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/user/record/VideoInfo.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/user/record/VideoInfo.java
 Mon Sep  1 09:08:56 2014
@@ -81,11 +81,14 @@ public class VideoInfo extends Panel {
                update(null, r);
        }
        
-       public VideoInfo update(AjaxRequestTarget target, FlvRecording r) {
-               rm.setObject(r == null ? new FlvRecording() : r);
+       public VideoInfo update(AjaxRequestTarget target, FlvRecording _r) {
+               FlvRecording r = _r == null ? new FlvRecording() : _r;
+               rm.setObject(r);
                try {
                        Room room = getBean(RoomDao.class).get(r.getRoomId());
-                       roomName.setObject(room.getName());
+                       if (room != null) {
+                               roomName.setObject(room.getName());
+                       }
                } catch (Exception e) {
                        //no-op
                }

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/util/UserDashboardPersister.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/util/UserDashboardPersister.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/util/UserDashboardPersister.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-web/src/main/java/org/apache/openmeetings/web/util/UserDashboardPersister.java
 Mon Sep  1 09:08:56 2014
@@ -48,7 +48,7 @@ public class UserDashboardPersister impl
                new XStreamDashboardPersister().save(dashboard);
        }
 
-       public class XStreamDashboardPersister implements DashboardPersister {
+       public static class XStreamDashboardPersister implements 
DashboardPersister {
                private File file;
                private XStream xstream;
                

Modified: 
openmeetings/trunk/singlewebapp/openmeetings-webservice/src/main/java/org/apache/openmeetings/webservice/UserWebService.java
URL: 
http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/openmeetings-webservice/src/main/java/org/apache/openmeetings/webservice/UserWebService.java?rev=1621711&r1=1621710&r2=1621711&view=diff
==============================================================================
--- 
openmeetings/trunk/singlewebapp/openmeetings-webservice/src/main/java/org/apache/openmeetings/webservice/UserWebService.java
 (original)
+++ 
openmeetings/trunk/singlewebapp/openmeetings-webservice/src/main/java/org/apache/openmeetings/webservice/UserWebService.java
 Mon Sep  1 09:08:56 2014
@@ -126,11 +126,8 @@ public class UserWebService {
                                return -1L;
                        }
                        
-                       Boolean bool = sessiondataDao.updateUser(SID, 
u.getId(), false, u.getLanguage_id());
-                       if (bool == null) {
-                               // Exception
-                               return -1L;
-                       } else if (!bool) {
+                       boolean bool = sessiondataDao.updateUser(SID, 
u.getId(), false, u.getLanguage_id());
+                       if (!bool) {
                                // invalid Session-Object
                                return -35L;
                        }


Reply via email to