minor refactoring (issue #NX-1289)
authorVictor Kirhenshtein <victor@netxms.org>
Tue, 28 Nov 2017 11:09:53 +0000 (13:09 +0200)
committerVictor Kirhenshtein <victor@netxms.org>
Tue, 28 Nov 2017 11:09:53 +0000 (13:09 +0200)
src/java/netxms-eclipse/ObjectManager/src/org/netxms/ui/eclipse/objectmanager/propertypages/ObjectPolling.java
src/server/core/interface.cpp
src/server/include/nms_objects.h
webui/webapp/ObjectManager/src/org/netxms/ui/eclipse/objectmanager/propertypages/ObjectPolling.java

index 4af14f7..343846a 100644 (file)
@@ -29,7 +29,6 @@ import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Control;
 import org.eclipse.swt.widgets.Group;
 import org.eclipse.swt.widgets.Label;
-import org.eclipse.swt.widgets.Spinner;
 import org.eclipse.ui.dialogs.PropertyPage;
 import org.netxms.client.NXCObjectModificationData;
 import org.netxms.client.NXCSession;
@@ -42,6 +41,7 @@ import org.netxms.ui.eclipse.objectmanager.Activator;
 import org.netxms.ui.eclipse.objectmanager.Messages;
 import org.netxms.ui.eclipse.shared.ConsoleSharedData;
 import org.netxms.ui.eclipse.tools.WidgetHelper;
+import org.netxms.ui.eclipse.widgets.LabeledSpinner;
 
 /**
  * "Polling" property page for nodes
@@ -56,7 +56,7 @@ public class ObjectPolling extends PropertyPage
        private Button radioAgentCacheDefault;
    private Button radioAgentCacheOn;
    private Button radioAgentCacheOff;
-   private Spinner pollCount;
+   private LabeledSpinner pollCount;
        private List<Button> flagButtons = new ArrayList<Button>();
        private List<Integer> flagValues = new ArrayList<Integer>();
        
@@ -78,7 +78,7 @@ public class ObjectPolling extends PropertyPage
       GridData gd = new GridData();
       
       /* poller node */
-      if(object.containPollerNode())
+      if (object.containPollerNode())
       {
          Group servicePollGroup = new Group(dialogArea, SWT.NONE);
          servicePollGroup.setText(Messages.get().NodePolling_GroupNetSrv);
@@ -108,6 +108,14 @@ public class ObjectPolling extends PropertyPage
                label.setLayoutData(gd);
       }
 
+      /* poll count */
+      if (object instanceof AbstractNode)
+      {
+         pollCount = new LabeledSpinner(dialogArea, SWT.NONE);
+         pollCount.setLabel("Required poll count for status change");
+         pollCount.setSelection(((AbstractNode)object).getRequredPollCount());
+      }
+
                /* options */
                Group optionsGroup = new Group(dialogArea, SWT.NONE);
                optionsGroup.setText(Messages.get().NodePolling_GroupOptions);
@@ -192,15 +200,6 @@ public class ObjectPolling extends PropertyPage
          radioAgentCacheOff.setSelection(object.getAgentCacheMode() == AgentCacheMode.OFF);
       }
       
-      /* poll count */
-      if (object instanceof AbstractNode)
-      {
-         gd = new GridData();
-         gd.verticalSpan = 1;
-         pollCount = WidgetHelper.createLabeledSpinner(dialogArea, SWT.BORDER, "Required poll count", 0, 1000, gd);
-         pollCount.setSelection(((AbstractNode)object).getRequredPollCount());
-      }
-
       return dialogArea;
        }
 
index 0f34b69..1cf506f 100644 (file)
@@ -642,7 +642,7 @@ void Interface::statusPoll(ClientSession *session, UINT32 rqId, Queue *eventQueu
        }
 
        int requiredPolls = (m_requiredPollCount > 0) ? m_requiredPollCount :
-                           ((getParentNode()->getRequiredPolls() > 0) ? getParentNode()->getRequiredPolls() : g_requiredPolls);
+                           ((getParentNode()->getRequiredPollCount() > 0) ? getParentNode()->getRequiredPollCount() : g_requiredPolls);
        sendPollerMsg(rqId, _T("      Interface is %s for %d poll%s (%d poll%s required for status change)\r\n"),
                      GetStatusAsText(newStatus, true), m_statusPollCount, (m_statusPollCount == 1) ? _T("") : _T("s"),
                      requiredPolls, (requiredPolls == 1) ? _T("") : _T("s"));
