Github user tliron commented on a diff in the pull request:

    https://github.com/apache/incubator-ariatosca/pull/207#discussion_r153011505
  
    --- Diff: 
extensions/aria_extension_tosca/simple_v1_0/modeling/data_types.py ---
    @@ -318,15 +318,15 @@ def report(message, constraint):
     #
     
     def get_data_type_value(context, presentation, field_name, type_name):
    -    the_type = get_type_by_name(context, type_name, 'data_types')
    -    if the_type is not None:
    -        value = getattr(presentation, field_name)
    -        if value is not None:
    +    value = getattr(presentation, field_name)
    +    if value is not None:
    +        the_type = get_type_by_name(context, type_name, 'data_types')
    --- End diff --
    
    +1 to `data_type`


---

Reply via email to