From: <pn...@hy...> - 2009-01-13 17:46:09
|
Author: pnguyen Date: 2009-01-13 09:46:05 -0800 (Tue, 13 Jan 2009) New Revision: 12257 URL: http://svn.hyperic.org/?view=rev&root=Hyperic+HQ&revision=12257 Modified: trunk/src/org/hyperic/hq/bizapp/server/session/AppdefBossEJBImpl.java Log: HHQ-2670: fixed issue with ConcurrentModificationException with the removeServer method (caused by changes to ServiceManagerEJBImpl.removeService(), revision 12049) Modified: trunk/src/org/hyperic/hq/bizapp/server/session/AppdefBossEJBImpl.java =================================================================== --- trunk/src/org/hyperic/hq/bizapp/server/session/AppdefBossEJBImpl.java 2009-01-13 17:44:58 UTC (rev 12256) +++ trunk/src/org/hyperic/hq/bizapp/server/session/AppdefBossEJBImpl.java 2009-01-13 17:46:05 UTC (rev 12257) @@ -1495,7 +1495,9 @@ throws VetoException, PermissionException { try { - for (Iterator it = server.getServices().iterator(); it.hasNext();) { + // Service manager will update the collection, so we need to copy + Collection services = new ArrayList(server.getServices()); + for (Iterator it = services.iterator(); it.hasNext();) { Service service = (Service) it.next(); removeService(subject, service); } |