From: <pn...@hy...> - 2009-01-28 01:11:58
|
Author: pnguyen Date: 2009-01-27 17:11:52 -0800 (Tue, 27 Jan 2009) New Revision: 12365 URL: http://svn.hyperic.org/?view=rev&root=Hyperic+HQ&revision=12365 Modified: trunk/src/org/hyperic/hq/measurement/server/session/MeasurementManagerEJBImpl.java Log: HHQ-2724: check enabled state before enabling Modified: trunk/src/org/hyperic/hq/measurement/server/session/MeasurementManagerEJBImpl.java =================================================================== --- trunk/src/org/hyperic/hq/measurement/server/session/MeasurementManagerEJBImpl.java 2009-01-28 01:09:12 UTC (rev 12364) +++ trunk/src/org/hyperic/hq/measurement/server/session/MeasurementManagerEJBImpl.java 2009-01-28 01:11:52 UTC (rev 12365) @@ -936,14 +936,20 @@ throws PermissionException { AppdefEntityID appId = new AppdefEntityID(r); checkModifyPermission(subj.getId(), appId); + boolean sendToAgent = false; List metrics = getMeasurementDAO().findDefaultsByResource(r); for (Iterator it = metrics.iterator(); it.hasNext(); ) { Measurement dm = (Measurement)it.next(); - dm.setEnabled(true); + if (!dm.isEnabled()) { + dm.setEnabled(true); + sendToAgent = true; + } } - sendAgentSchedule(appId); + if (sendToAgent) { + sendAgentSchedule(appId); + } } /** |