Legacy Dev Forum Posts

 View Only

Sign Up

Back to discussions
Expand all | Collapse all

Json serializer broke in 37.2.0

  • 1.  Json serializer broke in 37.2.0

    Posted 06-05-2025 18:05

    kellcomnet | 2018-07-09 21:39:17 UTC | #1

    In 37.1.3 the following code

    var request = new ConversationQuery( Interval: Data.Interval, Order: ConversationQuery.OrderEnum.Asc, OrderBy: ConversationQuery.OrderByEnum.Conversationstart, Paging: new PagingSpec(PageSize: 75, PageNumber: pageNumber), ConversationFilters: new List<AnalyticsQueryFilter>() { new AnalyticsQueryFilter(Type: AnalyticsQueryFilter.TypeEnum.And ,Predicates: new List<AnalyticsQueryPredicate>() { new AnalyticsQueryPredicate(Type: AnalyticsQueryPredicate.TypeEnum.Dimension, Dimension: AnalyticsQueryPredicate.DimensionEnum.Conversationend, _Operator: AnalyticsQueryPredicate.OperatorEnum.Exists) }) } ); logger.Trace("GetConversations: request {0}", request.ToJson());

    generates

    request { "order": "asc", "orderBy": "conversationStart", "interval": "2018-07-09T00:00:00.0000000Z/2018-07-09T21:30:00.0000000Z", "conversationFilters": [ { "type": "and", "predicates": [ { "type": "dimension", "dimension": "conversationEnd", "operator": "exists" } ] } ], "paging": { "pageSize": 75, "pageNumber": 4 } }

    however in 37.2.0 it generates

    request { "order": "asc", "orderBy": "conversationStart", "interval": "2018-07-09T00:00:00.0000000Z/2018-07-09T21:30:00.0000000Z", "conversationFilters": [ { "predicates": [ { "type": "dimension", "dimension": "conversationEnd", "operator": "exists" } ] } ], "paging": {} }

    tim.smith | 2018-07-09 22:00:00 UTC | #2

    Hm, I the changes in that version shouldn't have affected that, but I'll take a look. Thanks for narrowing it down to that specific version. I've created API-3516 to look into it.

    Is this happening to other classes or just these two (AnalyticsQueryFilter and PagingSpec)? Any pattern that you can see?

    kellcomnet | 2018-07-09 22:48:14 UTC | #3

    this was the only project i updated references on so i do not know if it is more widespread.

    tim.smith | 2018-07-11 13:57:11 UTC | #4

    This has been fixed in 37.2.1. I missed a condition in part of the template when removing checks for required properties from the models.

    system | 2018-08-11 13:57:12 UTC | #5

    This topic was automatically closed 31 days after the last reply. New replies are no longer allowed.

    This post was migrated from the old Developer Forum.

    ref: 3146



Related Content