Reverting member name/visibility linting

This will be tackled in a future PR

Author: edwardneal

src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlBulkCopy.cs

@@ -2458,16 +2458,16 @@ private Assembly ResolveTypeAssembly(AssemblyName asmRef, bool throwOnError)
 
         internal void CheckGetExtendedUDTInfo(SqlMetaDataPriv metaData, bool fThrow)
         {
-            if (metaData.Udt?.Type == null)
+            if (metaData.udt?.Type == null)
             { // If null, we have not obtained extended info.
-                Debug.Assert(!string.IsNullOrEmpty(metaData.Udt?.AssemblyQualifiedName), "Unexpected state on GetUDTInfo");
+                Debug.Assert(!string.IsNullOrEmpty(metaData.udt?.AssemblyQualifiedName), "Unexpected state on GetUDTInfo");
                 // Parameter throwOnError determines whether exception from Assembly.Load is thrown.
-                metaData.Udt.Type =
-                    Type.GetType(typeName: metaData.Udt.AssemblyQualifiedName, assemblyResolver: asmRef => ResolveTypeAssembly(asmRef, fThrow), typeResolver: null, throwOnError: fThrow);
+                metaData.udt.Type =
+                    Type.GetType(typeName: metaData.udt.AssemblyQualifiedName, assemblyResolver: asmRef => ResolveTypeAssembly(asmRef, fThrow), typeResolver: null, throwOnError: fThrow);
 
-                if (fThrow && metaData.Udt.Type == null)
+                if (fThrow && metaData.udt.Type == null)
                 {
-                    throw SQL.UDTUnexpectedResult(metaData.Udt.AssemblyQualifiedName);
+                    throw SQL.UDTUnexpectedResult(metaData.udt.AssemblyQualifiedName);
                 }
             }
         }
