Commit cb5e1ffc authored by Rafael Monnerat's avatar Rafael Monnerat

slapos_crm: Consider the default as "disabled" for monitor_scope

See merge request !600
parents 8967ca45 d127603b
Pipeline #32544 failed with stage
in 0 seconds
...@@ -3,7 +3,7 @@ portal = context.getPortalObject() ...@@ -3,7 +3,7 @@ portal = context.getPortalObject()
person = context.getSourceAdministrationValue(portal_type="Person") person = context.getSourceAdministrationValue(portal_type="Person")
if not person or \ if not person or \
context.getMonitorScope() == "disabled" or \ context.getMonitorScope("disabled") == "disabled" or \
portal.ERP5Site_isSupportRequestCreationClosed(): portal.ERP5Site_isSupportRequestCreationClosed():
return return
......
...@@ -3,7 +3,7 @@ portal = context.getPortalObject() ...@@ -3,7 +3,7 @@ portal = context.getPortalObject()
person = context.getSourceAdministrationValue(portal_type="Person") person = context.getSourceAdministrationValue(portal_type="Person")
if not person or \ if not person or \
context.getMonitorScope() == "disabled" or \ context.getMonitorScope("disabled") == "disabled" or \
portal.ERP5Site_isSupportRequestCreationClosed(): portal.ERP5Site_isSupportRequestCreationClosed():
return return
......
...@@ -4,7 +4,7 @@ from erp5.component.module.DateUtils import addToDate ...@@ -4,7 +4,7 @@ from erp5.component.module.DateUtils import addToDate
instance_tree = context instance_tree = context
portal = context.getPortalObject() 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 # Don't generate ticket if Monitor Scope is marked to disable
return return
......
...@@ -13,7 +13,7 @@ if document is None: ...@@ -13,7 +13,7 @@ if document is None:
aggregate_portal_type = document.getPortalType() aggregate_portal_type = document.getPortalType()
if aggregate_portal_type in ["Compute Node", "Instance Tree"] and \ if aggregate_portal_type in ["Compute Node", "Instance Tree"] and \
document.getMonitorScope() == "disabled": document.getMonitorScope("disabled") == "disabled":
return [] return []
if aggregate_portal_type == "Instance Tree": if aggregate_portal_type == "Instance Tree":
......
...@@ -14,7 +14,7 @@ if document is None: ...@@ -14,7 +14,7 @@ if document is None:
aggregate_portal_type = document.getPortalType() aggregate_portal_type = document.getPortalType()
if aggregate_portal_type == "Compute Node": 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() return "Monitor is disabled to the related %s." % document.getPortalType()
d = document.getAccessStatus() d = document.getAccessStatus()
...@@ -65,7 +65,7 @@ if aggregate_portal_type == "Compute Node": ...@@ -65,7 +65,7 @@ if aggregate_portal_type == "Compute Node":
if aggregate_portal_type == "Software Installation": if aggregate_portal_type == "Software Installation":
compute_node_title = document.getAggregateTitle() 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() return "Monitor is disabled to the related %s." % document.getPortalType()
if document.getSlapState() not in ["start_requested", "stop_requested"]: if document.getSlapState() not in ["start_requested", "stop_requested"]:
...@@ -87,7 +87,7 @@ if aggregate_portal_type == "Software Installation": ...@@ -87,7 +87,7 @@ if aggregate_portal_type == "Software Installation":
(document.getUrlString(), compute_node_title, document.getCreationDate()) (document.getUrlString(), compute_node_title, document.getCreationDate())
if aggregate_portal_type == "Instance Tree": 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() return "Monitor is disabled to the related %s." % document.getPortalType()
message_list = [] message_list = []
......
...@@ -689,9 +689,10 @@ class TestSlapOSHasError(SlapOSTestCaseMixin): ...@@ -689,9 +689,10 @@ class TestSlapOSHasError(SlapOSTestCaseMixin):
instance_tree.validate() instance_tree.validate()
new_id = self.generateNewId() new_id = self.generateNewId()
instance_tree.edit( instance_tree.edit(
title= "Test hosting sub ticket %s" % new_id, title="Test hosting sub ticket %s" % new_id,
reference="TESTHST-%s" % new_id, reference="TESTHST-%s" % new_id,
destination_section_value=person destination_section_value=person,
monitor_scope="enabled"
) )
return instance_tree return instance_tree
...@@ -1036,7 +1037,7 @@ class TestSupportRequestUpdateMonitoringState(SlapOSTestCaseMixin): ...@@ -1036,7 +1037,7 @@ class TestSupportRequestUpdateMonitoringState(SlapOSTestCaseMixin):
instance_tree.validate() instance_tree.validate()
new_id = self.generateNewId() new_id = self.generateNewId()
instance_tree.edit( instance_tree.edit(
title= "Test hosting sub ticket %s" % new_id, title="Test hosting sub ticket %s" % new_id,
reference="TESTHST-%s" % new_id, reference="TESTHST-%s" % new_id,
destination_section_value=person destination_section_value=person
) )
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment