slapos_crm: Consider the default as "disabled" for monitor_scope

master/bt5/slapos_crm/SkinTemplateItem/portal_skins/slapos_crm_monitoring/ComputeNode_checkSoftwareInstallationState.py

@@ -3,7 +3,7 @@ portal = context.getPortalObject()
 
 person = context.getSourceAdministrationValue(portal_type="Person")
 if not person or \
-   context.getMonitorScope() == "disabled" or \
+   context.getMonitorScope("disabled") == "disabled" or \
    portal.ERP5Site_isSupportRequestCreationClosed():
   return 
 

master/bt5/slapos_crm/SkinTemplateItem/portal_skins/slapos_crm_monitoring/ComputeNode_checkState.py

@@ -3,7 +3,7 @@ portal = context.getPortalObject()
 
 person = context.getSourceAdministrationValue(portal_type="Person")
 if not person or \
-   context.getMonitorScope() == "disabled" or \
+   context.getMonitorScope("disabled") == "disabled" or \
    portal.ERP5Site_isSupportRequestCreationClosed():
   return
 

master/bt5/slapos_crm/SkinTemplateItem/portal_skins/slapos_crm_monitoring/InstanceTree_checkSoftwareInstanceState.py

@@ -4,7 +4,7 @@ from erp5.component.module.DateUtils import addToDate
 instance_tree = context
 portal = context.getPortalObject()
 
-if instance_tree.getMonitorScope() == "disabled":
+if instance_tree.getMonitorScope("disabled") == "disabled":
   # Don't generate ticket if Monitor Scope is marked to disable
   return
 

master/bt5/slapos_crm/SkinTemplateItem/portal_skins/slapos_crm_monitoring/SupportRequest_getInstanceMessageList.py

@@ -13,7 +13,7 @@ if document is None:
 aggregate_portal_type = document.getPortalType()
 
 if aggregate_portal_type in ["Compute Node", "Instance Tree"] and \
-  document.getMonitorScope() == "disabled":
+  document.getMonitorScope("disabled") == "disabled":
   return []
 
 if aggregate_portal_type == "Instance Tree":

master/bt5/slapos_crm/SkinTemplateItem/portal_skins/slapos_crm_monitoring/SupportRequest_recheckMonitoring.py

@@ -14,7 +14,7 @@ if document is None:
 
 aggregate_portal_type = document.getPortalType()
 if aggregate_portal_type == "Compute Node":
-  if document.getMonitorScope() == "disabled":
+  if document.getMonitorScope("disabled") == "disabled":
     return "Monitor is disabled to the related %s." % document.getPortalType()
 
   d = document.getAccessStatus()
@@ -65,7 +65,7 @@ if aggregate_portal_type == "Compute Node":
 
 if aggregate_portal_type == "Software Installation":
   compute_node_title = document.getAggregateTitle()
-  if document.getAggregateValue().getMonitorScope() == "disabled":
+  if document.getAggregateValue().getMonitorScope("disabled") == "disabled":
     return "Monitor is disabled to the related %s." % document.getPortalType()
 
   if document.getSlapState() not in ["start_requested", "stop_requested"]:
@@ -87,7 +87,7 @@ if aggregate_portal_type == "Software Installation":
       (document.getUrlString(), compute_node_title, document.getCreationDate())
 
 if aggregate_portal_type == "Instance Tree":
-  if document.getMonitorScope() == "disabled":
+  if document.getMonitorScope("disabled") == "disabled":
     return "Monitor is disabled to the related %s." % document.getPortalType()
 
   message_list = []

master/bt5/slapos_crm/TestTemplateItem/portal_components/test.erp5.testSlapOSCRMSkins.py

@@ -689,9 +689,10 @@ class TestSlapOSHasError(SlapOSTestCaseMixin):
     instance_tree.validate()
     new_id = self.generateNewId()
     instance_tree.edit(
-        title= "Test hosting sub ticket %s" % new_id,
+        title="Test hosting sub ticket %s" % new_id,
         reference="TESTHST-%s" % new_id,
-        destination_section_value=person
+        destination_section_value=person,
+        monitor_scope="enabled"
     )
 
     return instance_tree
@@ -1036,7 +1037,7 @@ class TestSupportRequestUpdateMonitoringState(SlapOSTestCaseMixin):
     instance_tree.validate()
     new_id = self.generateNewId()
     instance_tree.edit(
-        title= "Test hosting sub ticket %s" % new_id,
+        title="Test hosting sub ticket %s" % new_id,
         reference="TESTHST-%s" % new_id,
         destination_section_value=person
     )