Fix updating of number of problems when filter items

Ref #309
Alignak-monitoring-contrib · Apr 12, 2018 · c457677 · c457677
1 parent f8b0d33
commit c457677
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/alignak_app/qobjects/alignak/problems.py b/alignak_app/qobjects/alignak/problems.py
@@ -84,7 +84,7 @@ def initialize(self, spy_widget):
 
         self.update_problems_data()
 
-    def get_curent_user_role_item(self):
+    def get_current_user_role_item(self):
         """
         Return current selected item by ``Qt.UserRole``
 
@@ -106,7 +106,7 @@ def update_action_buttons(self):
         """
 
         # Get item by UserRole
-        item = self.get_curent_user_role_item()
+        item = self.get_current_user_role_item()
 
         if item:
             # If the elements had been ack or downtimed, they would not be present
@@ -226,7 +226,7 @@ def add_spied_host(self):
         """
 
         # Get item by UserRole
-        item = self.get_curent_user_role_item()
+        item = self.get_current_user_role_item()
 
         if item:
             if 'service' in item.item_type:
@@ -256,6 +256,13 @@ def update_problems_data(self, item_type=''):
                 self.parent().parent().indexOf(self),
                 _("Problems (%d)") % len(problems_data['problems'])
             )
+            self.problems_title.setText(
+                _('There are %d problems to manage (hosts: %d, services: %d)') % (
+                    len(problems_data['problems']),
+                    problems_data['hosts_nb'],
+                    problems_data['services_nb']
+                )
+            )
 
         if self.filter_hosts_btn.is_checked() and not self.filter_services_btn.is_checked():
             item_type = 'host'
@@ -277,13 +284,6 @@ def update_problems_data(self, item_type=''):
 
         proxy_filter = self.problems_table.update_view(problems_data)
         if problems_data['problems']:
-            self.problems_title.setText(
-                _('There are %d problems to manage (hosts: %d, services: %d)') % (
-                    len(problems_data['problems']),
-                    problems_data['hosts_nb'],
-                    problems_data['services_nb']
-                )
-            )
             self.line_search.textChanged.connect(proxy_filter.setFilterRegExp)
         else:
             self.problems_title.setText(_('If problems are found, they will be displayed here.'))

diff --git a/alignak_app/qobjects/panel.py b/alignak_app/qobjects/panel.py
@@ -195,7 +195,7 @@ def set_host_from_problems(self):
 
         """
 
-        item = self.problems_widget.get_curent_user_role_item()
+        item = self.problems_widget.get_current_user_role_item()
         if item:
             if 'service' in item.item_type:
                 hostname = data_manager.get_item('host', item.data['host']).name

diff --git a/test/test_panel_widget.py b/test/test_panel_widget.py
@@ -207,7 +207,7 @@ def test_set_host_from_problems(self):
         under_test.initialize()
 
         self.assertEqual('', under_test.synthesis_widget.line_search.text())
-        self.assertIsNone(under_test.problems_widget.get_curent_user_role_item())
+        self.assertIsNone(under_test.problems_widget.get_current_user_role_item())
 
         # Make an item as current in problems table
         under_test.problems_widget.problems_table.update_view({'problems': [self.host_list[8]]})
@@ -217,7 +217,7 @@ def test_set_host_from_problems(self):
             QItemSelectionModel.SelectCurrent
         )
 
-        self.assertIsNotNone(under_test.problems_widget.get_curent_user_role_item())
+        self.assertIsNotNone(under_test.problems_widget.get_current_user_role_item())
         self.assertEqual('', under_test.synthesis_widget.line_search.text())
 
         under_test.set_host_from_problems()