[TA] Fix not passing domain=phi in sync and analyze endpoint plus some analyze cleanup (Azure#19104)

* fix in sync and analyze endpoint plus some analyze cleanup

* remove transform

Author: maririos

sdk/textanalytics/Azure.AI.TextAnalytics/CHANGELOG.md

@@ -9,7 +9,11 @@
 - Renamed `MinedOpinion` to `SentenceOpinion`.
 - Renamed `OpinionSentiment` to `AssessmentSentiment`.
 - For `PiiEntity.Category` the type of the property is now `PiiEntityCategory` instead of `EntityCategory`.
- 
+- `RecognizePiiEntitiesOptions.Domain` is now a nullable type.
+
+### Fixes
+- `RecognizePiiEntities` and `TextAnalyticsActions.RecognizePiiEntitiesOptions` were always passing `PiiEntityDomainType.PHI`. Now, it is only passed when requested by the user [19086](https://github.com/Azure/azure-sdk-for-net/issues/19086).
+
 ## 5.1.0-beta.4 (2021-02-10)
 ### New features
 - Added property `Length` to `CategorizedEntity`, `SentenceSentiment`, `LinkedEntityMatch`, `AspectSentiment`, `OpinionSentiment`, and `PiiEntity`.

sdk/textanalytics/Azure.AI.TextAnalytics/api/Azure.AI.TextAnalytics.netstandard2.0.cs

@@ -534,7 +534,7 @@ public partial class RecognizePiiEntitiesOptions : Azure.AI.TextAnalytics.TextAn
     {
         public RecognizePiiEntitiesOptions() { }
         public System.Collections.Generic.IList<Azure.AI.TextAnalytics.PiiEntityCategory> CategoriesFilter { get { throw null; } }
-        public Azure.AI.TextAnalytics.PiiEntityDomainType DomainFilter { get { throw null; } set { } }
+        public Azure.AI.TextAnalytics.PiiEntityDomainType? DomainFilter { get { throw null; } set { } }
     }
     public partial class RecognizePiiEntitiesResult : Azure.AI.TextAnalytics.TextAnalyticsResult
     {

sdk/textanalytics/Azure.AI.TextAnalytics/src/EntitiesTask.cs

@@ -1,25 +1,13 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
 
-using Azure.AI.TextAnalytics.Models;
 using Azure.Core;
 
-namespace Azure.AI.TextAnalytics
+namespace Azure.AI.TextAnalytics.Models
 {
     /// <summary>
     /// EntitiesTaskParameters class.
     /// </summary>
     [CodeGenModel("EntitiesTaskParameters")]
-    internal partial class EntitiesTaskParameters
-    {
-        /// <summary>
-        /// ModelVersion
-        /// </summary>
-        public string ModelVersion { get; set; } = "latest";
-
-        /// <summary>
-        /// StringIndexType
-        /// </summary>
-        public StringIndexType StringIndexType { get; set; } = StringIndexType.Utf16CodeUnit;
-    }
+    internal partial class EntitiesTaskParameters { }
 }