Index: lams_admin/conf/xdoclet/struts-forms.xml =================================================================== RCS file: /usr/local/cvsroot/lams_admin/conf/xdoclet/struts-forms.xml,v diff -u -r1.22.2.4 -r1.22.2.5 --- lams_admin/conf/xdoclet/struts-forms.xml 7 Jun 2007 01:26:09 -0000 1.22.2.4 +++ lams_admin/conf/xdoclet/struts-forms.xml 22 Jun 2007 05:17:20 -0000 1.22.2.5 @@ -112,6 +112,7 @@ + Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/action/FederationAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_admin/src/java/org/lamsfoundation/lams/admin/web/action/Attic/FederationAction.java,v diff -u -r1.1.2.2 -r1.1.2.3 --- lams_admin/src/java/org/lamsfoundation/lams/admin/web/action/FederationAction.java 5 Jun 2007 04:31:43 -0000 1.1.2.2 +++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/action/FederationAction.java 22 Jun 2007 05:17:20 -0000 1.1.2.3 @@ -3,6 +3,8 @@ */ package org.lamsfoundation.lams.admin.web.action; +import java.util.List; + import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; @@ -18,6 +20,7 @@ import org.lamsfoundation.lams.admin.service.AdminServiceProxy; import org.lamsfoundation.lams.federation.FederationServer; import org.lamsfoundation.lams.federation.service.IFederationService; +import org.lamsfoundation.lams.usermanagement.service.IUserManagementService; /** * @author jliew @@ -43,8 +46,11 @@ service = AdminServiceProxy.getFederationService(getServlet().getServletContext()); } + IUserManagementService uservice = AdminServiceProxy.getService(getServlet().getServletContext()); + ActionMessages errors = new ActionMessages(); DynaActionForm fedServerForm = (DynaActionForm)form; + Integer fedId = (Integer)fedServerForm.get("fedId"); String name = fedServerForm.getString("name"); String url = fedServerForm.getString("url"); String privatekey = fedServerForm.getString("privateKey"); @@ -53,6 +59,9 @@ if (submitted!=null && submitted.equals(new Integer(1))) { // form validation + if (fedId==null) { + errors.add("fedId", new ActionMessage("error.fedId.required")); + } if (name==null || name.trim().length()==0) { errors.add("name", new ActionMessage("error.name.required")); } @@ -65,9 +74,15 @@ // save form or display errors if (errors.isEmpty()) { - FederationServer newServer = new FederationServer(); - BeanUtils.copyProperties(newServer, fedServerForm); - service.insertFederationServer(newServer); + FederationServer fedServer; + List fedServers = uservice.findByProperty(FederationServer.class, "fedId", fedId); + if (fedServers==null || fedServers.isEmpty()) { + fedServer = new FederationServer(); + } else { + fedServer = (FederationServer)fedServers.get(0); + } + BeanUtils.copyProperties(fedServer, fedServerForm); + service.saveFederationServer(fedServer); fedServerForm.initialize(mapping); } else { saveErrors(request, errors); Index: lams_admin/web/federation/federation.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_admin/web/federation/Attic/federation.jsp,v diff -u -r1.1.2.3 -r1.1.2.4 --- lams_admin/web/federation/federation.jsp 5 Jun 2007 04:31:43 -0000 1.1.2.3 +++ lams_admin/web/federation/federation.jsp 22 Jun 2007 05:17:20 -0000 1.1.2.4 @@ -6,29 +6,31 @@

Federation Servers

+

Add servers for testing, should be overwritten when updated against the registry.

- + + -
nameinstitutionurlprivatekeyfedIdnameinstitutionurlprivatekey
+
-

">view federation xml

-

Update servers

+

" target="_blank">View XML from federation registry

+

Update servers

@@ -38,6 +40,7 @@ + @@ -47,7 +50,9 @@ + +
institution url privateKeyview wddx
" target="_blank">getFedServerGroups