@@ -2484,7 +2484,7 @@ internal object GetUdtValue(object value, SqlMetaDataPriv metaData, bool returnD
             // Since the serializer doesn't handle nulls...
             if (ADP.IsNull(value))
             {
-                Type t = metaData.Udt?.Type;
+                Type t = metaData.udt?.Type;
                 Debug.Assert(t != null, "Unexpected null of udtType on GetUdtValue!");
                 o = t.InvokeMember("Null", BindingFlags.Public | BindingFlags.GetProperty | BindingFlags.Static, null, null, Array.Empty<object>(), CultureInfo.InvariantCulture);
                 Debug.Assert(o != null);
@@ -2495,7 +2495,7 @@ internal object GetUdtValue(object value, SqlMetaDataPriv metaData, bool returnD
 
                 MemoryStream stm = new MemoryStream((byte[])value);
 
-                o = Server.SerializationHelperSql9.Deserialize(stm, metaData.Udt?.Type);
+                o = Server.SerializationHelperSql9.Deserialize(stm, metaData.udt?.Type);
 
                 Debug.Assert(o != null, "object could NOT be created");
                 return o;

src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlCommand.cs

@@ -21,19 +21,19 @@ internal SqlDbColumn(_SqlMetaData md)
         private void Populate()
         {
             AllowDBNull = _metadata.IsNullable;
-            BaseCatalogName = _metadata.CatalogName;
-            BaseColumnName = _metadata.BaseColumn;
-            BaseSchemaName = _metadata.SchemaName;
-            BaseServerName = _metadata.ServerName;
-            BaseTableName = _metadata.TableName;
-            ColumnName = _metadata.Column;
-            ColumnOrdinal = _metadata.Ordinal;
-            ColumnSize = (_metadata.MetaType.IsSizeInCharacters && (_metadata.Length != 0x7fffffff)) ? (_metadata.Length / 2) : _metadata.Length;
+            BaseCatalogName = _metadata.catalogName;
+            BaseColumnName = _metadata.baseColumn;
+            BaseSchemaName = _metadata.schemaName;
+            BaseServerName = _metadata.serverName;
+            BaseTableName = _metadata.tableName;
+            ColumnName = _metadata.column;
+            ColumnOrdinal = _metadata.ordinal;
+            ColumnSize = (_metadata.metaType.IsSizeInCharacters && (_metadata.length != 0x7fffffff)) ? (_metadata.length / 2) : _metadata.length;
             IsAutoIncrement = _metadata.IsIdentity;
             IsIdentity = _metadata.IsIdentity;
-            IsLong = _metadata.MetaType.IsLong;
+            IsLong = _metadata.metaType.IsLong;
 
-            if (SqlDbType.Timestamp == _metadata.Type)
+            if (SqlDbType.Timestamp == _metadata.type)
             {
                 IsUnique = true;
             }
@@ -42,18 +42,18 @@ private void Populate()
                 IsUnique = false;
             }
 
-            if (TdsEnums.UNKNOWN_PRECISION_SCALE != _metadata.Precision)
+            if (TdsEnums.UNKNOWN_PRECISION_SCALE != _metadata.precision)
             {
-                NumericPrecision = _metadata.Precision;
+                NumericPrecision = _metadata.precision;
             }
             else
             {
-                NumericPrecision = _metadata.MetaType.Precision;
+                NumericPrecision = _metadata.metaType.Precision;
             }
 
             IsReadOnly = _metadata.IsReadOnly;
 
-            UdtAssemblyQualifiedName = _metadata.Udt?.AssemblyQualifiedName;
+            UdtAssemblyQualifiedName = _metadata.udt?.AssemblyQualifiedName;
 
         }
 

src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlConnection.cs

@@ -698,8 +698,8 @@ public override string ServerVersion
         {
             get
             {
-                return (string.Format("{0:00}.{1:00}.{2:0000}", _loginAck.MajorVersion,
-                       (short)_loginAck.MinorVersion, _loginAck.BuildNum));
+                return (string.Format("{0:00}.{1:00}.{2:0000}", _loginAck.majorVersion,
+                       (short)_loginAck.minorVersion, _loginAck.buildNum));
             }
         }
 
@@ -722,7 +722,7 @@ protected override bool UnbindOnTransactionCompletion
         /// <summary>
         /// Validates if federated authentication is used, Access Token used by this connection is active for the value of 'accessTokenExpirationBufferTime'.
         /// </summary>
-        internal override bool IsAccessTokenExpired => _federatedAuthenticationInfoRequested && DateTime.FromFileTimeUtc(_fedAuthToken.ExpirationFileTime) < DateTime.UtcNow.AddSeconds(accessTokenExpirationBufferTime);
+        internal override bool IsAccessTokenExpired => _federatedAuthenticationInfoRequested && DateTime.FromFileTimeUtc(_fedAuthToken.expirationFileTime) < DateTime.UtcNow.AddSeconds(accessTokenExpirationBufferTime);
 
         ////////////////////////////////////////////////////////////////////////////////////////
         // GENERAL METHODS
@@ -1278,37 +1278,37 @@ private void Login(ServerInfo server, TimeoutTimer timeout, string newPassword,
                 }
             }
 
-            login.Authentication = ConnectionOptions.Authentication;
-            login.Timeout = timeoutInSeconds;
-            login.UserInstance = ConnectionOptions.UserInstance;
-            login.HostName = ConnectionOptions.ObtainWorkstationId();
-            login.UserName = ConnectionOptions.UserID;
-            login.Password = ConnectionOptions.Password;
-            login.ApplicationName = ConnectionOptions.ApplicationName;
+            login.authentication = ConnectionOptions.Authentication;
+            login.timeout = timeoutInSeconds;
+            login.userInstance = ConnectionOptions.UserInstance;
+            login.hostName = ConnectionOptions.ObtainWorkstationId();
+            login.userName = ConnectionOptions.UserID;
+            login.password = ConnectionOptions.Password;
+            login.applicationName = ConnectionOptions.ApplicationName;
 
-            login.Language = _currentLanguage;
-            if (!login.UserInstance)
+            login.language = _currentLanguage;
+            if (!login.userInstance)
             {
                 // Do not send attachdbfilename or database to SSE primary instance
-                login.Database = CurrentDatabase;
-                login.AttachDbFilename = ConnectionOptions.AttachDBFilename;
+                login.database = CurrentDatabase;
+                login.attachDBFilename = ConnectionOptions.AttachDBFilename;
             }
 
             // VSTS#795621 - Ensure ServerName is Sent During TdsLogin To Enable Sql Azure Connectivity.
             // Using server.UserServerName (versus ConnectionOptions.DataSource) since TdsLogin requires
             // serverName to always be non-null.
-            login.ServerName = server.UserServerName;
+            login.serverName = server.UserServerName;
 
-            login.UseReplication = ConnectionOptions.Replication;
-            login.UseSspi = ConnectionOptions.IntegratedSecurity  // Treat AD Integrated like Windows integrated when against a non-FedAuth endpoint
+            login.useReplication = ConnectionOptions.Replication;
+            login.useSSPI = ConnectionOptions.IntegratedSecurity  // Treat AD Integrated like Windows integrated when against a non-FedAuth endpoint
                                      || (ConnectionOptions.Authentication == SqlAuthenticationMethod.ActiveDirectoryIntegrated && !_fedAuthRequired);
-            login.PacketSize = _currentPacketSize;
-            login.NewPassword = newPassword;
-            login.ReadOnlyIntent = ConnectionOptions.ApplicationIntent == ApplicationIntent.ReadOnly;
-            login.Credential = _credential;
+            login.packetSize = _currentPacketSize;
+            login.newPassword = newPassword;
+            login.readOnlyIntent = ConnectionOptions.ApplicationIntent == ApplicationIntent.ReadOnly;
+            login.credential = _credential;
             if (newSecurePassword != null)
             {
-                login.NewSecurePassword = newSecurePassword;
+                login.newSecurePassword = newSecurePassword;
             }
 
             TdsEnums.FeatureExtension requestedFeatures = TdsEnums.FeatureExtension.None;
@@ -1338,9 +1338,9 @@ private void Login(ServerInfo server, TimeoutTimer timeout, string newPassword,
                 _fedAuthFeatureExtensionData =
                     new FederatedAuthenticationFeatureExtensionData
                     {
-                        LibraryType = TdsEnums.FedAuthLibrary.MSAL,
-                        Authentication = ConnectionOptions.Authentication,
-                        FedAuthRequiredPreLoginResponse = _fedAuthRequired
+                        libraryType = TdsEnums.FedAuthLibrary.MSAL,
+                        authentication = ConnectionOptions.Authentication,
+                        fedAuthRequiredPreLoginResponse = _fedAuthRequired
                     };
             }
 
@@ -1349,9 +1349,9 @@ private void Login(ServerInfo server, TimeoutTimer timeout, string newPassword,
                 requestedFeatures |= TdsEnums.FeatureExtension.FedAuth;
                 _fedAuthFeatureExtensionData = new FederatedAuthenticationFeatureExtensionData
                 {
-                    LibraryType = TdsEnums.FedAuthLibrary.SecurityToken,
-                    FedAuthRequiredPreLoginResponse = _fedAuthRequired,
-                    AccessToken = _accessTokenInBytes
+                    libraryType = TdsEnums.FedAuthLibrary.SecurityToken,
+                    fedAuthRequiredPreLoginResponse = _fedAuthRequired,
+                    accessToken = _accessTokenInBytes
                 };
                 // No need any further info from the server for token based authentication. So set _federatedAuthenticationRequested to true
                 _federatedAuthenticationRequested = true;
@@ -2134,14 +2134,14 @@ internal void OnLoginAck(SqlLoginAck rec)
             _loginAck = rec;
             if (_recoverySessionData != null)
             {
-                if (_recoverySessionData._tdsVersion != rec.TdsVersion)
+                if (_recoverySessionData._tdsVersion != rec.tdsVersion)
                 {
                     throw SQL.CR_TDSVersionNotPreserved(this);
                 }
             }
             if (_currentSessionData != null)
             {
-                _currentSessionData._tdsVersion = rec.TdsVersion;
+                _currentSessionData._tdsVersion = rec.tdsVersion;
             }
         }
 
@@ -2179,7 +2179,7 @@ internal void OnFedAuthInfo(SqlFedAuthInfo fedAuthInfo)
                 Debug.Assert(_dbConnectionPool.AuthenticationContexts != null);
 
                 // Construct the dbAuthenticationContextKey with information from FedAuthInfo and store for later use, when inserting in to the token cache.
-                _dbConnectionPoolAuthenticationContextKey = new DbConnectionPoolAuthenticationContextKey(fedAuthInfo.StsUrl, fedAuthInfo.Spn);
+                _dbConnectionPoolAuthenticationContextKey = new DbConnectionPoolAuthenticationContextKey(fedAuthInfo.stsurl, fedAuthInfo.spn);
 
                 // Try to retrieve the authentication context from the pool, if one does exist for this key.
                 if (_dbConnectionPool.AuthenticationContexts.TryGetValue(_dbConnectionPoolAuthenticationContextKey, out dbConnectionPoolAuthenticationContext))
@@ -2272,11 +2272,11 @@ internal void OnFedAuthInfo(SqlFedAuthInfo fedAuthInfo)
 
                 // If the code flow is here, then we are re-using the context from the cache for this connection attempt and not
                 // generating a new access token on this thread.
-                _fedAuthToken.AccessToken = dbConnectionPoolAuthenticationContext.AccessToken;
-                _fedAuthToken.ExpirationFileTime = dbConnectionPoolAuthenticationContext.ExpirationTime.ToFileTime();
+                _fedAuthToken.accessToken = dbConnectionPoolAuthenticationContext.AccessToken;
+                _fedAuthToken.expirationFileTime = dbConnectionPoolAuthenticationContext.ExpirationTime.ToFileTime();
             }
 
