Merge pull request #35 from zendtech/bartlomiej

ZSTD-4968 [Regression] Monitoring filter URLs are not saved
zendtech · May 8, 2015 · de08e6f · de08e6f
2 parents 035729a + a007283
commit de08e6f
Showing 1 changed file with 15 additions and 11 deletions.
diff --git a/...or.ui/src/org/zend/php/zendserver/monitor/ui/preferences/MonitoringCompositeFragment.java b/...or.ui/src/org/zend/php/zendserver/monitor/ui/preferences/MonitoringCompositeFragment.java
@@ -117,17 +117,21 @@ public void saveValues() {
 	@Override
 	public boolean performOk() {
 		Server server = getServer();
-		if (server != null && isDeploymentEnabled()) {
-			updateAttribute(MonitorManager.FILTERS_ATTRIBUTE, getValue(input),
-					server);
-			updateAttribute(MonitorManager.HIDE_ATTRIBUTE,
-					String.valueOf(hideButton.getSelection()), server);
-			updateAttribute(MonitorManager.HIDE_TIME_ATTRIBUTE,
-					delayText.getText(), server);
-			for (int i = 0; i < severityButtons.length; i++) {
-				String severityName = getSeverityName(severityButtons[i]);
-				updateAttribute(MonitorManager.SEVERITY_ATTRIBUTE
-						+ severityName, String.valueOf(severities[i]), server);
+		if (server != null) {
+			IZendTarget target = ServerUtils.getTarget(server);
+			if (isDeploymentEnabled() || target != null) {
+				updateAttribute(MonitorManager.FILTERS_ATTRIBUTE,
+						getValue(input), server);
+				updateAttribute(MonitorManager.HIDE_ATTRIBUTE,
+						String.valueOf(hideButton.getSelection()), server);
+				updateAttribute(MonitorManager.HIDE_TIME_ATTRIBUTE,
+						delayText.getText(), server);
+				for (int i = 0; i < severityButtons.length; i++) {
+					String severityName = getSeverityName(severityButtons[i]);
+					updateAttribute(MonitorManager.SEVERITY_ATTRIBUTE
+							+ severityName, String.valueOf(severities[i]),
+							server);
+				}
 			}
 		}
 		return true;