Is this a bug for chat feature? #1734

andry3ag created 8 years ago

private void CheckChatFeaturesInternal(int? sourceTenantId, int? targetTenantId, ChatSide side) { var localizationPosfix = side == ChatSide.Sender ? "ForSender" : "ForReceiver"; if (sourceTenantId.HasValue) { if (!_featureChecker.IsEnabled(sourceTenantId.Value, AppFeatures.ChatFeature)) { throw new UserFriendlyException(L("ChatFeatureIsNotEnabled" + localizationPosfix)); }

            if (targetTenantId.HasValue)
            {
                if (!_featureChecker.IsEnabled(sourceTenantId.Value, AppFeatures.TenantToTenantChatFeature))
                {
                    throw new UserFriendlyException(L("TenantToTenantChatFeatureIsNotEnabled" + localizationPosfix));
                }
            }

...

I am wondering if the sourceTenantId and targetTenantId are the same, shouldn't the chat be enabled, instead of throw exception in the above code.

Regards,

derek

1 Answer(s)