index 740ffb9..9d50973 100644 (file)
@@ -1904,6 +1904,7 @@ public:
    const uuid& getTunnelId() const { return m_tunnelId; }
    void setNewTunnelBindFlag() { m_runtimeFlags |= NDF_NEW_TUNNEL_BIND; }
    void removeNewTunnelBindFlag() { m_runtimeFlags &= ~NDF_NEW_TUNNEL_BIND; }
+   UINT32 getRequiredPollCount() const { return m_requiredPollCount; }
 
    bool isDown() { return (m_state & DCSF_UNREACHABLE) ? true : false; }
        time_t getDownTime() const { return m_downSince; }
@@ -2045,8 +2046,6 @@ public:
        void incSnmpTrapCount();
 
        static const TCHAR *typeName(NodeType type);
-
-       UINT32 getRequiredPolls() const { return m_requiredPollCount; }
 };
 
 /**
index f34e29b..343846a 100644 (file)
@@ -29,7 +29,6 @@ import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Control;
 import org.eclipse.swt.widgets.Group;
 import org.eclipse.swt.widgets.Label;
-import org.eclipse.swt.widgets.Spinner;
 import org.eclipse.ui.dialogs.PropertyPage;
 import org.netxms.client.NXCObjectModificationData;
 import org.netxms.client.NXCSession;
@@ -42,6 +41,7 @@ import org.netxms.ui.eclipse.objectmanager.Activator;
 import org.netxms.ui.eclipse.objectmanager.Messages;
 import org.netxms.ui.eclipse.shared.ConsoleSharedData;
 import org.netxms.ui.eclipse.tools.WidgetHelper;
+import org.netxms.ui.eclipse.widgets.LabeledSpinner;
 
 /**
  * "Polling" property page for nodes
@@ -50,13 +50,13 @@ public class ObjectPolling extends PropertyPage
 {
        private PollingTarget object;
        private ObjectSelector pollerNode;
-   private Spinner pollCount;
        private Button radioIfXTableDefault;
        private Button radioIfXTableEnable;
        private Button radioIfXTableDisable;
        private Button radioAgentCacheDefault;
    private Button radioAgentCacheOn;
    private Button radioAgentCacheOff;
+   private LabeledSpinner pollCount;
        private List<Button> flagButtons = new ArrayList<Button>();
        private List<Integer> flagValues = new ArrayList<Integer>();
        
@@ -78,7 +78,7 @@ public class ObjectPolling extends PropertyPage
       GridData gd = new GridData();
       
       /* poller node */
-      if(object.containPollerNode())
+      if (object.containPollerNode())
       {
          Group servicePollGroup = new Group(dialogArea, SWT.NONE);
          servicePollGroup.setText(Messages.get().NodePolling_GroupNetSrv);
@@ -108,6 +108,14 @@ public class ObjectPolling extends PropertyPage
                label.setLayoutData(gd);
       }
 
+      /* poll count */
+      if (object instanceof AbstractNode)
+      {
+         pollCount = new LabeledSpinner(dialogArea, SWT.NONE);
+         pollCount.setLabel("Required poll count for status change");
+         pollCount.setSelection(((AbstractNode)object).getRequredPollCount());
+      }
+
                /* options */
                Group optionsGroup = new Group(dialogArea, SWT.NONE);
                optionsGroup.setText(Messages.get().NodePolling_GroupOptions);
@@ -192,15 +200,6 @@ public class ObjectPolling extends PropertyPage
          radioAgentCacheOff.setSelection(object.getAgentCacheMode() == AgentCacheMode.OFF);
       }
       
-      /* poll count */
-      if (object instanceof AbstractNode)
-      {
-         gd = new GridData();
-         gd.verticalSpan = 1;
-         pollCount = WidgetHelper.createLabeledSpinner(dialogArea, SWT.BORDER, "Required poll count", 0, 1000, gd);
-         pollCount.setSelection(((AbstractNode)object).getRequredPollCount());
-      }
-
       return dialogArea;
        }