Fix Az.Sql XML comments (#18576)

* Fix Az.Sql XML comments

* Fix Az.Sql XML comments

* Update DatabaseEditions.cs

src/Sql/Sql.LegacySdk/Generated/Models/DatabaseEditions.cs

@@ -35,33 +35,33 @@ public static partial class DatabaseEditions
         /// and workgroup or departmental applications.
         /// </summary>
         public const string Web = "Web";
-        
+
         /// <summary>
         /// The Business Edition Database is best suited for independent
         /// software vendors (ISVs), line-of-business (LOB) applications, and
         /// enterprise applications.
         /// </summary>
         public const string Business = "Business";
-        
+
         /// <summary>
         /// The Basic Edition Database is best suited for small databases with
-        /// few transactions (<100) per minute and only a couple users.
+        /// few transactions (&lt;100) per minute and only a couple users.
         /// </summary>
         public const string Basic = "Basic";
-        
+
         /// <summary>
         /// The Standard Edition Database is best suited for a group of users
         /// makingfewer than 1000 transactions per minute.
         /// </summary>
         public const string Standard = "Standard";
-        
+
         /// <summary>
         /// The Premium Edition Database is for situations requiring a high
-        /// number of transactionsper second (>100).  Supports more users than
+        /// number of transactionsper second (&gt;100).  Supports more users than
         /// Basic and Standard editions.
         /// </summary>
         public const string Premium = "Premium";
-        
+
         /// <summary>
         /// The Free Edition Database is available for certain offers and
         /// product bundles.

src/Sql/Sql/AdvancedThreatProtection/Cmdlet/SqlManagedInstanceAdvancedDataSecurityCmdletBase.cs

@@ -71,7 +71,6 @@ protected override ManagedInstanceAdvancedDataSecurityPolicyModel GetEntity()
         /// <summary>
         /// Creation and initialization of the ModelAdapter object
         /// </summary>
-        /// <param name="subscription">The AzureSubscription in which the current execution is performed</param>
         /// <returns>An initialized and ready to use ModelAdapter object</returns>
         protected override SqlAdvancedDataSecurityAdapter InitModelAdapter()
         {

src/Sql/Sql/Data Masking/Cmdlet/BuildAzureSqlDatabaseDataMaskingRule.cs

@@ -71,7 +71,7 @@ public abstract class BuildAzureSqlDatabaseDataMaskingRule : SqlDatabaseDataMask
         /// <summary>
         /// Updates the given model element with the cmdlet specific operation 
         /// </summary>
-        /// <param name="model">A model object</param>
+        /// <param name="rules">A model object</param>
         protected override IEnumerable<DatabaseDataMaskingRuleModel> ApplyUserInputToModel(IEnumerable<DatabaseDataMaskingRuleModel> rules)
         {
             string errorMessage = ValidateOperation(rules);
@@ -215,7 +215,7 @@ private MaskingFunction ModelizeMaskingFunction()
         /// This method is responsible to call the right API in the communication layer that will eventually send the information in the 
         /// object to the REST endpoint
         /// </summary>
-        /// <param name="model">The model object with the data to be sent to the REST endpoints</param>
+        /// <param name="rules">The model object with the data to be sent to the REST endpoints</param>
         protected override IEnumerable<DatabaseDataMaskingRuleModel> PersistChanges(IEnumerable<DatabaseDataMaskingRuleModel> rules)
         {
             ModelAdapter.SetDatabaseDataMaskingRule(rules.First(IsModelOfRule));

src/Sql/Sql/Data Masking/Cmdlet/SqlDatabaseDataMaskingRuleCmdletBase.cs

@@ -78,7 +78,7 @@ protected override IEnumerable<DatabaseDataMaskingRuleModel> GetEntity()
         /// This method is responsible to call the right API in the communication layer that will eventually send the information in the 
         /// object to the REST endpoint
         /// </summary>
-        /// <param name="model">The model object with the data to be sent to the REST endpoints</param>
+        /// <param name="rules">The model object with the data to be sent to the REST endpoints</param>
         protected override IEnumerable<DatabaseDataMaskingRuleModel> PersistChanges(IEnumerable<DatabaseDataMaskingRuleModel> rules)
         {
             DatabaseDataMaskingRuleModel model = rules.First();

src/Sql/Sql/Data Sync/Model/AzureSqlSyncAgentModel.cs

@@ -81,6 +81,7 @@ public AzureSqlSyncAgentModel()
         /// Construct AzureSqlSyncAgentModel
         /// </summary>
         /// <param name="resourceGroup">resource group</param>
+        /// <param name="serverName">server name</param>
         /// <param name="syncAgent">sync agent</param>
         public AzureSqlSyncAgentModel(string resourceGroup, string serverName, SyncAgent syncAgent)
         {

src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaColumnModel.cs

@@ -23,7 +23,7 @@ namespace Microsoft.Azure.Commands.Sql.DataSync.Model
     /// </summary>
     public class AzureSqlSyncGroupSchemaColumnModel
     {
-        // <summary>
+        /// <summary>
         /// The quoted table name
         /// </summary>
         public string QuotedName { get; set; }

src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaModel.cs

@@ -28,7 +28,7 @@ public class AzureSqlSyncGroupSchemaModel
         /// </summary>
         public IList<AzureSqlSyncGroupSchemaTableModel> Tables { get; set; }
 
-        // <summary>
+        /// <summary>
         /// Master sync member name
         /// </summary>
         public string MasterSyncMemberName { get; set; }

src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaTableModel.cs

@@ -28,7 +28,7 @@ public class AzureSqlSyncGroupSchemaTableModel
         /// </summary>
         public IList<AzureSqlSyncGroupSchemaColumnModel> Columns { get; set; }
 
-        // <summary>
+        /// <summary>
         /// The quoted table name
         /// </summary>
         public string QuotedName { get; set; }

src/Sql/Sql/Data Sync/Model/AzureSqlSyncMemberModel.cs

@@ -118,10 +118,11 @@ public AzureSqlSyncMemberModel()
         /// <summary>
         /// Construct AzureSqlSyncMemberModel for Management.Sql.Models.syncMember object
         /// </summary>
-        /// <param name="resourceGroupName">Resource group name</param>
+        /// <param name="resourceGroup">Resource group name</param>
         /// <param name="serverName">Server name</param>
         /// <param name="databaseName">Databse name</param>
-        /// <param name="syncGroup">sync member object</param>
+        /// <param name="syncGroupName">The name of the sync group</param>
+        /// <param name="syncMember">sync member object</param>
         public AzureSqlSyncMemberModel(string resourceGroup, string serverName, string databaseName, string syncGroupName, SyncMember syncMember)
         {
             ResourceGroupName = resourceGroup;

src/Sql/Sql/Data Sync/Services/AzureSqlDataSyncAdapter.cs

@@ -161,6 +161,7 @@ public void InvokeSyncMemberSchemaRefresh(string resourceGroupName, string serve
         /// Create a sync group
         /// </summary>
         /// <param name="model">AzureSqlSyncGroupModel object</param>
+        /// <param name="syncDatabaseId">The sync database resource id</param>
         /// <returns>Created AzureSqlSyncGroupModel object</returns>
         internal AzureSqlSyncGroupModel CreateSyncGroup(AzureSqlSyncGroupModel model, string syncDatabaseId)
         {
@@ -295,8 +296,7 @@ public void RemoveSyncMember(string resourceGroupName, string serverName, string
         /// Create a sync member
         /// </summary>
         /// <param name="model">AzureSqlSyncMemberModel object</param>
-        /// <param name="databaseType">The type of member database</param>
-        /// <param name="password">The password of member database</param>
+        /// <param name="syncAgentId">The sync agent resource id</param>
         /// <returns>Created AzureSqlSyncGroupModel object</returns>
         internal AzureSqlSyncMemberModel CreateSyncMember(AzureSqlSyncMemberModel model, string syncAgentId)
         {
@@ -335,8 +335,6 @@ internal AzureSqlSyncMemberModel CreateSyncMember(AzureSqlSyncMemberModel model,
         /// Update a sync member
         /// </summary>
         /// <param name="model">AzureSqlSyncMemberModel object</param>
-        /// <param name="databaseType">The type of member database</param>
-        /// <param name="password">The password of member database</param>
         /// <returns>Updated AzureSqlSyncGroupModel object</returns>
         internal AzureSqlSyncMemberModel UpdateSyncMember(AzureSqlSyncMemberModel model)
         {
@@ -384,6 +382,7 @@ public AzureSqlSyncFullSchemaModel GetSyncMemberSchema(string resourceGroupName,
         /// Gets a sync agent by name
         /// </summary>
         /// <param name="resourceGroupName">The resource group the sync agent is in</param>
+        /// <param name="serverName">The name of the server</param>
         /// <param name="syncAgentName">The name of the sync agent</param>
         /// <returns>The sync agent object</returns>
         public AzureSqlSyncAgentModel GetSyncAgent(string resourceGroupName, string serverName, string syncAgentName)
@@ -412,6 +411,7 @@ internal ICollection<AzureSqlSyncAgentModel> ListSyncAgents(string resourceGroup
         /// Create a sync agent
         /// </summary>
         /// <param name="model">AzureSqlSyncAgentModel object</param>
+        /// <param name="syncDatabaseId">The sync database resource id</param>
         /// <returns>Created AzureSqlSyncAgentModel object</returns>
         internal AzureSqlSyncAgentModel CreateSyncAgent(AzureSqlSyncAgentModel model, string syncDatabaseId)
         {
@@ -458,6 +458,7 @@ internal AzureSqlSyncAgentKeyModel CreateSyncAgentKey(string resourceGroupName,
         /// Get all linked databases connected by a specified sync agent
         /// </summary>
         /// <param name="resourceGroupName">The resource group the sync agent is in</param>
+        /// <param name="serverName">The name of the server</param>
         /// <param name="syncAgentName">The name of the sync agent</param>
         internal ICollection<AzureSqlSyncAgentLinkedDatabaseModel> ListSyncAgentLinkedDatabases(string resourceGroupName, string serverName, string syncAgentName)
         {
@@ -497,6 +498,7 @@ public static AzureSqlSyncGroupLogModel CreateSyncGroupLogModelFromResponse(Sync
         /// <param name="resourceGroupName">The resource group the sync member is in</param>
         /// <param name="serverName">The name of the server</param>
         /// <param name="databaseName">The name of the database</param>
+        /// <param name="syncGroupName">The name of the sync group</param>
         /// <param name="syncMember">The sync member object from the response</param>
         /// <returns>The converted model</returns>
         public static AzureSqlSyncMemberModel CreateSyncMemberModelFromResponse(string resourceGroupName, string serverName, string databaseName, string syncGroupName, Management.Sql.Models.SyncMember syncMember)
@@ -508,7 +510,7 @@ public static AzureSqlSyncMemberModel CreateSyncMemberModelFromResponse(string r
         /// Converts the response from the service to a powershell sync agent object
         /// </summary>
         /// <param name="resourceGroupName">The resource group the agent is in</param>
-        /// <param name="server">The server name</param>
+        /// <param name="serverName">The server name</param>
         /// <param name="syncAgent">The sync agent object from the response</param>
         /// <returns>The converted model</returns>
         public static AzureSqlSyncAgentModel CreateSyncAgentModelFromResponse(string resourceGroupName, string serverName, SyncAgent syncAgent)

src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationAdapter.cs

@@ -38,8 +38,7 @@ public class AzureSqlDatabaseActivationAdapter
         /// <summary>
         /// Constructs a database activation adapter
         /// </summary>
-        /// <param name="profile">The current azure profile</param>
-        /// <param name="subscription">The current azure subscription</param>
+        /// <param name="context">The current azure context</param>
         public AzureSqlDatabaseActivationAdapter(IAzureContext context)
         {
             Context = context;

src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationCommunicator.cs

@@ -43,8 +43,7 @@ public class AzureSqlDatabaseActivationCommunicator
         /// <summary>
         /// Creates a communicator for Azure Sql Databases
         /// </summary>
-        /// <param name="profile"></param>
-        /// <param name="subscription"></param>
+        /// <param name="context">The current azure context</param>
         public AzureSqlDatabaseActivationCommunicator(IAzureContext context)
         {
             Context = context;

src/Sql/Sql/Database Backup/Cmdlet/CopyAzureSqlDatabaseLongTermRetentionBackup.cs

@@ -135,7 +135,7 @@ public class CopyAzureSqlDatabaseLongTermRetentionBackup : AzureSqlDatabaseLongT
         public string TargetServerFullyQualifiedDomainName { get; set; }
 
         /// <summary>
-        /// Gets or sets the resource ID of the target server (target subscription ID & target resource group name can be derived).  
+        /// Gets or sets the resource ID of the target server (target subscription ID &amp; target resource group name can be derived).
         /// </summary>
         [Parameter(Mandatory = false,
             HelpMessage = "The name of the target server.")]

src/Sql/Sql/Database Backup/Cmdlet/RestoreAzureRMSqlDatabase.cs

@@ -399,7 +399,7 @@ protected override AzureSqlDatabaseModel GetEntity()
                 model.Edition = Edition;
             }
 
-            /// get auth headers for cross-sub and cross-tenant restore operations
+            // get auth headers for cross-sub and cross-tenant restore operations
             string targetSubscriptionId = ModelAdapter.Context?.Subscription?.Id;
             string sourceSubscriptionId = ParseSubscriptionIdFromResourceId(ResourceId);
             bool isCrossSubscriptionRestore = false;

src/Sql/Sql/Database Backup/Model/AzureSqlDatabaseBackupShortTermRetentionPolicyModel.cs

@@ -48,6 +48,7 @@ public class AzureSqlDatabaseBackupShortTermRetentionPolicyModel
         /// </summary>
         /// <param name="resourceGroup"></param>
         /// <param name="serverName"></param>
+        /// <param name="databaseName"></param>
         /// <param name="policy"></param>
         public AzureSqlDatabaseBackupShortTermRetentionPolicyModel(string resourceGroup, string serverName, string databaseName, Management.Sql.Models.BackupShortTermRetentionPolicy policy)
         {