[jboss-svn-commits] JBL Code SVN: r15684 - in labs/jbosslabs/trunk/portal-extensions: forge-ejb3/src/java/org/jboss/forge/ejb3/projects and 2 other directories.

[jboss-svn-commits at lists.jboss.org]

Author: wrzep
Date: 2007-10-09 05:53:06 -0400 (Tue, 09 Oct 2007)
New Revision: 15684

Modified:
   labs/jbosslabs/trunk/portal-extensions/forge-common/src/java/org/jboss/forge/common/exceptions/PersistProjectException.java
   labs/jbosslabs/trunk/portal-extensions/forge-ejb3/src/java/org/jboss/forge/ejb3/projects/ProjectsServiceRWImpl.java
   labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/ProjectAdminBean.java
   labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/info/InfoAdminBean.java
Log:
JBLAB-869 project admin


Modified: labs/jbosslabs/trunk/portal-extensions/forge-common/src/java/org/jboss/forge/common/exceptions/PersistProjectException.java
===================================================================
--- labs/jbosslabs/trunk/portal-extensions/forge-common/src/java/org/jboss/forge/common/exceptions/PersistProjectException.java	2007-10-09 09:44:13 UTC (rev 15683)
+++ labs/jbosslabs/trunk/portal-extensions/forge-common/src/java/org/jboss/forge/common/exceptions/PersistProjectException.java	2007-10-09 09:53:06 UTC (rev 15684)
@@ -22,17 +22,15 @@
 
 package org.jboss.forge.common.exceptions;
 
-import java.lang.reflect.Method;
-
 public class PersistProjectException extends Exception {
 
 	private static final long serialVersionUID = 425234643743334451L;
 
-	public PersistProjectException(Method m) {
-		super(m.getDeclaringClass().getName() + " " + m.getName());
-	}
-
 	public PersistProjectException(Exception e) {
 		super(e);
 	}
+
+	public PersistProjectException(String m) {
+		super(m);
+	}
 }

Modified: labs/jbosslabs/trunk/portal-extensions/forge-ejb3/src/java/org/jboss/forge/ejb3/projects/ProjectsServiceRWImpl.java
===================================================================
--- labs/jbosslabs/trunk/portal-extensions/forge-ejb3/src/java/org/jboss/forge/ejb3/projects/ProjectsServiceRWImpl.java	2007-10-09 09:44:13 UTC (rev 15683)
+++ labs/jbosslabs/trunk/portal-extensions/forge-ejb3/src/java/org/jboss/forge/ejb3/projects/ProjectsServiceRWImpl.java	2007-10-09 09:53:06 UTC (rev 15684)
@@ -122,9 +122,13 @@
      * @throws PersistProjectException 
      */
     public synchronized void persistProject(Project project) throws PersistProjectException {
-        if (project.getId()==null) return;
+        if (project.getId()==null) {
+        	throw new PersistProjectException("null project id");
+        }
 
-        if (jc==null) return ;
+        if (jc==null) {
+        	throw new PersistProjectException("jc is null");
+        }
             
         try {
             

Modified: labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/ProjectAdminBean.java
===================================================================
--- labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/ProjectAdminBean.java	2007-10-09 09:44:13 UTC (rev 15683)
+++ labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/ProjectAdminBean.java	2007-10-09 09:53:06 UTC (rev 15684)
@@ -25,8 +25,6 @@
 
 import javax.faces.component.UIViewRoot;
 import javax.faces.context.FacesContext;
-import javax.management.MalformedObjectNameException;
-
 import org.jboss.forge.common.FacesTools;
 import org.jboss.forge.common.ejb3.LabsServices;
 import org.jboss.forge.common.projects.project.Project;
@@ -66,7 +64,7 @@
 		} catch (Exception e) {
 			
 			log.warn("Save project unsuccessful " + e.getMessage());
-			FacesTools.addMessage("Save failed");
+			FacesTools.addMessage("Save failed.");
 			return ProjectAdminConstants.MESSAGE_ACTION;
 		}
 		

Modified: labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/info/InfoAdminBean.java
===================================================================
--- labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/info/InfoAdminBean.java	2007-10-09 09:44:13 UTC (rev 15683)
+++ labs/jbosslabs/trunk/portal-extensions/forge-portlets/src/java/org/jboss/forge/projects/admin/info/InfoAdminBean.java	2007-10-09 09:53:06 UTC (rev 15684)
@@ -67,7 +67,7 @@
 		projectJems.addAll(jems);
 		
 		// proxy issue workaround
-		Developers d =project.getDevelopers();
+		Developers d = project.getDevelopers();
 		d.setLeadDeveloper(projectLead);
 		project.setDevelopers(d);