-            Debug.Assert(_fedAuthToken != null && _fedAuthToken.AccessToken != null, "fedAuthToken and fedAuthToken.accessToken cannot be null.");
+            Debug.Assert(_fedAuthToken != null && _fedAuthToken.accessToken != null, "fedAuthToken and fedAuthToken.accessToken cannot be null.");
             _parser.SendFedAuthToken(_fedAuthToken);
         }
 
@@ -2374,8 +2374,8 @@ internal SqlFedAuthToken GetFedAuthToken(SqlFedAuthInfo fedAuthInfo)
                 {
                     var authParamsBuilder = new SqlAuthenticationParameters.Builder(
                             authenticationMethod: ConnectionOptions.Authentication,
-                            resource: fedAuthInfo.Spn,
-                            authority: fedAuthInfo.StsUrl,
+                            resource: fedAuthInfo.spn,
+                            authority: fedAuthInfo.stsurl,
                             serverName: ConnectionOptions.DataSource,
                             databaseName: ConnectionOptions.InitialCatalog)
                         .WithConnectionId(_clientConnectionId)
@@ -2482,7 +2482,7 @@ internal SqlFedAuthToken GetFedAuthToken(SqlFedAuthInfo fedAuthInfo)
                             break;
                     }
 
-                    Debug.Assert(_fedAuthToken.AccessToken != null, "AccessToken should not be null.");
+                    Debug.Assert(_fedAuthToken.accessToken != null, "AccessToken should not be null.");
 #if DEBUG
                     if (_forceMsalRetry)
                     {
@@ -2569,13 +2569,13 @@ internal SqlFedAuthToken GetFedAuthToken(SqlFedAuthInfo fedAuthInfo)
             }
 
             Debug.Assert(_fedAuthToken != null, "fedAuthToken should not be null.");
