Changes between Initial Version and Version 1 of Ticket #34564, comment 5


Ignore:
Timestamp:
May 14, 2023, 5:02:14 AM (19 months ago)
Author:
Amin Aminian

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #34564, comment 5

    initial v1  
    1919    ...
    2020}}}
    21 And aggregate classes have implemented this. All of classes have `empty_result_set_value = None` instead of `Count` class and `RegrCount` class, which it is `empty_result_set_value = 0` in those classes. And because of this attr, we can't use `default=0`:
     21And aggregate classes have implemented this. All of classes have `empty_result_set_value = None` instead of `Count` class and `RegrCount` class, which it is `empty_result_set_value = 0` in those classes. And because of this attr, we can't use `default=0` in `Count`:
    2222
    2323{{{
    … …  
    3434      ...
    3535}}}
    36 So as far as I understand, we are considering `empty_result_set_value` as kind of a default value. So why don't we just return `empty_result_set_value` in case of being None in `convert_value` property ?
     36And in `Count` class, `empty_result_set_value=0` and we use default because of that. So as far as I understand, we are considering `empty_result_set_value` as kind of a default value. So why don't we just return `empty_result_set_value` in case of being None in `convert_value` property ?
Back to Top