-            Debug.Assert(_fedAuthToken.AccessToken != null && _fedAuthToken.AccessToken.Length > 0, "fedAuthToken.accessToken should not be null or empty.");
+            Debug.Assert(_fedAuthToken.accessToken != null && _fedAuthToken.accessToken.Length > 0, "fedAuthToken.accessToken should not be null or empty.");
 
             // Store the newly generated token in _newDbConnectionPoolAuthenticationContext, only if using pooling.
             if (_dbConnectionPool != null)
             {
-                DateTime expirationTime = DateTime.FromFileTimeUtc(_fedAuthToken.ExpirationFileTime);
-                _newDbConnectionPoolAuthenticationContext = new DbConnectionPoolAuthenticationContext(_fedAuthToken.AccessToken, expirationTime);
+                DateTime expirationTime = DateTime.FromFileTimeUtc(_fedAuthToken.expirationFileTime);
+                _newDbConnectionPoolAuthenticationContext = new DbConnectionPoolAuthenticationContext(_fedAuthToken.accessToken, expirationTime);
             }
             SqlClientEventSource.Log.TryTraceEvent("<sc.SqlInternalConnectionTds.GetFedAuthToken> {0}, Finished generating federated authentication token.", ObjectID);
             return _fedAuthToken;
@@ -2667,7 +2667,7 @@ internal void OnFeatureExtAck(int featureId, byte[] data)
 
                         Debug.Assert(_fedAuthFeatureExtensionData != null, "_fedAuthFeatureExtensionData must not be null when _federatedAuthenticationRequested == true");
 
-                        switch (_fedAuthFeatureExtensionData.LibraryType)
+                        switch (_fedAuthFeatureExtensionData.libraryType)
                         {
                             case TdsEnums.FedAuthLibrary.MSAL:
                             case TdsEnums.FedAuthLibrary.SecurityToken:
@@ -2682,7 +2682,7 @@ internal void OnFeatureExtAck(int featureId, byte[] data)
                             default:
                                 Debug.Fail("Unknown _fedAuthLibrary type");
                                 SqlClientEventSource.Log.TryTraceEvent("<sc.SqlInternalConnectionTds.OnFeatureExtAck|ERR> {0}, Attempting to use unknown federated authentication library", ObjectID);
-                                throw SQL.ParsingErrorLibraryType(ParsingErrorState.FedAuthFeatureAckUnknownLibraryType, (int)_fedAuthFeatureExtensionData.LibraryType);
+                                throw SQL.ParsingErrorLibraryType(ParsingErrorState.FedAuthFeatureAckUnknownLibraryType, (int)_fedAuthFeatureExtensionData.libraryType);
                         }
                         _federatedAuthenticationAcknowledged = true;